=== chihchun_afk is now known as chihchun | ||
=== chihchun is now known as chihchun_afk | ||
=== chihchun_afk is now known as chihchun | ||
mup | PR snapd#2807 opened: snap: add new `snap switch` command <Created by mvo5> <https://github.com/snapcore/snapd/pull/2807> | 07:08 |
---|---|---|
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:17 |
mup | PR snapd#3608 opened: cmd: rework reexec detection <Created by mvo5> <https://github.com/snapcore/snapd/pull/3608> | 08:28 |
Saviq | zyga-ubuntu: the crash: http://pastebin.ubuntu.com/25131529/ | 08:35 |
zyga-ubuntu | Saviq: thank you | 08:46 |
mup | PR snapcraft#1413 closed: core: minimal windows support <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1413> | 09:35 |
=== jospoortvliet_ is now known as jospoortvliet | ||
mup | PR snapd#3609 opened: Introduce the kvm interface <Created by Saviq> <https://github.com/snapcore/snapd/pull/3609> | 10:21 |
mup | PR snapd#3610 opened: snap: do not always quote the snap info summary <Created by mvo5> <https://github.com/snapcore/snapd/pull/3610> | 10:35 |
* Chipaca in a warren of undoes | 11:06 | |
mup | PR snapcraft#1417 opened: Move cross-compiling from kernel to kbuild plugin <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1417> | 11:08 |
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:36 |
=== chihchun is now known as chihchun_afk | ||
Chipaca | zyga-ubuntu: sure | 11:42 |
Chipaca | zyga-ubuntu: how will this work in places where it's not /snap? | 11:43 |
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:49 |
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> | 11:52 |
Chipaca | pedronis: how're you feeling today, vis-a-vis being asked for reviews? | 12:00 |
pedronis | Chipaca: I'm ok | 12:04 |
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:09 |
Chipaca | meanwhile I'm fixing a bug in snapstate's copydata thing | 12:11 |
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" | 12:18 |
mup | PR snapd#3611 opened: overlord/snapstate/backend: some copydata improvements <Created by chipaca> <https://github.com/snapcore/snapd/pull/3611> | 13:12 |
pedronis | Chipaca: done | 13:22 |
Chipaca | pedronis: thanks! | 13:29 |
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:36 |
mup | Bug #1705486 opened: SPI not working on Raspberry Pi 2 with ubuntu core <Snappy:New> <https://launchpad.net/bugs/1705486> | 13:42 |
niemeyer | zyga-ubuntu: github.com/rogpeppe/govers | 13:55 |
zyga-ubuntu | thanks! | 13:58 |
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 | 13:59 |
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:02 |
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:03 |
Chipaca | pedronis: you're right | 14:04 |
Chipaca | there's no reason this needs to be like this | 14:04 |
* Chipaca fixes | 14:04 | |
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:05 |
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:13 |
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:19 |
=== chihchun_afk is now known as chihchun | ||
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:28 |
Chipaca | zyga-ubuntu: Dude. | 14:30 |
zyga-ubuntu | yes? | 14:30 |
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:31 |
cachio | tx | 14:33 |
zyga-ubuntu | cachio: commented, | 14:34 |
=== chihchun is now known as chihchun_afk | ||
cachio | zyga-ubuntu, i see this error | 14:36 |
cachio | https://paste.ubuntu.com/25133113/ | 14:36 |
cachio | when LDFLAGS contain a static link to seccomp | 14:37 |
cachio | zyga-ubuntu, any idea how to deal with it? | 14:38 |
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:42 |
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:51 |
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 | 14:52 | |
cachio | zyga-ubuntu, not sure but in snap-seccomp/main.go we are linking to it | 15:05 |
cachio | in opensuse libseccomp.a is not included in any package that we install | 15:10 |
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:11 |
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:12 |
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:13 |
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:14 |
cachio | zyga-ubuntu, why we need to link statically to libseccomp in ubuntu? | 15:16 |
pedronis | cachio: because of reexec | 15:18 |
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:19 |
cachio | pedronis, thanks | 15:20 |
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:36 |
kyrofa | jdstrand, moos-kyrofa is now in the review queue | 15:45 |
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:33 |
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:40 |
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:43 |
pedronis | Chipaca: generally +1 with a comment about expected in the parser | 16:54 |
Chipaca | pedronis: gotcha. Yeah, it's a little convoluted as is | 16:58 |
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 | 16:59 |
Chipaca | hmmm | 17:00 |
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> | 20:17 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!