[05:11] <mborzecki> morning
[06:33] <zyga-mbp> good morning :)
[06:38] <mardy> zyga-mbp: hi!
[06:38] <zyga-mbp> hey, how are you doing
[06:38] <zyga-mbp> it's pretty nasty outside, I prefer when it's actually freezing and dry and not when it's ~5C and damp
[06:51] <mborzecki> zyga-mbp: mardy: hey
[06:51] <zyga-mbp> good morning mborzecki
[06:51] <mborzecki> zyga-mbp: yeah, late autumn started pretty early this year
[06:51] <zyga-mbp> haha
[06:51] <zyga-mbp> pretty early this summer
[06:52] <zyga-mbp> isn't technically summer not over yet?
[06:52] <mborzecki> hahah right
[06:52] <mborzecki> zyga-mbp: no, 1 more day afaik
[06:52] <mborzecki> or 3?
[06:52] <mborzecki> anyways, too early for my taste
[07:01] <mup> PR snapd#10809 closed: .github/workflows/test.yaml: bump action-build to 1.0.9 <Simple 😃> <Skip spread> <Created by anonymouse64> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10809>
[07:02] <mborzecki> https://github.com/snapcore/snapd/pull/10801 needs a 2nd review
[07:02] <mup> PR #10801: cmd/libsnap-confine-private: use root when necessary for BPF related operations <cgroupv2> <cgroupv2-impish> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10801>
[07:11] <pstolowski> morning
[07:11] <zyga-mbp> hey pstolowski  :)
[07:11] <mvo> mborzecki: 10801 looks ready, shall I merge (have not looked at the failures yet though)
[07:11] <mvo> hey pstolowski and zyga-mbp
[07:20] <mvo> amurray: hey, if you have a few cycles, a review of this particular patch  https://github.com/bboozzoo/snapd/commit/b6f6a0f388880b6a732ce0a8c184878966a95c44  in 10803 would be great, that would unblock me uploading cgroups v2 into impish
[07:20] <mborzecki> mvo: yes, please merge 10801, the failures are the usual bunch, nothing related
[07:21] <mvo> mborzecki: thanks!
[07:23] <mborzecki> mvo: thank you!
[07:26] <mup> PR snapd#10801 closed: cmd/libsnap-confine-private: use root when necessary for BPF related operations <cgroupv2> <cgroupv2-impish> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10801>
[07:27] <mborzecki> so https://github.com/snapcore/snapd/pull/10803 needs a 2nd review now 😉 (and a security one for the docker-support patch)
[07:28] <mup> PR #10803: tests, interfaces/builtin: introduce 21.10 cgroupv2 variant, tweak tests for cgroupv2, update builtin interfaces <Security-High> <Needs security review> <cgroupv2> <cgroupv2-impish> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10803>
[07:30] <amurray> mvo: yep no worries - I'll try take a look soon
[07:31] <mvo> thanks so much!
[07:41] <mup> PR snapd#10790 closed: gadget: add VolumeName to Volume and VolumeStructure <Simple 😃> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10790>
[07:46] <mup> PR snapd#10782 closed: o/ifacestate: do not create stray task in batchConnectTasks if there are no connections <Bug> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10782>
[07:49] <mardy> hi mborzecki, pstolowski, mvo!
[07:49] <pstolowski> o/
[07:50] <mvo> hey mardy !
[08:07] <mardy> mvo: can you please land this? https://github.com/snapcore/snapd/pull/10772
[08:07] <mup> PR #10772: kernel/fde: mock systemd-run in unit test <Created by mardy> <https://github.com/snapcore/snapd/pull/10772>
[08:14] <mvo> mardy: sure
[08:15] <mvo> looks like 10805 needs a second review and will make tests happier
[08:15] <mvo> oh, nevermind, already got one
[08:16] <mardy> thanks!
[08:16] <mup> PR snapd#10772 closed: kernel/fde: mock systemd-run in unit test <Created by mardy> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10772>
[08:16] <mup> PR snapd#10805 closed: overlord/devicestate: make settle wait longer in remodel tests <Squash-merge> <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10805>
[08:39] <mborzecki> mvo: can you land https://github.com/snapcore/snapd/pull/10802 too?
[08:39] <mup> PR #10802: data/selinux: update the policy to allow s-c to manipulate BPF map and programs <Simple 😃> <cgroupv2> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10802>
[08:41] <mup> PR snapd#10802 closed: data/selinux: update the policy to allow s-c to manipulate BPF map and programs <Simple 😃> <cgroupv2> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10802>
[09:11] <abeato_> good morning!
[09:11] <mup> PR snapd#10806 closed: tests: rename interfaces-hooks-misbehaving spread test to install-hook-misbehaving <Simple 😃> <Skip spread> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10806>
[09:12] <abeato_> mvo, hi there, I would need sponsorship for the modifications to get static libs for some packages. I've created LP#1943859, LP#1943984, and LP#1944107 for that
[09:15] <mvo> abeato_: cool, looking
[09:16] <abeato_> thanks!
[09:38] <mvo> abeato_: diffs look very nice, thanks! all of this is synced from debian so things are even more complicated :/ however the libdrm patch is something that tjaalton probably should review, we need to find a friendly way to ask him about this :) and the pango1.0 one is probably something that seb128 could look at. ideally all of this goes into debian so that we don't  need to maintain a delta in ubuntu. graphite2 I'm not sure about if we have experts but it seems
[09:38] <mvo> it's not changing that much so a delta is ok. for that we probably need a debian bugreport. sorry for the extra level of indirection here
[09:40] <abeato_> mvo, no prob, thanks for the guidance. Yeah, I will follow the debian bugreport route.
[10:07] <mup> PR snapd#10810 opened: o/assertstate, api: update validation set assertions only when updating all snaps <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10810>
[10:34] <mborzecki> hmm 2021-09-20 07:52:12 Cannot allocate google:ubuntu-21.10-64: cannot allocate or deallocate (!) new Google server google:ubuntu-21.10-64 (sep200751-577426): cannot provision google:ubuntu-21.10-64 (sep200751-577426): quota 'CPUS' exceeded.  Limit: 600.0 in region us-east1.
[12:07] <mup> PR snapd#10811 opened: tests: update the time tolerance to fix the snapd-state test <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10811>
[13:07] <mup> PR snapd#10812 opened: o/ifacestate: don't fail remove if disconnect hook fails <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10812>
[13:57] <mup> PR snapd#10813 opened: .github/workflows/test.yaml: revert #10809 <Simple 😃> <Skip spread> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10813>
[15:03] <ijohnson[m]> pstolowski: any insight into the "snap connections disappeared" after snap mount units disappear ?
[15:03] <ijohnson[m]> *issue
[15:04] <pstolowski> ijohnson[m]: yes, reproduced with a spread test, trying to find a sensible fix (to avoid loosing conns, missing mounts is a different story)
[15:07] <pstolowski> ijohnson[m]: the issue is caused by reloadConnections as Samuele guessed
[15:08] <ijohnson[m]> mmh I see
[15:08] <ijohnson[m]> well nice that we found that particular issue, since it has come up before that interface connections mysteriously disappeared too but less frequently than the mount units disappearing I think
[15:23] <mup> PR snapd#10813 closed: .github/workflows/test.yaml: revert #10809 <Simple 😃> <Skip spread> <Created by anonymouse64> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10813>
[15:26] <pstolowski> ijohnson[m]: to be clear afaict loosing connections is directly related to not having snaps mountuned
[15:39]  * cachio_ lunch
[16:38] <mup> PR snapd#10811 closed: tests: update the time tolerance to fix the snapd-state test <Simple 😃> <Created by sergiocazzolato> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/10811>
[16:40] <ijohnson[m]> cachio_: have you seen this one? https://pastebin.ubuntu.com/p/bbqydN7WkN/ this is really weird, it waits until the snap command exists, then finds it exists at /usr/bin/snap, then tries to run snap wait ... and it can't find the snap command somehow...
[17:01] <cachio_> ijohnson[m], hi, first time I see this one
[17:01] <cachio_> agree it is very very weird
[17:02] <ijohnson[m]> must be some sort of race condition I suppose
[17:02]  * ijohnson[m] -> lunch
[17:02] <cachio_> yes, I'll re-run it in a loop
[17:02] <cachio_> perhaps I am lucky
[17:03] <mup> PR snapd#10814 opened: [RFC] o/ifacestate: don't loose connections if snaps are broken <Created by stolowski> <https://github.com/snapcore/snapd/pull/10814>
[18:33] <ijohnson[m]> cachio_: hey I was pinged about verification of https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1929842 for focal, can you take a look?
[18:33] <mup> Bug #1929842: [SRU] 2.51 <verification-done-bionic> <verification-done-groovy> <verification-done-hirsute> <verification-needed> <verification-needed-focal> <snapd (Ubuntu):Fix Released> <snapd (Ubuntu Bionic):Fix Committed> <snapd (Ubuntu Focal):Fix Committed> <snapd (Ubuntu Groovy):Fix Committed>
[18:33] <mup> <snapd (Ubuntu Hirsute):Fix Released> <https://launchpad.net/bugs/1929842>
[18:34] <cachio_> ijohnson[m], sure
[18:34] <ijohnson[m]> thanks