[01:22] PR snapd#9300 opened: boot: build bootchains data for sealing <⛔ Blocked> [02:22] PR snapd#9301 opened: tests/lib/cla_check.py: don't allow users.noreply.github.com commits to pass CLA [04:57] PR snapd#9273 closed: boot: mark successful with boot assets [05:09] morning [05:13] PR snapd#9298 closed: boot: represent boot chains, helpers for marshalling and equivalence checks [05:13] yay [05:15] good morning mborzecki [05:24] mvo: hey [05:25] need to drive me daughter to school, back in a bit [05:27] sure thing [06:26] re [06:51] morning [07:03] PR snapd#9301 closed: tests/lib/cla_check.py: don't allow users.noreply.github.com commits to pass CLA [07:09] mvo: mborzecki: hi, should we have a chat in a bit? [07:10] spread job on 20.10 is required now? [07:11] mborzecki: I think mvo did that change [07:11] pedronis: sure, when do you want to have the chat? [07:11] in 10 minutes? [07:13] works for me, mvo? [07:15] merged master into https://github.com/snapcore/snapd/pull/9300 easier to see the actual changes [07:15] PR #9300: boot: build bootchains data for sealing <⛔ Blocked> [07:22] mborzecki: I'm joining the standup [07:24] pedronis: hey, only now saw this message [07:24] pedronis: joining [07:43] good morning [07:43] I'll get ready for the call with JJ [07:43] last night I got stuck on task ordering weirdness [07:44] I need to look at the code with fresh eyes [07:44] how are core20 bits? [07:54] zyga-kaveri: good morning [07:54] zyga-kaveri: things are progressing [07:55] from the outside it looks like a lot of concentrated effort [07:55] happy to chat later today if you have time [08:08] mborzecki: I think you asked about ubuntu-20.10 as required earlier - it is now but if that is a problem I can disable that again [08:08] mvo: thanks for the comments under remove-many, was really simple change [08:09] mvo: it failed in https://github.com/snapcore/snapd/pull/9297 but I see it's green now after i pushed a merge from master there [08:09] PR #9297: boot: add "rootdir" to baseBootenvSuite and use in tests [08:10] mborzecki: ta [08:13] back from call [08:13] this thing is making progress :) [08:13] PR snapd#9237 closed: devicestate: enable cloud-init on uc20 for grade signed and secured [08:13] PR snapd#9297 closed: boot: add "rootdir" to baseBootenvSuite and use in tests [08:14] okay, onto exports [08:23] PR snapd#9302 opened: tests: use `nested_exec` in core{20,}-early-config test <⚠ Critical> [09:48] mborzecki: mvo: proposed, https://github.com/snapcore/snapd/pull/9303 [09:48] PR #9303: boot,secboot: switch to expose and use snapcore/secboot load event trees [09:49] PR snapd#9303 opened: boot,secboot: switch to expose and use snapcore/secboot load event trees [09:51] pstolowski: one small remark for 9294, thanks for this, looking really good [09:51] * mvo has some barely controlled rage against nested tests currently [09:52] mvo: watch out, nested is dangerous for mental health [09:54] pedronis: thanks, i've opened the tweaks too: https://github.com/snapcore/snapd/pull/9304 (cc mvo) [09:54] PR #9304: boot: tweak boot chains to support a list of kernel command lines, keep track of model and kernel boot file [09:54] pstolowski: yeah, I was wondering if I should play hockey tonight or not but I think I will just to get rid of this agression that is building up [09:55] pstolowski: to be fair, a lot of it is shell [09:57] yeah, one simple typo grants you 40 minutes of waiting for another run ;) [09:58] and fun with quoting [09:58] mborzecki: thanks, I'll review it after lunch [09:58] pstolowski: yeah, that's part of it. anyway, *rage* [09:59] pstolowski: debugging it is also impossible, ps afx shows "sleep 1" [09:59] PR snapd#9304 opened: boot: tweak boot chains to support a list of kernel command lines, keep track of model and kernel boot file [09:59] pstolowski: like why/where etc? *furryandrage* [09:59] pstolowski: anyway, sorry, I will see that at least debugging can be improved a little bit [10:00] mvo: oh, what's up with nested? [10:00] mhm [10:00] zyga-kaveri: some tests in there are broken [10:00] zyga-kaveri: and it's unclear why and they are kind-of-important for our work :) [10:00] :( [10:00] I'm sorry this is frustrating [10:00] can I help? [10:01] zyga-kaveri: maybe in some minutes, I plan to push some small tweaks [10:01] zyga-kaveri: review/ideas about those will be great [10:01] mvo: updated #9294 [10:01] PR #9294: o/snapstate: check available disk space in RemoveMany [10:01] pstolowski: \o/ [10:03] we will probably need to think about --purge flag support with remove-many; with the above check the client will hint at --purge which cannot be applied to multiple snaps at once [10:04] not super problematic but slightly confusing [10:36] * mvo will note that qemu is a piece of magic [10:37] mvo: large piece [10:37] mvo: would the snapshot / restore feature help with nested tests perhaps? [10:37] * zyga-kaveri doesn't understand TestEnableRunThrough [10:37] zyga-kaveri: maybe but it's really more fundamental at this point [10:38] zyga-kaveri: I mean, the nested vm is not creating a user right now for me [10:38] zyga-kaveri: and afaict I have no way to debug this [10:38] zyga-kaveri: so I need to collect some wood first to build a crafting table it seems [10:38] can you mount the image and look at logs? [10:39] you can use the virt tools package for that [10:39] unless crypted [10:40] zyga-kaveri: this is what I'm currently doing http://paste.ubuntu.com/p/ZkGtr2pMpb/ [10:40] zyga-kaveri: I could but I really want to have a better way, see the new -sertial option there [10:41] zyga-kaveri: it started with qemu-nested not working for me and then I gradually crawled my way out [10:41] I can see where the frustration is coming from :) [10:42] mvo: should sleep_reason echo the reason? [10:49] zyga-kaveri: it could, I mostly did it for ps afx [10:50] maybe I miunderstand but now the || part will always fail [10:50] as you run it in a subshell and give it function names [10:55] * pstolowski lunch [10:59] zyga-kaveri: it's just there so that a "ps afx" in the qemu shows why we wait. the first sleep xx will never fail so the right side of the || never runs [11:01] mborzecki: I'm not sure I understand your question in 9303, if next was always one element there would be no point in doing this [11:07] mborzecki: I answered in the PR [11:12] nested.sh defines PARAM_SERIAL 3 times in the same way [11:22] mvo: after prepare.sh nested.sh is our largest shell file [11:22] and prepare.sh has probably a simpler flow than nested.sh [11:23] pedronis: yeah, did I mention I'm full of rage and furry? [11:23] pedronis: anyway, I think there will be some good outcomes at least [11:27] is uc20-recovery known to be flaky? [11:27] or known to fail? [11:27] it's failing in my PR [11:28] pedronis: not know to me, let me look [11:29] pedronis: this "2020-09-09T10:40:40.0118864Z retry: command sh -c test "$(wc -l < /tmp/mock-shutdown.calls)" = "2" keeps failing after 30 attempts" is a failure ijohnson fixed recently in 9299 - it's not in master yet [11:38] pedronis: yeah, i was a bit confused by the tests, but it's clear now, suggested little tweaks to formatting that makes the nesting clearer [11:43] mborzecki: thx, I reviewed your PR [11:44] pedronis: thanks! [11:50] ijohnson: an interessting observation - I am debugging a failed nested uc20 minimal smoke test right now and it looks like the /run/mnt/ubuntu-seed/data/etc/cloud.cfg.d/data.cfg did not get copied into the system, it's a bit hard to see why because we have no persistent journal right now. note that this is before merging your uc20 changes. anyway, I guess that is not much, I will keep digging [11:53] mborzecki: pushed your suggestions, please double check [11:53] pedronis: lgtm [11:56] mborzecki: should we chat about the next step? [12:00] mvo yeah that would explain why the user didn't run [12:01] ijohnson: anything I can/should debug while I'm inside the nested vm? if not I will re-run with persistent jounral enabled [12:01] mvo I don't remember if I proposed the branch or not but I added a bit of code to make the journal persistent in nested VMs via the gadget.yanl defaults [12:01] ijohnson: yeah, I just added the same bit :) [12:01] ijohnson: well, don't know if it's the same [12:01] ijohnson: but same goal [12:02] mvo in this system do you see the disabled file in /etc/cloud ? [12:02] ijohnson: yes [12:02] ijohnson: and the timestamp is some seconds before my snapd log starts [12:02] ijohnson: so I don't know what it did before :/ [12:03] Yeah probably just need to re-run with persistent journal [12:03] After breakfast I will propose that branch if I haven't already proposed it [12:03] ijohnson: ok, only if it's no hassle for you. i have similar code already locally [12:04] ijohnson: part of some bigger and slightly messier work [12:04] ijohnson: that I will cleanup :) [12:05] Yeah I just checked I haven't proposed it yet, will do so when available this morning [12:06] no rush, I rerun and get some lunch [12:08] pedronis: sry, yes, 1430 maybe? does not look like cmatsuoka is online yet [12:09] mborzecki: ok, works for me [12:29] mborzecki: going into the standup [12:41] PR snapcraft#3281 closed: build providers: hide systemd setup for LXD [14:00] ijohnson, cachio just FYI, I'm poking at nested right now and this is my in progress stuff http://paste.ubuntu.com/p/6CTYJn3ZYs/ [14:01] ijohnson, cachio I will probably throw some away and hopefully push some things as their own PRs [14:02] mvo, nice, thanks [14:04] cachio: i triggered adt for i386/focal [14:05] mvo, tx [14:10] PR snapd#9302 closed: tests: use `nested_exec` in core{20,}-early-config test <⚠ Critical> [14:12] hi, i'm trying to run mattermost-desktop snap inside of an lxc container, but I get a weird error it seems related to snap-confine [14:13] an strace shows this: execve("/snap/snapd/8790/usr/lib/snapd/snap-confine", ["/snap/snapd/8790/usr/lib/snapd/snap-confine", "--base", "core18", "snap.mattermost-desktop.mattermost-desktop", "/usr/lib/snapd/snap-exec", "mattermost-desktop"], 0xc4202a57c0 /* 38 vars */) = -1 EACCES (Permission denied) [14:13] and it results in a segfault [14:14] micah: can you please look at apparmor DENIED messages in journald? [14:14] micah: usually dmesg | grep DENIED is a good start [14:15] PR snapd#9305 opened: tests: fix nested to work with qemu and kvm [14:17] zyga-kaveri: i found these: https://paste.debian.net/1163177/ [14:19] micah: I think those are a bit over my head, definitely related to LXD's own apparmor profile [14:19] what kind of setup is this? a debian host with lxd and a container that runs mattermost desktop with forwarded X? [14:20] zyga-kaveri: debian host, running lxd with snap, and an ubuntu 20.4 container running mattermost desktop with forwarded X, yes [14:21] micah: I think that may be a little bit beyond our test matrix [14:21] micah: but the denial itself is related to lxd, maybe stgraber can help or delegate? [14:27] making tiny bit of progress by digging into fake backend [14:37] thanks mvo [14:37] * ijohnson is back now [14:37] ijohnson: the gadget default is wrong (as you probably already noticed, fixed locally) [14:42] yeah I like the bit about checking for a failed vm state immediately, I think that would make things die a lot quicker with some simple typos that can be rather frustrating :-) [14:45] ijohnson: yeah [14:45] ijohnson: I particularly like the change that I can telnet to the serial port of the nested qemu, qemu is really magic [14:46] mvo: yes though that is less useful for spread runs via i.e. github actions where you don't get dropped to a debug shell [14:46] mvo: I experimented with a pipe fd / tee solution where you would both get all of the serial output in a file for automated / non-interactive runs AND also still be able to login via serial somehow too if running an interactive debug shell, but I didn't spend enough time on it to make it all work properly unfortunately [14:47] perhaps there's a clean way with qemu to tell it to send to multiple things at once, but I don't think there is when I looked [14:47] cachio: google-sru:ubuntu-16.04-64:tests/main/snap-run-gdbserver passed for me with release/2.46 [14:48] pstolowski, yes [14:48] cachio: not sure if there is anything special about running these tests? [14:48] but [14:48] I'll show you how to run it [14:48] ok [14:48] pstolowski, you need to use this env [14:48] SPREAD_MODIFY_CORE_SNAP_FOR_REEXEC=0 SPREAD_TRUST_TEST_KEYS=false SPREAD_SNAP_REEXEC=0 SPREAD_CORE_CHANNEL=stable SPREAD_SRU_VALIDATION=1 [14:49] pedronis: cmatsuoka: opened https://github.com/snapcore/snapd/pull/9306 with the helper [14:49] PR #9306: boot: helper for generating secboot load chains from a given boot asset sequence [14:49] cachio: ah ok [14:50] pstolowski, https://paste.ubuntu.com/p/HJpXJ9QrCG/ [14:50] PR snapd#9306 opened: boot: helper for generating secboot load chains from a given boot asset sequence [14:50] cachio: thanks [14:50] pstolowski, yaw [14:53] mvo: sorry can you clarify if you are running the qemu-nested:ubuntu-20.04-64:tests/nested/manual/minimal-smoke or qemu-nested:ubuntu-20.04-64:tests/core20/basic ? [14:53] I forgot to ask which specific test you were debugging [14:58] ijohnson: minimal-smoke [14:59] ack thanks [14:59] ijohnson: I'm in the nested vm right now [14:59] ijohnson: but still have no persistent log from the time of the install :/ [14:59] ijohnson: I have the /var/log/journal dir though [14:59] ijohnson: and no data.cfg in /etc/cloud/cloud.cfg.d [14:59] mvo: hmm how did you apply the journal persistence logging? [15:00] ijohnson: via the gadget default, one sec, let me pastebin my latest diff [15:00] ijohnson: this is on top of sergios PR 9284 (just to avoid duplicating fixes) [15:00] PR #9284: tests: some fixes and improvements for nested execution [15:00] r9ght [15:00] *right [15:01] ijohnson: http://paste.ubuntu.com/p/p2NdDvHX9M/ - persistent log is in line 82pp [15:02] mmm fair enough I used a heredoc, but that's neither here nor there [15:02] ijohnson: well, the config apply did work [15:02] ijohnson: I can see the dir [15:02] mvo: did you confirm that the data.cfg file is in the right partition in the image and named correctly ? [15:02] I've disabled more and more of the state engine and I still see auto-connect [15:03] with only the snap manager [15:03] I need to break for PT soon but at some point things to disable will run out [15:03] ijohnson: it's under /run/mnt/ubuntu-seed/data/etc/cloud/cloud.cfg.d/data.cfg [15:03] k, looks correct [15:03] ijohnson: so looks correct, snap model --assertion also give me a dangerous assertion so the copy should happen [15:03] * ijohnson is still waiting to reproduce [15:03] * cachio lunch [15:04] mvo: can you pastebin logs from the previous boot ? [15:04] ijohnson: however this is not master, so maybe I should merge in master to get your latest stuff [15:04] err the first boot I suppose [15:04] mvo: probably not necessary to merge in master [15:04] ijohnson: sure, I can try, it's layers of vms [15:04] it's vms all the way down [15:04] * ijohnson is a vm running a vm inside a vm on a vm [15:05] * zyga-kaveri afk [15:09] mvo, cachio: re gdbserver test failure, usr/bin/gdbsever is there on 16.04. the error "cannot snap-exec: cannot use "gdbserver" command" suggests that snap-exec doesn't recognize --gdbserver [15:10] mvo: see snap-exec/main.go, the default case of findCommand [15:10] pstolowski: oh, that's confusing, isn't that our snap-exec? [15:11] pstolowski: or does it first need to land inside the core stable snap or something? [15:11] mborzecki: thanks, will check after lunch [15:11] mvo: yeah i'm wondering if that means it's running old snap-exec. fwtw that code change was made 3 months ago [15:12] mvo: one sec, let me check something [15:13] need to taxi the kids around for a bit, will check back in later [15:13] mborzecki: good luck [15:15] mvo: so yeah, the core there is 16-2.45.3.1 9804, release 2.45 didn't have "gdbserver" flag in snap-exec [15:15] (afaict) [15:16] pstolowski: ok, mysery solved then [15:16] strings over snap-exec confirms that [15:25] PR snapd#9307 opened: interfaces/tee: add TEE/OPTEE interface [15:46] tests/main/searching strikes again (in #9305) [15:46] PR #9305: tests: fix nested to work with qemu and kvm [16:05] pedronis: updated 9306, will need a merge from master when you PR lands [16:37] cmatsuoka: https://github.com/snapcore/snapd/pull/9306 needs a review, also we used bootloader.Role now [16:37] PR #9306: boot: helper for generating secboot load chains from a given boot asset sequence [16:53] pedronis: yes, I just noticed that, that was an interesting solution [16:54] encoding/json does the right thing, though reading the docs is not very obvious [16:54] I had to read the code to double check [16:55] cmatsuoka: related to your comment, yes I think in a follow up we should now use bootloade constants for Role [16:55] to avoid confusion [17:02] pedronis: which pr's for resealing should I look at today [17:02] I'm about to break for lunch but can start looking after I get back [17:04] ijohnson: we want to land my 9303 (that one has two reviews but tests are giving trouble) https://github.com/snapcore/snapd/pull/9304 and https://github.com/snapcore/snapd/pull/9306 [17:04] PR #9304: boot: tweak boot chains to support a list of kernel command lines, keep track of model and kernel boot file [17:04] PR #9306: boot: helper for generating secboot load chains from a given boot asset sequence [17:05] ijohnson: I'm dong a run to see if your uc20-recovery PR helps with 9303 [17:07] * pedronis dinner [17:25] ijohnson: no, even with your PR merged uc20-recovery is failing on my PR but I'm not quite sure why, the files contains reboot requests [17:29] pedronis: hmm ok let me have a look [17:30] ijohnson: I don't think the fix is completely right [17:30] -r +10 reboot scheduled to update the system [17:30] stop serial-getty@ttyS0 [17:30] stop getty@tty1 [17:30] is more than two lines [17:30] but won't match one of the patterns [17:31] pedronis: right that's why afterwards we check for the two reboot lines we care about [17:31] yes, but we might check too early [17:31] ah I see what you mean [17:31] can't we add some filtering to mock-shutdown itself [17:31] mm, I suppose I could just put the retry's on checking the MATCH [17:31] that would work instead [17:32] so tha we get only significative lines [17:33] sure I can make it such that only lines for "-r ..." go to the file we grep for [17:33] yes [17:41] PR snapd#9308 opened: tests: allow root to connect to ubuntu core using a certificate [18:14] back from PT [18:14] food and back to work [18:16] pedronis: fixed #9299 [18:16] PR #9299: tests/core/uc20-recovery: fix check for at least specific calls to mock-shutdown [18:25] * cachio -> kinesiologist [18:33] ijohnson: thx, reviewed [18:37] cmatsuoka: ijohnson: so as I said, we should try to merge 9303 9304 9306, the two latter needs 2nd review, also merging 9299 if it passed uc20-recovery would be good [18:37] PR snapcraft#3282 opened: v2 plugins: add catkin-tools plugin [18:38] pedronis: ok yes 9304 and 9306 are in my queue now [18:38] pedronis: 9303 is failing on tests but it seems that it's not related to the changes there [18:38] cmatsuoka: are you blocked atm? the next step is essentially reworking 9300 and 9287 on top of 9304 and 9306 [18:39] pedronis: I'm using these changes in my staging branch but having them on master would be better [18:39] cmatsuoka: yes, 9303 failure should be fixed by 9299, I will force merge when the test have finished [18:40] * cmatsuoka checking 9304 now... [18:40] but yes, best would be landing them but we need to start from 9303 [18:41] tests have been pain today [18:41] also the store had/has some problems [18:41] yes, I tried to re-run on 9303 a couple of times but they keep failing [18:43] cmatsuoka: we should close #9278, right? is supersed by new logic and 9300 ? [18:43] PR #9278: boot: seal using boot chains from bootloader [18:43] pedronis: yes, it has been obsoleted by the new PRs, we should close it [18:47] thx [18:51] PR snapd#9278 closed: boot: seal using boot chains from bootloader [18:55] * cmatsuoka will make a short break [19:06] I force merged 9303 [19:11] PR snapd#9303 closed: boot,secboot: switch to expose and use snapcore/secboot load event trees [19:16] thanks [19:43] so arch seems to fail sometimes on google:arch-linux-64:tests/main/cgroup-tracking:root [19:51] pedronis yes I've seen that a few times and mentioned it in the SU doc but I don't understand the error [19:56] PR snapd#9304 closed: boot: tweak boot chains to support a list of kernel command lines, keep track of model and kernel boot file [20:01] ijohnson: cmatsuoka: I merged 9304 ^ and I fixed conflicts in 9306 [20:03] pedronis: ah should I just skip striaght to 9306 then ? [20:03] pedronis: thanks! it will make things easier to reconcile here [20:07] ijohnson: yes [20:07] ack [20:14] ijohnson: cmatsuoka: I made #9309 on top #9306, it's just a cleanup but likely conflict prone [20:14] PR #9309: boot: be consistent using bootloader.Role* consts instead of strings [20:14] PR #9306: boot: helper for generating secboot load chains from a given boot asset sequence [20:15] pedronis: ack, checking this one... [20:16] PR snapd#9309 opened: boot: be consistent using bootloader.Role* consts instead of strings [20:17] cmatsuoka: it's related to your comment in the other one [20:17] pedronis: ah, yes, I just checked the relevant commit. thanks. [20:32] why are tests so slooooow today?? === jdstrand_ is now known as jdstrand === ahayzen_ is now known as ahayzen [20:38] ijohnson: what channel did you use for core20 when built your image to test nested? [20:38] pedronis: edge [20:38] ? [20:39] what? [20:39] ijohnson: afaict your fix https://github.com/snapcore/core20/pull/84/files is not in edge [20:39] PR core20#84: static/writable-paths: use transition instead of none for /etc/cloud [20:39] ... it should be? [20:40] pedronis: the fix for that is in edge [20:40] pedronis: at least for revision 819 [20:41] I also double checked and the image built for the nested test here has the fix as well [20:41] (and the same revision of the core20 snap too) [20:46] ijohnson: mmh, notice that here https://github.com/snapcore/snapd/pull/9098 mimimal-smoke failed on 18 [20:46] PR #9098: tests: new organization for nested tests [20:46] it was merged anyway [20:47] pedronis: mmm also 16.04 failed on minimal-smoke too [20:47] this would confirm my theory that something in nested.sh is broken [20:47] why did we merge that though? are they super flaky anyway? [20:48] well the nested tests in general are slightly flaky but there were just too many other fixes in that PR, in the future we should really be more careful to split up changes [20:49] the nested tests are just suffering from so much churn right now [20:51] I'm now trying a fresh build with just snapd from that branch to see if the snapd snap is somehow broken from nested.sh now [20:53] ijohnson: this run worked https://github.com/snapcore/snapd/runs/1038225314 13 days ago [20:55] this one also workd: https://github.com/snapcore/snapd/runs/1052967599 [20:56] right [20:56] also this one: https://github.com/snapcore/snapd/runs/1067305138 [20:56] 6 days ago [20:58] but this from today failed https://github.com/snapcore/snapd/pull/9302 [20:58] PR #9302: tests: use `nested_exec` in core{20,}-early-config test <⚠ Critical> [20:58] so yes if I just use the snapd snap that was produced for the nested run it fails [20:59] i.e. I build my own image and boot it with my tools, and the only snap I replace is the snapd snap, then cloud-init is not run [20:59] err rather it is immediately disabled instead of importing from ubuntu-seed [20:59] so is snapd code, not nested.sh ? [21:00] some of the sysconfig changes? [21:01] the only changes are here https://pastebin.ubuntu.com/p/m9SvCkkmkM/ [21:02] it could be the sysconfig changes I suppose [21:02] somehow? [21:02] I notice we didn't run them with nested on [21:03] yeah I guess I didn't figure that sysconfig changes would need nested runs :-( [21:03] tbh I think we should have had the uc20 tag also trigger nested tests too for all pr's [21:03] a bit late for that now though [21:07] ijohnson: I suspect https://github.com/snapcore/snapd/pull/9253/files [21:07] PR #9253: sysconfig/cloudinit.go: add AllowCloudInit and use GadgetDir for cloud.conf [21:07] do we set to true in the relevant places? [21:07] that PR wasn't setting it anywhere [21:08] so I suspect it just turned it off? [21:08] mmm [21:08] maybe [21:08] pedronis: 9306 tests finished with the usual fails [21:08] pedronis: could you force-merge it too? [21:09] pedronis: yeah I think you're right [21:09] we landed your last one, that use it sometimes but not sure we set it in all the places [21:09] this is a bit the risk of splitting things [21:09] sometimes [21:10] pedronis: ok, so I rebuilt snapd exe from master with my last PR merged and just replacing snapd binary in the spread snapd I extracted now cloud-init works [21:10] so theoretically it should have been broken by the first PR's landed, then fixed with the last PR that was merged [21:10] also of interest is the fact that mvo was debugging things based on Sergio's PR which does not have the master merge from my final PR [21:10] ijohnson: so the question is, to be have only one place right calling the relevant function? [21:10] so [21:10] yes we only have one place calling the sysconfig function [21:11] err to be more precise [21:11] we set sysconfig.Options for cloud-init in one place [21:11] and on master that is correct [21:11] so I think if I trigger a run of this on master it should pass actually [21:11] I need to go pick up my car though, so I will trigger a run on master to see what happens [21:11] maybe it passes [21:12] cmatsuoka: I hoped ijohnson would also look at 9306 [21:12] pedronis: cmatsuoka: yes sorry I was midway through a review and got sidetracked with nested tests [21:13] I will have to finish my review in a bit like an hour or so from now though [21:13] * ijohnson afk little bit [21:13] * zyga-kaveri EODs [21:13] good night! [21:13] ijohnson: cmatsuoka: I can land it and we apply feedback after the fact (it does have two +1) [21:14] pedronis: i would be ok with that [21:20] cmatsuoka: ijohnson: done [21:21] pedronis: thanks [21:22] PR snapd#9306 closed: boot: helper for generating secboot load chains from a given boot asset sequence [21:29] cmatsuoka: I merged #9309 as well (it was just replacing strings in tests anyway) and had your +1 [21:29] PR #9309: boot: be consistent using bootloader.Role* consts instead of strings [21:30] cmatsuoka: so master should have all the bits now that you need [21:30] merged [21:30] pedronis: yes, I'm running an integrated test with load chains now [21:32] PR snapd#9309 closed: boot: be consistent using bootloader.Role* consts instead of strings [22:53] PR snapcraft#3282 closed: v2 plugins: add catkin-tools plugin