[07:06] <mborzecki> morning
[07:58] <mup> PR snapd#11023 opened: cmd/snap-confine: do not include libglvnd libraries from the host system <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/11023>
[08:06] <pstolowski> morning
[08:06] <mvo> good morning pstolowski 
[08:07] <mborzecki> pstolowski: mvo: hey
[08:08] <mborzecki> gopls is fun, it's taking mere 2gb of ram to provide completions 🙂
[08:14] <zyga-mbp> hey guys
[08:15] <zyga-mbp> jamesh thank you for working on nvidia support!
[08:15] <pstolowski> hey zyga-mbp 
[08:16] <mvo> hey mborzecki and good morning zyga-mbp ! 
[08:16] <mvo> zyga-mbp: how are you feeling?
[08:16]  * mvo hugs jamesh for his nvidia branch
[08:16] <zyga-mbp> mvo hey Michael 
[08:17] <zyga-mbp> a little better, oxygen stabilized, just tired through and through 
[08:25] <mvo> zyga-mbp: get well! did you had to go to the hospital?
[08:32] <zyga-mbp> no, hospital is only used in emergencies 
[08:32] <zyga-mbp> if I feel bad or cannot breathe, I'm supposed to call my doctor or 112 directly and wait for an ambulance 
[08:32] <zyga-mbp> being vaccinated means that the illness is less severe for sure
[08:33]  * zyga-mbp I read that only 3% of people who had to doses and were past the vaccine resistance build-up phase were confirmed to have contracted the illness 
[08:33] <zyga-mbp> so I'm unlucky perhaps, but even those 3% had much less chance of needing to be hospitalized or to have died
[08:34] <mvo> zyga-mbp: thanks for sharing, good to hear that it's less servere for you
[08:34] <zyga-mbp> I think the only scary part is that you just cannot breathe normally at times, that is weird, like being underwater on an empty tank
[08:35] <zyga-mbp> plus you're tired from doing nothing at all, but well, I take that over being dead
[08:36] <jamesh> zyga-mbp: no problem. I'm just glad that we can fix the immediate problems without massively rearchitecting things (i.e. implementing your hardware support snap proposal)
[08:36]  * zyga-mbp when we were making all those changes, it did not occur to me that some of the libraries are not made by nvidia
[08:37] <jamesh> zyga-mbp: things were quite different in 16.04
[08:37] <zyga-mbp> really good outcome, I don't use a lot of nvidia but my thinkpad technically has that gpu 
[08:37] <zyga-mbp> yes
[08:49] <mvo> mborzecki: I see some test failures related to the new memory limits (e.g. in 11022 in the 20.04 run). do we know more what is going on here?
[08:50] <mborzecki> mvo: let me see
[08:53] <mborzecki> mvo: hm not sure if that's related to the limit which is 100MB, but the snap is supposed to be 8k, tbh the test is quite confusing, does it expect that snap install will succeed?
[08:53] <mup> PR snapd#11020 closed: timeutil,deviceutil: fix unit tests on systems without dbus or without ntp-sync <⚠ Critical> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/11020>
[08:58] <mardy> hi mborzecki, mvo, pstolowski, zyga-mbp 
[08:58] <mborzecki> mvo: can you merge https://github.com/snapcore/snapd/pull/11017 ?
[08:58] <mup> PR #11017: data/selinux: update the policy to allow snapd to talk to org.freedesktop.timedate1 <⚠ Critical> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/11017>
[08:58] <mborzecki> mardy: hey
[08:58] <pstolowski> hey mardy 
[09:27] <mborzecki> pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/11019 ?
[09:27] <mup> PR #11019: overlord/state: add a unit test for a kernel+base refresh like sequence <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/11019>
[09:28] <mborzecki> it's a simple one
[09:29] <pstolowski> sure
[09:44] <mup> PR snapd#10920 closed: osutil/disks: add AllPhysicalDisks()  <Simple 😃> <Skip spread> <Created by anonymouse64> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10920>
[10:13] <zyga-mbp> stgraber I'm sorry for asking here, not sure if there's an LXD channel - I'm using lxd virtual machines with usb pass-through, it seems to work well as long as you only have one matching device, if you have several, only one (at random) is passed to the VM, is that a known issue?
[10:29] <mup> Bug #1948872 changed: Should not document download-by-revision until it's fixed <snapd:Triaged> <https://launchpad.net/bugs/1948872>
[10:40] <pstolowski> mvo: could you land https://github.com/snapcore/snapd/pull/10867 ? only seccomp syscalls & selinux failures
[10:40] <mup> PR #10867: desktop, usersession: observe notifications <Created by stolowski> <https://github.com/snapcore/snapd/pull/10867>
[10:40] <mvo> pstolowski: sure
[10:41] <pstolowski> ty!
[10:44] <mup> PR snapd#10867 closed: desktop, usersession: observe notifications <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10867>
[11:29] <mup> PR snapd#11019 closed: overlord/state: add a unit test for a kernel+base refresh like sequence <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/11019>
[12:11]  * zyga-mbp stgraber with a bit of debugging I've found that the usb logic correctly finds the devices, but they are not rendered into qemu.conf 
[12:11] <zyga-mbp> only one device is
[12:12] <zyga-mbp> with an extra printf at the end of lxd/lxd/device/usb.go:183 (in startVM), I can see that all the devices are collected:
[12:13] <zyga-mbp> looking at the template side
[12:18] <zyga-mbp> so in driver_qemu.go, in addUSBDeviceConfig we get an slice of RunConfigItem (usbConfig) but only act on one, the last one?
[12:19] <zyga-mbp> stgraber is this intentional or shall I report a bug for it and send patches?
[12:41] <zyga-mbp> stgraber I have a patch that I've tested locally
[12:42] <zyga-mbp> stgraber I'll send a PR and we can move the discussion there
[12:48] <zyga-mbp> stgraber https://github.com/lxc/lxd/pull/9497
[12:48] <mup> PR lxc/lxd#9497: lxd/instances/qemu: fix usb pass-through with more than one device <Created by zyga> <https://github.com/lxc/lxd/pull/9497>
[14:46] <miguelpires> mvo: can you merge https://github.com/snapcore/snapd/pull/10987 please?
[14:46] <mup> PR #10987: o/snapstate: keep old revision if install doesn't add new one <Squash-merge> <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10987>
[14:55] <mup> PR snapd#10987 closed: o/snapstate: keep old revision if install doesn't add new one <Squash-merge> <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10987>
[15:55] <mup> PR snapd#11017 closed: data/selinux: update the policy to allow snapd to talk to org.freedesktop.timedate1 <⚠ Critical> <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/11017>
[16:51] <pstolowski> mvo: this would be great to land - https://github.com/snapcore/snapd/pull/10962 - only seccomp and selinux failures
[16:51] <mup> PR #10962: o/assertstate: functions for handling validation sets tracking history <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10962>
[16:55] <mup> PR snapd#11024 opened: boot: expand on the fact that reseal taking modeenv is very intentional <Remodel 🚋> <Skip spread> <Created by pedronis> <https://github.com/snapcore/snapd/pull/11024>
[17:05] <mup> PR snapd#11025 opened: Allow access to hunspell dictionaries for spellchecking <Created by gunnarhj> <https://github.com/snapcore/snapd/pull/11025>
[18:26] <mup> PR snapd#11018 closed: cmd/snap-seccomp/syscalls: update syscalls to match libseccomp abad8a8f4 <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/11018>
[18:31] <mup> PR snapd#11024 closed: boot: expand on the fact that reseal taking modeenv is very intentional <Remodel 🚋> <Skip spread> <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/11024>
[18:46] <mup> PR snapd#11014 closed: tests: add ntp related debug around "auto-refresh" test <Test Robustness> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/11014>
[18:46] <mup> PR snapd#11026 opened: tests: (partially) revert the memory limits PR#r10241 <Flaky Test> <Created by mvo5> <https://github.com/snapcore/snapd/pull/11026>
[19:01] <mup> PR snapd#10973 closed: interfaces/network-manager-observe: Update for libnm / dart clients <cherry-picked> <Created by alexmurray> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10973>
[22:27] <mup> PR snapd#11027 opened: osutil/disks: support device-unlock encrypted device mapper nodes in disks pkg [easy] <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/11027>
[22:32] <mup> PR snapd#11028 opened: osutil/disks: support device-unlock encrypted device mapper nodes in disks pkg [hard] <Needs Samuele review> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/11028>
[23:22] <mup> PR snapd#11008 closed: tests: some improvements for the spread log parser <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/11008>