/srv/irclogs.ubuntu.com/2020/06/30/#snappy.txt

mupPR snapd#8943 opened: wrappers: generate D-Bus service activation files <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8943>03:33
mborzeckimorning05:23
mborzeckihmm 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:10
mborzeckimvo: hey06:21
mvomborzecki: good morning06:29
mupPR 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:39
mborzeckimvo: there was an interesting failur in prepare of uc20: https://paste.ubuntu.com/p/XrZQbS6mFf/06:40
mborzeckimvo: happens right at the beginning when we purge snapd on the host06:40
mborzeckisurprisingly, haven't seen it in regular 20.04 tests06:41
mvomborzecki: looking06:48
mvomborzecki: so purge did not actually work and we have leftover data?06:49
mvomborzecki: my theory is that we need to stop snapd earlier in postrm06:49
mborzeckimvo: that's your pr right?06:49
mvomborzecki: still does not explain everything, but something like 888306:49
mvomborzecki: I was thinking about it06:49
mvomborzecki: maybe we need to do the cleanup in two phases06:49
mvomborzecki: or N phases, cleanup, stop snapd, cleanup again if anything is left06:50
mvomborzecki: to ensure we catch stuff in flight - just a theory right now06:50
mborzeckimhm06:50
mvomborzecki: would be cool to get the state.json content06:51
mvomborzecki: 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 it06:51
mborzeckialso, debug section is apparently not executed  when prepare fails06:51
mvomborzecki: oh no06:53
mvomborzecki: meh, ok06:53
mupPR 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
mborzeckimvo: ^^ ok, let's see if anythign comes up06:59
mvomborzecki: \o/ thank you so much07:01
pstolowskimorning07:02
mvomborzecki: 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 it07:02
mvo(do it this way)07:03
mvohey pstolowski good morning!07:03
mborzeckipstolowski: hey07:03
mborzeckimvo: hmm it should be failry empty at this point, it's a clean system07:03
mborzecki(or at least should be)07:03
mborzeckimvo: i don't want to interrupt this run, so let's wait to see what happens and i can push the python bit next07:04
mvomborzecki: aha, indeed, it will just have the seeding for lxd/core18 - good point07:04
mvomborzecki: 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 similar07:05
zygahello07:49
zygasorry for being offline, I was working on the other laptop and I haven't set IRC up there yet07:49
zygaupdated refresh backend branch and fixed two failing tests07:49
zygahow are things?07:50
mvohey zyga07:53
zygahey mvo, welcome back07:53
mborzeckizyga: hey07:56
mborzeckimvo: haha https://github.com/snapcore/snapd/pull/8944 passed on uc2008:04
mupPR #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
mupPR 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:04
mvomborzecki: of course it did ;)08:08
mvomborzecki: fun with races08:08
mvomborzecki: I guess the rule is "if there is a race you always loose - even when you win!"08:08
mvomborzecki: let's just merge it and we will get good debug in one of the subsequent failures, wdyt?08:08
mborzeckiyup08:09
pedronismvo: mborzecki: hi, #8817 needs 2nd reviews08:09
mupPR #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:09
mborzeckizyga: pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/8944 ?08:10
mupPR #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
zygasure08:10
pstolowskiyep08:11
zygamborzecki merge it!08:11
pstolowskii made a syggestion... to late08:12
pstolowski*too08:12
mborzeckipstolowski: hah, do we have a wrapper for -m json.tool?08:12
pstolowskihmm nope, also very few test use json.tool; no tests use json_pp08:14
mupPR 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:14
* zyga pushed to https://github.com/snapcore/snapd/pull/7825 - only ~ 600 lines change now08:18
mupPR #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:18
zygapstolowski I added one more comment to https://github.com/snapcore/snapd/pull/8939/files#r44750015308:20
mupPR #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:20
zygamborzecki I made one more, super small this time, branch carved out of the big backend branch: https://github.com/snapcore/snapd/pull/893808:22
mupPR #8938: sandbox/cgroup: extend SnapNameFromPid with tracking cgroup data <Created by zyga> <https://github.com/snapcore/snapd/pull/8938>08:22
mvopedronis: reviewing something else right now but will put it on my list08:26
mupPR 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:34
mupPR 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:39
mborzeckineed to run an errand, i'll probably work from some cafe or sth for a while, bbl08:42
zygabrb08:51
mupPR snapd#8940 closed: tests: fix "restart.service" <Simple 😃> <Test Robustness> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8940>09:19
mupPR snapd#8945 opened: tests: fix leaked dbus-daemon in selinux-clean <Simple 😃> <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8945>09:59
mborzeckiand re, at least for some time10:21
mupPR 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:30
mborzeckipedronis: added a comment under https://github.com/snapcore/snapd/pull/8930#discussion_r44689748610:34
mupPR #8930: many: managed boot config during run mode setup <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8930>10:34
mborzeckiit should probably be both flags tbh (if we ever have a script that's not aware of kernel.efi at some point)10:35
pedronismborzecki: answered, still we should add the TODO10:47
mborzeckiok10:48
zygare11:06
=== msalvatore_ is now known as msalvatore
mupPR snapd#8947 opened: many: update managd boot config when refreshing snapd <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8947>12:00
pstolowskizyga: two independent reports for a namespace error when refreshing gimp: https://bugs.launchpad.net/snapd/+bug/188484912:03
mupBug #1884849: Error message trying to refresh or uninstalling a snap ( gimp ) <snapd:Triaged by zyga> <https://launchpad.net/bugs/1884849>12:03
mupBug #1873452 changed: 'snap install' | "INFO Waiting for restart..." <Snappy:Fix Released> <https://launchpad.net/bugs/1873452>12:16
mupBug #1835342 changed: disconnect fails if plug is gone from snap meta <Snappy:Fix Released by stolowski> <https://launchpad.net/bugs/1835342>12:22
mupPR 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:30
mupBug #1835342 opened: disconnect fails if plug is gone from snap meta <Snappy:Fix Released by stolowski> <https://launchpad.net/bugs/1835342>12:37
mupBug #1835342 changed: disconnect fails if plug is gone from snap meta <Snappy:Fix Released by stolowski> <https://launchpad.net/bugs/1835342>12:43
mborzeckiijohnson: 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 yet14:06
mupPR #8918: many: make nested spread tests more reliable <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8918>14:06
ijohnsonmborzecki: no unfortunately I haven't had time to apply those yet, so feel free to do so, thanks!14:08
mborzeckiijohnson: cool, will do14:09
cachiozyga, hey, about the rename of the functions on the core-config helper, do you have something in mind?14:29
cachiozyga, I though prepare_model -> prepare_core_model14:29
cachiozyga, does it make sense?14:30
mupPR snapd#8948 opened: cmd/snap-update-ns: detect unmounted mounts <Created by zyga> <https://github.com/snapcore/snapd/pull/8948>14:36
cachiozyga, I already updated the names,. perhaps this is better14:49
mborzeckihttps://github.com/snapcore/snapd/pull/8918 needs a 2nd review14:50
mupPR #8918: many: make nested spread tests more reliable <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8918>14:50
* cachio lunch15:22
mupPR 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:16
dingus9hey recently I had an issue where after a dist upgrade all my snaps lost there connections... e.g. plugs and slots all disconnected17:39
dingus9so 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.yaml17:40
dingus92. 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 indicated17:42
ograit just connects them at install time ... so snap connections will just show the default ones as connected17:43
dingus93. 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 things17:43
dingus9ogra: so defaults as connected currently right?17:43
ograanything you connected later will be tagged as "manual"17:43
ogra(in the most right column of the connections output)17:43
dingus9it's interesting, my output shows - for notes on all of them with all of them disconnected17:43
dingus9naturally the app crashes17:44
ograogra@pi4:~$ snap connections fabrica17:44
ograInterface       Plug                    Slot             Notes17:44
ogralxd             fabrica:lxd             lxd:lxd          manual17:44
ogramount-observe   fabrica:mount-observe   :mount-observe   manual17:44
ogranetwork         fabrica:network         :network         -17:44
ogranetwork-bind    fabrica:network-bind    :network-bind    -17:44
ograssh-keys        fabrica:ssh-keys        :ssh-keys        manual17:44
ograsystem-observe  fabrica:system-observe  :system-observe  manual17:44
ograbah ...17:44
ograso here (even though the formatting is totally screwed) you can see that network and network-bind were connected by default17:45
dingus9in my case if the default output lists a plug with no slot... is it listed because it was defaulted?17:45
dingus9Interface        Plug                                              Slot                                           Notes17:45
dingus9alsa             google-play-music-desktop-player:alsa             -                                              -17:45
dingus9audio-playback   google-play-music-desktop-player:audio-playback   -                                              -17:46
dingus9...17:46
dingus9desktop          google-play-music-desktop-player:desktop          :desktop                                       manual17:46
ograhmm, audio-playback should definitely autoconnect ... as should desktop17:46
dingus9I manually started reconnecting but I thought it an odd thing to have to do17:46
ograit is17:46
dingus9the obvious thing here is to just re-install17:46
dingus9the snap17:47
ograeither open a topic in the forum (see channel topic) or open a bug on launchpad17:47
dingus9but I thought it an interesting thing to have happened17:47
ograthis should definitely not happen17:47
dingus9ogra, thanks just wanted a sanity check17:47
ograFSVO "interesting" ... 🙂17:47
dingus9I thought maybe I was missing something17:47
ogranah, thats surely a bug17:48
dingus9interesting thing here is I've not idea how to reproduce the issue as this happened after a dist upgrade of popos17:48
dingus9who the hell knows what happened during that process17:48
dingus9LOL17:48
ogra"popos" ... /me grins17:48
ogra(popo is a nickly term for "butt" in german ... )17:49
dingus9haha17:50
dingus9I'll have to avoid that with my sister in law17:50
dingus9our family likes to use mushy for things that are squishy and she gets a kick out of that too17:50
ograhaha17:51
mupPR snapd#8949 opened: tests: new fs-tool which replaces the files.sh helper <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8949>18:21
mupPR snapd#8950 opened: tests: new str-tool which replaces the strings.sh helper <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8950>20:22
mupPR 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
mupPR snapd#8951 opened: gadget/install: move udev trigger to gadget/install <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8951>20:27
=== pedronis_ is now known as pedronis
mupPR snapcraft#3192 closed: Flutter <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3192>20:45
mupPR snapcraft#3193 opened: extensions: plug the opengl interface for GNOME <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3193>21:05
mupPR 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:18
mupPR snapd#8953 opened: tests: enable system-snap-refresh test on uc20 <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8953>23:58

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!