[03:33] <mup> PR snapd#8943 opened: wrappers: generate D-Bus service activation files <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8943>
[05:23] <mborzecki> morning
[06:10] <mborzecki> hmm ubuntu-core-20 prepare sometimes fails right at the start, before we transtion from 20.04 host to actual uc20 https://paste.ubuntu.com/p/XrZQbS6mFf/
[06:21] <mborzecki> mvo: hey
[06:29] <mvo> mborzecki: good morning
[06:39] <mup> PR snapd#8912 closed: tests: Remove unity test from nightly test suite <Skip spread> <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8912>
[06:40] <mborzecki> mvo: there was an interesting failur in prepare of uc20: https://paste.ubuntu.com/p/XrZQbS6mFf/
[06:40] <mborzecki> mvo: happens right at the beginning when we purge snapd on the host
[06:41] <mborzecki> surprisingly, haven't seen it in regular 20.04 tests
[06:48] <mvo> mborzecki: looking
[06:49] <mvo> mborzecki: so purge did not actually work and we have leftover data?
[06:49] <mvo> mborzecki: my theory is that we need to stop snapd earlier in postrm
[06:49] <mborzecki> mvo: that's your pr right?
[06:49] <mvo> mborzecki: still does not explain everything, but something like 8883
[06:49] <mvo> mborzecki: I was thinking about it
[06:49] <mvo> mborzecki: maybe we need to do the cleanup in two phases
[06:50] <mvo> mborzecki: or N phases, cleanup, stop snapd, cleanup again if anything is left
[06:50] <mvo> mborzecki: to ensure we catch stuff in flight - just a theory right now
[06:50] <mborzecki> mhm
[06:51] <mvo> mborzecki: would be cool to get the state.json content
[06:51] <mvo> mborzecki: maybe that's the next debug output and/or snapd journal output, then we can see if e.g. snapd is restarting whle we purge it
[06:51] <mborzecki> also, debug section is apparently not executed  when prepare fails
[06:53] <mvo> mborzecki: oh no
[06:53] <mvo> mborzecki: meh, ok
[06:59] <mup> PR snapd#8944 opened: tests/lib/prepare-restore: collect debug info when prepare purge fails <Precious Logs> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8944>
[06:59] <mborzecki> mvo: ^^ ok, let's see if anythign comes up
[07:01] <mvo> mborzecki: \o/ thank you so much
[07:02] <pstolowski> morning
[07:02] <mvo> mborzecki: the state.json maybe a bit unwieldy but pushing it through python3 -m json.tool will just make it incredible long :/ and jq seems overkill (plus we may important info) so let's do it
[07:03] <mvo> (do it this way)
[07:03] <mvo> hey pstolowski good morning!
[07:03] <mborzecki> pstolowski: hey
[07:03] <mborzecki> mvo: hmm it should be failry empty at this point, it's a clean system
[07:03] <mborzecki> (or at least should be)
[07:04] <mborzecki> mvo: i don't want to interrupt this run, so let's wait to see what happens and i can push the python bit next
[07:04] <mvo> mborzecki: aha, indeed, it will just have the seeding for lxd/core18 - good point
[07:05] <mvo> mborzecki: yeah, no python bits needed I think, was mostly thinking alound/wanted to talk it over. I think we can paste locally and run through jq or similar
[07:49] <zyga> hello
[07:49] <zyga> sorry for being offline, I was working on the other laptop and I haven't set IRC up there yet
[07:49] <zyga> updated refresh backend branch and fixed two failing tests
[07:50] <zyga> how are things?
[07:53] <mvo> hey zyga
[07:53] <zyga> hey mvo, welcome back
[07:56] <mborzecki> zyga: hey
[08:04] <mborzecki> mvo: haha https://github.com/snapcore/snapd/pull/8944 passed on uc20
[08:04] <mup> PR #8944: tests/lib/prepare-restore: collect debug info when prepare purge fails <Precious Logs> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8944>
[08:04] <mup> PR snapd#8925 closed: bootloader: allow managed bootloader to update its boot config <Needs Samuele review> <UC20> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8925>
[08:08] <mvo> mborzecki: of course it did ;)
[08:08] <mvo> mborzecki: fun with races
[08:08] <mvo> mborzecki: I guess the rule is "if there is a race you always loose - even when you win!"
[08:08] <mvo> mborzecki: let's just merge it and we will get good debug in one of the subsequent failures, wdyt?
[08:09] <mborzecki> yup
[08:09] <pedronis> mvo: mborzecki: hi, #8817 needs 2nd reviews
[08:09] <mup> PR #8817: tests: new test to validate refresh and revert of kernel and gadget on uc20 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8817>
[08:10] <mborzecki> zyga: pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/8944 ?
[08:10] <mup> PR #8944: tests/lib/prepare-restore: collect debug info when prepare purge fails <Precious Logs> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8944>
[08:10] <zyga> sure
[08:11] <pstolowski> yep
[08:11] <zyga> mborzecki merge it!
[08:12] <pstolowski> i made a syggestion... to late
[08:12] <pstolowski> *too
[08:12] <mborzecki> pstolowski: hah, do we have a wrapper for -m json.tool?
[08:14] <pstolowski> hmm nope, also very few test use json.tool; no tests use json_pp
[08:14] <mup> PR snapd#8944 closed: tests/lib/prepare-restore: collect debug info when prepare purge fails <Precious Logs> <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8944>
[08:18]  * zyga pushed to https://github.com/snapcore/snapd/pull/7825 - only ~ 600 lines change now
[08:18] <mup> PR #7825: many: use transient scope for tracking apps and hooks <Needs security review> <Security-High> <Created by zyga> <https://github.com/snapcore/snapd/pull/7825>
[08:20] <zyga> pstolowski I added one more comment to https://github.com/snapcore/snapd/pull/8939/files#r447500153
[08:20] <mup> PR #8939: [RFC] snap-confine: don't die if a device from sysfs path cannot be found by udev <Bug> <Needs security review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8939>
[08:22] <zyga> mborzecki I made one more, super small this time, branch carved out of the big backend branch: https://github.com/snapcore/snapd/pull/8938
[08:22] <mup> PR #8938: sandbox/cgroup: extend SnapNameFromPid with tracking cgroup data <Created by zyga> <https://github.com/snapcore/snapd/pull/8938>
[08:26] <mvo> pedronis: reviewing something else right now but will put it on my list
[08:34] <mup> PR snapd#8817 closed: tests: new test to validate refresh and revert of kernel and gadget on uc20 <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8817>
[08:39] <mup> PR snapd#8923 closed: wrappers: helper for enabling services (8/9) <Services ⚙️> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/8923>
[08:42] <mborzecki> need to run an errand, i'll probably work from some cafe or sth for a while, bbl
[08:51] <zyga> brb
[09:19] <mup> PR snapd#8940 closed: tests: fix "restart.service" <Simple 😃> <Test Robustness> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8940>
[09:59] <mup> PR snapd#8945 opened: tests: fix leaked dbus-daemon in selinux-clean <Simple 😃> <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8945>
[10:21] <mborzecki> and re, at least for some time
[10:30] <mup> PR snapd#8946 opened: client: move all error kinds into errors.go and add doc strings <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8946>
[10:34] <mborzecki> pedronis: added a comment under https://github.com/snapcore/snapd/pull/8930#discussion_r446897486
[10:34] <mup> PR #8930: many: managed boot config during run mode setup <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8930>
[10:35] <mborzecki> it should probably be both flags tbh (if we ever have a script that's not aware of kernel.efi at some point)
[10:47] <pedronis> mborzecki: answered, still we should add the TODO
[10:48] <mborzecki> ok
[11:06] <zyga> re
[12:00] <mup> PR snapd#8947 opened: many: update managd boot config when refreshing snapd <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8947>
[12:03] <pstolowski> zyga: two independent reports for a namespace error when refreshing gimp: https://bugs.launchpad.net/snapd/+bug/1884849
[12:03] <mup> Bug #1884849: Error message trying to refresh or uninstalling a snap ( gimp ) <snapd:Triaged by zyga> <https://launchpad.net/bugs/1884849>
[12:16] <mup> Bug #1873452 changed: 'snap install' | "INFO Waiting for restart..." <Snappy:Fix Released> <https://launchpad.net/bugs/1873452>
[12:22] <mup> Bug #1835342 changed: disconnect fails if plug is gone from snap meta <Snappy:Fix Released by stolowski> <https://launchpad.net/bugs/1835342>
[12:30] <mup> PR snapd#8899 closed: tests: add servicestate.Control tests (7/9) <Services ⚙️> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/8899>
[12:37] <mup> Bug #1835342 opened: disconnect fails if plug is gone from snap meta <Snappy:Fix Released by stolowski> <https://launchpad.net/bugs/1835342>
[12:43] <mup> Bug #1835342 changed: disconnect fails if plug is gone from snap meta <Snappy:Fix Released by stolowski> <https://launchpad.net/bugs/1835342>
[14:06] <mborzecki> ijohnson: there's some little tweaks in https://github.com/snapcore/snapd/pull/8918 and it could land, i can push a patch there if you haven't started with the fixes yet
[14:06] <mup> PR #8918: many: make nested spread tests more reliable <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8918>
[14:08] <ijohnson> mborzecki: no unfortunately I haven't had time to apply those yet, so feel free to do so, thanks!
[14:09] <mborzecki> ijohnson: cool, will do
[14:29] <cachio> zyga, hey, about the rename of the functions on the core-config helper, do you have something in mind?
[14:29] <cachio> zyga, I though prepare_model -> prepare_core_model
[14:30] <cachio> zyga, does it make sense?
[14:36] <mup> PR snapd#8948 opened: cmd/snap-update-ns: detect unmounted mounts <Created by zyga> <https://github.com/snapcore/snapd/pull/8948>
[14:49] <cachio> zyga, I already updated the names,. perhaps this is better
[14:50] <mborzecki> https://github.com/snapcore/snapd/pull/8918 needs a 2nd review
[14:50] <mup> PR #8918: many: make nested spread tests more reliable <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8918>
[15:22]  * cachio lunch
[17:16] <mup> PR snapd#8945 closed: tests: fix leaked dbus-daemon in selinux-clean <Simple 😃> <Test Robustness> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8945>
[17:39] <dingus9> hey recently I had an issue where after a dist upgrade all my snaps lost there connections... e.g. plugs and slots all disconnected
[17:40] <dingus9> so I ran into some interesting things while poking around... 1.  I can't find a snap command to list the default connections from meta/snap.yaml
[17:42] <dingus9> 2. I was thinking that somewhere in connections it might list the default/installed state but I can't seem to see from the output where that might be indicated
[17:43] <ogra> it just connects them at install time ... so snap connections will just show the default ones as connected
[17:43] <dingus9> 3. is there a command that can restore a snap to it's default connections, might be helpful for users that want to reset a snap if they've manually connected or disconnect things
[17:43] <dingus9> ogra: so defaults as connected currently right?
[17:43] <ogra> anything you connected later will be tagged as "manual"
[17:43] <ogra> (in the most right column of the connections output)
[17:43] <dingus9> it's interesting, my output shows - for notes on all of them with all of them disconnected
[17:44] <dingus9> naturally the app crashes
[17:44] <ogra> ogra@pi4:~$ snap connections fabrica
[17:44] <ogra> Interface       Plug                    Slot             Notes
[17:44] <ogra> lxd             fabrica:lxd             lxd:lxd          manual
[17:44] <ogra> mount-observe   fabrica:mount-observe   :mount-observe   manual
[17:44] <ogra> network         fabrica:network         :network         -
[17:44] <ogra> network-bind    fabrica:network-bind    :network-bind    -
[17:44] <ogra> ssh-keys        fabrica:ssh-keys        :ssh-keys        manual
[17:44] <ogra> system-observe  fabrica:system-observe  :system-observe  manual
[17:44] <ogra> bah ...
[17:45] <ogra> so here (even though the formatting is totally screwed) you can see that network and network-bind were connected by default
[17:45] <dingus9> in my case if the default output lists a plug with no slot... is it listed because it was defaulted?
[17:45] <dingus9> Interface        Plug                                              Slot                                           Notes
[17:45] <dingus9> alsa             google-play-music-desktop-player:alsa             -                                              -
[17:46] <dingus9> audio-playback   google-play-music-desktop-player:audio-playback   -                                              -
[17:46] <dingus9> ...
[17:46] <dingus9> desktop          google-play-music-desktop-player:desktop          :desktop                                       manual
[17:46] <ogra> hmm, audio-playback should definitely autoconnect ... as should desktop
[17:46] <dingus9> I manually started reconnecting but I thought it an odd thing to have to do
[17:46] <ogra> it is
[17:46] <dingus9> the obvious thing here is to just re-install
[17:47] <dingus9> the snap
[17:47] <ogra> either open a topic in the forum (see channel topic) or open a bug on launchpad
[17:47] <dingus9> but I thought it an interesting thing to have happened
[17:47] <ogra> this should definitely not happen
[17:47] <dingus9> ogra, thanks just wanted a sanity check
[17:47] <ogra> FSVO "interesting" ... 🙂
[17:47] <dingus9> I thought maybe I was missing something
[17:48] <ogra> nah, thats surely a bug
[17:48] <dingus9> interesting thing here is I've not idea how to reproduce the issue as this happened after a dist upgrade of popos
[17:48] <dingus9> who the hell knows what happened during that process
[17:48] <dingus9> LOL
[17:48] <ogra> "popos" ... /me grins
[17:49] <ogra> (popo is a nickly term for "butt" in german ... )
[17:50] <dingus9> haha
[17:50] <dingus9> I'll have to avoid that with my sister in law
[17:50] <dingus9> our family likes to use mushy for things that are squishy and she gets a kick out of that too
[17:51] <ogra> haha
[18:21] <mup> PR snapd#8949 opened: tests: new fs-tool which replaces the files.sh helper <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8949>
[20:22] <mup> PR snapd#8950 opened: tests: new str-tool which replaces the strings.sh helper <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8950>
[20:27] <mup> PR snapd#8924 closed: gadget, bootloader: preserve managed boot assets during gadget updates <UC20> <Created by bboozzoo> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8924>
[20:27] <mup> PR snapd#8951 opened: gadget/install: move udev trigger to gadget/install <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8951>
[20:45] <mup> PR snapcraft#3192 closed: Flutter <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3192>
[21:05] <mup> PR snapcraft#3193 opened: extensions: plug the opengl interface for GNOME <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3193>
[23:18] <mup> PR snapd#8952 opened: tests: enable tests on uc20 which now work with the real model assertion <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8952>
[23:58] <mup> PR snapd#8953 opened: tests: enable system-snap-refresh test on uc20 <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8953>