/srv/irclogs.ubuntu.com/2017/04/19/#snappy.txt

=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
mupPR snapcraft#1262 opened: replace : with _ in file names <Created by andyli> <https://github.com/snapcore/snapcraft/pull/1262>03:13
=== chihchun_afk is now known as chihchun
=== ahoneybun is now known as Guest36814
=== chihchun is now known as chihchun_afk
mupBug #1576500 changed: Plasma fails to load:  "all shell packages missing" <amd64> <apport-bug> <xenial> <Snappy:Expired> <snapd (Ubuntu):Expired> <https://launchpad.net/bugs/1576500>04:18
mupPR 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:12
mupBug #1684014 opened: LibreOffice in snap won't print <Snappy:New> <https://launchpad.net/bugs/1684014>06:13
morphismvo: thanks for merging!06:15
mvomorphis: my pleasure, thanks for doing the PR in the first place :)06:15
mupPR snapd#3048 closed: interfaces: add media-hub interface <Created by alfonsosanchezbeato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3048>06:39
mupPR 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>06:46
zygagood morning07:24
zygamvo: 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 inside07:25
mupPR snapd#3198: interfaces/mount: pass mount.Profile to mount.NeededChanges <Created by zyga> <https://github.com/snapcore/snapd/pull/3198>07:25
zygamvo: I should have documented that inside the PR07:25
mupPR 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:39
mupPR snapd#3203 opened: interfaces/mount: add ReadMountInfo and LoadMountInfo <Created by zyga> <https://github.com/snapcore/snapd/pull/3203>07:58
pstolowskimvo, hey, 3197 has a conflict. it can land once fixed08:12
mupPR 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:14
zygapstolowski: something for you https://forum.snapcraft.io/t/snapd-service-doesnt-start/31908:16
mvopstolowski: sure, I will push a fix, I also adding a testh08:16
mvotest08:16
* zyga -> break / errands08:18
mupBug #1684041 opened: In core /var/lib/systemd/backlight should be bind mounted to writable <Snappy:New> <https://launchpad.net/bugs/1684041>08:19
davmor2Someone needs to fix the snapcraft bot on rocketchat08:24
mupPR snapd#3205 opened: tests: add openvswitch interface spread test <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3205>08:49
ogra_urgh09:01
ogra_niemeyer, mup is going crazy in the #snapcraft channel on rocket09:01
ogra_sending the same bug notification every 2sec09:02
mupPR 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:04
niemeyerogra_: Stopped and investigating. Thanks for the note.09:05
ogra_niemeyer, wow, now i'm impressed ... wouldnt have expected you awake!09:05
niemeyerogra_: Me neither..09:05
niemeyer:)09:05
ogra_lol09:06
niemeyerogra_: Looks like RocketChat broke the protocol again09:06
ogra_ouch09:06
niemeyerI bet it was just updated..09:07
niemeyerYep..09:07
niemeyer1 hours, 13 seconds09:07
zygamvo: are we releasing 2.24 again?09:36
zygawe need a 2nd review for https://github.com/snapcore/snapd/pull/314909:37
mvozyga: just a sru fix09:37
zygaaha09:37
mvozyga: 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 snap09:37
mvozyga: but this only affects the debs, we could even do it as a distro patch09:38
mvip\quit09:42
mupPR 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:48
zygamvo: thanks for merging the panic fix09:55
zygamvo: I was thinking, could we call {plug,slot}.Sanitize and register a panic handler so that we fail sanitization if the method misbehaves09:55
zygamvo: that part is naturally exposed to user input09:55
zygamvo: and while it cannot harm snapd as go is a safe language (except for the unsafe package we don't use here)09:56
zygamvo: malicious input can include denial of service attacks, like this one09:56
mvozyga: thanks for fixing the panic so quickly09:59
mvozyga: a good question about the handler, probably best to check with gustavo, I think it would be fine09:59
mvozyga: depends a bit how clean it will be (codewise) I htink09:59
zygamvo: I think that's pretty clean, let me make a quick prototype10:00
mupPR snapd#3207 opened: tests: relax network-bind interface regexps <Created by mvo5> <https://github.com/snapcore/snapd/pull/3207>10:01
Son_Gokumvo: if this is ubuntu specific, please don't re-release the tarball10:14
Son_Gokuit's not the end of the world if you need to distro patch temporarily10:15
mvoSon_Goku: yeah, I think we should not even do a regular release, its strictly fixes for autopkgtests that are pretty specific10:15
Son_Gokuindeed10:16
zygahey Son_Goku, how are you doing10:17
Son_Gokuwell, I just woke up, so kinda cottonmouthy10:18
zygamvo: ok, took longer than expected but is pretty short actually10:50
mupPR snapd#3208 opened: interfaces: recover panics when sanitizing plugs and slots <Created by zyga> <https://github.com/snapcore/snapd/pull/3208>10:51
zygamvo: details on the forum https://forum.snapcraft.io/t/snapd-service-doesnt-start/319/510:52
* zyga takes a break and goes to eat something10:52
zygattyl10:52
zygare11:10
ogra_zyga, an opinion on bug 1684063 would be appreciated11:11
mupBug #1684063: core snap/amd64: dynamic linker unusable in classic confinement when using Zesty <snapd:New> <https://launchpad.net/bugs/1684063>11:11
zygaogra_: looking11:11
zygaogra_: replied11:13
zygaogra_: look at the reply please11:13
ogra_zyga, seeing it ... hmm, but that means i need to mangle the link ... not great :/11:14
zygaogra_: I think there should be no links11:17
zygaogra_: the way snapcraft links snaps should give them the realpath of the linker11:18
ogra_zyga, well, thats a default setup coming from libc11:18
zygaogra_: not some symlink11:18
zygaogra_: IMO that is irrelevant11:18
zygaogra_: if the linker is in /snap/core/current/foo-ld.so11:18
zygaogra_: then whatever other symlinks point to it11:18
zygaogra_: that is what snapcraft should use11:18
ogra_zyga, well, the problem is that the core snap uses an absolute link to /11:20
ogra_or rather libc does11:20
ogra_so it links outside of the core snap11:20
morphisSon_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:21
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 binary11:22
zygaogra_: absolute link to / ?11:26
zygaogra_: sorry, I don't understand11:26
zygaogra_: that is correct11:27
zygaogra_: the answer is to rebuild those11:27
zygaogra_: that's how it is designed to work11:27
zygaogra_: (until we come and re-design it)11:27
ogra_ugh11:27
ogra_but that means you can not use any stage-packages at all in classic snaps11:27
ogra_you would have to build all dependencies from source11:27
zygaogra_: yes, until we figure out a better way to have classic confinement working11:28
Son_Gokumorphis: some of the issues are related to the socket not having its label applied correctly11:28
zygaogra_: it's a complex problem - there's no silver bullet11:28
Son_GokuI wonder if systemd can create the socket with the correct label...11:28
zygaogra_: what did you mean about symlink mangling?11:28
ogra_zyga, hmm, could you note that on the big as well ? i think thats unexpected11:28
morphisSon_Goku: can we fix that somehow?11:28
zygaSon_Goku: yes, we can11:28
zygaSon_Goku: you mean selinux label?11:28
Son_Gokuyes11:28
zygaSon_Goku: sure we can, it's a feature in systemd for ages11:28
ogra_zyga:11:29
ogra_ogra@styx:~$ ls -l /snap/core/current/lib64/ld-linux-x86-64.so.211: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.so11:29
ogra_zyga, see where the link points to ?11:29
ogra_that is my hosts linker11:29
ogra_zyga, if my host has a newer libc ... i.e. 2.24, that link points nowhere11:30
morphiszyga: is there a config item for this in the service unit?11:30
zygamorphis: SELinuxContext=11:30
zygamorphis: AFAIR11:30
morphisah11:30
zygaogra_: aha11:30
zygaogra_: I see11:30
zygaogra_: right,11:30
morphisSon_Goku: so SELinuxContext= is what we should try11:31
Son_Gokuzyga: doesn't look like it for .socket files?11:31
morphisSon_Goku: https://www.freedesktop.org/software/systemd/man/systemd.socket.html11:31
Son_Gokuthere's SELinuxContextFromNet=11:31
morphisyeah11:31
morphisthat will use the connection from the daemon to determine the label11:32
Son_Gokuah, that might work11: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
morphisso we need to touch snapd.service and snapd.socket11:32
Son_Gokusnapd itself is already properly labeled, I think11:32
Son_Gokuthe issue is that the socket doesn't exist at the point of the policy getting applied11:33
zygaogra_: unless sergiusens disagrees I don't think we support prebuilt binaries yet11:33
Son_Gokuso I think snapd.socket is the only thing that needs to be changed11:33
ogra_zyga, you mean no stage-packages in classic ? thats news to me11:33
ogra_(but that would indeed solve the issue)11:34
kotVaskahallo, spricht hier jemand deutsch?11:36
zygaogra_: yes that's what I mean11:37
morphisSon_Goku: ok, so who labels snapd if not through the .service file?11:37
zygaogra_: how did you expect stage-packages to work/11:37
morphiszyga: https://github.com/snapcore/snapd/pull/3156 gets really close :-)11:37
mupPR snapd#3156: Support debian in our CI <Created by morphis> <https://github.com/snapcore/snapd/pull/3156>11:37
zygamorphis: let me see :)11:37
Son_Gokumorphis: the policy, when it is applied, labels /usr/libexec/snapd as snappy_exec_t11:37
ogra_zyga, dunno, i never built a classic snap ever :P11:37
Son_Gokuthat happens during the package install11:37
* ogra_ thinks thats an insanity anyway 11:38
zygamorphis: 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 switch11:38
Son_Gokuor at least, it should...11:38
morphiszyga: two tests are failing which I need to look into but all others should be ok now11:38
morphiszyga: nice :-)11:38
morphisSon_Goku: ah I see11:38
* Son_Goku needs to set up a clean box and test something11:38
Son_Gokusomething about these denials is bugging me, because it looks like at least a good chunk of them shouldn't be issues anymore...11:39
zygamorphis: commented quickly11:39
zygamorphis: I'll comment again after reading it fully11:39
kotVaskaplease help my. i have follow error with "sudo snap install nextcloud"-command -> https://pastebin.com/DsherkXt11:39
zygakotVaska: hey, you may have better luck on forum.snapcraft.io11:39
zygakotVaska: I'll gladly help you there11:40
zygakotVaska: because others can benefit from this11:40
morphiszyga: thanks11:40
zygakotVaska: please open a new thread under the "snap" category11:40
zygathanks!11:40
kotVaskazyga: I don't want to register there.11:42
* zyga though we already had various social network login available11:43
Son_Gokuwe still don't11:44
zygakotVaska: can you please run `snap version`11:44
* ogra_ bets the kernel is a mess ... the "vps" in the hostname kind of indicates that :)11:46
kotVaskazyga: 2.22.611:46
ogra_kotVaska, the full output please11:46
kotVaskasnap    2.22.611:46
kotVaskasnapd   2.22.611:46
kotVaskaseries  1611:46
kotVaskaubuntu  16.0411:46
kotVaskakernel  2.6.32-042stab120.711:46
ogra_(should be 5 lines)11:46
ogra_lol11:46
Son_Gokuoh god11:46
ogra_2.6.32-042stab120.711:46
Son_Gokustab is right11:46
ogra_kotVaska, that cant work11:46
Son_Gokuit's an OpenVZ system11:46
kotVaskayes11:46
ogra_kotVaska, talk to your vps provider to use a proper kernel11:47
Son_Gokuyou can't run snaps on OpenVZ11:47
zygakotVaska: snapd won't work on that kernel11:47
zygakotVaska: your kernel is too old, I'm sorry11:47
Son_Gokuthey need to upgrade to Virtuozzo 711:47
ogra_this is like decades outdated11:47
Son_Gokuif they do that, snaps will work again11:47
kotVaskai have a vServer11:47
* zyga recalls his first job that had a 2.6 kernel ;)11:47
Son_GokukotVaska: your hosting provider needs to upgrade to Virtuozzo 7: https://virtuozzo.com/products/virtuozzo/11:47
mupPR snapcraft#1263 opened: Pass through commands into the lxd container <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1263>11:48
kotVaskamy provider have "LXC - Linux vServer", " Qemu-KVM vServer" and " OpenVZ vServer"11:51
kotVaskai rent openvz vserver11:51
morphiszyga: where did you still see is-forced-devmode on the PR?11:52
Son_GokuQemu-KVM will work, as you'll use the distro's kernel11:52
morphisthat should be gone11:52
zygamorphis: hmm,11:52
Son_Gokuand LXC may work, if the host is modern11:52
zygamorphis: not sure what you mean?11:52
zygamorphis: I commented on the blacklisting of tests on Debian11:52
zygamorphis: that those had no comments explaining why11:52
morphisgot 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
zygamorphis: ah11:52
zygamorphis: that's ancient :)11:52
zygamorphis: it was for a command you wanted to add11:52
zygamorphis: to check if a distro is in forced devmode11:53
morphisyeah, I will do that in a second step11:53
zygamorphis: we (very long time ago) defined some rules of how help for commands must look like11:53
zygamorphis: including specific wording11:53
zygamorphis: see all the commands, they have consistent help wording11:53
morphisok, any link to that?11:53
zygamorphis: no, it was before we had anything like a forum/public ML11:53
zygamorphis: it was in a private email thread11:53
zygamorphis: just use your imagination, compare to how other commands have help worded11: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
morphiszyga: worth to bring that to the forum or any other documentation place now :-)11:54
zygamorphis: yeah11:57
zygamorphis: if you ask the question I'll go through my archive and try to find it11:57
sergiusensogra_: zyga we don't support doing the right thing with runnables from prebuilts when on classic confinement, that is correct11:57
kotVaskathanks11:57
Son_Gokuogra_: it's not hacked up11:58
Son_Gokuit's a container11:58
Son_Gokudon't be misleading11:58
ogra_Son_Goku, dude ... to make 16.04 run on top of a 2.x kernel you will have to massively patch it11:58
Son_Gokuunless the 2.6 kernel itself has been patched and backported11:58
morphiszyga: thanks11:59
Son_Gokuwhich is exactly how that is even working11:59
ogra_no matter if it is a container or raw HW11:59
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 here12:00
Son_Gokuthe OpenVZ kernel used supports multiple ptys12:00
ogra_this can not legally be called ubuntu12:00
Son_Gokuthen the same goes for all your docker images12:00
Son_Gokuif I load a docker image into an OpenVZ system, I didn't touch Ubuntu12:01
ogra_depends if they are blessed by canonical or not ... but i doubt there are doker installs using 2.x anywhere12:01
Son_Gokua docker image doesn't have to be used by docker12:01
Son_Gokurkt, nspawn, lxc, openvz, etc. can all consume them12:01
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 then12:02
ogra_*sell12:02
ogra_the kernel is a part of the brand12:03
ogra_(you cant sell RH CDs with a replaced kernel either and call it RH)12:04
zygaanother part of update-ns for review: https://github.com/snapcore/snapd/pull/320912:16
mupPR snapd#3209: interfaces/mount: add partial implementation of Change.Needed <Created by zyga> <https://github.com/snapcore/snapd/pull/3209>12:16
mupPR snapd#3209 opened: interfaces/mount: add partial implementation of Change.Needed <Created by zyga> <https://github.com/snapcore/snapd/pull/3209>12:16
zygaI'll switch to code reviews now and work on chipaca's tab-completion PR now12:16
zygaogra_, morphis: ^^12:17
zygalow level, feedback appreciated12:17
ogra_zyga, why do you nered all the options for umount  ? if it is mounted the mountpoint should be enough for unmounting12:21
ogra_*need12:21
zygaogra_: interesting point; I guess we don't want to unmount something _else_12:25
zygaogra_: but not sure if that captures it12:25
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_*to12:26
zygamorphis: did you cancel the call just now?12:28
zygaogra_: yes, I think you are right12:28
ogra_beyond that i think it looks fine12:28
zygaogra_: I think the devil is in mountpoint details though (like bind mounts making stuff more complex)12:28
ogra_i also wonder if you carry over the mount options for filesystems from the original mount when bind mounting ...12:29
zygaogra_: yes, those are unchanged12:29
zygaogra_: well, there are two sets as you see12:29
ogra_good12:29
zygaogra_: superblock and per-mount12:29
ogra_right, i'm rather cobncerned about "noatime,nodiratime,data=ordered" etc ... stuff that makes sure we dont wear out SSDs or eMMCs12:30
ogra_but i guess that is carried forward anyway ... or rather physically used from the original mount12:31
zygaogra_: I think that right now it doesn't matter12:33
zygaogra_: we only do bind mounts12:33
zygaogra_: and the code as written is only a stab at the generic functionality12:33
ogra_yeah12:33
mupPR snapd#3210 opened:  tests: relax network-bind interface regexps for 2.24 <Created by mvo5> <https://github.com/snapcore/snapd/pull/3210>12:54
=== caribou_ is now known as caribou
mupPR snapcraft#1255 closed: Update rust plugin to set RUSTFLAGS <Created by ChrisMacNaughton> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1255>13:21
mupPR snapcraft#1250 closed: Fixed links to doc <Created by andyli> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1250>13:24
mupPR 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:27
NicolinoCurallihi 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 output13:46
ogra_NicolinoCuralli, well, depends on the context13:47
NicolinoCurallinaturally snap connect don0t work for that interface13:47
ogra_NicolinoCuralli, on a classic install where network-manager is pre-installed for your desktoop, the interface is provided by this ...13:47
NicolinoCuralliogra_ : i run in a strict mode13: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 snap13:48
ogra_so the interface is context dependant13:48
NicolinoCuralliogra_ : i understood , thanks13:49
ogra_:)13:49
davidcalleNicolinoCuralli: although, you can install the network-manager snap https://docs.ubuntu.com/core/en/stacks/network/network-manager/docs/installation13:50
ogra_indeed13:50
donofrioanyone know how to force a refresh that actually loads snappy reposities?  I tried but I just get this https://apaste.info/8eil14:07
ogra_donofrio, that is not how snaps work14:09
ogra_there are no local ppackage lists you need to refresh like you have in apt14:09
ogra_(also ubuntu-core is obsolete ... the snap is only called "core" nowadays)14:09
donofrioogra_, 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:10
donofriouh yah14: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:11
ogra_donofrio, oh, also ... nevver use "sudo su -" it will unset a lot of environment ... use sudo -s or sudo -i and not su14:16
ogra_(snapd definitely uses some env that would be dropped by "su -" )14:16
ogra_(but thats a general rule not snap specific ... su - is always bad unless you want to be in a new env)14:19
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:21
ogra_i wonder if something regressed in 2.24 in that regard14:22
niemeyerogra_: I think the idea was always to have a stock list of interesting snaps there.. if they're too few, probably something needs fixing14:22
niemeyerogra_: Can you open a topic on forum under "store"?14:22
kyrofaogra_, do you know if ubuntu core 16 has an image in AWS?14:22
ogra_kyrofa, i dont actually ... but i dont see why the normal x86 image wouldnt work ...14:23
kyrofaYeah, 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 himself14:24
ogra_niemeyer, https://forum.snapcraft.io/t/empty-snap-find-suddenly-returns-four-snaps-but-not-more/323/114:25
ogra_kyrofa, it was hard back then ... it needed some special packages added ... but that need was dropped14:25
ogra_(we used to actually build special AWS images in 15.04 snappy ...)14:26
kyrofaogra_, yeah, the only results I'm seeing are in the community AMIs, and that's only 15.0414:26
kyrofaniemeyer, do you know anything about ubuntu core on AWS?14:28
ogra_kyrofa, rharper does perhaps ... being in the cloud team14:28
niemeyerkyrofa: Not much to be honest14:28
kyrofaniemeyer, trying to figure out how to test a CUDA setup without buying an expensive video card :P14:29
ogra_all i know was that i was asked to drop the AWS hacks before 16 because it would "just work" now14:29
kyrofaogra_, good deal, thank you14:29
niemeyerogra_: Thanks for the topic14:33
sergiusensjdstrand: tyhicks can I get your opinion on https://forum.snapcraft.io/t/asset-recording-for-a-built-snap/317?u=sergiusens14:35
tyhickssergiusens: nice! I'll need a little bit of time to think it over14:38
sergiusenstyhicks: no rush14:38
mupPR snapcraft#1260 closed: meta: version from git support <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1260>14:39
jdstrandsergiusens: I saw it and like tyhicks, plan to think about it and comment14:48
mupPR snapcraft#1264 opened: tests: do not print coverage check message <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1264>14:48
morphiszyga: can you have a quick look at https://paste.ubuntu.com/24414168/ ; not an expert yet in reading those expect scripts14:52
morphiszyga: look at the bottom14:53
zygamorphis: looking14:54
zygamorphis: those expect-based tests are very unreliable14:55
zygamorphis: just run it a few times, if it fails still it's interesting14:55
morphisthey reliable fail on debian :-)14:55
zygamorphis: in that case run those manually in a debug shell14:55
zygamorphis: and see what you get14:55
zygamorphis: maybe PS1 discrepancy or something14:55
morphishm, that is a good idea14:56
* zyga goes for a walk, ttyl14:57
morphiszyga: that's it14:59
zygamorphis: aha, PS* differences?14:59
morphis$ bash --norc -i14:59
morphisthat gives different prompts on debian vs. ubuntu15:00
* zyga completed backup15:01
zygattyl :)15:01
mupPR 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:12
mupPR snapd#3211 opened: assertions: add "repair" assertion  <Created by mvo5> <https://github.com/snapcore/snapd/pull/3211>15:24
Pharaoh_Atemmorphis: please open a review for snapd-xdg-open15:24
Pharaoh_Atemat this point, I don't care anymore and I want this in Fedora now15:24
Pharaoh_Atemzyga: have you had a chance to respond in the email thread started by mattdm?15:41
zygaPharaoh_Atem: no, you are right, I should do it now15:52
Facusergiusens, anybody, my snapcraft branch in travis failed with:15:59
FacuThe 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
Facudo you have any idea about what is it? why a CLA check may have failed? it's not my first branch for snapcraft15:59
sergiusensFacu: because you authored the commit with an email that has not signed the CLA or is not @canonical.com15:59
Facusergiusens, mmm, that may be it (I'm not @canonical.com in github), but it would suprise me how the other branch got in16:01
* Facu checks16:01
sergiusenswe started enforcing the check recently to avoid these oversights16:01
sergiusensFacu: looks like you used gmail in this last one and "Author: Facundo Batista <facundo@taniquetil.com.ar>" in one that passed16:02
Facugmail????16:03
sergiusenshmm, I don't know what is going on, which is why I assigned to elopio16:04
Facusergiusens, "git log" in my computer says the Author is facundo@taniquetil.com.ar (expected)16:04
Facusergiusens, where are you finding the gmail mail address?16:05
sergiusensI am not... anymore ;-)16:05
elopioFacu: hello.16:05
elopioFacu: I think that the error will go away after rebase with master. I will hit the update button in your pr.16:09
Facu:o16:10
Facuelopio, thanks16:11
Facuhere https://launchpad.net/~facundo it says I signed the contributor agreement on April 3rd16:12
renatHi, guys! It's Renat from Screenly.16:12
mupPR snapcraft#1265 opened: shell_utils: code cleanup <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1265>16:13
zygaPharaoh_Atem: replied16:14
renatI have a question related to the warning message from the store16:16
renat(NEEDS REVIEW) 'daemon' should not be used with 'browser-support' security-snap-v2_daemon_with_browser-support (viewer)16:16
renatIs it possible to make the store accept a daemon service with a browser support plug?16:17
renatTo make it automatically.16:17
zygarenat: 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:17
zygarenat: browser-support is a special interface that we really only expect to hand out to specific browsers16:18
zygarenat: and nobody else16:18
zygarenat: as it is exceptionally open to support various things modern browsers do16:18
ogra_zyga, well, there might be browsers in kiosk mode ...16:18
ogra_which i assume this boils down to16:18
zygaogra_: aha16:18
zygaogra_: daemon mode browser is an interesting case16:18
ogra_yep16:18
zygastill feel like a hack16:19
ogra_but i guess the seutrity team had reasons to not allow this combo yet16:19
zygabecause runs as root16:19
zygayes, it's super powerful and open16:19
ogra_*security16:19
renatzyga, ogra_, we use it for digital signage.16:19
renatAnd it's really running in a daemon mode.16:19
zygarenat: please open a thread on the forum, I will reply tere16:20
zygathere*16:20
pstolowskiit would be great to have second review on 3159 and land it16:20
renatzyga, forum? You mean - write to the mailing list?16:21
zygarenat: see forum.snapcraft.io :-)16:21
zygapstolowski: done16:21
renatWow. Okay.16:21
mupPR 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
pstolowskizyga, cool, thanks16:21
zygapstolowski: I'd love to land https://github.com/snapcore/snapd/pull/320316:22
mupPR snapd#3203: interfaces/mount: add ReadMountInfo and LoadMountInfo <Created by zyga> <https://github.com/snapcore/snapd/pull/3203>16:22
zygamvo: I added some comments on https://github.com/snapcore/snapd/pull/3211/16:26
mupPR snapd#3211: assertions: add "repair" assertion  <Created by mvo5> <https://github.com/snapcore/snapd/pull/3211>16:26
mupPR snapd#3212 opened: api, ifacemgr: resolve disconnect early <Created by stolowski> <https://github.com/snapcore/snapd/pull/3212>16:26
zygamvo: quick question, aren't models specific to a brand?16:27
zygamvo: 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:27
mupPR snapcraft#1266 opened: tests: report coverage only in unit tests <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1266>16:28
renatDone: https://forum.snapcraft.io/t/suppress-the-security-snap-v2-daemon-with-browser-support-warning-for-the-snap/32516:28
zygarenat: thanks!16:28
* zyga -> break16:32
pstolowskizyga, +116:33
* pstolowski eod16:33
pstolowskio/16:33
elopioFacu: 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
elopioif you sync with master, it will just pull your commits, and not check the CLA on things that were already merged.16:46
Facuelopio, thanks16:47
jdstrandrenatu (cc ogra_ and zyga): I responded to the forum post17:34
mupPR snapcraft#1264 closed: tests: do not print coverage check message <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1264>17:37
=== JanC_ is now known as JanC
mupPR snapcraft#1265 closed: shell_utils: code cleanup <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1265>18:19
mupPR snapcraft#1267 opened: meta: override the version with version-script <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1267>20:37
mupPR snapcraft#1195 closed: [experimental] run unit tests in osx <Created by elopio> <Closed by elopio> <https://github.com/snapcore/snapcraft/pull/1195>22:55

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