[01:37] <mup> PR snapd#8304 opened: usersession/userd: add zoommtg url support <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8304>
[06:41] <mborzecki> morning
[07:06] <zyga> hey :)
[07:06] <zyga> coffee
[07:06] <zyga> I'm a bit tired today
[07:06] <zyga> cannot stop working till NaN o clock
[07:13] <zyga> yawn
[07:31] <zyga> I need a break today
[07:31] <zyga> I'll be back later
[07:31] <zyga> I just need to rest
[07:34] <mborzecki> zyga: fwiw it's officially spring now
[07:57] <mborzecki> hmm broke snapd somehow
[08:00] <zyga> oh spring
[08:01] <zyga> too bad we cannot leave home
[08:02] <zyga> I need to burn overtime
[08:05] <pstolowski> mornings
[08:05] <mvo> hey pstolowski
[08:05] <mvo> and good morning mborzecki
[08:08] <mborzecki> pstolowski: mvo : hey!
[08:08] <zyga> hey mvo and pawel
[08:08] <pstolowski> o/
[08:08] <zyga> mvo: I filed for the day off, I just need to slow down and not work till 22
[08:08] <zyga> plus, it's spring as maciek told me
[08:31] <mup> PR snapd#8251 closed: overlord: remove unneeded overlord.MockPruneInterval() mocks <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8251>
[08:31] <mup> PR snapd#8302 closed: interfaces/greengrass-support: fix typo <Simple 😃> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8302>
[08:31] <mup> PR snapd#8303 closed: interfaces/greengrass-support: fix typo <Simple 😃> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8303>
[09:05] <pstolowski> zyga: do you know if the focal can be safely updated now?
[09:42] <pedronis> pstolowski: snap-preseed -reset in actually in beta now I think
[09:44] <pstolowski> pedronis: but they need the deb for snap-preseed command (or build it themselves)
[09:44] <pedronis> pstolowski: yes, just pointing out that is not just in edge
[09:46] <pstolowski> pedronis: i see, ok, i'll send an update, core from beta + ppa for snap-preseed
[09:51] <mvo> pedronis, pstolowski it's in 2.44 which is in focal right now if that helps
[09:53] <pedronis> I suppose it might not, but mostly to show some progress in releasing this
[09:53] <zyga> pstolowski: yes it can
[09:53] <pstolowski> mvo: that's nice, thanks, i suppose they need xenial though
[09:53] <zyga> I just fixed my laptop
[09:53] <pstolowski> (nonetheless i'll mention this)
[09:53] <pstolowski> zyga: ack, ty
[09:59] <zyga> pstolowski: yah, now back in graphical mode
[09:59] <zyga> if you need to fix but don't have a bootable helper
[09:59] <zyga> you can enable debug shell
[09:59] <zyga> set static IP address over eth0
[10:00] <zyga> and upgrade libcrypt
[10:00] <zyga> then it works
[10:03] <pstolowski> zyga: i just upgraded, didn't have to do anything (but i haven't updated for a while, had ~250 packages to upd)
[10:12] <zyga> pstolowski: it was only bad if you upgraded over that weekend when it was broken
[10:12] <pstolowski> zyga: yeah, i just avoided a bad package
[10:19] <mup> PR snapd#8305 opened: cmd/snap-recovery-chooser: add recovery chooser <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8305>
[10:19] <mborzecki> mvo: ^^
[10:20] <mborzecki> time to tweak do the tweaks in 8298
[10:22] <mup> PR snapd#8200 closed: [RFC] cmd/snap-chooser-ui-demo: a demo of recovery chooser UI <Skip spread> <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/8200>
[10:24] <mvo> mborzecki: thank you
[10:27] <pedronis> #8208 still needs a 2nd review
[10:27] <mup> PR #8208: boot_test: add many boot robustness tests for UC20 kernel MarkBootSuccessul and SetNextBoot <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8208>
[10:32] <axino> hey bdx have you seen https://github.com/snapcrafters/sentry/issues/29 ?
[10:32] <mvo> pedronis: looking at this now
[10:48] <mup> PR snapd#8208 closed: boot_test: add many boot robustness tests for UC20 kernel MarkBootSuccessul and SetNextBoot <Test Robustness> <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8208>
[10:53] <zyga> I'm technically off today but +1 on https://github.com/snapcore/snapd/pull/8242 - let's land it
[10:53] <mup> PR #8242: many: improve environment handling, fixing duplicate entries <Bug> <Needs Samuele review> <Squash-merge> <Created by zyga> <https://github.com/snapcore/snapd/pull/8242>
[10:54] <zyga> I have a moment while lucy is sleeping
[10:54] <zyga> shall I just merge it?
[10:57] <mvo> zyga: it has two +1 so should be fine
[10:57] <zyga> ok
[10:57] <zyga> squashing and merging
[10:57] <mvo> thanks!
[10:58] <zyga> I *love* fixing bugs
[10:58] <mup> PR snapd#8242 closed: many: improve environment handling, fixing duplicate entries <Bug> <Needs Samuele review> <Squash-merge> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8242>
[11:06] <pedronis> mvo: about cloud-init and u-i, are you thinking of making --cloud-init work somehow for 20, at least for dangerous?
[11:07] <pedronis> I'm not quite sure what it does for 16/18
[11:08] <ackk> hi, when an automated snap refresh fails, does hook stdout/stderr get logged somewhere?
[11:20] <mup> PR snapd#8306 opened: snap-bootstrap: add partition creation helper <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8306>
[11:36] <pedronis> mvo: master has formatting issues now, I think the env pr had still those test off so we didn't notice:  https://travis-ci.org/github/snapcore/snapd/jobs/664797858?utm_medium=notification&utm_source=github_status
[11:40] <axino> hi, does anyone have any idea if this is a snap bug or a snapd bug ? https://github.com/snapcrafters/sentry/issues/30
[11:48] <mvo> pedronis: uh, ok. is someone on it already or should I do a quick PR?
[11:50] <ackk> axino, snaps can only access shm files under /dev/shm/$SNAP_NAME.*
[11:50] <ackk> axino, snapcraft-preload can help there
[11:51] <ackk> axino, https://github.com/sergiusens/snapcraft-preload
[11:56] <mup> PR snapd#8307 opened: snap: run gofmt -s <Simple 😃> <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8307>
[12:01] <pedronis> mvo: thanks, I was having lunch
[12:02] <mvo> pedronis: me too
[12:06] <jdstrand> cjwatson: ah, thanks for confirming! hey mwhudson, we weren't too far off :)
[12:19] <mup> PR snapd#8307 closed: snap: run gofmt -s <Simple 😃> <Skip spread> <Created by mvo5> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/8307>
[13:16] <ijohnson> hello folks
[13:17] <mup> PR snapcraft#2987 opened: spread tests: set appropriate default base in snapcraft.yamls <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2987>
[13:26] <pedronis> ijohnson: hi
[13:32] <ijohnson> hey pedronis
[13:32] <ijohnson> you were up pretty late last night :-)
[13:32] <pedronis> indeed I was
[13:33] <ijohnson> any more thoughts on what else might be triggering snapd/snap things to run in early boot?
[13:33] <ijohnson> as I mentioned in the SU doc, I don't think it's necessarily the autoimport, because with the removal of that it still took a long time to boot, suggesting that there's something else that needs lots of entropy from 2.44 in early boot
[13:33] <pedronis> ijohnson: I think really early boot was probably "snap auto-import", after that I would say any snap service
[13:34] <pedronis> because of the implied snap run
[13:34] <pedronis> ijohnson: the syslog thing would tell us for sure
[13:34] <ijohnson> I see you have a branch in the SU doc, shall I try that one?
[13:35] <pedronis> ijohnson: yes, that's the real fix I would do for 2.44 , if it still works
[13:35] <pedronis> it might not, depending
[13:35] <ijohnson> should I do that before the syslog branch then?
[13:35] <ijohnson> in any case, probs won't get results til after SU, I have to re-reserve the system
[13:36] <pedronis> ijohnson: I thought you needed help for the syslog one, whatever order works best for you
[13:37] <ijohnson> well for the syslog branch if I add that with the removal of the autoimport, the system does eventually boot, so I think we should be able to get the data there, it just will take a while
[13:38] <pedronis> ijohnson: ah, yes combining would work well, we should be able to see what is the first requester
[13:38] <pedronis> knowing that otherwise it would be one of the auto-import
[13:39] <ijohnson> yes, but I also need to turn on persistent logs, because I noticed from the auto-import system journal logs I added to the SU doc that they didn't go all the way back
[13:39] <pedronis> ok
[13:40] <pedronis> ijohnson: anyway as I said in the SU doc, it's good we have likely a fix, but it would be also good to understand the full problem because other it might get back later
[13:40] <pedronis> ijohnson: also mvo is pursuing seeing if the kernel can help, because we control snapd, but other things might start wanting entropy early
[13:44] <mup> PR snapcraft#2988 opened: tests: add two more workers to the 18.04 systems <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2988>
[14:00] <mup> PR snapd#8308 opened: tests: umount partitions which are not umounted after remount gadget <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8308>
[14:57] <ijohnson> pedronis: still waiting for that syslog change + no autoimport to boot, been like 20 minutes now so it should be any minute now, but it also could be racy that the no autoimport change from yesterday booted :-/
[15:05] <pedronis> ijohnson: worse case we can mix my full fix and logging
[15:07] <ijohnson> yes
[15:07] <pedronis> ijohnson: I can propose something if you want
[15:07] <ijohnson> if you want to sure, but I think I'll give it some more time
[15:30] <pedronis> ijohnson: https://github.com/pedronis/snappy/tree/randutil-syslog-fixed-sim-2.44
[15:38] <ijohnson> pedronis: yeah the boot failed, I gave up on it, I'm gonna retry one more time
[15:38] <ijohnson> then I'll try your branch
[16:10] <pedronis> pstolowski: fyi #8277 is now green and I have applied your comments
[16:11] <mup> PR #8277: asserts,o/devicestate: support model specified alternative serial-authority <Created by pedronis> <https://github.com/snapcore/snapd/pull/8277>
[16:11] <pstolowski> pedronis: ok
[16:29] <ijohnson> it would be great to land #8308 asap, since a number of open PR's are hitting the issued fixed there
[16:29] <mup> PR #8308: tests: umount partitions which are not umounted after remount gadget <⚠ Critical> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8308>
[16:47] <mup> PR snapd#8308 closed: tests: umount partitions which are not umounted after remount gadget <⚠ Critical> <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8308>
[16:55] <mup> PR snapd#8309 opened: o/configcore: implement Apply method for early configuration of core <Created by stolowski> <https://github.com/snapcore/snapd/pull/8309>
[17:17] <abeato> ijohnson, hey, iptables is not included in the core20 snap, is that a bug?
[17:18] <abeato> ijohnson, firewall-control gives permissions to use it
[17:18] <ijohnson> abeato: yes sounds like a bug, perhaps xnox can fix that easily
[17:19] <ijohnson> core20 is also on github if you wanted to take a look you're self and file a MP
[17:19] <ijohnson> abeato: https://github.com/snapcore/core20
[17:19] <abeato> ijohnson, thanks, I'll create an issue there
[17:24] <mup> Issue core20#27 opened: iptables is missing <Created by alfonsosanchezbeato> <https://github.com/snapcore/core20/issue/27>
[17:30] <xnox> hm https://github.com/snapcore/core20/issue/27 is 404 for me?!
[17:31] <mup> Issue core20#27: iptables is missing <Created by alfonsosanchezbeato> <https://github.com/snapcore/core20/issue/27>
[17:31] <xnox> oh
[17:31] <xnox> can we fix mup?
[17:31] <xnox> it needs to be "issues" not "issue"
[17:33] <ijohnson> xnox: I've mentioned it before to niemeyer
[17:34] <niemeyer> We can.. I've been working on mup in the last couple of weeks.. that will certainly be addressed
[17:35] <niemeyer> I'm replacing the whole database backend.. that's why it's taking a bit.. apologies for that
[18:06] <mup> PR snapd#8310 opened: randutil: ask for less entrop during boot  <⚠ Critical> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8310>
[18:20] <mup> PR snapcraft#2989 opened: tests: only run catkin based snap on 16.04 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2989>
[19:39] <cachio> mvo, hey, is cloud init PR already merged?
[19:39] <cachio> mvo, or it is #8299 ?
[19:40] <mup> PR #8299: devicestate,sysconfig: support "cloud.cfg.d" in uc20 for grade: dangerous <Squash-merge> <UC20> <⛔ Blocked> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8299>
[19:43] <ijohnson> cachio: #8278 was merged, but that just disables cloud-init for uc20, 8299 as you see is the PR which supports cloud-init from the ubuntu-seed partition
[19:44] <mup> PR #8278: devicestate: disable cloud-init by default on uc20 <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8278>
[19:45] <cachio> ijohnson, any idea when it is going to be merged?
[19:45] <cachio> because the user assertion alternative is taking to much time
[19:45] <ijohnson> cachio: I think it is close, afaik just needs pedronis to approve the new dir
[19:46] <ijohnson> cachio: although maybe we need to wait for the cloud folks to approve the new dir location, not sure
[19:46] <cachio> ijohnson, ah, ok
[19:46] <ijohnson> i.e. rharper and company
[20:08] <mvo> cachio: unfortunately not yet, it pending
[20:11] <mup> PR snapd#8311 opened: randutil: don't consume kernel entropy at init, just mix more info to try to avoid fleet collisions <Squash-merge> <Created by pedronis> <https://github.com/snapcore/snapd/pull/8311>
[20:11] <pedronis> mvo: ijohnson: ^
[20:11] <ijohnson> thanks looking now
[20:13] <mvo> pedronis: \o/
[20:13] <mvo> pedronis: looking
[20:18] <mvo> pedronis: added a small question, looks very good
[20:19] <pedronis> mvo: I answered
[20:20] <mvo> pedronis: aha, so seed is cumulative? it does not reset the state of the crng (sorry if that is a stupid question)?
[20:20] <pedronis> mvo: the contrary, it resets it
[20:20] <pedronis> but we use a value derived from the old one to make the new one
[20:21] <mvo> pedronis: aha, of course, sorry, misread the code. makes perfect sense
[20:22] <pedronis> mvo: and to be clear I'm not doing all this in init because I don't want by chance to slow down snap run
[20:22] <pedronis> which shouldn't care
[20:22] <mvo> pedronis: +1
[20:38] <pedronis> mvo: cachio: I don't we should be blocked on the cloud people, they just need to be aware of the slightly different location, the other one was anyway different from 16/18
[20:38] <pedronis> *I don't think
[20:38] <pedronis> mvo: cachio: but I need to re-review it, which I will first thing Monday
[20:39] <cachio> sure
[20:39] <cachio> pedronis, is it any way to workaroud it while I am testing the new smoke suite executen in a nested uc20?
[20:40] <cachio> pedronis, I mean, using cloudinit
[20:40] <mvo> pedronis: thanks, enjoy your weeknd
[20:42] <pedronis> mvo: I'm still around, waiting for the other results, but I don't think I can reviews right now
[20:53] <mup> PR snapd#8310 closed: randutil: ask for less entropy during boot  <Squash-merge> <⚠ Critical> <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/8310>
[23:27] <mup> PR snapcraft#2989 closed: tests: only run catkin based snap on 16.04 <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2989>