[07:08] <mup> PR snapd#2807 opened: snap: add new `snap switch` command <Created by mvo5> <https://github.com/snapcore/snapd/pull/2807>
[08:17] <mup> PR snapd#3607 opened: cmd: fix re-exec bug when starting from snapd 2.21 <Created by mvo5> <https://github.com/snapcore/snapd/pull/3607>
[08:28] <mup> PR snapd#3608 opened: cmd: rework reexec detection <Created by mvo5> <https://github.com/snapcore/snapd/pull/3608>
[08:35] <Saviq> zyga-ubuntu: the crash: http://pastebin.ubuntu.com/25131529/
[08:46] <zyga-ubuntu> Saviq: thank you
[09:35] <mup> PR snapcraft#1413 closed: core: minimal windows support <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1413>
[10:21] <mup> PR snapd#3609 opened: Introduce the kvm interface <Created by Saviq> <https://github.com/snapcore/snapd/pull/3609>
[10:35] <mup> PR snapd#3610 opened: snap: do not always quote the snap info summary <Created by mvo5> <https://github.com/snapcore/snapd/pull/3610>
[11:06]  * Chipaca in a warren of undoes
[11:08] <mup> PR snapcraft#1417 opened: Move cross-compiling from kernel to kbuild plugin <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1417>
[11:36] <zyga-ubuntu> Chipaca: hey, can you have a look at a small pr for cmd.go please?
[11:36] <zyga-ubuntu> https://github.com/snapcore/snapd/pull/3608
[11:36] <mup> PR snapd#3608: cmd: rework reexec detection <Created by mvo5> <https://github.com/snapcore/snapd/pull/3608>
[11:42] <Chipaca> zyga-ubuntu: sure
[11:43] <Chipaca> zyga-ubuntu: how will this work in places where it's not /snap?
[11:49] <zyga-ubuntu> https://lwn.net/Articles/698073/
[11:49] <zyga-ubuntu> Chipaca: it's not yet, we're aware
[11:49] <zyga-ubuntu> Chipaca: but that's for future
[11:52] <mup> PR snapd#3607 closed: cmd: fix re-exec bug when starting from snapd 2.21 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3607>
[12:00] <Chipaca> pedronis: how're you feeling today, vis-a-vis being asked for reviews?
[12:04] <pedronis> Chipaca: I'm ok
[12:09] <Chipaca> pedronis: when you have a slot, I'd appreciate a look at snapd#3600
[12:09] <mup> PR snapd#3600: many: expose service status in 'snap info' <Created by chipaca> <https://github.com/snapcore/snapd/pull/3600>
[12:11] <Chipaca> meanwhile I'm fixing a bug in snapstate's copydata thing
[12:18] <zyga-ubuntu> lip 20 14:17:55 fyke kernel: audit: type=1400 audit(1500553075.963:253): apparmor="DENIED" operation="capable" profile="/snap/core/2445/usr/lib/snapd/snap-confine" pid=10030 comm="snap-confine" capability=4  capname="fsetid"
[13:12] <mup> PR snapd#3611 opened: overlord/snapstate/backend: some copydata improvements <Created by chipaca> <https://github.com/snapcore/snapd/pull/3611>
[13:22] <pedronis> Chipaca: done
[13:29] <Chipaca> pedronis: thanks!
[13:36] <Chipaca> pedronis: I'll address your comments right now (as i just finished the copydata pr)
[13:36] <Chipaca> "just" == "just now before reading your comments and making me some tea"
[13:42] <mup> Bug #1705486 opened: SPI not working on Raspberry Pi 2 with ubuntu core <Snappy:New> <https://launchpad.net/bugs/1705486>
[13:55] <niemeyer> zyga-ubuntu: github.com/rogpeppe/govers
[13:58] <zyga-ubuntu> thanks!
[13:59] <pedronis> Chipaca: to clarify, I thing you were trying to solve that  client and deamon API are the same, so why two struct, but now you are positing client == daemon == systemd, which I'm not sure about
[14:02] <Chipaca> fair 'nuf
[14:02] <Chipaca> thing is
[14:02] <Chipaca> bah, forward-thinking to the actual services work
[14:02] <Chipaca> not sure this distinction will continue to be as clear, but ok
[14:03] <pedronis> which distinction?
[14:03] <pedronis> about systemd?
[14:03] <pedronis> the package
[14:03] <Chipaca> bah
[14:03] <Chipaca> i'm probably hung up on details of the past implementation
[14:03] <mup> PR snapd#3606 closed: cmd: fix broken double re-executiong from snapd 2.21 <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/3606>
[14:03] <mup> PR snapd#3608 closed: cmd: rework reexec detection <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3608>
[14:04] <Chipaca> pedronis: you're right
[14:04] <Chipaca> there's no reason this needs to be like this
[14:04]  * Chipaca fixes
[14:05] <pedronis> I understand that having two structs is more annoying than 1, but reading client in the signatures of systemd is really strange, a bit unclear if the relationship will hold
[14:05] <pedronis> but I agree we don't want/need 3 structs for this
[14:05] <zyga-ubuntu> niemeyer: we don't have to switch, upstream just merged the fix :)
[14:13] <mup> PR snapd#3612 opened: vendor: update go-flags to address crash in "snap debug" <Created by zyga> <https://github.com/snapcore/snapd/pull/3612>
[14:19] <zyga-ubuntu> Chipaca: FYI https://forum.snapcraft.io/t/snap-remove-doesnt-remove-data-from-root-snap-snap-name/1387
[14:19] <zyga-ubuntu> maybe true, maybe red herring
[14:19] <zyga-ubuntu> maybe red dwarf
[14:28] <mup> PR snapd#3525 closed: interfaces: add password-manager-service implicit classic interface (LP: #1653769) <Created by jdstrand> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3525>
[14:30] <Chipaca> zyga-ubuntu: Dude.
[14:30] <zyga-ubuntu>  yes?
[14:31] <cachio> zyga-ubuntu, I have updated PR 3604 to make seccomp work for opensuse as you requested
[14:31] <cachio> zyga-ubuntu, could you take a look when you have some time?
[14:31] <zyga-ubuntu> yes, looking
[14:33] <cachio> tx
[14:34] <zyga-ubuntu> cachio: commented,
[14:36] <cachio> zyga-ubuntu, i see this error
[14:36] <cachio> https://paste.ubuntu.com/25133113/
[14:37] <cachio> when LDFLAGS contain a static link to seccomp
[14:38] <cachio> zyga-ubuntu, any idea how to deal with it?
[14:42] <mup> PR snapd#3548 closed: interfaces: Add /run/uuid/request to openvswitch <Created by coreycb> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3548>
[14:51] <zyga-ubuntu> cachio: looking
[14:51] <zyga-ubuntu> cachio: are we installing a package that ships libseccomp.a?
[14:51] <Chipaca> zyga-ubuntu: wrt "dude", commented in the forum
[14:51] <zyga-ubuntu> Chipaca: aha, thank you
[14:52] <zyga-ubuntu> :D
[14:52] <zyga-ubuntu> thanks!
[14:52] <zyga-ubuntu> so pawel reported it too :D
[14:52] <zyga-ubuntu> it must have been him then
[14:52]  * zyga-ubuntu has rusty memory
[15:05] <cachio> zyga-ubuntu, not sure but in snap-seccomp/main.go we are linking to it
[15:10] <cachio> in opensuse libseccomp.a is not included in any package that we install
[15:11] <kyrofa> Hey jdstrand, you wanted to ping you about that classic snap I'm playing with
[15:11] <cachio> zyga-ubuntu, but we need to link statically to libseccomp
[15:12] <kyrofa> I'm not awake yet and thus can't type
[15:12] <cachio> zyga-ubuntu, at least there is a test for that
[15:13] <kyrofa> jdstrand, it contains libraries for building its clients, and a tool for running several clients in a specific configuration based on a text file
[15:13] <kyrofa> It looks for those clients in the PATH
[15:14] <kyrofa> And even if the PATH is accessible in confinement, snap-confine strips it off
[15:14] <kyrofa> So if the snap is strictly confined, it can't find anything to orchestrate in e.g. the home dir
[15:14] <kyrofa> Thus classic
[15:16] <cachio> zyga-ubuntu, why we need to link statically to libseccomp in ubuntu?
[15:18] <pedronis> cachio: because of reexec
[15:19] <cachio> pedronis, to make sure both are using the same one?
[15:19] <pedronis> yes (though we don't do that with libudev because there the libudev<->kernel part is the more fragile one, but is not ideal)
[15:19] <cachio> ok
[15:20] <cachio> pedronis, thanks
[15:36] <pedronis> zyga-ubuntu: not only it was reported, but we discussed not to touch it until we have snapshots, at least that was the conclusion in London
[15:45] <kyrofa> jdstrand, moos-kyrofa is now in the review queue
[16:33] <Chipaca> pedronis: fixed snapd#3600 as per your review; thanks!
[16:33] <mup> PR snapd#3600: many: expose service status in 'snap info' <Created by chipaca> <https://github.com/snapcore/snapd/pull/3600>
[16:40] <mup> PR snapd#3596 closed: tests: disable snapd-notify for the external backend <Created by fgimenez> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/3596>
[16:43] <mup> PR snapd#3503 closed: tests: add browser-support interface test <Created by fgimenez> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/3503>
[16:54] <pedronis> Chipaca: generally +1 with a comment about expected in the parser
[16:58] <Chipaca> pedronis: gotcha. Yeah, it's a little convoluted as is
[16:59] <Chipaca> pedronis: by having expected be an array, you mean then doing strutil.ListContains to check for unexpected stuff?
[16:59] <Chipaca> bah, no real need, the switch handles that
[17:00] <Chipaca> hmmm
[20:17] <mup> PR snapd#3600 closed: many: expose service status in 'snap info' <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/3600>