[05:48] <mborzecki> morning
[07:11] <mardy> hi mborzecki, mvo 
[07:14] <mborzecki> mardy: heya
[07:14] <mvo> good morning mardy and mborzecki !
[07:15] <zyga-mbp> good morning :)
[07:24] <mvo> good morning zyga-mbp !
[07:24]  * zyga-mbp hey :)
[07:24] <zyga-mbp> I'm venturing into automation land 
[07:24] <zyga-mbp> with industrial control boxes and what not 
[07:24] <zyga-mbp> it's a weird place
[07:24] <zyga-mbp> how are you doing mvo?
[07:24] <zyga-mbp> Impish is almost upon us
[07:25] <mvo> zyga-mbp: woah, nice. I'm doing fine but not really as exciting things as you, trying to make sure things are delivered on time currently
[07:26] <zyga-mbp> serious things are meant to be like that
[07:26] <zyga-mbp> I'm very happy to see you and samuele manage snapd
[07:52] <mup> PR snapd#10888 closed: tests: support running all spread tests with experimental features <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10888>
[08:02] <mardy> 'morning zyga-mbp 
[08:03] <zyga-mbp> hey mardy :)
[08:08] <mardy> find squashfs-root/ -name configure
[08:09] <mardy> squashfs-root/hooks/configure
[08:09] <mardy> squashfs-root/meta/hooks/configure
[08:09] <mardy> squashfs-root/snap/hooks/configure
[08:09] <mardy> I guess I'll edit all of them :-D
[08:10] <zyga-mbp> woah
[08:10] <zyga-mbp> that's a lot of hooks :)
[08:10] <zyga-mbp> technically snapd needs meta/hooks/
[08:10] <zyga-mbp> snap/hooks is probably (guessing) from snapcraft
[08:10] <zyga-mbp> the top-level is dunno
[08:11] <mardy> zyga-mbp: thanks, I'll ask the developers of the snap to do some housecleaning :-)
[08:13] <zyga-mbp> at least configure is not a directory :)
[08:13] <zyga-mbp> that would confuse snapd
[08:13] <mardy> or a socket ;-)
[08:39] <miguelpires> mvo: hi, can you merge https://github.com/snapcore/snapd/pull/10875 please?
[08:39] <mup> PR #10875: client: fail fast on non-retryable errors <Simple 😃> <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10875>
[08:41] <miguelpires> ty
[08:42] <mup> PR snapd#10875 closed: client: fail fast on non-retryable errors <Simple 😃> <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10875>
[09:00] <mborzecki> mvo: hi, do you have the source tarballs to pus on the github releases page?
[09:05] <mvo> mborzecki: for 2.53? not yet, sry! I can upload after the meeting
[09:07] <mborzecki> mvo: that's fine, i'll update the arch and opensuse packages when the sources are available
[09:23] <mvo> mborzecki: yeah, sry again, will get to it later but in meetings this morning 
[09:43] <mborzecki> got an errand, back in 1,5h or so
[10:21] <mvo> mborzecki: tarfiles updated
[11:02] <mborzecki> re
[11:02] <mborzecki> mvo: thanks!
[11:14] <mardy> mvo: we have a tests/smoke/ directory, should I move the microk8s test there?
[11:21] <mborzecki> meh lxd broke tests https://paste.ubuntu.com/p/2nsXcy5Fyg/
[11:28] <mvo> mardy: the name is a bit misleading, that testsuite is really for smoke testing snapd itself and it is expected to be quick(ish)
[11:29] <mvo> mardy: maybe I should document this :/
[11:38] <mup> PR snapd#10827 closed: fde: add new device-setup support to fde-setup <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10827>
[11:38] <mup> PR snapd#10830 closed: gadget: add `encryptedDevice` and add encryptedDeviceLUKS <Simple 😃> <Skip spread> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10830>
[11:38] <mup> PR snapd#10854 closed: spread: use `bios: uefi` for uc20 <Simple 😃> <Skip spread> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10854>
[12:01] <mup> PR snapcraft#3588 closed: snap: patch patchelf on riscv64 (CRAFT-566) <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3588>
[13:08] <mup> PR snapd#10896 opened: tests: remove "features" from fde-setup.go example <Simple 😃> <Run nested> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10896>
[13:22] <ogra> mwhudson, are you still doing console-conf ? i'm running into a weird bug here ... after installing network-manager on a fully set up UC20 the screen shows "<no ip address>" afterwards while the device actually *does* have an IP ... does console-conf not simply use "ip addr" to find the default IP ?
[13:23] <zyga-mbp> mvo pedronis is not around today?
[13:23]  * zyga-mbp mvo I ran into an issue where snapd cannot remove the core snap (on focal) because it's a part of the model
[13:23] <zyga-mbp> but on classic that is mostly meaningless
[13:24] <zyga-mbp> and it's also a very confusing error message, models are relatively unknown to general public
[13:24] <zyga-mbp> is this by design? 
[13:24] <zyga-mbp> The exact interaction is:
[13:25]  * zyga-mbp $ sudo snap remove core
[13:25]  * zyga-mbp error: cannot remove "core": snap "core" is not removable: snap is used by the model
[13:32] <ogra> mwhudson, ah, ian pointed me to https://bugs.launchpad.net/subiquity/+bug/1898583 (which is sadly still not fix-released thanks to core20 stable not having been updated since april (!!!))
[13:32] <mup> Bug #1898583: console-conf with wlan connection does not ever report ip via tty message, even though it does have ip <uc20> <subiquity:Fix Committed> <https://launchpad.net/bugs/1898583>
[13:36] <dbungert> ogra: this one may also be relevant LP: #1903050
[13:36] <mup> Bug #1903050: consoleconf main is broken (core22) <core22> <fr-896> <uc22> <subiquity:Fix Released> <https://launchpad.net/bugs/1903050>
[13:38] <ogra> dbungert, well, it is fixed in core20 candidate (just confirmed) ... the shocking bit is that core20 has not been released to stable since april ... which means there might be a gazillion open CVEs out there on devices 😞
[13:38] <ogra> that bug is for UC22 ... 
[13:39] <ogra> (which is still a year away, luckily 🙂 )
[13:42] <ogra> oh, weird ... after the switch to candidate snap info core20 shows something completely different for the stable channel ... the stable image i downloaded had an april-released core20 
[13:42] <ogra> (and did not see an update for it with snap refresh)
[13:45] <mvo> zyga-mbp: in a meeting but pedronis should be around
[13:46] <zyga-mbp> perhaps not on irc
[13:46] <mvo> ogra: snap info core20 shows july which is not much better but a bit
[13:46] <mvo> ogra: I think sil2100 knows when core20 is promoted to stable, I expect RSN
[13:47] <ogra> mvo, yeah, this is really weird, i am on a freshly installed UC20 stable image where snap list showed "core20  20210429  1030  latest/stable  canonical✓  base" even after several snap refresh runs that i manually did ... switching the channel to candidate and back for core20 now gives me the correct one 
[13:48] <mvo> ogra: oh, that is odd. what image did you use, I would love to test this 
[13:48] <ogra> http://cdimage.ubuntu.com/ubuntu-core/20/stable/current/ubuntu-core-20-arm64+raspi.img.xz
[13:48] <ogra> on a Pi4
[13:48] <mvo> ogra: ta
[13:49] <sil2100> We are phasing out core20 right now
[13:50] <zyga-mbp> sil2100 phasing out as in replacing it with something or phase upgrade?
[13:50] <sil2100> We're using automated progressive releases for core20, and it's being phased out right now - the new version that is
[13:50] <mvo> nice
[13:50] <sil2100> But yeah, 20210429 seems like something VERY old
[13:50] <mvo> ogra: there you go, I guess the phasing is the reason you did not get hte update
[13:50] <sil2100> So that shouldn't have been listing at all
[13:51] <ogra> mvo, ah, okay ... though april still seems a bit oldish for the image (which has an image timestamp from june)
[13:52] <ogra> i'd have expected a newer core20 on that image 
[13:52] <mvo> ogra: I think we had this big jump https://people.canonical.com/~mvo/core20-changes/html/stable/
[13:53] <ogra> oha !
[13:53] <ogra> okay
[13:53] <mvo> ogra: I guess we need to make sure to keep an eye on this and make sure we release more often
[13:53] <ogra> yeah, we need to make sure CVEs are closed in time 🙂
[13:54] <ogra> else i'll have some customers with torches and pitchforks showing up at my doorstep i fear 
[13:55] <mvo> +100
[13:56] <ogra> nah, i dont want more than 100 customers with pitchforks 😛
[13:56] <ijohnson[m]> -95
[13:56] <ijohnson[m]> 5 customers with pitchforks seems like a reasonable bunch to manage
[13:56] <ogra> yeah, 5 i can handle with a garden hose 
[13:56] <mvo> hahaha
[14:16] <flotter> Following HACKING.md to go build snap: I feel like I am missing something, as I have two broken deps. (1) secboot - which I was able to bypass with a -tag and (2) dbus *Match* functions, which does not appear to exist in godbus. Any hints - sorry im new
[14:16] <mborzecki> invoking `/snap/snapd/current/lib/x86_64-linux-gnu/ld-2.23.so --library-path /snap/snapd/current/usr/local/lib:/snap/snapd/current/lib/x86_64-linux-gnu:/snap/snapd/current/usr/lib/x86_64-linux-gnu /snap/snapd/current/usr/bin/mksquashfs` fails on opensuse
[14:17] <mborzecki> the usual: libgcc_s.so.1 must be installed for pthread_cancel to work
[14:17] <ijohnson[m]> bboozzoo: hey funny you mention ld
[14:17] <ijohnson[m]> I just pinged you on MM about libc haha
[14:17] <mborzecki> fwiw i think we should not invoke the tooling mksquashfs in tests
[14:17] <ijohnson[m]> I think we need another change to the snap-confine profile for s390x
[14:18] <flotter> GOFLAGS='-tags=nosecboot' go build -o /tmp/snap github.com/snapcore/snapd/cmd/snap
[14:19] <flotter> fdo.go:127:18: undefined: dbus.MatchOption
[14:41] <mborzecki> flotter: is this a new checkout? go.mod should already mention the right version of go-dbus
[14:42] <mborzecki> and that version should have been fetched when go build was invoked
[14:42] <flotter> I did find the functions exist in the downloaded godbus repo
[14:42] <flotter> I did a fresh go get, an hour ago
[14:44] <flotter> go: extracting github.com/godbus/dbus v4.1.0+incompatible
[14:45] <flotter> The name is not very inspiring :D
[14:45] <flotter> The name is not very inspiring :)
[14:47] <flotter> work/pkg/mod/github.com/snapcore/snapd@v0.0.0-20210902070205-9fe87efa1b36/desktop/notification/fdo.go:127:18: undefined: dbus.MatchOption
[14:48] <zyga-mbp> flotter that's not right
[14:48] <zyga-mbp> you should not do that manually
[14:48] <zyga-mbp> go build ./... should do it
[14:48] <zyga-mbp> you're not supposed to compete with go.mod by checking out things by hand
[14:48] <zyga-mbp> here you are running into module versions that use a somewhat elaborate scheme, snapd uses /v5
[14:49] <zyga-mbp> flotter git clone && go build ./... should be enough
[14:49]  * zyga-mbp (apart from C world)
[14:50] <flotter> I only did the go get command (from HACKING.md) and then "GOFLAGS='-tags=nosecboot' go build -o /tmp/snap github.com/snapcore/snapd/cmd/snap"
[14:50]  * zyga-mbp most likely hacking is stale
[14:50] <zyga-mbp> it predates go-mod days
[14:50] <flotter> understood
[14:51] <flotter> so you are saying just check out the git, and build
[14:51] <zyga-mbp> yes, at least that's the start
[14:51] <zyga-mbp> once you check out _and_ are on a debian-like distro you can apt-get build-dep . 
[14:51] <zyga-mbp> to get C world side
[14:51] <zyga-mbp> you will be much closer to being able to iterate
[14:51] <flotter> OK let me try, thanks!
[14:52] <zyga-mbp> send patches to fix HACKING.md
[14:52] <flotter> I will do!
[14:52]  * zyga-mbp loves the go.mod world where $GOPATH/src is gone
[14:53]  * zyga-mbp thank you :)
[15:09] <mup> PR snapd#10897 opened: osutil: ensure parent dir is opened and sync'd <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10897>
[15:17] <mup> PR core20#116 closed: hooks: adjtime: add adjtime file to etc <question> <Created by stulluk> <Closed by stulluk> <https://github.com/snapcore/core20/pull/116>
[15:34] <mup> PR snapd#10896 closed: tests: remove "features" from fde-setup.go example <Simple 😃> <Run nested> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10896>
[15:37] <dbungert> so the LP: #1943077 saga continues - I'm trying to judge if snapd from the deb having problems with `snap pack` is release critical for impish.  Retesting adt with snapd from impish-proposed still shows the problem with libgcc.
[15:37] <mup> Bug #1943077: snapd fails to autopkgtest on mksquashfs, which is looking for libgcc_s <impish> <rls-ii-incoming> <update-excuse> <snapd:Fix Committed> <openssh (Ubuntu):New> <squashfs-tools (Ubuntu):New> <https://launchpad.net/bugs/1943077>
[15:39] <mup> PR snapd#10881 closed: tests: merge coverage results <Skip spread> <Created by mardy> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10881>
[17:17] <flotter> Hi. GOOS=linux GOARCH=arm64 go build -v -a -o /tmp/build/arm64 ./...
[17:17] <flotter> I am attempting to cross compile snapd for arm and arm64. 
[17:19] <flotter> I get a compiler failure for golang-evdev - undefined: ioctl
[17:19] <ijohnson[m]> flotter: golang-evdev is only necessary for building snap-bootstrap which is only used on Ubuntu Core so you should just skip building that binary
[17:20] <dbungert> ijohnson[m]: may I get your thoughts on the matter above?
[17:20] <ijohnson[m]> dbungert: sorry I missed your message above let me re-read the bug again
[17:21] <dbungert> buried in bot updates :)
[17:23] <ijohnson[m]> yeah it gets annoying since the links get expanded and so I don't see individual messages as clearly, meh communication is hard
[17:24] <flotter> If I do want to build the snap-bootstrap, any tip(s) on what I should be looking at to get the deps resolved?
[17:29] <ijohnson[m]> flotter: you need to setup a C cross compiler and use it when compiling, something like this
[17:29] <ijohnson[m]> oh hmm I don't seem to have that in my bash history anymore, darn
[17:34] <ijohnson[m]> flotter: this is what you want essentially
[17:34] <ijohnson[m]> CC=aarch64-linux-gnu-gcc GOARCH=arm64 CGO_ENABLED=1  go build -o snap-bootstrap-arm64 ./cmd/snap-bootstrap/
[17:35] <ijohnson[m]> dbungert: so the situation really hasn't changed at all on our side as it pertains to snap pack and impish, one wrinkle that I wasn't aware of (or more accurately I had forgotten) is that `snap pack` will always unconditionally attempt to re-exec to use mksquashfs from the core/snapd _snaps_ even when the version of snapd itself from the debian package is newer. This is to ensure that snaps that are created are always created with a
[17:35] <ijohnson[m]> supported and working mksquashfs
[17:36] <ijohnson[m]> so this means that we need to have the fix you landed in the snapd/core snaps before it can work in these autopkgtests
[17:36] <ijohnson[m]> and also that any impish users who try to use `snap pack` from i.e. snapcraft will be broken and it won't work until the core/snapd snaps are updated
[17:37] <dbungert> that part I'm not so wild about, but I don't have a feel for how common it would be
[17:37] <ijohnson[m]> unfortunately, while the fix is in the pipeline as part of snapd 2.52.1, it is probably 2 weeks away now I think from going to stable for all users to be able to get that fix
[17:39] <ijohnson[m]> we have 2.52 ready to go to stable very shortly, and the next release after 2.52 will be 2.52.1, but we need to let 2.52.1 bake a bit in the beta channel before we can push it through
[17:40] <ijohnson[m]> and in the meantime, real users trying to use snapcraft on impish can use the edge (possibly beta) channel of the snapd snap until the fix has made its way through the pipeline
[17:42] <flotter> Thank you very much, understood
[17:43] <ijohnson[m]> dbungert: does that make sense?
[17:43] <dbungert> It does, yes.  The real question here is - do we believe action needs taken prior to impish release, and I think the answer is "no".  We plan to point affected people to snapd edge/beta.
[17:44] <ijohnson[m]> yeah there is no action to be taken at this time unfortunately other than to wait
[17:45] <dbungert> would we have expected the snapd in impish-proposed to have the libgcc_s fix?
[17:47] <ijohnson[m]> dbungert: the issue is that snapd as a feature will purposefully execute mksquashfs from the snap ignoring whatever version is in the deb world
[17:47] <dbungert> ok, so no change to the deb can conceivably help
[17:49] <dbungert> ijohnson[m]: appreciate the feedback, I think we're good
[17:49] <ijohnson[m]> well we could disable re-exec in the deb so it stops doing that, but it's sort of a feature, so we would want to disable it in a very specific manner, like basically "try re-exec, it if fails specifically because of segfaulting, then use mksquashfs from the host", but that would take time that we don't have a lot of at the moment unfortunately
[17:50] <dbungert> fair
[20:19] <flotter> x86_64 (native)
[20:19] <flotter> ~/snapd$ CGO_ENABLED=1 GOOS=linux GOARCH=amd64 go build -a -o /tmp/build/amd64 ./...
[20:19] <flotter> ARMv8_64 (crosscompile)
[20:19] <flotter> ~/libseccomp$ ./configure --host=aarch64-linux-gnu --prefix=/home/flotter/libseccomp/root
[20:19] <flotter> ~/libseccomp$ make && make install
[20:19] <flotter> ~/snapd$ CC=aarch64-linux-gnu-gcc CGO_ENABLED=1 CGO_LDFLAGS="-L/home/flotter/libseccomp/root/lib" GOOS=linux GOARCH=arm64 go build -a -o /tmp/build/arm64 ./...
[20:19] <flotter> ARMv7 (crosscompile)
[20:19] <flotter> ~/libseccomp$ ./configure --host=arm-linux-gnueabihf --prefix=/home/flotter/libseccomp/root
[20:19] <flotter> ~/libseccomp$ make && make install
[20:19] <flotter> ~/snapd$ CC=arm-linux-gnueabihf-gcc CGO_ENABLED=1 CGO_LDFLAGS="-L/home/flotter/libseccomp/root/lib" GOOS=linux GOARCH=arm go build -a -o /tmp/build/arm ./...
[20:20] <flotter> I have been able to now build snapd on amd64 / arm and arm64. The only dependency was libseccomp, which I cross-compiled for the to non-native builds. 
[20:21] <flotter> Is there anything I have done wrong here (conceptionally) ? I verified the ELF files and the headers match the target arch.
[20:22] <flotter> I am asking in the context of trying to update HACKING.md with steps that could help others.
[20:49] <ijohnson[m]> flotter: did you have any problems? I don't see anything conceptually wrong there but the best test would be to see if it just works
[21:35] <mup> PR snapd#10898 opened: cmd/snap-confine/snap-confine.apparmor.in: update ld rule for s390x impish <⚠ Critical> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10898>