[03:13] <mup> PR snapcraft#1262 opened: replace : with _ in file names <Created by andyli> <https://github.com/snapcore/snapcraft/pull/1262>
[04:18] <mup> Bug #1576500 changed: Plasma fails to load:  "all shell packages missing" <amd64> <apport-bug> <xenial> <Snappy:Expired> <snapd (Ubuntu):Expired> <https://launchpad.net/bugs/1576500>
[06:12] <mup> PR snapd#3195 closed: interfaces/builtin: allow full access to properties iface of the udisks service <Created by morphis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3195>
[06:13] <mup> Bug #1684014 opened: LibreOffice in snap won't print <Snappy:New> <https://launchpad.net/bugs/1684014>
[06:15] <morphis> mvo: thanks for merging!
[06:15] <mvo> morphis: my pleasure, thanks for doing the PR in the first place :)
[06:39] <mup> PR snapd#3048 closed: interfaces: add media-hub interface <Created by alfonsosanchezbeato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3048>
[06:46] <mup> PR snapd#3179 closed: packaging: add `built-using` header for 16.04 packaging <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3179>
[07:24] <zyga> good morning
[07:25] <zyga> mvo: about https://github.com/snapcore/snapd/pull/3198 -- it is an update of existing function to use the new type that was introduced later; it used to take []Entry but now it takes a struct with that inside
[07:25] <mup> PR snapd#3198: interfaces/mount: pass mount.Profile to mount.NeededChanges <Created by zyga> <https://github.com/snapcore/snapd/pull/3198>
[07:25] <zyga> mvo: I should have documented that inside the PR
[07:39] <mup> PR snapd#3198 closed: interfaces/mount: pass mount.Profile to mount.NeededChanges <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3198>
[07:58] <mup> PR snapd#3203 opened: interfaces/mount: add ReadMountInfo and LoadMountInfo <Created by zyga> <https://github.com/snapcore/snapd/pull/3203>
[08:12] <pstolowski> mvo, hey, 3197 has a conflict. it can land once fixed
[08:14] <mup> PR snapd#3204 opened: interfaces/builtin: don't panic if content plug has nil attrs <Created by zyga> <https://github.com/snapcore/snapd/pull/3204>
[08:16] <zyga> pstolowski: something for you https://forum.snapcraft.io/t/snapd-service-doesnt-start/319
[08:16] <mvo> pstolowski: sure, I will push a fix, I also adding a testh
[08:16] <mvo> test
[08:18]  * zyga -> break / errands
[08:19] <mup> Bug #1684041 opened: In core /var/lib/systemd/backlight should be bind mounted to writable <Snappy:New> <https://launchpad.net/bugs/1684041>
[08:24] <davmor2> Someone needs to fix the snapcraft bot on rocketchat
[08:49] <mup> PR snapd#3205 opened: tests: add openvswitch interface spread test <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3205>
[09:01] <ogra_> urgh
[09:01] <ogra_> niemeyer, mup is going crazy in the #snapcraft channel on rocket
[09:02] <ogra_> sending the same bug notification every 2sec
[09:04] <mup> PR snapd#3204 closed: interfaces/builtin: don't panic if content plug has nil attrs <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3204>
[09:05] <niemeyer> ogra_: Stopped and investigating. Thanks for the note.
[09:05] <ogra_> niemeyer, wow, now i'm impressed ... wouldnt have expected you awake!
[09:05] <niemeyer> ogra_: Me neither..
[09:05] <niemeyer> :)
[09:06] <ogra_> lol
[09:06] <niemeyer> ogra_: Looks like RocketChat broke the protocol again
[09:06] <ogra_> ouch
[09:07] <niemeyer> I bet it was just updated..
[09:07] <niemeyer> Yep..
[09:07] <niemeyer> 1 hours, 13 seconds
[09:36] <zyga> mvo: are we releasing 2.24 again?
[09:37] <zyga> we need a 2nd review for https://github.com/snapcore/snapd/pull/3149
[09:37] <mvo> zyga: just a sru fix
[09:37] <zyga> aha
[09:37] <mvo> zyga: the autopkgtests are unhappy currently and the SRU team wants us to do some small tweaks, I'm also preparing a fix for a failure with the stable core snap
[09:38] <mvo> zyga: but this only affects the debs, we could even do it as a distro patch
[09:42] <mvip> \quit
[09:48] <mup> PR snapd#3206 opened: [2.24] Add maintscript helper to remove usr.lib.snapd.snap-confine in snap-confine <Created by mvo5> <https://github.com/snapcore/snapd/pull/3206>
[09:55] <zyga> mvo: thanks for merging the panic fix
[09:55] <zyga> mvo: I was thinking, could we call {plug,slot}.Sanitize and register a panic handler so that we fail sanitization if the method misbehaves
[09:55] <zyga> mvo: that part is naturally exposed to user input
[09:56] <zyga> mvo: and while it cannot harm snapd as go is a safe language (except for the unsafe package we don't use here)
[09:56] <zyga> mvo: malicious input can include denial of service attacks, like this one
[09:59] <mvo> zyga: thanks for fixing the panic so quickly
[09:59] <mvo> zyga: a good question about the handler, probably best to check with gustavo, I think it would be fine
[09:59] <mvo> zyga: depends a bit how clean it will be (codewise) I htink
[10:00] <zyga> mvo: I think that's pretty clean, let me make a quick prototype
[10:01] <mup> PR snapd#3207 opened: tests: relax network-bind interface regexps <Created by mvo5> <https://github.com/snapcore/snapd/pull/3207>
[10:14] <Son_Goku> mvo: if this is ubuntu specific, please don't re-release the tarball
[10:15] <Son_Goku> it's not the end of the world if you need to distro patch temporarily
[10:15] <mvo> Son_Goku: yeah, I think we should not even do a regular release, its strictly fixes for autopkgtests that are pretty specific
[10:16] <Son_Goku> indeed
[10:17] <zyga> hey Son_Goku, how are you doing
[10:18] <Son_Goku> well, I just woke up, so kinda cottonmouthy
[10:50] <zyga> mvo: ok, took longer than expected but is pretty short actually
[10:51] <mup> PR snapd#3208 opened: interfaces: recover panics when sanitizing plugs and slots <Created by zyga> <https://github.com/snapcore/snapd/pull/3208>
[10:52] <zyga> mvo: details on the forum https://forum.snapcraft.io/t/snapd-service-doesnt-start/319/5
[10:52]  * zyga takes a break and goes to eat something
[10:52] <zyga> ttyl
[11:10] <zyga> re
[11:11] <ogra_> zyga, an opinion on bug 1684063 would be appreciated
[11:11] <mup> Bug #1684063: core snap/amd64: dynamic linker unusable in classic confinement when using Zesty <snapd:New> <https://launchpad.net/bugs/1684063>
[11:11] <zyga> ogra_: looking
[11:13] <zyga> ogra_: replied
[11:13] <zyga> ogra_: look at the reply please
[11:14] <ogra_> zyga, seeing it ... hmm, but that means i need to mangle the link ... not great :/
[11:17] <zyga> ogra_: I think there should be no links
[11:18] <zyga> ogra_: the way snapcraft links snaps should give them the realpath of the linker
[11:18] <ogra_> zyga, well, thats a default setup coming from libc
[11:18] <zyga> ogra_: not some symlink
[11:18] <zyga> ogra_: IMO that is irrelevant
[11:18] <zyga> ogra_: if the linker is in /snap/core/current/foo-ld.so
[11:18] <zyga> ogra_: then whatever other symlinks point to it
[11:18] <zyga> ogra_: that is what snapcraft should use
[11:20] <ogra_> zyga, well, the problem is that the core snap uses an absolute link to /
[11:20] <ogra_> or rather libc does
[11:20] <ogra_> so it links outside of the core snap
[11:21] <morphis> Son_Goku: you had success fixing a few few SELinux denials yesterday?
[11:21] <ogra_> the binaires would have to use /snap/core/current/lib/x86_64-linux-gnu/ld-2.23.so ...
[11:22] <ogra_> instead of /snap/core/current/lib64/ld-linux-x86-64.so.2 ... (which is the default everywhere)
[11:22] <ogra_> so you would have to re-build each and every binary
[11:26] <zyga> ogra_: absolute link to / ?
[11:26] <zyga> ogra_: sorry, I don't understand
[11:27] <zyga> ogra_: that is correct
[11:27] <zyga> ogra_: the answer is to rebuild those
[11:27] <zyga> ogra_: that's how it is designed to work
[11:27] <zyga> ogra_: (until we come and re-design it)
[11:27] <ogra_> ugh
[11:27] <ogra_> but that means you can not use any stage-packages at all in classic snaps
[11:27] <ogra_> you would have to build all dependencies from source
[11:28] <zyga> ogra_: yes, until we figure out a better way to have classic confinement working
[11:28] <Son_Goku> morphis: some of the issues are related to the socket not having its label applied correctly
[11:28] <zyga> ogra_: it's a complex problem - there's no silver bullet
[11:28] <Son_Goku> I wonder if systemd can create the socket with the correct label...
[11:28] <zyga> ogra_: what did you mean about symlink mangling?
[11:28] <ogra_> zyga, hmm, could you note that on the big as well ? i think thats unexpected
[11:28] <morphis> Son_Goku: can we fix that somehow?
[11:28] <zyga> Son_Goku: yes, we can
[11:28] <zyga> Son_Goku: you mean selinux label?
[11:28] <Son_Goku> yes
[11:28] <zyga> Son_Goku: sure we can, it's a feature in systemd for ages
[11:29] <ogra_> zyga:
[11:29] <ogra_> ogra@styx:~$ ls -l /snap/core/current/lib64/ld-linux-x86-64.so.2
[11:29] <ogra_> lrwxrwxrwx 1 root root 32 Mär 21 21:06 /snap/core/current/lib64/ld-linux-x86-64.so.2 -> /lib/x86_64-linux-gnu/ld-2.23.so
[11:29] <ogra_> zyga, see where the link points to ?
[11:29] <ogra_> that is my hosts linker
[11:30] <ogra_> zyga, if my host has a newer libc ... i.e. 2.24, that link points nowhere
[11:30] <morphis> zyga: is there a config item for this in the service unit?
[11:30] <zyga> morphis: SELinuxContext=
[11:30] <zyga> morphis: AFAIR
[11:30] <morphis> ah
[11:30] <zyga> ogra_: aha
[11:30] <zyga> ogra_: I see
[11:30] <zyga> ogra_: right,
[11:31] <morphis> Son_Goku: so SELinuxContext= is what we should try
[11:31] <Son_Goku> zyga: doesn't look like it for .socket files?
[11:31] <morphis> Son_Goku: https://www.freedesktop.org/software/systemd/man/systemd.socket.html
[11:31] <Son_Goku> there's SELinuxContextFromNet=
[11:31] <morphis> yeah
[11:32] <morphis> that will use the connection from the daemon to determine the label
[11:32] <Son_Goku> ah, that might work
[11:32] <morphis> " When true, systemd will attempt to figure out the SELinux label used for the instantiated service from the information handed by the peer over the network."
[11:32] <morphis> so we need to touch snapd.service and snapd.socket
[11:32] <Son_Goku> snapd itself is already properly labeled, I think
[11:33] <Son_Goku> the issue is that the socket doesn't exist at the point of the policy getting applied
[11:33] <zyga> ogra_: unless sergiusens disagrees I don't think we support prebuilt binaries yet
[11:33] <Son_Goku> so I think snapd.socket is the only thing that needs to be changed
[11:33] <ogra_> zyga, you mean no stage-packages in classic ? thats news to me
[11:34] <ogra_> (but that would indeed solve the issue)
[11:36] <kotVaska> hallo, spricht hier jemand deutsch?
[11:37] <zyga> ogra_: yes that's what I mean
[11:37] <morphis> Son_Goku: ok, so who labels snapd if not through the .service file?
[11:37] <zyga> ogra_: how did you expect stage-packages to work/
[11:37] <morphis> zyga: https://github.com/snapcore/snapd/pull/3156 gets really close :-)
[11:37] <mup> PR snapd#3156: Support debian in our CI <Created by morphis> <https://github.com/snapcore/snapd/pull/3156>
[11:37] <zyga> morphis: let me see :)
[11:37] <Son_Goku> morphis: the policy, when it is applied, labels /usr/libexec/snapd as snappy_exec_t
[11:37] <ogra_> zyga, dunno, i never built a classic snap ever :P
[11:37] <Son_Goku> that happens during the package install
[11:38]  * ogra_ thinks thats an insanity anyway 
[11:38] <zyga> morphis: I have a fedora system that I plan to use for daily development in the next 2-3 weeks; I need to buy some extra hardware and then I can switch
[11:38] <Son_Goku> or at least, it should...
[11:38] <morphis> zyga: two tests are failing which I need to look into but all others should be ok now
[11:38] <morphis> zyga: nice :-)
[11:38] <morphis> Son_Goku: ah I see
[11:38]  * Son_Goku needs to set up a clean box and test something
[11:39] <Son_Goku> something about these denials is bugging me, because it looks like at least a good chunk of them shouldn't be issues anymore...
[11:39] <zyga> morphis: commented quickly
[11:39] <zyga> morphis: I'll comment again after reading it fully
[11:39] <kotVaska> please help my. i have follow error with "sudo snap install nextcloud"-command -> https://pastebin.com/DsherkXt
[11:39] <zyga> kotVaska: hey, you may have better luck on forum.snapcraft.io
[11:40] <zyga> kotVaska: I'll gladly help you there
[11:40] <zyga> kotVaska: because others can benefit from this
[11:40] <morphis> zyga: thanks
[11:40] <zyga> kotVaska: please open a new thread under the "snap" category
[11:40] <zyga> thanks!
[11:42] <kotVaska> zyga: I don't want to register there.
[11:43]  * zyga though we already had various social network login available
[11:44] <Son_Goku> we still don't
[11:44] <zyga> kotVaska: can you please run `snap version`
[11:46]  * ogra_ bets the kernel is a mess ... the "vps" in the hostname kind of indicates that :)
[11:46] <kotVaska> zyga: 2.22.6
[11:46] <ogra_> kotVaska, the full output please
[11:46] <kotVaska> snap    2.22.6
[11:46] <kotVaska> snapd   2.22.6
[11:46] <kotVaska> series  16
[11:46] <kotVaska> ubuntu  16.04
[11:46] <kotVaska> kernel  2.6.32-042stab120.7
[11:46] <ogra_> (should be 5 lines)
[11:46] <ogra_> lol
[11:46] <Son_Goku> oh god
[11:46] <ogra_> 2.6.32-042stab120.7
[11:46] <Son_Goku> stab is right
[11:46] <ogra_> kotVaska, that cant work
[11:46] <Son_Goku> it's an OpenVZ system
[11:46] <kotVaska> yes
[11:47] <ogra_> kotVaska, talk to your vps provider to use a proper kernel
[11:47] <Son_Goku> you can't run snaps on OpenVZ
[11:47] <zyga> kotVaska: snapd won't work on that kernel
[11:47] <zyga> kotVaska: your kernel is too old, I'm sorry
[11:47] <Son_Goku> they need to upgrade to Virtuozzo 7
[11:47] <ogra_> this is like decades outdated
[11:47] <Son_Goku> if they do that, snaps will work again
[11:47] <kotVaska> i have a vServer
[11:47]  * zyga recalls his first job that had a 2.6 kernel ;)
[11:47] <Son_Goku> kotVaska: your hosting provider needs to upgrade to Virtuozzo 7: https://virtuozzo.com/products/virtuozzo/
[11:48] <mup> PR snapcraft#1263 opened: Pass through commands into the lxd container <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1263>
[11:51] <kotVaska> my provider have "LXC - Linux vServer", " Qemu-KVM vServer" and " OpenVZ vServer"
[11:51] <kotVaska> i rent openvz vserver
[11:52] <morphis> zyga: where did you still see is-forced-devmode on the PR?
[11:52] <Son_Goku> Qemu-KVM will work, as you'll use the distro's kernel
[11:52] <morphis> that should be gone
[11:52] <zyga> morphis: hmm,
[11:52] <Son_Goku> and LXC may work, if the host is modern
[11:52] <zyga> morphis: not sure what you mean?
[11:52] <zyga> morphis: I commented on the blacklisting of tests on Debian
[11:52] <zyga> morphis: that those had no comments explaining why
[11:52] <morphis> got a mail with a comment "I suspect this part doesn't respect the rules for command help and long description. Have a look at other commands and what they do."
[11:52] <zyga> morphis: ah
[11:52] <zyga> morphis: that's ancient :)
[11:52] <zyga> morphis: it was for a command you wanted to add
[11:53] <zyga> morphis: to check if a distro is in forced devmode
[11:53] <morphis> yeah, I will do that in a second step
[11:53] <zyga> morphis: we (very long time ago) defined some rules of how help for commands must look like
[11:53] <zyga> morphis: including specific wording
[11:53] <zyga> morphis: see all the commands, they have consistent help wording
[11:53] <morphis> ok, any link to that?
[11:53] <zyga> morphis: no, it was before we had anything like a forum/public ML
[11:53] <zyga> morphis: it was in a private email thread
[11:54] <zyga> morphis: just use your imagination, compare to how other commands have help worded
[11:54] <ogra_> kotVaska, you sould re-consider that ... this kernel is definitely something very hacked up ... running ubuntu 16.04 on a 2.x kernel was never officially possible .... would be surprising if there were not also other issues that do not surface as easily as this one (not to forget there might be horrid security holes)
[11:54] <morphis> zyga: worth to bring that to the forum or any other documentation place now :-)
[11:57] <zyga> morphis: yeah
[11:57] <zyga> morphis: if you ask the question I'll go through my archive and try to find it
[11:57] <sergiusens> ogra_: zyga we don't support doing the right thing with runnables from prebuilts when on classic confinement, that is correct
[11:57] <kotVaska> thanks
[11:58] <Son_Goku> ogra_: it's not hacked up
[11:58] <Son_Goku> it's a container
[11:58] <Son_Goku> don't be misleading
[11:58] <ogra_> Son_Goku, dude ... to make 16.04 run on top of a 2.x kernel you will have to massively patch it
[11:58] <Son_Goku> unless the 2.6 kernel itself has been patched and backported
[11:59] <morphis> zyga: thanks
[11:59] <Son_Goku> which is exactly how that is even working
[11:59] <ogra_> no matter if it is a container or raw HW
[12:00] <ogra_> were there even things like multiple pty's possible in 2.x ? i dont think it was ... there is a ton of corner cases that will affect the behaviour of the userspace here
[12:00] <Son_Goku> the OpenVZ kernel used supports multiple ptys
[12:00] <ogra_> this can not legally be called ubuntu
[12:00] <Son_Goku> then the same goes for all your docker images
[12:01] <Son_Goku> if I load a docker image into an OpenVZ system, I didn't touch Ubuntu
[12:01] <ogra_> depends if they are blessed by canonical or not ... but i doubt there are doker installs using 2.x anywhere
[12:01] <Son_Goku> a docker image doesn't have to be used by docker
[12:01] <Son_Goku> rkt, nspawn, lxc, openvz, etc. can all consume them
[12:02] <ogra_> sure,. i can also consume am ubuntu-base tarball o top of a monolithic 1.x kernel and still cant seel it as ubuntu then
[12:02] <ogra_> *sell
[12:03] <ogra_> the kernel is a part of the brand
[12:04] <ogra_> (you cant sell RH CDs with a replaced kernel either and call it RH)
[12:16] <zyga> another part of update-ns for review: https://github.com/snapcore/snapd/pull/3209
[12:16] <mup> PR snapd#3209: interfaces/mount: add partial implementation of Change.Needed <Created by zyga> <https://github.com/snapcore/snapd/pull/3209>
[12:16] <mup> PR snapd#3209 opened: interfaces/mount: add partial implementation of Change.Needed <Created by zyga> <https://github.com/snapcore/snapd/pull/3209>
[12:16] <zyga> I'll switch to code reviews now and work on chipaca's tab-completion PR now
[12:17] <zyga> ogra_, morphis: ^^
[12:17] <zyga> low level, feedback appreciated
[12:21] <ogra_> zyga, why do you nered all the options for umount  ? if it is mounted the mountpoint should be enough for unmounting
[12:21] <ogra_> *need
[12:25] <zyga> ogra_: interesting point; I guess we don't want to unmount something _else_
[12:25] <zyga> ogra_: but not sure if that captures it
[12:26] <ogra_> well, that would only affect things that are double-mounted on the same mountpoint (which you likely dont want top happen in the first place)
[12:26] <ogra_> *to
[12:28] <zyga> morphis: did you cancel the call just now?
[12:28] <zyga> ogra_: yes, I think you are right
[12:28] <ogra_> beyond that i think it looks fine
[12:28] <zyga> ogra_: I think the devil is in mountpoint details though (like bind mounts making stuff more complex)
[12:29] <ogra_> i also wonder if you carry over the mount options for filesystems from the original mount when bind mounting ...
[12:29] <zyga> ogra_: yes, those are unchanged
[12:29] <zyga> ogra_: well, there are two sets as you see
[12:29] <ogra_> good
[12:29] <zyga> ogra_: superblock and per-mount
[12:30] <ogra_> right, i'm rather cobncerned about "noatime,nodiratime,data=ordered" etc ... stuff that makes sure we dont wear out SSDs or eMMCs
[12:31] <ogra_> but i guess that is carried forward anyway ... or rather physically used from the original mount
[12:33] <zyga> ogra_: I think that right now it doesn't matter
[12:33] <zyga> ogra_: we only do bind mounts
[12:33] <zyga> ogra_: and the code as written is only a stab at the generic functionality
[12:33] <ogra_> yeah
[12:54] <mup> PR snapd#3210 opened:  tests: relax network-bind interface regexps for 2.24 <Created by mvo5> <https://github.com/snapcore/snapd/pull/3210>
[13:21] <mup> PR snapcraft#1255 closed: Update rust plugin to set RUSTFLAGS <Created by ChrisMacNaughton> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1255>
[13:24] <mup> PR snapcraft#1250 closed: Fixed links to doc <Created by andyli> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1250>
[13:27] <mup> PR snapcraft#1251 closed: cli: remove empty lines in the unclean parts message <Created by EduardoVega> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1251>
[13:46] <NicolinoCuralli> hi all: it seem that there is a misalignement between  https://snapcraft.io/docs/reference/interfaces and the result of snap interfaces on  a snap core 1580; ex network-manager is on wiki page but not in snap interfaces output
[13:47] <ogra_> NicolinoCuralli, well, depends on the context
[13:47] <NicolinoCuralli> naturally snap connect don0t work for that interface
[13:47] <ogra_> NicolinoCuralli, on a classic install where network-manager is pre-installed for your desktoop, the interface is provided by this ...
[13:48] <NicolinoCuralli> ogra_ : i run in a strict mode
[13:48] <ogra_> on a core image on some developer board there is no network-manager, thus there is no interface for it until you install the network-manager snap
[13:48] <ogra_> so the interface is context dependant
[13:49] <NicolinoCuralli> ogra_ : i understood , thanks
[13:49] <ogra_> :)
[13:50] <davidcalle> NicolinoCuralli: although, you can install the network-manager snap https://docs.ubuntu.com/core/en/stacks/network/network-manager/docs/installation
[13:50] <ogra_> indeed
[14:07] <donofrio> anyone know how to force a refresh that actually loads snappy reposities?  I tried but I just get this https://apaste.info/8eil
[14:09] <ogra_> donofrio, that is not how snaps work
[14:09] <ogra_> there are no local ppackage lists you need to refresh like you have in apt
[14:09] <ogra_> (also ubuntu-core is obsolete ... the snap is only called "core" nowadays)
[14:10] <donofrio> ogra_, how do I use this good project then....when I try "snap install --classic anbox-installer" from https://github.com/anbox/anbox/blob/master/README.md#installation I just get no snap found ;(
[14:10] <ogra_> donofrio, thats a normal ubuntu install ?
[14:11] <donofrio> uh yah
[14:11] <ogra_> (i dont think classic snaps work anywhere else currently)
[14:11] <ogra_> whats the output of "snap version" (shoudl return 5 lines)
[14:16] <ogra_> donofrio, oh, also ... nevver use "sudo su -" it will unset a lot of environment ... use sudo -s or sudo -i and not su
[14:16] <ogra_> (snapd definitely uses some env that would be dropped by "su -" )
[14:19] <ogra_> (but thats a general rule not snap specific ... su - is always bad unless you want to be in a new env)
[14:21] <ogra_> niemeyer, since when does an empty "snap find" actually return one/two packages ? thats new ...
[14:21]  * ogra_ gets lxd, docker, mongo32 and rocketchat-server as returns here 
[14:22] <ogra_> i wonder if something regressed in 2.24 in that regard
[14:22] <niemeyer> ogra_: I think the idea was always to have a stock list of interesting snaps there.. if they're too few, probably something needs fixing
[14:22] <niemeyer> ogra_: Can you open a topic on forum under "store"?
[14:22] <kyrofa> ogra_, do you know if ubuntu core 16 has an image in AWS?
[14:23] <ogra_> kyrofa, i dont actually ... but i dont see why the normal x86 image wouldnt work ...
[14:24] <kyrofa> Yeah, I just see blog posts from 2014 saying "hey, it's on AWS!" like it was hard, so...
[14:24]  * kyrofa actually goes to check for himself
[14:25] <ogra_> niemeyer, https://forum.snapcraft.io/t/empty-snap-find-suddenly-returns-four-snaps-but-not-more/323/1
[14:25] <ogra_> kyrofa, it was hard back then ... it needed some special packages added ... but that need was dropped
[14:26] <ogra_> (we used to actually build special AWS images in 15.04 snappy ...)
[14:26] <kyrofa> ogra_, yeah, the only results I'm seeing are in the community AMIs, and that's only 15.04
[14:28] <kyrofa> niemeyer, do you know anything about ubuntu core on AWS?
[14:28] <ogra_> kyrofa, rharper does perhaps ... being in the cloud team
[14:28] <niemeyer> kyrofa: Not much to be honest
[14:29] <kyrofa> niemeyer, trying to figure out how to test a CUDA setup without buying an expensive video card :P
[14:29] <ogra_> all i know was that i was asked to drop the AWS hacks before 16 because it would "just work" now
[14:29] <kyrofa> ogra_, good deal, thank you
[14:33] <niemeyer> ogra_: Thanks for the topic
[14:35] <sergiusens> jdstrand: tyhicks can I get your opinion on https://forum.snapcraft.io/t/asset-recording-for-a-built-snap/317?u=sergiusens
[14:38] <tyhicks> sergiusens: nice! I'll need a little bit of time to think it over
[14:38] <sergiusens> tyhicks: no rush
[14:39] <mup> PR snapcraft#1260 closed: meta: version from git support <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1260>
[14:48] <jdstrand> sergiusens: I saw it and like tyhicks, plan to think about it and comment
[14:48] <mup> PR snapcraft#1264 opened: tests: do not print coverage check message <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1264>
[14:52] <morphis> zyga: can you have a quick look at https://paste.ubuntu.com/24414168/ ; not an expert yet in reading those expect scripts
[14:53] <morphis> zyga: look at the bottom
[14:54] <zyga> morphis: looking
[14:55] <zyga> morphis: those expect-based tests are very unreliable
[14:55] <zyga> morphis: just run it a few times, if it fails still it's interesting
[14:55] <morphis> they reliable fail on debian :-)
[14:55] <zyga> morphis: in that case run those manually in a debug shell
[14:55] <zyga> morphis: and see what you get
[14:55] <zyga> morphis: maybe PS1 discrepancy or something
[14:56] <morphis> hm, that is a good idea
[14:57]  * zyga goes for a walk, ttyl
[14:59] <morphis> zyga: that's it
[14:59] <zyga> morphis: aha, PS* differences?
[14:59] <morphis> $ bash --norc -i
[15:00] <morphis> that gives different prompts on debian vs. ubuntu
[15:01]  * zyga completed backup
[15:01] <zyga> ttyl :)
[15:12] <mup> PR snapcraft#1203 closed: Add some initial checks to intelligently build the initial snapcraft.yaml <Created by mhall119> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1203>
[15:24] <mup> PR snapd#3211 opened: assertions: add "repair" assertion  <Created by mvo5> <https://github.com/snapcore/snapd/pull/3211>
[15:24] <Pharaoh_Atem> morphis: please open a review for snapd-xdg-open
[15:24] <Pharaoh_Atem> at this point, I don't care anymore and I want this in Fedora now
[15:41] <Pharaoh_Atem> zyga: have you had a chance to respond in the email thread started by mattdm?
[15:52] <zyga> Pharaoh_Atem: no, you are right, I should do it now
[15:59] <Facu> sergiusens, anybody, my snapcraft branch in travis failed with:
[15:59] <Facu> The command "if [ ! -z $CHECK_CLA ] && [ "$TRAVIS_PULL_REQUEST" != "false" ] && [ "$TRAVIS_EVENT_TYPE" != 'cron' ]; then docker exec -i builder ./tools/cla_check_in_travis.sh; fi" exited with 1.
[15:59] <Facu> do you have any idea about what is it? why a CLA check may have failed? it's not my first branch for snapcraft
[15:59] <sergiusens> Facu: because you authored the commit with an email that has not signed the CLA or is not @canonical.com
[16:01] <Facu> sergiusens, mmm, that may be it (I'm not @canonical.com in github), but it would suprise me how the other branch got in
[16:01]  * Facu checks
[16:01] <sergiusens> we started enforcing the check recently to avoid these oversights
[16:02] <sergiusens> Facu: looks like you used gmail in this last one and "Author: Facundo Batista <facundo@taniquetil.com.ar>" in one that passed
[16:03] <Facu> gmail????
[16:04] <sergiusens> hmm, I don't know what is going on, which is why I assigned to elopio
[16:04] <Facu> sergiusens, "git log" in my computer says the Author is facundo@taniquetil.com.ar (expected)
[16:05] <Facu> sergiusens, where are you finding the gmail mail address?
[16:05] <sergiusens> I am not... anymore ;-)
[16:05] <elopio> Facu: hello.
[16:09] <elopio> Facu: I think that the error will go away after rebase with master. I will hit the update button in your pr.
[16:10] <Facu> :o
[16:11] <Facu> elopio, thanks
[16:12] <Facu> here https://launchpad.net/~facundo it says I signed the contributor agreement on April 3rd
[16:12] <renat> Hi, guys! It's Renat from Screenly.
[16:13] <mup> PR snapcraft#1265 opened: shell_utils: code cleanup <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1265>
[16:14] <zyga> Pharaoh_Atem: replied
[16:16] <renat> I have a question related to the warning message from the store
[16:16] <renat> (NEEDS REVIEW) 'daemon' should not be used with 'browser-support' security-snap-v2_daemon_with_browser-support (viewer)
[16:17] <renat> Is it possible to make the store accept a daemon service with a browser support plug?
[16:17] <renat> To make it automatically.
[16:17] <zyga> renat: most likely no, why do you need a daemon to have browser support permissions?
[16:17]  * ogra_ guesses thats a jdstrand question ... there are surely reasons to not allow browser daemons 
[16:18] <zyga> renat: browser-support is a special interface that we really only expect to hand out to specific browsers
[16:18] <zyga> renat: and nobody else
[16:18] <zyga> renat: as it is exceptionally open to support various things modern browsers do
[16:18] <ogra_> zyga, well, there might be browsers in kiosk mode ...
[16:18] <ogra_> which i assume this boils down to
[16:18] <zyga> ogra_: aha
[16:18] <zyga> ogra_: daemon mode browser is an interesting case
[16:18] <ogra_> yep
[16:19] <zyga> still feel like a hack
[16:19] <ogra_> but i guess the seutrity team had reasons to not allow this combo yet
[16:19] <zyga> because runs as root
[16:19] <zyga> yes, it's super powerful and open
[16:19] <ogra_> *security
[16:19] <renat> zyga, ogra_, we use it for digital signage.
[16:19] <renat> And it's really running in a daemon mode.
[16:20] <zyga> renat: please open a thread on the forum, I will reply tere
[16:20] <zyga> there*
[16:20] <pstolowski> it would be great to have second review on 3159 and land it
[16:21] <renat> zyga, forum? You mean - write to the mailing list?
[16:21] <zyga> renat: see forum.snapcraft.io :-)
[16:21] <zyga> pstolowski: done
[16:21] <renat> Wow. Okay.
[16:21] <mup> PR snapd#3159 closed: store: retry once on hashsum mismatches in a Download() <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3159>
[16:21] <pstolowski> zyga, cool, thanks
[16:22] <zyga> pstolowski: I'd love to land https://github.com/snapcore/snapd/pull/3203
[16:22] <mup> PR snapd#3203: interfaces/mount: add ReadMountInfo and LoadMountInfo <Created by zyga> <https://github.com/snapcore/snapd/pull/3203>
[16:26] <zyga> mvo: I added some comments on https://github.com/snapcore/snapd/pull/3211/
[16:26] <mup> PR snapd#3211: assertions: add "repair" assertion  <Created by mvo5> <https://github.com/snapcore/snapd/pull/3211>
[16:26] <mup> PR snapd#3212 opened: api, ifacemgr: resolve disconnect early <Created by stolowski> <https://github.com/snapcore/snapd/pull/3212>
[16:27] <zyga> mvo: quick question, aren't models specific to a brand?
[16:27] <zyga> mvo: so would we need a list of pairs there or would a single repair be associated with the publisher and somehow implying which brand is affected?
[16:28] <mup> PR snapcraft#1266 opened: tests: report coverage only in unit tests <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1266>
[16:28] <renat> Done: https://forum.snapcraft.io/t/suppress-the-security-snap-v2-daemon-with-browser-support-warning-for-the-snap/325
[16:28] <zyga> renat: thanks!
[16:32]  * zyga -> break
[16:33] <pstolowski> zyga, +1
[16:33]  * pstolowski eod
[16:33] <pstolowski> o/
[16:46] <elopio> Facu: yes, it's green now. So, we had a commit from a private github email and thus we couldn't check CLA there. If you have an outdated PR, travis will add on top all the commits that you are missing from master, and then run the tests.
[16:46] <elopio> if you sync with master, it will just pull your commits, and not check the CLA on things that were already merged.
[16:47] <Facu> elopio, thanks
[17:34] <jdstrand> renatu (cc ogra_ and zyga): I responded to the forum post
[17:37] <mup> PR snapcraft#1264 closed: tests: do not print coverage check message <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1264>
[18:19] <mup> PR snapcraft#1265 closed: shell_utils: code cleanup <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1265>
[20:37] <mup> PR snapcraft#1267 opened: meta: override the version with version-script <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1267>
[22:55] <mup> PR snapcraft#1195 closed: [experimental] run unit tests in osx <Created by elopio> <Closed by elopio> <https://github.com/snapcore/snapcraft/pull/1195>