/srv/irclogs.ubuntu.com/2019/12/16/#snappy.txt

mupBug #1674847 changed: produce package lists for Ubuntu Core versions on the web <Snappy:Expired> <https://launchpad.net/bugs/1674847>04:20
mborzeckimorning06:32
mvohey mborzecki !07:07
mvomborzecki: good morning, how are you?07:07
mborzeckimvo: hey07:07
zygagood morning guys07:25
zygamvo: last night I realized I have holidays to burn07:25
mborzeckizyga: hey hey07:25
mborzeckizyga: you can carry over up to 5 days iirc07:25
zygamvo: I was thinking about EOWking and just soft-working on few loose ends07:25
zygamborzecki: IIRC I have 12-15 ish07:25
mborzeckiwow07:25
mvohey zyga07:28
mvozyga: uh, GO07:29
mvozyga: like seriously, that is a lot07:29
* zyga files for a week off07:32
mborzeckimvo: we need a trello card for zyga to remind him of taking vacation07:37
mvomborzecki: haha - indeed!07:39
zyga:D07:45
mupPR snapd#7909 opened: snap: improve squashfs.ReadFile() error <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7909>07:50
pedronismorning, mvo: let me know when/if you want to chat about #790808:02
mupPR #7908: [RFC] boot,devicestate: update modenv and extract kernel <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/7908>08:03
mvopedronis: not right now, debugging a issue in the initramfs right now :(08:03
pstolowskimorning08:04
mvohey pstolowski08:05
mborzeckipstolowski: hey08:06
pedronismvo: ok, I'll start looking at its bits in handlers_install.go08:06
mvopedronis: thank you!08:07
mvopedronis: yeah, let's sync, I think the install steps (the minimal ones) are not that much work08:07
* zyga goes to check out alternative school for Janek09:02
mupPR 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:16
pedronismvo: I pushed some improvements to 7908, not quite fully there, we need to chat a bit about /boot layout09:31
mvopedronis: ok, should we have a HO in some minutes?09:34
pedronismvo: yes09:35
mvopedronis: ok, I am ready in 3min or so09:36
pedronismborzecki: 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 well10:10
mupPR #7905: cmd/snap-bootstrap: actually parse snapd_recovery_system label <UC20> <Created by xnox> <https://github.com/snapcore/snapd/pull/7905>10:10
mborzeckipedronis: sure, will do10:11
pedronisthx10:11
mupPR 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:22
pedronispstolowski: hi, do we need to chat about services?10:27
pstolowskipedronis: hi, yes, that would be great10:32
* Chipaca steps back from the forum and goes for coffee10:33
pedronispstolowski: now, in a couple of mins?10:39
pstolowskipedronis: in 10 ?10:39
pedronisok10:39
pstolowskipedronis: i'm in the standup ho10:49
pedronisok10:49
pedronisjoining10:49
mupPR snapd#7910 opened: boot,image: add skeleton boot.makeBootable20RunMode <Created by mvo5> <https://github.com/snapcore/snapd/pull/7910>11:20
mborzeckimeh, looks like we'll need to pass Device to wrappers11:20
mborzeckibut probably as a later cleanup11:20
mupPR 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:24
mupPR 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:33
pedronismvo: Q in 791011:48
mvopedronis: looking11:49
mborzeckimvo: updated #777211:50
mupPR #7772: wrappers: write and undo snapd services on core <Remodel 🚋> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7772>11:50
mvopedronis: of course, I'm a muppet, fixing and adding a test11:51
mvomborzecki: nice!11:51
mupPR snapd#7911 opened: Uc20 shutdown <Created by xnox> <https://github.com/snapcore/snapd/pull/7911>11:58
cachiomborzecki, hey12:12
cachiomborzecki, about lp:185607312:13
cachioyou think it is be manually cleaned during tests?12:13
mborzeckicachio: hi, yes, left it in the comment https://bugs.launchpad.net/snapd/+bug/1856073/comments/212:15
mupBug #1856073: Failed service state recorded by systemd even after snap removal <snapd:Confirmed> <https://launchpad.net/bugs/1856073>12:15
cachiomborzecki, yes, I saw that12:16
cachioI'll make the change in the tests in that case12:16
mborzeckiok12:16
cachiomborzecki, thakns12:17
cachiozyga, hey, could you please give a second round to #787712:17
mupPR #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
cachiothanks12:17
cachiomborzecki, another thing12:18
cachiomborzecki, I can reproduce this in arch https://paste.ubuntu.com/p/XCwW2VnpXG/12:19
cachiomborzecki, I think at some point you reported something similat12:19
cachiosimilar12:19
pedronismvo: reviewed 7910 with some suggestions12:26
mvopedronis: cool, thank you12:28
mborzeckicachio: no that was antoher test checking whether access to the socket is mediated by apparmor12:31
=== zyga-laptop is now known as zyga-holidays
pedronispstolowski: reviewed #765812:44
mupPR 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:44
mupPR #7658: cmd/snap-preseed: add snap-preseed executable <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7658>12:45
mupPR snapd#7912 opened: boot: write modeenv when creating the run mode <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7912>12:45
pstolowskipedronis: ty!12:45
* Chipaca gets lunch12:53
=== ricab is now known as ricab|lunch
ijohnsonmorning folks13:55
mupPR 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:22
cachiomborzecki, hey14:38
pstolowskipedronis, degville i've sent you an email invitation to edit snap-preseed help doc14:38
cachioafter many tests we left a systemd unit sys-devices-virtual-block-loop1.device14:38
cachiomborzecki, is it created by snapd?14:38
cachioany idea?14:38
pedronispstolowski: thanks, I'll try to get to it today or tomorrow morning14:41
mvopedronis: 7912 should be a simple one14:41
mborzeckicachio: it's probably a loopback device that's left behind, any chance you could run `losetup -l` there?14:41
pedronismvo: ok, I need a short break frist though14:41
mupPR 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*first14:42
cachiomborzecki, yes, I'll add more info for next run14:42
mvopedronis: yeah, no worries14:43
mborzeckicachio: it might be the case that libloop leaves one free device behind as an optimization so that it doesn't need to allocate it again14:43
cachiomborzecki, thanks, I added a new check for loop devices14:48
cachioand triggered a new run14:49
cachioit will be ready on 1 hour aprox14:49
cachioafer lunch14:49
* cachio lunch14:49
Chipacaneed to go to the boys school for a bit, will bbiab14:54
mupPR 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:17
=== ricab|lunch is now known as ricab
mborzeckihmm MATCH with multiline input behaves somewhat weird15:39
mborzeckierr MATCH -v actually15:40
Chipacamborzecki: weird how?15:41
mborzeckiChipaca: https://paste.ubuntu.com/p/P3hyRbDmcy/15:42
mborzeckiChipaca: we actually have code in tests that does MATCH -v with multiline input15:44
Chipacamborzecki: 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 wrong15:47
mborzeckiChipaca: heh sooo https://paste.ubuntu.com/p/GD63G4S2sX/15:48
mborzeckihmm that lsmod, snap list, virsh checks are likely wrong15:50
mborzeckiprobably the same in cgroups test :/15:50
mupPR snapd#7913 opened: boot,bootloader: extract kernel in makeBootable20RunMode  <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7913>15:58
mupPR snapd#7914 opened: bootstrap: reduce runmode mounts from 5 to 2 steps <Created by xnox> <https://github.com/snapcore/snapd/pull/7914>16:07
pedronismvo: does 7913 works or we think it works?16:11
mvopedronis: I'm testing it now16:11
mvopedronis: you can mark it blocked if you want16:12
mvopedronis: but it's essentially what I had in mind and tried to describe16:12
pedronismvo: yes, but I said it makes sense to land only if works16:12
pedronisalso it needs more XXX to it :)16:12
mvopedronis: heh :) yeah, I need to update the other bits from claudios first16:13
mvopedronis: doing this now16:13
pedronismvo: sounds like bootloader.Options.Recovery needs a different name16:13
pedronisbut that's less of an issue16:13
mvopedronis: yeah, its a bit too confusing, ideas welcome!16:14
pedronisI need to look again exactly how it is used16:14
mvopedronis: \o/16:15
pedronismvo: working?16:16
pedronisdoes it seed again?16:17
mvopedronis: yeah, testing if it seeding16:19
pedroniswhat initramfs is it using?16:19
mvopedronis: I mean, unless I miss something (entirely possible) with this PR and the other changes from claudio we should be seeding now16:19
mvopedronis: except that snap-bootstrap in run-mode in initramfs is buggy16:19
mvopedronis: but xnox is on this :)16:20
pedronismvo: but what is putting the kernel and the base in the right place?16:20
mvopedronis: 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/base16:21
mupPR #7913: boot,bootloader: extract kernel in makeBootable20RunMode  <UC20> <⛔ Blocked> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7913>16:21
mvopedronis: we also have the seed16:21
pedronisI probably don't remember what snap-bootstrap run mode is doing then16:22
pedronisis probably wrong16:22
pedronissomehow16:22
pedronismvo: to be clear, I'm talking about ubuntu-data/var/lib/snapd/snaps16:23
xnoxmvo:  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
mvopedronis: we setup the uc20 seed there and go through normal seeding iirc, no?16:23
pedronismvo: no, something is confusing me16:23
pedronisor maybe is right for the first boot but not the other16:23
pedronisI need to stare at snap-bootstrap code16:24
pedronisI suppose16:24
pedronislet me look at it again16:25
pedronismvo: snap-bootstrap expects the base and kernel to be in  filepath.Join(dataDir, "system-data", dirs.SnapBlobDir, modeEnv.Base)16:28
pedronisthat's the bit I don't undersand16:28
pedroniswhen do we put them there16:28
pedronis??16:28
pedronisextracting the kernel doesn't do that16:30
mvopedronis: oh, right - now I see what you mean16:31
pedronisso I'm not sure how it works16:31
pedronisexcept magic :) ?16:31
mvopedronis: well, we are not there yet :/16:31
pedronisyou said it worked ?16:31
pedroniswork != work16:32
mvopedronis: sorry, I think I miswrote, I wanted to say that I'm trying to see if it works16:33
mvopedronis: and that I need bits from cladio first and that it's a bit of work to get to the point16:33
pedronismvo: heh, it's ok, I got very confused16:33
pedronismvo: I gave some comments in 791316:36
mvopedronis: but it does sound like another wrench in the gears16:36
mvopedronis: oh well! so much things to fix still16:36
pedronismvo: well, it's easy to fix, we need to copy those snaps16:36
pedronisthat we knew16:36
mvopedronis: in makebootable16:36
pedronisyes16:36
* mvo nods16:36
pedronislike the UC16 makes symlinkes, this one need to do copies16:37
mvoright16:37
mvopedronis: sorry, brain is a bit fried - I added a todo for myself here16:37
pedronismvo: anything I can help with?16:41
mvopedronis: 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 now16:47
mvopedronis: I mean, I try to work through the stuff from claudio16:47
pedronismvo: sorry, "point the extra" ? address the extra? add an extra ?16:48
mvopedronis: sorry again! I mean, you mentioned earlier that 7913 will need a bunch of extra XXX beyond the ones I added there16:55
mvopedronis: if you haven't done so already, pointing them out in the PR would be great16:56
ijohnsonis ubuntu-image expected to build partitions from gadget.yaml for core20 pc gadget yet?17:01
ijohnsonI 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 missing17:02
pedronisijohnson: that's expected17:09
pedronisthe images have only seed17:09
pedronisboot is added at installation17:09
ijohnsonpedronis: 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 partition17:11
* cachio afk 10 min 19:05
mupPR snapd#7915 opened: lkenv.go: adjust for new location of include file <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7915>19:21
mupPR snapd#7916 opened: interfaces/browser-support: add more product/vendor paths <Created by Erick555> <https://github.com/snapcore/snapd/pull/7916>20:03

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