/srv/irclogs.ubuntu.com/2021/04/12/#snappy.txt

jameshmwhudson: looks like the problem https://github.com/snapcore/snapcraft/pull/3498 is supposed to fix.  edge snapcraft might let you close the channel04:54
mupPR snapcraft#3498: storeapi: set content-type and accept headers for close <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3498>04:54
jameshseems like a bit of a problem that the tool is trailing the changes in the store rather than leading04:55
mborzeckimorning05:49
zygagood morning07:14
mborzeckizyga: hey07:16
mvogood morning zyga and mborzecki. happy monday!07:16
mborzeckimvo: hey, good morning07:16
mborzeckimvo: fyi, this PR can land https://github.com/snapcore/snapd/pull/1012807:17
mupPR #10128: tests: add 20.04 to systems for nested/core <Run nested> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10128>07:17
mvomborzecki: cool07:18
mborzeckimvo: can you take a look at https://github.com/snapcore/snapd/pull/10120 ?07:19
mupPR #10120: boot: derive boot variables for kernel command lines <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10120>07:19
mvomborzecki: sure07:19
mborzeckishould be fairly simple, it there's some post merge comments from pedronis we can address those in a followup07:20
mupPR snapd#10132 closed: boot,tests: move where we write boot-flags one level up <Squash-merge> <UC20> <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10132>07:21
mborzeckimvo: thanks! would you like to take a look at https://github.com/snapcore/snapd/pull/10129 too? builds on top of the previous one07:29
mupPR #10129:  boot: set extra kernel command line arguments when making a recovery system bootable <Run nested> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10129>07:29
mvosure07:30
mborzeckithanks!07:30
mupPR snapd#10120 closed: boot: derive boot variables for kernel command lines <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10120>07:31
mvomborzecki: done, how much extra work will run-mode support be?07:33
mborzeckimvo: i need this to land to open the next PR: https://github.com/snapcore/snapd/pull/10130 (or at least https://github.com/snapcore/snapd/pull/10119 so that there wont' be too many prs open)07:38
mupPR #10130: boot: load bits of kernel command line from gadget snaps <Run nested> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10130>07:38
mupPR #10119: boot: pass gadget path to command line helpers, load gadget from seed <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10119>07:38
mvomborzecki: anything there I could review and help with?07:45
mborzeckimvo: maybe you could also take a look at https://github.com/snapcore/snapd/pull/10119  then?07:46
mupPR #10119: boot: pass gadget path to command line helpers, load gadget from seed <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10119>07:46
jameshanyone up for a second review of https://github.com/snapcore/snapd/pull/10126 ? It's a fairly short and simple cleanup08:28
mupPR #10126: daemon: make ucrednetGet() return a *ucrednet structure <Simple 😃> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/10126>08:28
pedroniswas about to mention it as well08:35
pedronisjamesh: it's merged now08:39
jameshpedronis, mborzecki: thanks!08:40
mupPR snapd#10126 closed: daemon: make ucrednetGet() return a *ucrednet structure <Simple 😃> <Created by jhenstridge> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10126>08:41
mborzeckipedronis: can you take another look at https://github.com/snapcore/snapd/pull/10119 ?08:51
mupPR #10119: boot: pass gadget path to command line helpers, load gadget from seed <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10119>08:51
pedronismborzecki: yes, was about to, it's near the top of my queue08:51
mborzeckipedronis: cool, thank you08:52
mupPR snapd#10131 closed: tests: new os.paths tool <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10131>08:56
mupPR snapd#10123 closed: boot,gadget,devicestate: create "aux-key" for fde-hooks v2 <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/10123>09:01
pedronismborzecki: could you merge master into https://github.com/snapcore/snapd/pull/1012909:11
mupPR #10129:  boot: set extra kernel command line arguments when making a recovery system bootable <Run nested> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10129>09:11
mborzeckipedronis: sure, just a sec09:12
mborzeckipedronis: done09:13
pedronismborzecki: it's the extra args case complete with 10130 or there is more even for that?09:35
mborzeckipedronis: there's a change in MakeSystemRunnable(), another one in update boot config, a helper for handling update of bits that contribute to command line, and changes in devicestate, although all of that is for both extra and full cmdlines09:37
pedronisah, ok, so still quite a bit of stuff09:38
mupPR snapd#10119 closed: boot: pass gadget path to command line helpers, load gadget from seed <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10119>09:41
mborzeckiheh running a nested test with tpm and secure boot, the qemu cpu is resetting10:35
mupPR snapd#10129 closed:  boot: set extra kernel command line arguments when making a recovery system bootable <Run nested> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10129>10:37
mupPR snapd#10134 opened: boot: set extra command line arguments when preparing run mode <Run nested> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10134>11:12
mborzeckihmmm 10134 works locally with secure boot, but fails on google11:23
mborzeckimvo: can you take a look at https://github.com/snapcore/snapd/pull/10135 ? i started doubting myself tbh11:45
mupPR #10135: tests/lib/nested: proper status return for tpm/secure boot checks <Run nested> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10135>11:45
mupPR snapd#10135 opened: tests/lib/nested: proper status return for tpm/secure boot checks <Run nested> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10135>11:47
mvomborzecki: will do after lunch11:51
mborzeckithanks!11:51
mborzeckicachio_: have you seen: 2021-04-12 10:03:19 Cannot allocate google-nested:ubuntu-18.04-64: cannot find any Google image matching "ubuntu-1804-64-virt-enabled" on project "computeengine" or "ubuntu-os-cloud"11:54
mborzeckiit's in https://github.com/snapcore/snapd/runs/232268246711:54
cachio_mborzecki, hi, let me fix that11:54
mborzeckicachio_: also os.query seems to be failing on debian sid, as if it's not sid anymore 😉 https://paste.ubuntu.com/p/C9xsyhZ8Jd/11:55
mborzeckimaybe they've bumped os-release or something11:55
mborzeckior we're using the wrong image :/11:56
cachio_mborzecki, images fixed11:56
cachio_let me check sic image11:57
cachio_sid11:57
cachio_thanks for let me know11:57
cachio_mborzecki, so now debian sid is VERSION_CODENAME=bullseye12:03
cachio_this is why os.query is failing12:03
cachio_I need to create a new image I see12:05
cachio_mvo, hi, should we support debian 11?12:06
mborzeckihmm, maybe there's other way tell it's debian sid?12:06
mborzeckimerging some of my branches in isolation,  broke master https://github.com/snapcore/snapd/pull/10136 fixes it12:10
mupPR #10136: boot: drop gadget snap yaml which is already defined elsewhere in the tests <Skip spread> <⚠ Critical> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10136>12:10
mupPR snapd#10136 opened: boot: drop gadget snap yaml which is already defined elsewhere in the tests <Skip spread> <⚠ Critical> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10136>12:12
mupPR snapd#10136 closed: boot: drop gadget snap yaml which is already defined elsewhere in the tests <Skip spread> <⚠ Critical> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10136>13:27
mupPR snapd#10047 closed: tests: remove the snap profiler from the test suite <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10047>13:42
mupPR snapd#10137 opened: tests: update os.query check to match new bullseye codename used on sid images <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10137>14:02
* cachio_ afk14:22
ijohnsoncachio_: do you know if our ubuntu core spread GCE environments would have virtual network interfaces that don't have MAC addresses? if so then the tests/core/netplan failures we are seeing may be from https://bugs.launchpad.net/ubuntu/+source/netplan.io/+bug/191306214:43
mupBug #1913062: netplan apply backtrace <amd64> <apport-bug> <focal> <uec-images> <netplan.io (Ubuntu):Triaged> <https://launchpad.net/bugs/1913062>14:43
ijohnsoncachio_: if so I have a workaround from paul that might avoid the issue before the fix is released15:02
=== pedronis_ is now known as pedronis
cachio_ijohnson, hey, sorry, didn't see what youwrote15:56
ijohnsoncachio_: sure let me re-send it15:57
cachio_let me check15:57
ijohnsonoh did you see the message ?15:57
cachio_didn't see the notificatin15:57
cachio_let me check the mac address for uc15:57
cachio_ijohnson, uc16 right?15:58
ijohnsoncachio_: I have seen it on uc20 too I think15:58
cachio_I see in the bug DistroRelease: Ubuntu 20.0416:00
cachio_I am creating vms16:00
cachio_I'll check the network configuration when vms are ready16:00
ijohnsonthanks cachio16:06
ijohnsonyeah I think the bug only effects uc20, I don't know that I've seen the netplan failure on uc16 or uc1816:06
ijohnsonmvo: pedronis: https://github.com/snapcore/snapd/pull/10133 is ready for review again16:16
mupPR #10133: wrappers/services.go: introduce EnsureSnapServices() <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10133>16:16
ijohnsonmmm now our mac os tests are failing16:22
ijohnsonhomebew seems unhappy for some reason16:22
ijohnson*homebrew16:22
cachio_ijohnson, https://paste.ubuntu.com/p/jVj7Nf4Pnx/16:23
cachio_ijohnson, so it has a mac address}16:24
ijohnsoncachio_: right that makes sense, so probably not the same bug unfortunately16:46
cachio_mvo, hey, about the test tests/core/gadget-kernel-refs-update-pc/task.yaml17:01
cachio_does it make sense to run it on arm right?17:01
cachio_because the kernel snap is harcoded to pc-kernel17:02
=== ijohnson is now known as ijohnson|lunch
mvocachio_: yeah, should not run on the arm machines17:43

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