| mborzecki | morning | 05:11 |
|---|---|---|
| zyga-mbp | good morning :) | 06:33 |
| 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:38 |
| 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:51 |
| 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 | 06:52 |
| 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:01 |
| 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:02 |
| 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:11 |
| 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:20 |
| mvo | mborzecki: thanks! | 07:21 |
| mborzecki | mvo: thank you! | 07:23 |
| 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:26 |
| mborzecki | so https://github.com/snapcore/snapd/pull/10803 needs a 2nd review now 😉 (and a security one for the docker-support patch) | 07:27 |
| 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:28 |
| amurray | mvo: yep no worries - I'll try take a look soon | 07:30 |
| mvo | thanks so much! | 07:31 |
| 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:41 |
| 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:46 |
| mardy | hi mborzecki, pstolowski, mvo! | 07:49 |
| pstolowski | o/ | 07:49 |
| mvo | hey mardy ! | 07:50 |
| 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:07 |
| mvo | mardy: sure | 08:14 |
| mvo | looks like 10805 needs a second review and will make tests happier | 08:15 |
| mvo | oh, nevermind, already got one | 08:15 |
| 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:16 |
| 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:39 |
| 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> | 08:41 |
| 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:11 |
| 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:12 |
| mvo | abeato_: cool, looking | 09:15 |
| abeato_ | thanks! | 09:16 |
| 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:38 |
| abeato_ | mvo, no prob, thanks for the guidance. Yeah, I will follow the debian bugreport route. | 09:40 |
| 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:07 |
| 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. | 10:34 |
| 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> | 12: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:07 |
| mup | PR snapd#10813 opened: .github/workflows/test.yaml: revert #10809 <Simple 😃> <Skip spread> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10813> | 13:57 |
| ijohnson[m] | pstolowski: any insight into the "snap connections disappeared" after snap mount units disappear ? | 15:03 |
| ijohnson[m] | *issue | 15:03 |
| 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:04 |
| pstolowski | ijohnson[m]: the issue is caused by reloadConnections as Samuele guessed | 15:07 |
| 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:08 |
| 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:23 |
| pstolowski | ijohnson[m]: to be clear afaict loosing connections is directly related to not having snaps mountuned | 15:26 |
| * cachio_ lunch | 15:39 | |
| 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:38 |
| 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... | 16:40 |
| cachio_ | ijohnson[m], hi, first time I see this one | 17:01 |
| cachio_ | agree it is very very weird | 17:01 |
| 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:02 |
| 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> | 17:03 |
| 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:33 |
| cachio_ | ijohnson[m], sure | 18:34 |
| ijohnson[m] | thanks | 18:34 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!