[06:03] <mborzecki> morning
[06:03] <mvo> hey mborzecki - good morning
[06:03] <mborzecki> mvo: hey
[06:04] <mborzecki> mvo: did i miss anything on friday?
[06:10] <mvo> mborzecki: not much was relatively quiet
[06:12] <mborzecki> mvo: i like quiet :)
[06:12] <mvo> mee too
[06:41] <mvo> zyga: 6575 has some conflicts
[07:35] <zyga> 89.§34-[\8p]
[07:35] <zyga> =
[07:35] <zyga> good morning
[07:35] <zyga> mvo: thank you, looking now
[07:35] <mvo> zyga: good morning!
[07:36] <zyga> it's good to be back
[08:04] <pedronis> mborzecki: Hi, I made a couple of suggestions in 6576
[08:04] <mborzecki> pedronis: thanks, reading now
[08:06] <pstolowski> morning
[08:07] <mborzecki> pedronis: tbh i've alawys used determinant in the math sense, but i suppose your suggestion works too
[08:08] <pedronis> mborzecki: an other option is interfaceConstraint
[08:08] <pedronis> mborzecki: it's command impl, not an API or something used everywhere
[08:08] <pedronis> so mostly trying to unblock this
[08:09] <mvo> hey pstolowski - good morning
[08:09] <mborzecki> pstolowski: hey
[08:12] <zyga> mvo: quick question about https://github.com/snapcore/snapd/pull/6574#discussion_r264106985 - can you explain that again please?
[08:12] <mup> PR #6574: cmd/snap-confine: track per-app and per-hook processes <Created by zyga> <https://github.com/snapcore/snapd/pull/6574>
[08:12] <zyga> mvo: is the goal to see more than one process at a time?
[08:13] <mborzecki> pedronis: reading dict output a bit more, 'determinant' sounds good
[08:15] <mvo> zyga: it was mostly to ensure that the cgroup really tracks all the pids, we know this so maybe silly
[08:15] <mvo> zyga: otoh *might* be useful when we go to cgroup v2 - otoh there will probably be different interfaces then so maybe again not super useful
[08:16] <zyga> mvo: wait wait, I'm not sure I understand the comment you made
[08:16] <mvo> zyga: i.e. to ensure that any children of the test-snapd-tools.sh app are also part of the cgroup that tracks the activity of "sh"
[08:16] <zyga> mvo: is it really to track two processes?
[08:16] <zyga> ah
[08:16] <zyga> thanks!
[08:16] <zyga> I understand now
[08:16] <zyga> mvo: I will look at conjuring  something that checks that
[08:16] <mvo> zyga: great! sorry for the slightly winded way of explaining it
[08:17] <zyga> thank you for clarifying :)
[08:17] <mvo> zyga: cool! but don't kill yourself I think its a bit optional, cgroups tend to work
[08:17] <zyga> I think it's not hard, just a small extra chunk after the two tests there
[08:18] <mvo> mborzecki: quick question about 6238 - this (probably) needs a master merge to make tests happy and a full review from pstolowski - is that accurate?
[08:18] <mvo> zyga: nice
[08:22] <mborzecki> mvo: and a small update, some new denials were found in the tests which need a little bit of investigation
[08:25] <mborzecki> pedronis: pushed an update
[08:27] <pedronis> mborzecki: thx
[08:29] <mvo> mborzecki: thank you
[09:20] <Chipaca> mborzecki: #6576 seems to be GTG
[09:20] <mup> PR #6576: cmd/snap, client, daemon, ifacestate: show a leading attribute of a connection <Squash-merge> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6576>
[09:21] <mborzecki> aand merged
[09:21] <Chipaca> mborzecki: squash-merged?
[09:21] <mborzecki> Chipaca: yup
[09:22] <Chipaca> phew
[09:22] <Chipaca> :-)
[09:22] <mup> PR snapd#6576 closed: cmd/snap, client, daemon, ifacestate: show a leading attribute of a connection <Squash-merge> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6576>
[09:23] <mborzecki> mvo: opening a branch with a cherry picke in a minute
[09:25] <mup> PR snapd#6578 opened: cmd/snap, client, daemon, ifacestate: show a leading attribute of a connection (2.38) <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6578>
[09:43] <mvo> mborzecki: if it can be cherry-picked without a conflcit no need to open a PR
[09:44] <mvo> mborzecki: thanks for landing this so timely!
[09:44] <mborzecki> mvo: ah, opened the PR anyway, feel free to close :)
[09:45] <pedronis> pstolowski: thanks for applying the last round of comment on timings
[09:45] <mvo> mborzecki: thats fine
[09:45] <mvo> mborzecki: I just wanted to make your life easier :)
[09:46] <pstolowski> pedronis: sure, thanks for the suggestions
[09:46] <pedronis> Chipaca: hi, thanks again for the epochs docs,  there is a couple of things maybe to clarify and degville has some input too, when is a good time to chat for you and and degville
[09:46] <pedronis> ?
[09:47] <Chipaca> pedronis: anytime before 6pm :-)
[09:47] <degville> pedronis: Chipaca: same for me :)
[09:48] <pedronis> Chipaca: degville: can we have a HO (same as standup) in 10 mins then?
[09:48] <Chipaca> sure
[09:48] <degville> Chipaca: pedronis: good for me!
[09:59] <mborzecki> zyga: updated #6329, please take a look
[09:59] <mup> PR #6329: cmd/snap-confine, packaging: support SELinux <SELinux> <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6329>
[09:59] <zyga> mborzecki: ack, enqueued
[10:00] <mborzecki> zyga: thanks!
[10:05] <zyga> mvo: added the test to https://github.com/snapcore/snapd/pull/6574
[10:05] <mup> PR #6574: cmd/snap-confine: track per-app and per-hook processes <Created by zyga> <https://github.com/snapcore/snapd/pull/6574>
[10:08] <mvo> zyga: nice, thank you
[10:13] <mborzecki> mvo: #6578 is green
[10:13] <mup> PR #6578: cmd/snap, client, daemon, ifacestate: show a leading attribute of a connection (2.38) <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6578>
[10:16] <mvo> mborzecki: yay - thanks
[10:17] <mup> PR snapd#6578 closed: cmd/snap, client, daemon, ifacestate: show a leading attribute of a connection (2.38) <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6578>
[10:23] <Chipaca> pedronis: https://pastebin.ubuntu.com/p/2wMJ6DtjNz/
[10:30] <Chipaca> pedronis: because the check is done in mount-snap, which isn't done for reverts
[10:39] <pedronis> Chipaca: ok, so we have a problem
[11:00] <zyga> brb
[11:01] <zyga> mborzecki: is https://github.com/snapcore/snapd/pull/6329/files#r261618980 done?
[11:01] <mup> PR #6329: cmd/snap-confine, packaging: support SELinux <SELinux> <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6329>
[11:02] <mborzecki> zyga: line 64?
[11:02] <zyga> ahh
[11:02] <zyga> thanks, I missed that (sorry)
[11:02] <mborzecki> zyga: how about we switch all of the code to clang-format in a separate PR?
[11:04] <mvo> mborzecki: the downside of doing this is that we loose a lot of history for git blame
[11:04] <pedronis> of snap-confine?
[11:05] <pedronis> I fear we cannot afford that
[11:08] <mborzecki> hmn wonder if git blame -w would work around that
[11:08] <mvo> mborzecki: might be worth a quick experiment
[11:08] <mvo> (in a local PR)
[11:08] <mvo> eh, local branch
[11:12] <Chipaca> brb, gym
[11:14] <zyga> mborzecki: let's not
[11:14] <zyga> mborzecki: I wanted that but then considered it a bad move
[11:14] <zyga> mborzecki: it's better to evolve code over time
[11:15] <zyga> mborzecki: this is why I use new files, things slowly move to clang-format
[11:16] <mborzecki> zyga: actually git blame -w looks quite good locally, but github blame is all bs now and there's no way to switch it
[11:17] <zyga> mborzecki: more reasons to refactor snap-confine tree :)
[11:18] <zyga> mborzecki: sent review on selinux branch
[11:23] <mborzecki> zyga: not sure how at address your question here https://github.com/snapcore/snapd/pull/6329#discussion_r264181494
[11:23] <mup> PR #6329: cmd/snap-confine, packaging: support SELinux <SELinux> <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6329>
[11:30] <pedronis> zyga: is that correct in the typical case we expect the /usr hierarchy to come from the base snap?
[11:38] <mvo> pedronis: I am experimenting a bit more with remodel right now, one thing I noticed is that refresh of the kernel track will now trigger a re-refresh check which is not compatible with our requirement that we can't have things talking to the network in the install phase of the remodel. thats seems to be a bit of a tricky one
[11:48] <mborzecki> zyga: updated #6329
[11:48] <mup> PR #6329: cmd/snap-confine, packaging: support SELinux <SELinux> <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6329>
[11:54] <zyga> mborzecki: thanks!
[11:55] <zyga> pedronis: yes
[12:17] <zyga> mborzecki: https://github.com/snapcore/snapd/pull/6329#discussion_r264181494
[12:17] <mup> PR #6329: cmd/snap-confine, packaging: support SELinux <SELinux> <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6329>
[12:18] <zyga> ah
[12:18] <zyga> odd
[12:18] <zyga> github doesn't  show the replies in some  views
[12:18] <pedronis> mvo: that's probably easiest to deal with a flag for now
[12:18] <zyga> mborzecki: approved
[12:19] <mborzecki> zyga: thanks!
[12:23] <mup> PR snapd#6579 opened: cmd/snap-confine: make sc_args helpers const-correct <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6579>
[12:23] <zyga> mborzecki, mvo: quick one please ^
[12:24] <mvo> pedronis: ok
[12:25] <pedronis> mvo: we probably want canarying for (automatic) remodeling at some point but is quite premature to have that bit there
[12:25] <pedronis> we need to decide what it really means
[12:27] <mvo> pedronis: yeah, a flag for now is fine for me
[13:01] <mborzecki> off to pick up the kids
[13:08] <pedronis> Chipaca: could you review #6568 when you have a bit of time?
[13:08] <mup> PR #6568: overlord/snapstate: fix restoring of "old-current" revision config in undoLinkSnap <Created by stolowski> <https://github.com/snapcore/snapd/pull/6568>
[13:10] <zyga> mvo: https://github.com/snapcore/snapd/pull/6575 needs a 2nd review now
[13:10] <mup> PR #6575: cmd/snap-confine: pass sc_invocation instead of numerous args around <Created by zyga> <https://github.com/snapcore/snapd/pull/6575>
[13:14] <pedronis> mvo: probably want to look at #6574 a 2nd time
[13:14] <mup> PR #6574: cmd/snap-confine: track per-app and per-hook processes <Created by zyga> <https://github.com/snapcore/snapd/pull/6574>
[13:16] <mup> PR snapd#6580 opened: cmd/snap-confine: drop unused dependency on libseccomp <Created by zyga> <https://github.com/snapcore/snapd/pull/6580>
[13:17] <zyga> mborzecki: ^ that one might be interesting for you
[13:20] <zyga> mvo: quick trivial, 2nd review: https://github.com/snapcore/snapd/pull/6579
[13:20] <mup> PR #6579: cmd/snap-confine: make sc_args helpers const-correct <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6579>
[13:25] <mup> PR snapd#6581 opened: daemon: move a struct def that was between an other struct and its methods <Simple 😃> <Created by pedronis> <https://github.com/snapcore/snapd/pull/6581>
[13:26] <pedronis> trivial PR ^
[13:32] <Chipaca> hehe
[13:32] <Chipaca> pedronis: +1 just from the description
[13:33] <Chipaca> pedronis: of course github thinks you moved a func def, not a struct def
[13:34] <pedronis> Chipaca: no,  it's a func,  is just that my brain conceptuall thought it's a struct
[13:34] <pedronis> (because it's kind of response)
[13:35] <Chipaca> daemon/ does kinda blur the lines
[13:35] <Chipaca> there are things in there that are funcs that have methods on 'em
[13:36] <pedronis> Chipaca: I fixed the descr/commit now
[13:36] <Chipaca> pedronis: about daemon, I was thinking of slowly (oportunistically) moving api* chunks out to daemon/api/, to separate the api from the daemon bits, and make refactoring the one without breaking the other easier
[13:37] <Chipaca> pedronis: do you think that might be a good approach for that?
[13:37] <pedronis> Chipaca: I don't know, I need to understand a bit more what it entails
[13:37] <pedronis> we don't seem close to that
[13:37] <pedronis> the tests use daemon often
[13:37] <pedronis> and they don't use exported things only either
[13:38] <Chipaca> yeah
[13:38] <Chipaca> blackboxing has only just started, in daemon
[13:38] <Chipaca> or is it whiteboxing
[13:38]  * Chipaca is colourblind
[13:38] <pedronis> Chipaca: more small scale, I'm about to split out one api_*[_test].go file for asserts
[13:38] <Chipaca> pedronis: nice
[13:38] <Chipaca> pedronis: and make it daemon_test?
[13:39] <pedronis> I don't know is that possible?
[13:39] <pedronis> debug stuff isn't doing that for example
[13:39] <pedronis> Chipaca: sorry, to be clear, I'm not doing this to cleanup,  I want to add a feature (returning json stuff for asserts if asked)
[13:40] <pedronis> but I might as well do that first
[13:46] <Chipaca> pedronis: I've been moving to _test opportunistically :-)
[13:47] <Chipaca> pedronis: I need to do something about apiBaseSuite to be able to move to the package
[13:47] <Chipaca> pedronis: it'll come in time
[13:48] <Chipaca> might be as simple as adding an exported type alias for it
[13:48] <Chipaca> but, too fiddly to do as an on-the-fly refactor imho
[13:56] <pedronis> pstolowski: seems 6568 can be landed
[14:01] <Chipaca> mvo: *cough*
[14:01] <Chipaca> ah there you are
[14:02] <mup> PR snapd#6568 closed: overlord/snapstate: fix restoring of "old-current" revision config in undoLinkSnap <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6568>
[14:32] <mborzecki> zyga: is there a request to openSUSE:Factory with snapd already open?
[14:33] <zyga> mborzecki: no, it was  automatically closed
[14:33] <zyga> we need to fix our "badness" score by allowing setuid root and polkit policy first
[14:33] <mborzecki> hmm
[14:33] <zyga> that's why I opened those two bugs about that
[14:34] <mborzecki> zyga: do you post links to the bugs?
[14:35] <zyga> https://bugzilla.suse.com/show_bug.cgi?id=1127366 and https://bugzilla.suse.com/show_bug.cgi?id=1127368
[14:35] <mborzecki> zyga: thanks!
[14:42]  * zyga goes for lunch
[14:59] <mup> PR snapd#6581 closed: daemon: move a function that was between an other struct and its methods <Simple 😃> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/6581>
[15:14] <ogra> Chipaca, mvo, i'm booting my first core18 image (beaglebone) here and the first boot looks pretty odd doing interactive snap animations (mounting/installing with the spinner and such) on the serial console
[15:14] <ogra> is that intentional ?
[15:16] <pedronis> ogra: that sounds odd
[15:16] <ogra> i mean ... it looks fancy ... but a bit out of place between all the system startup messages
[15:17] <ogra> :)
[15:18] <alan_g> cachio, are you the guy to refresh the rawhide image? https://github.com/MirServer/mir/pull/749#pullrequestreview-212741294
[15:19] <mup> PR MirServer/mir#749: release/1.1 <Created by AlanGriffiths> <https://github.com/MirServer/mir/pull/749>
[15:20] <mvo> ogra: its intentinal but if its too odd its easy to remove
[15:21] <mvo> ogra: it was added mostly to show that things are happing but if its too out-of-place its easy to change
[15:22] <pedronis> mvo: I wasn't aware of this
[15:22] <ogra> mvo, well, it looks unusual
[15:22] <mborzecki> zyga: https://forum.snapcraft.io/t/plans-for-sharing-a-gl-lib/10298/4
[15:22] <ogra> but i'm not sure if i actually find it too odd after overcoming the first shock ;)
[15:25] <mvo> ogra, pedronis http://paste.ubuntu.com/p/Sjm4j6jXK5/ in core18 is all that is needed to remove the progress
[15:26] <pedronis> ah
[15:26] <pedronis> I see
[15:26] <mup> PR core#38 closed: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>
[15:26] <mup> PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>
[15:26] <ogra> mvo, after letting it sink in i think it is fine ... unless you have a slow enough board that they kick in during console-conf
[15:27] <mup> PR core#38 opened: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38>
[15:27] <mup> PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83>
[15:27] <ogra> ppisati, FYI
[15:27] <ogra> ogra@localhost:~$ uname -a
[15:27] <ogra> Linux localhost 4.15.0-46-generic #49-Ubuntu SMP Wed Feb 6 09:34:18 UTC 2019 armv7l armv7l armv7l GNU/Linux
[15:27] <ogra> ogra@localhost:~$ snap list |grep kernel
[15:27] <ogra> pc-kernel  4.15.0-46.49            194   18/edge   canonical*  kernel
[15:27] <mvo> ogra: console-conf will wait for those
[15:27] <mup> PR #49: allow (optional) snappy update $pkgname <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/49>
[15:27] <ogra> mvo, ah, is that new in core18 ? it didnt in core16
[15:27] <mvo> ogra: correct
[15:27] <ogra> awesome
[15:30] <zyga> re
[15:31] <zyga> mborzecki: ack
[15:33] <mup> PR snapd#6579 closed: cmd/snap-confine: make sc_args helpers const-correct <Simple 😃> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6579>
[15:33] <zyga> thanks!
[15:34] <mup> PR snapd#6574 closed: cmd/snap-confine: track per-app and per-hook processes <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6574>
[15:40]  * cachio lunch
[15:43] <ogra> ogra@localhost:~$ sudo hostnamectl set-hostname beaglebone
[15:43] <ogra> Could not set property: Failed to set static hostname: Read-only file system
[15:43] <ogra> ogra@localhost:~$ ls -l /etc/writable/
[15:43] <ogra> total 0
[15:43] <ogra> ogra@localhost:~$ ls -l /etc/hostname
[15:43] <ogra> lrwxrwxrwx 1 root root 17 Mar  8 10:46 /etc/hostname -> writable/hostname
[15:43] <ogra> mvo, ^^^
[15:44] <zyga> mborzecki: replied on that thread
[15:46] <zyga> mvo: if you have some time https://github.com/snapcore/snapd/pull/6575 would be enough for me to propose the final bit of the core16 fixup
[15:46] <mup> PR #6575: cmd/snap-confine: pass sc_invocation instead of numerous args around <Created by zyga> <https://github.com/snapcore/snapd/pull/6575>
[15:46] <zyga> mborzecki: can you please look at https://github.com/snapcore/snapd/pull/6580?
[15:46] <mup> PR #6580: cmd/snap-confine: drop unused dependency on libseccomp <Created by zyga> <https://github.com/snapcore/snapd/pull/6580>
[15:48] <ogra> $ sudo timedatectl set-timezone Europe/Berlin
[15:48] <ogra> Failed to set time zone: Failed to set time zone: Read-only file system
[15:48] <ogra> mvo, same for timezone :(
[15:49] <mvo> ogra: *cough*
[15:49] <zyga> ogra: hey, I got a ping about the pi3 gadget snap
[15:49] <zyga> ogra: are builds working there?
[15:50] <zyga> ogra: after fixing the spi interface name supposedly there are no more builds in edge
[15:50] <ogra> zyga, no idea, thats foundation nowadays
[15:50] <mvo> ogra: https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1778936
[15:50] <zyga> ogra: and in addition, someone should plan to release that
[15:50] <mup> Bug #1778936: please re-add Support-system-image-read-only-etc.patch <patch> <verification-needed> <verification-needed-bionic> <systemd (Ubuntu):Fix Released> <systemd (Ubuntu Bionic):Fix Committed> <systemd (Ubuntu Cosmic):Fix Released> <https://launchpad.net/bugs/1778936>
[15:50] <ogra> zyga, same ... also foundations
[15:50] <zyga> ogra: who should I ping?
[15:50] <mvo> ogra: it tells you something about how well our SRUs are working :/
[15:50] <mvo> ogra: note the date of the patch
[15:50] <ogra> zyga, sil2100 is the one i always ping (priobably wrongly, who knows *g* )
[15:51] <zyga> sil2100: ^ pi3 gadget snap
[15:51]  * zyga hugs sil2100 
[15:51] <zyga> thank you!
[15:51] <sil2100> On my TODO list, yes o/
[15:51] <ogra> mvo, well, not a year yet ... let it ripen a bit more :P
[15:51] <mvo> ogra: yeah! I'm not bitter, no no
[15:52] <sil2100> DMB meeting now...
[15:54]  * zyga wishes for a snap of https://github.com/sharkdp/hexyl
[15:55]  * ogra points zyga to https://forum.snapcraft.io/t/snap-wishlist-suggestions-wanted/567
[15:56] <zyga> wow that's a LONG thread
[15:56] <ogra> our longest :)
[15:56] <ogra> i blame popey
[15:57] <zyga> we should open a thread "brexit is good for the economy" to beat that ;)
[15:57] <ogra> haha
[15:58] <ogra> ondra, did you ever install avahi alongside lxd on a core image ? ... bad things happen :)
[15:58] <ondra> ogra I think I did
[15:58] <ondra> but tell me more
[15:59] <ogra> well, when lxd brings up its lxdbrX devices somehow avahi considers it needs to attach a -2 to the MDNS hostname
[16:00] <ogra> there seems to be some race somewhere between the two
[16:00] <ogra> and avahi restarts avahid with a $hostname-2.local entry
[16:06] <mup> PR snapd#6583 opened: cmd/snap-confine: move ubuntu-core fallback checks <Created by zyga> <https://github.com/snapcore/snapd/pull/6583>
[16:07] <zyga> mvo, pedronis: ^ that's the last of the fixes needed for core16
[16:07] <zyga> (to allow core16  fallback to core)
[16:08] <zyga> I need to EOD now
[16:08] <zyga> I can check the status of stuff and merge things in the evening
[16:11] <mup> PR snapd#6584 opened: spread.yaml: bump delta referece <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6584>
[16:27] <cachio> mvo, I see errors like this one during the execution https://paste.ubuntu.com/p/WWJR8dhB7Z/
[16:29] <cachio> mvo, it breaks all the tests restore
[16:35] <mvo> cachio: hm, hm, this one looks like no snapd is running and system key changed that the same time - if you have access to the system you can probably check if snapd is running
[16:35] <cachio> mvo, cgecjubg
[16:35] <cachio> mvo, is not running
[16:36] <cachio> https://paste.ubuntu.com/p/B9sy66TxMr/
[16:36] <cachio> mvo, this could help
[16:36] <mvo> yeah
[16:44] <pedronis> Chipaca: shouldn't the export_snapshots_test.go etc be called export_api_snapshots_test.go ?
[16:44] <Chipaca> pedronis: yeah, probably yes
[16:45] <Chipaca> 'twas the first split one so i might've gotten the pattern wrong ¯\_(ツ)_/¯
[16:45] <pedronis> snap_file is the same
[16:48] <mup> PR snapcraft#2497 opened: Improved error message for specific cases (type error and bad length) <Created by facundobatista> <https://github.com/snapcore/snapcraft/pull/2497>
[16:54] <mup> PR snapd#6585 opened: tests: add undo test with hanging stop command <Created by mvo5> <https://github.com/snapcore/snapd/pull/6585>
[16:55] <cachio> mvo, could you identify the problem with that log, do you need anything else?
[16:56] <cachio> mvo, otherwise I'll kill that vms
[16:56] <zyga> (re from a coffee shop)
[16:56] <zyga> pedronis: replied on https://github.com/snapcore/snapd/pull/6583#discussion_r264330080
[16:56] <mup> PR #6583: cmd/snap-confine: move ubuntu-core fallback checks <Created by zyga> <https://github.com/snapcore/snapd/pull/6583>
[16:57] <pedronis> zyga: I don't understand the reply
[16:57] <mvo> cachio: I know what I need to know, we need to make sure sandp runs or the system-key is updated when running the rsync code
[16:58] <cachio> mvo, nice
[16:58] <cachio> mvo, that made fail some runs today testing 2.38
[16:59] <cachio> thanks
[17:00] <zyga> pedronis: oh, sorry, I wanted to say that perhaps the desire to swap the order of the functions to achieve normal mode on core16 bases is correcet but leads to the correct behavior only happening if the fallback logic is  triggered. If you don't have any  fallback at all  (core16 is installed) then then normal mode should be enabled as well, currently is would not. Therefore we need to ensure *that* happens regardless of the
[17:00] <zyga> fallback.
[17:00] <pedronis> ?
[17:01] <pedronis> maybe we should chat tomorrow
[17:01] <zyga> pedronis: https://github.com/snapcore/snapd/commit/44e59fbd243b7f64d23a47a87e40c0c977274bdf#diff-0c384f3cd817f18339705204e8e7b788R304 should not depend on https://github.com/snapcore/snapd/commit/44e59fbd243b7f64d23a47a87e40c0c977274bdf#diff-0c384f3cd817f18339705204e8e7b788R299
[17:01] <zyga> pedronis: sure
[17:02] <pedronis> zyga: there is only one case where we want no pivot, which is on core 16 if the base is core
[17:03] <mvo> cachio: let me know if you need help with the fix but I need to run now
[17:03] <zyga> pedronis: yes, I agree
[17:03] <zyga> pedronis: note, perhaps I was assuming this implicitly: this patch doesn't implement core16 specific behaviour - it only moves the existing ubuntu-core / core transition logic.
[17:04] <pedronis> I know
[17:04] <zyga> I also added a comment to the thread about this
[17:04] <pedronis> but it does extra things that seems uneeded
[17:04] <zyga> which things?
[17:04] <pedronis> (everything being the same, less code is better)
[17:08] <zyga> pedronis: let's chat tomorrow then, perhaps mvo can merge https://github.com/snapcore/snapd/pull/6575 so that the final diff is short
[17:08] <mup> PR #6575: cmd/snap-confine: pass sc_invocation instead of numerous args around <Created by zyga> <https://github.com/snapcore/snapd/pull/6575>
[17:08] <zyga> https://github.com/snapcore/snapd/pull/6584 is green, ok to merge?
[17:08] <mup> PR #6584: spread.yaml: bump delta reference <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6584>
[17:09] <zyga> I will squash and fix the typo in the history
[17:15] <zyga> nobody to complain so I'll just do it
[17:15] <mup> PR snapd#6584 closed: spread.yaml: bump delta reference <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6584>
[17:26] <mup> PR snapd#6586 opened: daemon: extract assertions api endpoint implementation into api_asserts.go <Created by pedronis> <https://github.com/snapcore/snapd/pull/6586>
[17:27] <pedronis> Chipaca: ^
[17:28] <Chipaca> pedronis: nice :-)
[17:28] <Chipaca> pedronis: I've been unable to make a table / grid thing for epochs
[17:28] <Chipaca> pedronis: best i could do was a flowchart
[17:30] <Chipaca> pedronis: https://snapforum.s3.amazonaws.com/original/2X/6/6fb9858b52acd50165b31a0ba514467790f996e7.png
[17:31] <pedronis> Chipaca: per user is always going to be tricky right? because home dirs can be not around
[17:31] <Chipaca> pedronis: yarp
[17:31] <Chipaca> pedronis: should I make that distinction?
[17:32] <Chipaca> "this will probably never work" vs "this might work in the future"?
[17:32] <pedronis> Chipaca: I think we should make the two other work at some point
[17:32] <pedronis> Chipaca: we can chat on that a bit more tomorrow
[17:32] <Chipaca> I guess if I make the distinction from right now, when we do it all we need to do is add a "from rNNNN"
[17:37] <mup> PR snapd#6587 opened: interfaces/apparmor: factor out test boilerplate <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6587>
[17:44] <Chipaca> pedronis: https://snapforum.s3.amazonaws.com/original/2X/5/5561c2c011ecf21bc0992ec407c18cf550d2ed63.png fwiw
[17:45] <pedronis> Chipaca: thx
[17:51] <pedronis> Chipaca: I suppose there will text to clarify what we mean help with migration?
[17:52] <Chipaca> pedronis: nah, I'll just post the flowchart (in graphviz language) as the documentation
[17:52] <Chipaca> :-D
[17:52] <pedronis> :)
[17:52] <Chipaca> pedronis: (yes, I doubt I'll get it done before EOD tho)
[17:52] <pedronis> np
[18:03] <zyga> Chipaca: I remember using sphinx extension that handled that, that was neat :)
[18:03] <Chipaca> zyga: https://github.com/discourse/discourse-graphviz
[18:03]  * Chipaca runs
[18:04] <Chipaca> anyway, I need to make dinner before I am dinner
[18:04]  * Chipaca has full-on pacman-level teenagers
[18:06] <zyga> pedronis: FYI, https://github.com/snapcore/pi3-gadget/pull/22 is interesting
[18:07] <mup> PR pi3-gadget#22: Adding empty configure hook to enable configuration for gadget <Created by kubiko> <https://github.com/snapcore/pi3-gadget/pull/22>
[18:07] <zyga> there are number of similar PRs across gadget snaps
[18:07] <zyga> CC ondra (thank you)
[18:09] <zyga> Chipaca: FYI: I added the special case of encrypted home while user is logged out to https://forum.snapcraft.io/t/limitations-in-snapd/9718
[18:15] <pedronis> zyga: yes, I have pinged about those
[18:15] <pedronis> s/I have/I was/
[18:15] <pedronis> zyga: didn't you EOD a while ago, I misread?
[18:15] <zyga> yeah
[18:16] <zyga> I'm in starbucks waiting for $wife
[18:16] <zyga> (and she is actually coming now, ttyl)
[18:39] <mup> PR snapd#6586 closed: daemon: extract assertions api endpoint implementation into api_asserts.go <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/6586>
[19:40] <mup> PR snapcraft#2498 opened: python plugin: graceful ret when no packages set <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2498>
[19:56] <Chipaca> zyga: tweaked it a little
[20:31] <mup> PR snapcraft#2499 opened: many: support for "base: core" in snapcraft.yaml <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2499>
[21:32] <mup> PR snapd#6582 closed: daemon, snap: screenshots _only_ shows the deprecation notice, from 2.39 <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/6582>