[04:20] <mup> Bug #1674847 changed: produce package lists for Ubuntu Core versions on the web <Snappy:Expired> <https://launchpad.net/bugs/1674847>
[06:32] <mborzecki> morning
[07:07] <mvo> hey mborzecki !
[07:07] <mvo> mborzecki: good morning, how are you?
[07:07] <mborzecki> mvo: hey
[07:25] <zyga> good morning guys
[07:25] <zyga> mvo: last night I realized I have holidays to burn
[07:25] <mborzecki> zyga: hey hey
[07:25] <mborzecki> zyga: you can carry over up to 5 days iirc
[07:25] <zyga> mvo: I was thinking about EOWking and just soft-working on few loose ends
[07:25] <zyga> mborzecki: IIRC I have 12-15 ish
[07:25] <mborzecki> wow
[07:28] <mvo> hey zyga
[07:29] <mvo> zyga: uh, GO
[07:29] <mvo> zyga: like seriously, that is a lot
[07:32]  * zyga files for a week off
[07:37] <mborzecki> mvo: we need a trello card for zyga to remind him of taking vacation
[07:39] <mvo> mborzecki: haha - indeed!
[07:45] <zyga> :D
[07:50] <mup> PR snapd#7909 opened: snap: improve squashfs.ReadFile() error <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7909>
[08:02] <pedronis> morning, mvo: let me know when/if you want to chat about #7908
[08:03] <mup> PR #7908: [RFC] boot,devicestate: update modenv and extract kernel <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/7908>
[08:03] <mvo> pedronis: not right now, debugging a issue in the initramfs right now :(
[08:04] <pstolowski> morning
[08:05] <mvo> hey pstolowski
[08:06] <mborzecki> pstolowski: hey
[08:06] <pedronis> mvo: ok, I'll start looking at its bits in handlers_install.go
[08:07] <mvo> pedronis: thank you!
[08:07] <mvo> pedronis: yeah, let's sync, I think the install steps (the minimal ones) are not that much work
[09:02]  * zyga goes to check out alternative school for Janek
[09:16] <mup> PR pc-amd64-gadget#27 opened: grub.cfg-recovery: add run mode chainloading <Created by mvo5> <https://github.com/snapcore/pc-amd64-gadget/pull/27>
[09:31] <pedronis> mvo: I pushed some improvements to 7908, not quite fully there, we need to chat a bit about /boot layout
[09:34] <mvo> pedronis: ok, should we have a HO in some minutes?
[09:35] <pedronis> mvo: yes
[09:36] <mvo> pedronis: ok, I am ready in 3min or so
[10:10] <pedronis> mborzecki: hi, can you do a review of https://github.com/snapcore/snapd/pull/7905 when you have a moment , then mvo can do one as well
[10:10] <mup> PR #7905: cmd/snap-bootstrap: actually parse snapd_recovery_system label <UC20> <Created by xnox> <https://github.com/snapcore/snapd/pull/7905>
[10:11] <mborzecki> pedronis: sure, will do
[10:11] <pedronis> thx
[10:22] <mup> PR pc-amd64-gadget#28 opened: grub.cfg-normal: set snapd_recovery_mode= run in run mode <Created by mvo5> <https://github.com/snapcore/pc-amd64-gadget/pull/28>
[10:27] <pedronis> pstolowski: hi, do we need to chat about services?
[10:32] <pstolowski> pedronis: hi, yes, that would be great
[10:33]  * Chipaca steps back from the forum and goes for coffee
[10:39] <pedronis> pstolowski: now, in a couple of mins?
[10:39] <pstolowski> pedronis: in 10 ?
[10:39] <pedronis> ok
[10:49] <pstolowski> pedronis: i'm in the standup ho
[10:49] <pedronis> ok
[10:49] <pedronis> joining
[11:20] <mup> PR snapd#7910 opened: boot,image: add skeleton boot.makeBootable20RunMode <Created by mvo5> <https://github.com/snapcore/snapd/pull/7910>
[11:20] <mborzecki> meh, looks like we'll need to pass Device to wrappers
[11:20] <mborzecki> but probably as a later cleanup
[11:24] <mup> PR pc-amd64-gadget#27 closed: grub.cfg-recovery: add run mode chainloading <Created by mvo5> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/27>
[11:33] <mup> PR pc-amd64-gadget#28 closed: grub.cfg-normal: set snapd_recovery_mode= run in run mode <Created by mvo5> <Closed by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/28>
[11:48] <pedronis> mvo: Q in 7910
[11:49] <mvo> pedronis: looking
[11:50] <mborzecki> mvo: updated #7772
[11:50] <mup> PR #7772: wrappers: write and undo snapd services on core <Remodel 🚋> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7772>
[11:51] <mvo> pedronis: of course, I'm a muppet, fixing and adding a test
[11:51] <mvo> mborzecki: nice!
[11:58] <mup> PR snapd#7911 opened: Uc20 shutdown <Created by xnox> <https://github.com/snapcore/snapd/pull/7911>
[12:12] <cachio> mborzecki, hey
[12:13] <cachio> mborzecki, about lp:1856073
[12:13] <cachio> you think it is be manually cleaned during tests?
[12:15] <mborzecki> cachio: hi, yes, left it in the comment https://bugs.launchpad.net/snapd/+bug/1856073/comments/2
[12:15] <mup> Bug #1856073: Failed service state recorded by systemd even after snap removal <snapd:Confirmed> <https://launchpad.net/bugs/1856073>
[12:16] <cachio> mborzecki, yes, I saw that
[12:16] <cachio> I'll make the change in the tests in that case
[12:16] <mborzecki> ok
[12:17] <cachio> mborzecki, thakns
[12:17] <cachio> zyga, hey, could you please give a second round to #7877
[12:17] <mup> PR #7877: tests: avoid mask rsyslog service in case is not enabled on the system <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/7877>
[12:17] <cachio> thanks
[12:18] <cachio> mborzecki, another thing
[12:19] <cachio> mborzecki, I can reproduce this in arch https://paste.ubuntu.com/p/XCwW2VnpXG/
[12:19] <cachio> mborzecki, I think at some point you reported something similat
[12:19] <cachio> similar
[12:26] <pedronis> mvo: reviewed 7910 with some suggestions
[12:28] <mvo> pedronis: cool, thank you
[12:31] <mborzecki> cachio: no that was antoher test checking whether access to the socket is mediated by apparmor
[12:44] <pedronis> pstolowski: reviewed #7658
[12:44] <mup> PR core20#14 closed: run-snapd-from-snap: check for snapd.service existing too <Created by mvo5> <Merged by xnox> <https://github.com/snapcore/core20/pull/14>
[12:45] <mup> PR #7658: cmd/snap-preseed: add snap-preseed executable <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7658>
[12:45] <mup> PR snapd#7912 opened: boot: write modeenv when creating the run mode <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7912>
[12:45] <pstolowski> pedronis: ty!
[12:53]  * Chipaca gets lunch
[13:55] <ijohnson> morning folks
[14:22] <mup> PR snapd#7658 closed: cmd/snap-preseed: add snap-preseed executable <Preseeding 🍞> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7658>
[14:38] <cachio> mborzecki, hey
[14:38] <pstolowski> pedronis, degville i've sent you an email invitation to edit snap-preseed help doc
[14:38] <cachio> after many tests we left a systemd unit sys-devices-virtual-block-loop1.device
[14:38] <cachio> mborzecki, is it created by snapd?
[14:38] <cachio> any idea?
[14:41] <pedronis> pstolowski: thanks, I'll try to get to it today or tomorrow morning
[14:41] <mvo> pedronis: 7912 should be a simple one
[14:41] <mborzecki> cachio: it's probably a loopback device that's left behind, any chance you could run `losetup -l` there?
[14:41] <pedronis> mvo: ok, I need a short break frist though
[14:42] <mup> PR snapd#7910 closed: boot,image: add skeleton boot.makeBootable20RunMode <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7910>
[14:42] <pedronis> *first
[14:42] <cachio> mborzecki, yes, I'll add more info for next run
[14:43] <mvo> pedronis: yeah, no worries
[14:43] <mborzecki> cachio: it might be the case that libloop leaves one free device behind as an optimization so that it doesn't need to allocate it again
[14:48] <cachio> mborzecki, thanks, I added a new check for loop devices
[14:49] <cachio> and triggered a new run
[14:49] <cachio> it will be ready on 1 hour aprox
[14:49] <cachio> afer lunch
[14:49]  * cachio lunch
[14:54] <Chipaca> need to go to the boys school for a bit, will bbiab
[15:17] <mup> PR snapd#7852 closed: devicestate: call boot.MakeRunnable() in devicestate <UC20> <⛔ Blocked> <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/7852>
[15:39] <mborzecki> hmm MATCH with multiline input behaves somewhat weird
[15:40] <mborzecki> err MATCH -v actually
[15:41] <Chipaca> mborzecki: weird how?
[15:42] <mborzecki> Chipaca: https://paste.ubuntu.com/p/P3hyRbDmcy/
[15:44] <mborzecki> Chipaca: we actually have code in tests that does MATCH -v with multiline input
[15:47] <Chipaca> mborzecki: MATCH -v means "error if there are no lines that don't match" which is a little weird to think about so I wouldn't be surprised if it's used wrong
[15:48] <mborzecki> Chipaca: heh sooo https://paste.ubuntu.com/p/GD63G4S2sX/
[15:50] <mborzecki> hmm that lsmod, snap list, virsh checks are likely wrong
[15:50] <mborzecki> probably the same in cgroups test :/
[15:58] <mup> PR snapd#7913 opened: boot,bootloader: extract kernel in makeBootable20RunMode  <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7913>
[16:07] <mup> PR snapd#7914 opened: bootstrap: reduce runmode mounts from 5 to 2 steps <Created by xnox> <https://github.com/snapcore/snapd/pull/7914>
[16:11] <pedronis> mvo: does 7913 works or we think it works?
[16:11] <mvo> pedronis: I'm testing it now
[16:12] <mvo> pedronis: you can mark it blocked if you want
[16:12] <mvo> pedronis: but it's essentially what I had in mind and tried to describe
[16:12] <pedronis> mvo: yes, but I said it makes sense to land only if works
[16:12] <pedronis> also it needs more XXX to it :)
[16:13] <mvo> pedronis: heh :) yeah, I need to update the other bits from claudios first
[16:13] <mvo> pedronis: doing this now
[16:13] <pedronis> mvo: sounds like bootloader.Options.Recovery needs a different name
[16:13] <pedronis> but that's less of an issue
[16:14] <mvo> pedronis: yeah, its a bit too confusing, ideas welcome!
[16:14] <pedronis> I need to look again exactly how it is used
[16:15] <mvo> pedronis: \o/
[16:16] <pedronis> mvo: working?
[16:17] <pedronis> does it seed again?
[16:19] <mvo> pedronis: yeah, testing if it seeding
[16:19] <pedronis> what initramfs is it using?
[16:19] <mvo> pedronis: I mean, unless I miss something (entirely possible) with this PR and the other changes from claudio we should be seeding now
[16:19] <mvo> pedronis: except that snap-bootstrap in run-mode in initramfs is buggy
[16:20] <mvo> pedronis: but xnox is on this :)
[16:20] <pedronis> mvo: but what is putting the kernel and the base in the right place?
[16:21] <mvo> pedronis: maybe more is missing and I'm missing this - the kernel will be put in place byhttps://github.com/snapcore/snapd/pull/7913/files#diff-06592349bc3e8eaeecc7bc0cafcb3cacR471  and the snap-bootstrap should open the right base in run mode and make it available in /run/mnt/base
[16:21] <mup> PR #7913: boot,bootloader: extract kernel in makeBootable20RunMode  <UC20> <⛔ Blocked> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7913>
[16:21] <mvo> pedronis: we also have the seed
[16:22] <pedronis> I probably don't remember what snap-bootstrap run mode is doing then
[16:22] <pedronis> is probably wrong
[16:22] <pedronis> somehow
[16:23] <pedronis> mvo: to be clear, I'm talking about ubuntu-data/var/lib/snapd/snaps
[16:23] <xnox> mvo:  so for runmode I will do two things, reduce runmode steps from 5 to 2 => see my PR. And actually make it just step 1, as initrd will try to mount the "basic" things meaning in runmode, snap-bootstrap is done and dusted in one step.
[16:23] <mvo> pedronis: we setup the uc20 seed there and go through normal seeding iirc, no?
[16:23] <pedronis> mvo: no, something is confusing me
[16:23] <pedronis> or maybe is right for the first boot but not the other
[16:24] <pedronis> I need to stare at snap-bootstrap code
[16:24] <pedronis> I suppose
[16:25] <pedronis> let me look at it again
[16:28] <pedronis> mvo: snap-bootstrap expects the base and kernel to be in  filepath.Join(dataDir, "system-data", dirs.SnapBlobDir, modeEnv.Base)
[16:28] <pedronis> that's the bit I don't undersand
[16:28] <pedronis> when do we put them there
[16:28] <pedronis> ??
[16:30] <pedronis> extracting the kernel doesn't do that
[16:31] <mvo> pedronis: oh, right - now I see what you mean
[16:31] <pedronis> so I'm not sure how it works
[16:31] <pedronis> except magic :) ?
[16:31] <mvo> pedronis: well, we are not there yet :/
[16:31] <pedronis> you said it worked ?
[16:32] <pedronis> work != work
[16:33] <mvo> pedronis: sorry, I think I miswrote, I wanted to say that I'm trying to see if it works
[16:33] <mvo> pedronis: and that I need bits from cladio first and that it's a bit of work to get to the point
[16:33] <pedronis> mvo: heh, it's ok, I got very confused
[16:36] <pedronis> mvo: I gave some comments in 7913
[16:36] <mvo> pedronis: but it does sound like another wrench in the gears
[16:36] <mvo> pedronis: oh well! so much things to fix still
[16:36] <pedronis> mvo: well, it's easy to fix, we need to copy those snaps
[16:36] <pedronis> that we knew
[16:36] <mvo> pedronis: in makebootable
[16:36] <pedronis> yes
[16:36]  * mvo nods
[16:37] <pedronis> like the UC16 makes symlinkes, this one need to do copies
[16:37] <mvo> right
[16:37] <mvo> pedronis: sorry, brain is a bit fried - I added a todo for myself here
[16:41] <pedronis> mvo: anything I can help with?
[16:47] <mvo> pedronis: hm, maybe (if you haven't done already) point the extra xxx in 7913, other than that I think it's mostly working through stuff now
[16:47] <mvo> pedronis: I mean, I try to work through the stuff from claudio
[16:48] <pedronis> mvo: sorry, "point the extra" ? address the extra? add an extra ?
[16:55] <mvo> pedronis: sorry again! I mean, you mentioned earlier that 7913 will need a bunch of extra XXX beyond the ones I added there
[16:56] <mvo> pedronis: if you haven't done so already, pointing them out in the PR would be great
[17:01] <ijohnson> is ubuntu-image expected to build partitions from gadget.yaml for core20 pc gadget yet?
[17:02] <ijohnson> I just tried building a uc20 image with ubuntu-image with 20/edge snaps and what seems to be a correct dangerous model and I only got the ubuntu-seed partition, the ubuntu-boot partition seems to be missing
[17:09] <pedronis> ijohnson: that's expected
[17:09] <pedronis> the images have only seed
[17:09] <pedronis> boot is added at installation
[17:11] <ijohnson> pedronis: ah ok so for my purposes then I'll manually create the ubuntu-boot partition since it doesn't seem like trying to get install mode to work just to create that partition
[19:05]  * cachio afk 10 min 
[19:21] <mup> PR snapd#7915 opened: lkenv.go: adjust for new location of include file <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7915>
[20:03] <mup> PR snapd#7916 opened: interfaces/browser-support: add more product/vendor paths <Created by Erick555> <https://github.com/snapcore/snapd/pull/7916>