mup | PR snapd#8304 opened: usersession/userd: add zoommtg url support <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8304> | 01:37 |
---|---|---|
mborzecki | morning | 06:41 |
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:06 |
zyga | yawn | 07:13 |
zyga | I need a break today | 07:31 |
zyga | I'll be back later | 07:31 |
zyga | I just need to rest | 07:31 |
mborzecki | zyga: fwiw it's officially spring now | 07:34 |
mborzecki | hmm broke snapd somehow | 07:57 |
zyga | oh spring | 08:00 |
zyga | too bad we cannot leave home | 08:01 |
zyga | I need to burn overtime | 08:02 |
pstolowski | mornings | 08:05 |
mvo | hey pstolowski | 08:05 |
mvo | and good morning mborzecki | 08:05 |
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:08 |
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> | 08:31 |
pstolowski | zyga: do you know if the focal can be safely updated now? | 09:05 |
pedronis | pstolowski: snap-preseed -reset in actually in beta now I think | 09:42 |
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:44 |
pstolowski | pedronis: i see, ok, i'll send an update, core from beta + ppa for snap-preseed | 09:46 |
mvo | pedronis, pstolowski it's in 2.44 which is in focal right now if that helps | 09:51 |
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:53 |
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 | 09:59 |
zyga | and upgrade libcrypt | 10:00 |
zyga | then it works | 10:00 |
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:03 |
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:12 |
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:19 |
mborzecki | time to tweak do the tweaks in 8298 | 10:20 |
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:22 |
mvo | mborzecki: thank you | 10:24 |
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:27 |
axino | hey bdx have you seen https://github.com/snapcrafters/sentry/issues/29 ? | 10:32 |
mvo | pedronis: looking at this now | 10:32 |
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:48 |
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:53 |
zyga | I have a moment while lucy is sleeping | 10:54 |
zyga | shall I just merge it? | 10:54 |
=== pedronis_ is now known as pedronis | ||
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:57 |
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> | 10:58 |
pedronis | mvo: about cloud-init and u-i, are you thinking of making --cloud-init work somehow for 20, at least for dangerous? | 11:06 |
pedronis | I'm not quite sure what it does for 16/18 | 11:07 |
ackk | hi, when an automated snap refresh fails, does hook stdout/stderr get logged somewhere? | 11:08 |
mup | PR snapd#8306 opened: snap-bootstrap: add partition creation helper <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8306> | 11:20 |
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:36 |
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:40 |
mvo | pedronis: uh, ok. is someone on it already or should I do a quick PR? | 11:48 |
ackk | axino, snaps can only access shm files under /dev/shm/$SNAP_NAME.* | 11:50 |
ackk | axino, snapcraft-preload can help there | 11:50 |
ackk | axino, https://github.com/sergiusens/snapcraft-preload | 11:51 |
mup | PR snapd#8307 opened: snap: run gofmt -s <Simple 😃> <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8307> | 11:56 |
pedronis | mvo: thanks, I was having lunch | 12:01 |
mvo | pedronis: me too | 12:02 |
jdstrand | cjwatson: ah, thanks for confirming! hey mwhudson, we weren't too far off :) | 12:06 |
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> | 12:19 |
=== Facu is now known as facubatista | ||
ijohnson | hello folks | 13:16 |
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:17 |
pedronis | ijohnson: hi | 13:26 |
ijohnson | hey pedronis | 13:32 |
ijohnson | you were up pretty late last night :-) | 13:32 |
pedronis | indeed I was | 13:32 |
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:33 |
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:34 |
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:35 |
pedronis | ijohnson: I thought you needed help for the syslog one, whatever order works best for you | 13:36 |
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:37 |
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:38 |
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:39 |
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:40 |
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> | 13:44 |
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:00 |
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 :-/ | 14:57 |
pedronis | ijohnson: worse case we can mix my full fix and logging | 15:05 |
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:07 |
pedronis | ijohnson: https://github.com/pedronis/snappy/tree/randutil-syslog-fixed-sim-2.44 | 15:30 |
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 | 15:38 |
pedronis | pstolowski: fyi #8277 is now green and I have applied your comments | 16:10 |
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:11 |
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:29 |
=== ijohnson is now known as ijohnson|lunch | ||
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:47 |
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> | 16:55 |
=== ijohnson|lunch is now known as ijohnson | ||
abeato | ijohnson, hey, iptables is not included in the core20 snap, is that a bug? | 17:17 |
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:18 |
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:19 |
mup | Issue core20#27 opened: iptables is missing <Created by alfonsosanchezbeato> <https://github.com/snapcore/core20/issue/27> | 17:24 |
xnox | hm https://github.com/snapcore/core20/issue/27 is 404 for me?! | 17:30 |
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:31 |
ijohnson | xnox: I've mentioned it before to niemeyer | 17:33 |
niemeyer | We can.. I've been working on mup in the last couple of weeks.. that will certainly be addressed | 17:34 |
niemeyer | I'm replacing the whole database backend.. that's why it's taking a bit.. apologies for that | 17:35 |
mup | PR snapd#8310 opened: randutil: ask for less entrop during boot <âš Critical> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8310> | 18:06 |
mup | PR snapcraft#2989 opened: tests: only run catkin based snap on 16.04 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2989> | 18:20 |
cachio | mvo, hey, is cloud init PR already merged? | 19:39 |
cachio | mvo, or it is #8299 ? | 19:39 |
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:40 |
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:43 |
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:44 |
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:45 |
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 | 19:46 |
mvo | cachio: unfortunately not yet, it pending | 20:08 |
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:11 |
mvo | pedronis: \o/ | 20:13 |
mvo | pedronis: looking | 20:13 |
mvo | pedronis: added a small question, looks very good | 20:18 |
pedronis | mvo: I answered | 20:19 |
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:20 |
mvo | pedronis: aha, of course, sorry, misread the code. makes perfect sense | 20:21 |
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:22 |
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:38 |
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:39 |
cachio | pedronis, I mean, using cloudinit | 20:40 |
mvo | pedronis: thanks, enjoy your weeknd | 20:40 |
pedronis | mvo: I'm still around, waiting for the other results, but I don't think I can reviews right now | 20:42 |
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> | 20:53 |
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> | 23:27 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!