/srv/irclogs.ubuntu.com/2016/10/04/#snappy.txt

elopiocprov: sergiusens: hey, sorry, I couldn't access quassel for quite a while. Indeed, I had a typo on my exported var :/00:30
elopiosorry for the noise.00:30
mupPR snapd#2071 opened: interfaces,overlord/interfaces: switch to use declaration-based checking for auto-connect <Created by pedronis> <https://github.com/snapcore/snapd/pull/2071>00:39
mupPR snapd#2072 opened: cmd/snap: have 'snap autoimport' consider unmounted devices <Created by mwhudson> <https://github.com/snapcore/snapd/pull/2072>01:26
mupPR snapd#2068 closed: many: preparations for switching most of autoconnect to use the declarations <Critical> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/2068>01:29
sergiusenselopio cprov I am not oposed to a tools/staging_env.sh that one can source setting up the env and PS1 ;-)01:50
cprovsergiusens: that's a grand idea01:51
sergiusenscprov elopio I am on and off, but here's a bug just in case it falls off the radar LP: #163008301:53
mupBug #1630083: Provide sourceable script to work with the staging environment <Snapcraft:New> <https://launchpad.net/bugs/1630083>01:53
=== JanC_ is now known as JanC
mupPR snapd#2073 opened: client, cmd/snap, daemon: add is-managed command <Created by mwhudson> <https://github.com/snapcore/snapd/pull/2073>04:07
pittizyga: udev rules are all about events; so both a boot aand udevadm trigger will run every rule, so just attach it to teh "add" event of a device which provides the GPIO pins?06:25
pittizyga: /sys/class/gpio/export, is that really a global thing? that doesn't sound like something you should change in udev rules then06:26
pittior was that just simplifid/a template path?06:26
mupPR snapd#2074 opened: add /users endpoint and auto run `snap create-user --known` for unowned devices <Created by mvo5> <https://github.com/snapcore/snapd/pull/2074>06:33
* kalikiana found bug 1576303 whilst trying to listen to Japanese music on the VLC snap06:37
mupBug #1576303: Needs fontconfig integration <snap-desktop-issue> <Snapcraft:New> <snapcraft (Ubuntu):Confirmed> <https://launchpad.net/bugs/1576303>06:37
dholbachgood morning06:39
shuduomorning06:53
zygapitti: it wasn't simplified, that's the actual thing06:57
shuduocould anyone educate me how to call /sbin/ifconfig in a snap app? i see some relative discussion in https://bugs.launchpad.net/snappy/+bug/1615124 but no idea how to use and which interface is right one. thanks06:57
mupBug #1615124: apps should be able to run /usr/bin/shuf by default <snapd-interface> <Snappy:Fix Committed by jdstrand> <https://launchpad.net/bugs/1615124>06:57
mupPR snapd#2073 closed: client, cmd/snap, daemon: add is-managed command <Created by mwhudson> <Closed by niemeyer> <https://github.com/snapcore/snapd/pull/2073>07:17
mupBug #1630145 opened: out of disk space needs better handling <Snappy:New> <https://launchpad.net/bugs/1630145>07:23
mupPR snapd#2075 opened: cmd/snap: add version command, same as --version <Created by niemeyer> <https://github.com/snapcore/snapd/pull/2075>07:43
mupPR snapd#2026 closed: client, cmd: connect fixes <Created by stolowski> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2026>07:46
morphis_zyga: ping07:50
arapitti, hello! would you approve snap-confine into xenial-proposed, please? https://launchpad.net/ubuntu/xenial/+queue?queue_state=1&queue_text=snap-confine07:56
zygamorphis_: hey08:12
morphis_zyga: hey!08:12
zygamorphis_: sorry, I'm laggy, not slept much last night08:12
morphis_zyga: np :-)08:13
morphis_zyga: you already did the PR to fix the mount problem we have with the udisks2 snap?08:13
morphis_was looking for it but didn't found one?08:13
morphis_s/one/one?/08:13
zygamorphis_: no, though almost, I need to work on the all-snap side of the problem08:13
morphis_zyga: what is different there?08:13
zygamorphis_: the initial fs is /08:14
zygamorphis_: https://github.com/snapcore/snap-confine/commit/43e47b35a13786aa7a7aabe1a45d78febae49aa908:15
zygamorphis_: I need to finish one more bit for snapd and then I'm back to confine, we're freezing this week08:15
zyga(well, today)08:15
morphis_zyga: actually this is required to be fixed before the freeze08:15
zygamorphis_: we can still change images after today so it will get fixed08:17
zygamorphis_: it will be in the released images08:17
Skuggendpkg: error processing archive /var/cache/apt/archives/snapd_2.15.2ubuntu1_amd64.deb (--unpack): trying to overwrite '/usr/share/man/man1/snap.1.gz', which is also in package snap 2013-11-29-1ubuntu208:17
Skuggen:)08:17
morphis_zyga: so by when? last week you said there will be a PR later that day08:18
zygaSkuggen: I think it is a known issue, I think it was fixed but perhaps that's not the case08:18
zygamorphis_: I spent literelly two weeks on this issue08:18
zygamorphis_: there's lots of hard edges there08:18
Skuggenzyga: https://bugs.launchpad.net/ubuntu/+source/snap/+bug/1589037 just confirmed so far08:18
mupBug #1589037: package snap (not installed) failed to install/upgrade: trying to overwrite '/usr/share/man/man1/snap.1.gz', which is also in package snapd 2.0.5 <amd64>08:18
mup<apport-package> <package-conflict> <xenial> <One Hundred Papercuts:Confirmed> <snap (Ubuntu):Invalid> <snapd (Ubuntu):Confirmed> <https://launchpad.net/bugs/1589037>08:18
morphis_zyga: good, so do you need any further help? Lean suggested to ask Seth to help you with any further problems on the mount namespace08:19
zygamorphis_: I'd love some help on that if you want to, we know how it works now but the all-snap is significantly different that someone has to sit on it and push through the issues08:19
zygamorphis_: well, the mount namespace has been released, this is /media sharing08:19
zygamorphis_: it's not a problem of "how do we do it" anymore, that part is solved08:20
zygamorphis_: this is just some extra coding time that's required now08:20
morphis_zyga: so what the actual issues?08:20
zygamorphis_: take that branch and iterate on an all-snap system, it needs to be tweaked to use different rootfs_dir and the apparmor profile needs to be adjusted08:21
zygamorphis_: then if it doesn't crash the kernel and /proc/self/mountinfo looks sane then we're good08:21
zygamorphis_: apparmor profile needs to be cleaned up a little as it it now full of redundant rules08:22
zygamorphis_: it just need polish08:22
morphis_zyga: which branch?08:22
zygamorphis_: https://github.com/snapcore/snap-confine/commits/media-sharing08:22
zygamorphis_: I don't recommend anthing but a full blown VM08:22
zygamorphis_: one line wrong and you hose your box08:23
zygamorphis_: it would be amazing if we managed to run spread on master (not this branch) on an all-snap system08:25
zygamorphis_: in qemu or linode08:25
zygamorphis_: if you could get that, it would save me a lot of time08:25
zygamorphis_: I think qemu is easier and I would find it sufficient08:25
morphis_zyga: I must say I don't see where the actual rootfs change would come into this code change08:27
zygamorphis_: look at the new struct08:27
zygait is called...08:27
zygarootfs_dir :)08:28
zygahttps://github.com/snapcore/snap-confine/commit/43e47b35a13786aa7a7aabe1a45d78febae49aa9#diff-0ec19215f6a6c97084d2e4eb5e0b54ccR54608:28
morphis_ah08:28
morphis_zyga: there seem to be a lot other TODO's08:28
zygamorphis_: specifically?08:29
morphis_I count atleast three from the commit message08:29
zygamorphis_: ah, some of those are stale08:29
zygamorphis_: if spread passes you're in good shape08:29
zygamorphis_: I think /etc needs love on actual sharing08:29
zygamorphis_: it works but I think it's setup wrong08:30
morphis_zyga: so what else do you have on your plate which prevents you from finishing this?08:32
zygamorphis_: just one branch on snapd, carry over from last night08:32
zygahttps://github.com/snapcore/snapd/pull/2066 specifically08:33
mupPR snapd#2066: many: abbreviated forms of disconnect <Created by zyga> <https://github.com/snapcore/snapd/pull/2066>08:33
morphis_zyga: which would mean you will get back to the snap-confine later today?08:36
zygamorphis_: yes, totally08:37
zygamorphis_: nothing else on my plate08:37
morphis_zyga: I don't really have much time to support here and it looks like you're the best to get this fixed quickly08:37
morphis_zyga: so how much more time you need to finish this?08:37
zygamorphis_: I think one more day, I bet the non-all snap version will be 100% done today, including nice aa profile, the all-snap is probably trivial but maybe there are dragons, I didn't try yet08:39
morphis_zyga: non-all snap is not really the problem, we need this for all-snap08:40
morphis_in the end we want both but all-snap should be IMHO the priority08:40
zygamorphis_: it needs to work in both cases but I think we're very very close08:41
morphis_yeah08:42
zygamorphis_: just figuring out the sequence of do's and don'ts was the hard part :)08:42
SkuggenHow does snappy figure out dependencies for simple binary inclusions? run ldd?09:04
mupPR snapd#2076 opened: asserts: limit model to only lowercase until we can implement case insensitive/preserving headers <Created by pedronis> <https://github.com/snapcore/snapd/pull/2076>09:17
mwhudsonSkuggen: yes, i think so09:19
morphis_sergiusens: ping09:49
shuduocould anyone educate me how to call /sbin/ifconfig in a snap app? i see some relative discussion in https://bugs.launchpad.net/snappy/+bug/1615124 but no idea how to use and which interface is right one. thanks10:05
mupBug #1615124: apps should be able to run /usr/bin/shuf by default <snapd-interface> <Snappy:Fix Committed by jdstrand> <https://launchpad.net/bugs/1615124>10:05
zygashuduo: hey10:08
shuduozyga: hi10:09
zygashuduo: please use the network-control interface10:09
zyga:-)10:09
shuduozyga: yes, i added it but snappy-debug still report "Log: apparmor="ALLOWED" operation="create" profile="snap.connman.connman//null-/sbin/ifconfig" pid=7741 comm="ifconfig" family="unix" sock_type="dgram" protocol=0 requested_mask="create" denied_mask="create" addr=none"10:10
shuduoso i guess some lines needed in snapcraft.yaml too10:11
zygahmmm10:11
zygathat may depend on what you use ifconfig for10:11
zygacan you pease file a bug on snappy (on launchpad) with more informatio10:11
zygait is much better than figuring it out here, most of the team is busy working on the release10:11
zygaand some last few bits for it10:11
shuduozyga: okay. understand. i will. do you know if any exist snap use ifconfig then i can refer to?10:12
zygashuduo: I don't know, sorry10:12
shuduozyga: no worry. thanks.10:13
ogra_ppisati, regarding the question in bug 1627643 i think you ned to ask mwhudson or pitti10:19
mupBug #1627643: oops on pi3 (seemingly wlan related) <Snappy:New> <linux-raspi2 (Ubuntu):New for p-pisati> <https://launchpad.net/bugs/1627643>10:19
ogra_(not sure it is actually possible at all to run nplan on a xeianl classic install)10:20
ogra_*xenial10:20
ppisatiogra_: does it need yakkety?10:24
pittiogra_: it  will be soon, see bug 162764110:24
mupBug #1627641: Backport netplan to xenial <verification-needed> <network-manager (Ubuntu):Fix Released> <nplan (Ubuntu):Fix Released> <systemd (Ubuntu):Fix Released> <network-manager (Ubuntu Xenial):Fix Committed by pitti> <nplan (Ubuntu Xenial):Fix Committed by pitti> <systemd (Ubuntu Xenial):In10:24
mupProgress by pitti> <https://launchpad.net/bugs/1627641>10:24
ppisatiogra_: i can build a yakkey img if needed10:24
ogra_ppisati, well, the userspace would be totally different and we dont have yakkety snappy10:25
ogra_(i mean snappy images ... classic + snap might work)10:25
ppisatiogra_: yeah, but you said 'not sure it is actually possible at all to run nplan on a xeianl classic install' so i thought you needed some yakkety bits10:27
ogra_ppisati, yes, netplan :)10:27
ogra_(and its dependencies)10:27
ogra_ppisati, i also have the impression it is somehow related to firstboot  i seem to not get the oops on second boot either here10:40
=== hikiko is now known as hikiko|ln
sergiusensmorphis hello hello10:46
ppisatiogra_: but do you actually get an oops? because on first boot, my wifi doesn't get any ip and nothing is printed out10:50
ppisatiogra_: just a generic 'Fail to blabla' from nplan10:50
ppisatiogra_: and upon reboot, my wifi associated immediately10:50
ogra_ppisati, i only see the wifi interface very rarely (every 10th boot or so)10:50
ppisatibut no oops at all10:50
ogra_most of the time there is the oops and the device isnt shown at all in console-conf10:50
ppisatiso it's before the config phase the oops10:51
ogra_the prob iss that a reboto breaks the image completely10:51
ogra_*reboot10:51
ogra_(the snappy firstboot stuff only runs after console-conf completed ... and it breaks if it gets re-run on an already booted image)10:52
ogra_niemeyer, mvo, bug 1630145 made me think ... unrelated to diskspace, snapd should probably run with an oom_score_adj "- 500" value ... so it doesnt get killed early on OOM events11:01
mupBug #1630145: out of disk space needs better handling <Snappy:New> <https://launchpad.net/bugs/1630145>11:01
mupPR snapd#2077 opened: overlord/devicestate,docs/hooks.md:  switch prepare-device/registration opts to use nesting, some hook docs  <Critical> <Created by pedronis> <https://github.com/snapcore/snapd/pull/2077>11:15
ogra_mvo, niemeyer, i filed bug 1630208 for the above11:20
mupBug #1630208: snapd should be excluded from OOM <Snappy:New> <https://launchpad.net/bugs/1630208>11:20
zygaoh interesting11:21
mupBug #1630208 opened: snapd should be excluded from OOM <Snappy:New> <https://launchpad.net/bugs/1630208>11:21
=== hikiko|ln is now known as hikiko
mupPR snapd#2064 closed: interfaces/builtin: add autopilot-introspection <Blocked> <Created by heber013> <Closed by heber013> <https://github.com/snapcore/snapd/pull/2064>12:00
didrockssergiusens: hey! small quetsion, I can't find the bug reference anymore, but I'm pretty sure there was one on organize: not supporting globbing (so that you can dump the copy plugin in favor of… dump!), do you think you have time working on tihs?12:01
didrocksthis*12:01
sergiusensdidrocks it is already released12:02
didrocksoh12:02
didrocksit's the version in -proposed?12:02
didrocksthat's probably why I couldn't find it :)12:03
didrocksdidn't check for closed ones12:03
sergiusensdidrocks https://github.com/snapcore/snapcraft/blob/master/snapcraft/tests/test_pluginhandler.py#L67612:04
sergiusensdidrocks should be in -updates since 3 or 4 weeks ago12:04
didrockssergiusens: interesting, I was sure I tried a couple of weeks ago12:05
didrockssergiusens: so "*" : "www/" should work from what I see in the test12:06
sergiusensdidrocks yes12:06
sergiusensdidrocks if not, we have  anew bug :-(12:06
didrockssergiusens: oh, I did try ".": "www" as in the copy plugin12:07
didrocksprobably12:07
* didrocks gives it a try12:07
sergiusensoh, different semantics indeed12:08
didrocksI prefer *, makes more sense IMHO12:08
didrockssergiusens: so, I'm getting: Trying to organize file '*' to 'www', but 'www' already exists12:12
didrockssomething (I guess snapcraft) did create indeed a "www" file12:13
didrocksI want to put everyting in www12:13
didrocksmaybe the syntax is rather: "*": "www/"12:13
didrockssergiusens: that did it! Thanks a lot :)12:16
sergiusensdidrocks \o/12:30
jdstrandshuduo: please file a bug at https://bugs.launchpad.net/snappy/+filebug with steps to reproduce. you should be using 'plugs: [network-control]' to use ifconfig, however, connman is doing something different with ifconfig that isn't included in the policy atm12:32
abeatojdstrand, hi, thanks for the thorough review. I have pushed changes to the PR, with a couple of questions too12:33
jdstrandok12:35
jdstrandthanks :)12:35
abeatosergiusens, I was wondering about your opinion on using config flags to set file paths when compiling using autotools snapcraft plugin12:35
shuduojdstrand: it's a customer's app intend to do what connman doing. i'm trying to get source code.12:35
abeatosergiusens, we are checking $SNAP in some programs to know if it is part of a snap or not12:35
abeatosergiusens, maybe using a standard configure flag would be better and make changes more palatable for merging in upstreams12:36
jdstrandshuduo: it's possible to debug via irc. do you have time now?12:36
shuduojdstrand: i have time but don't have source code. is it possible? :)12:38
sergiusensabeato that's what we did for click; a nice configure flag we can standarize on would be great practice; all these configure flags are just convention after all12:38
jdstrandshuduo: yes, I'll give you rules to add and you tell me what you see in the logs12:39
shuduojdstrand: good12:39
abeatosergiusens, promoting a couple of standard configure flags would certainly be great :)12:39
jdstrandshuduo: first off, does your app require devmode for other things or are you using devmode only for this one denial?12:39
abeatosergiusens, maybe paths to data, to daemon data, and to current snap12:40
shuduojdstrand: i already use devmode in yaml and also installation.12:41
jdstrandshuduo: you misunderstand. must you use devmode? I'd prefer to use strict mode for policy debugging but if your app doesn't work at all in strict mode, then we can leave it12:42
shuduojdstrand: okay. let me modify it.12:42
shuduojdstrand: i thought if devmode works then i can try strict mode.12:43
jdstrandshuduo: you are doing the right thing. but for security policy development I prefer strict mode12:45
shuduojdstrand: i changed it to strict and scanlog output is http://pastebin.ubuntu.com/23274922/.12:46
jdstrandshuduo: can you give the output of 'snap interfaces'?12:47
shuduojdstrand: http://pastebin.ubuntu.com/23274935/12:48
jdstrandshuduo: please connect the interfaces with:12:48
jdstrandsudo snap connect connman:firewall-control ubuntu-core:firewall-control12:49
jdstrandsudo snap connect connman:network-control ubuntu-core:network-control12:49
jdstrandsudo snap connect connman:network-manager ubuntu-core:network-manager12:49
jdstrand(though I wonder about that last one-- is it supposed to talk to NetworkManager?12:50
jdstrand)12:50
shuduojdstrand: now output is http://pastebin.ubuntu.com/23274944/12:53
shuduoseems no complain for ifconfig12:54
jdstrandshuduo: ok, so your snap has a little bit of work to do to make ListOfServers.txt relocatable, but we can hopefully work around that12:54
jdstrandshuduo: oh, would you have seen the ifconfig denial by this point?12:54
shuduojdstrand: no ifconfig denial now.12:56
jdstrandshuduo: ok, so maybe you just needed to connect the interfaces. you need to do that in devmode too12:57
shuduojdstrand: in ubuntu desktop, the app will create a file ListOfServvers.txt in working directory.12:57
jdstrand(in devmode if you don't connect, the accesses that are against policy are allowed, but logged. to make the logging go away, connect the interfaces)12:57
jdstrandshuduo: you should 'cd "$SNAP_DATA"' before starting-- or choose one of the other SNAP_* variables12:59
jdstrandshuduo: and by 'you', I mean your snap12:59
shuduojdstrand: so i guess i need a wrapper to 'cd "$SNAP_DATA"', right?13:00
jdstrandshuduo: yes13:01
shuduojdstrand: in strict mode, the app will exit with "Failed to open default display". In devmode, it exit with "XOpenIM() failed. Unable to find fonts. check your FontConfig configuration." i claimed unity7 in snapcraft.yaml. anything missed?13:04
mupPR snapd#2078 opened: daemon: fix login API to return local macaroons <Created by matiasb> <https://github.com/snapcore/snapd/pull/2078>13:04
jdstrandshuduo: we are getting out of my area of expertise (I work on security policy). I suspect you need to use the desktop/gtk3 part13:06
jdstrandshuduo: eg:13:06
jdstrandparts:13:06
zygajdstrand: hey, how are you13:06
jdstrand  something:13:06
jdstrand  ...13:06
jdstrand  after: [desktop/gtk3]13:06
jdstrandmeh13:06
jdstrandparts:13:06
jdstrand  something:13:06
jdstrand    ...13:06
jdstrand    after: [desktop/gtk3]13:07
jdstrandshuduo: something like that ^, but will need others in the channel to help if that doesn't work for you13:07
jdstrandzyga: hi! :)13:07
shuduojdstrand: okay. thanks a lot!13:08
mupPR snapd#2079 opened: tests: use test snap from the store in the content interface test <Created by fgimenez> <https://github.com/snapcore/snapd/pull/2079>13:17
=== rokf_ is now known as rokf
mupPR snapd#2080 opened: daemon: do not hardcode UID in userLookup <Created by mvo5> <https://github.com/snapcore/snapd/pull/2080>13:51
loologra_: would you by any chance now where the ubuntu-core snaps are built for stable and edge?13:52
ogra_lool, we only build edge and propagate it ATM ...13:55
ogra_lool, https://code.launchpad.net/~snappy-dev/+snap/ubuntu-core13:55
ogra_lool, plan is to have all PPA bits SRUed by GA and then build beta/candidate without the PPA ... but keep edge on PPA with daily builds13:56
loologra_: is there a cron scheduling this daily?13:56
ogra_lool, yep13:56
looloh we even build it on s390x and ppc64el13:57
ogra_ogra@lillypilly:~$ crontab -l13:57
ogra_30 4 * * * /home/ogra/ubuntu-core-builds/build-ubuntu-core >>/home/ogra/ubuntu-core-builds/build.log 2>&113:57
ogra_0,30 * * * * /home/ogra/public_html/ubuntu-core-builds/generate-core.sh >>/home/ogra/public_html/ubuntu-core-builds/generate.log 2>&113:57
ogra_ogra@lillypilly:~$13:57
loologra_: ah I see we do the extra ppa thing in livecd-rootfs13:57
loolsnappy-dev/image snappy-dev/edge13:57
ogra_right13:57
ogra_for beta/candidate we'll have a branch with this dropped13:58
looland edge has daily github builds of master I guess13:59
loolhttps://launchpad.net/~snappy-dev/+archive/ubuntu/edge13:59
loolouch loads of stuff in https://launchpad.net/~snappy-dev/+archive/ubuntu/image13:59
ogra_lool, https://wiki.ubuntu.com/QATeam/OSSnapPromotion13:59
ogra_thats the long term plan13:59
ogra_lool, well, you want https://launchpad.net/~snappy-dev/+archive/ubuntu/image?field.series_filter=xenial14:00
loologra_: if I look at xenial, there are 3 uploads from you which are superseded that we might be able to drop14:01
loolhttps://launchpad.net/~snappy-dev/+archive/ubuntu/image/+packages?field.name_filter=&field.status_filter=published&field.series_filter=xenial14:01
ogra_lool, not yet ... they are in -propposed (LP UI lies)14:01
ogra_apart from snapcraft i think14:01
ogra_that can actually be dropped14:01
ogra_ah, no, not yet14:02
ogra_but yeah, i check it every day14:02
loologra_: ok, nice14:02
ogra_for initramfs-tools i'm waiting til cyphermox and lamont are done with theit ipv6 hackery before even attempting to land the fixtrc fix14:02
ogra_*their14:03
ogra_i dont want to cause mid-air clashes of uploads there14:03
loolI wonder if we should have snap-confine from edge too14:03
loolit's recent in the image PPA though14:03
mvopitti: did you had a chance to check https://code.launchpad.net/~jocave/netplan/+git/netplan/+merge/307531 ? sorry for the nudge, its somewhat urgent for the team that works on that14:03
ogra_lool, perhaps14:04
ogra_lool, thats a zyga decision :)14:04
ogra_lool, oh, and on a sidenote we also have daily images built from the edge snap http://people.canonical.com/~ogra/snappy/all-snaps/daily/current/14:05
ogra_s/snap/snaps/14:05
loolah that's useful14:06
loolalbeit I can just switch to edge ubuntu-core if I need that14:06
ogra_(these builds happen on my home desktop and get rsynced though ... waiting for infinity to get use dailies on cdimage)14:06
ogra_s/use/us/14:06
mupPR snapd#2080 closed: daemon: do not hardcode UID in userLookup <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2080>14:29
mupPR snapd#2078 closed: daemon: fix login API to return local macaroons <Created by matiasb> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2078>14:31
jocpitti: hi, i would appreciate some feedback on this as soon as you can https://code.launchpad.net/~jocave/netplan/+git/netplan/+merge/30753114:32
joczyga: did you get the extra info you requested on the udev rules?14:35
mupPR snapd#2077 closed: overlord/devicestate,docs/hooks.md: nest prepare-device configuration options <Critical> <Created by pedronis> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2077>14:37
elopioogra_: have you tried the new snapcraft in proposed?14:44
ogra_elopio, not yet, i'll pull it in for the next build before EODing14:45
ogra_thanks for the remonder (totally forgot about that)14:45
ogra_*reminder14:45
mupPR snapcraft#848 opened: `sign-build` to prompt users for key selection <Created by cprov> <https://github.com/snapcore/snapcraft/pull/848>14:47
morphis_kyrofa: ping15:02
ogra_elopio, https://code.launchpad.net/~snappy-dev/+snap/ubuntu-core is now building with the new snapcraft ... if s390x finishes without failure the fix is good to go15:06
pittijoc: replied15:07
mupPR snapd#2081 opened: snap: relax auto-import test <Created by mvo5> <https://github.com/snapcore/snapd/pull/2081>15:11
elopioogra_: thanks. Let us know when done to add the verification-done tag15:15
mupPR snapd#2082 opened: overlord,daemon,snap: support gadget config defaults <Created by niemeyer> <https://github.com/snapcore/snapd/pull/2082>15:16
ogra_elopio, done :) https://code.launchpad.net/~snappy-dev/+snap/ubuntu-core/+build/598515:16
ogra_(all arches ... so i dont think there are regressions either)15:16
mupBug #1630281 opened: fltk-based app snap exit with "XOpenIM() failed" <Snappy:New> <https://launchpad.net/bugs/1630281>15:26
kyrofamorphis_, pong15:30
morphis_kyrofa: hey!15:31
morphis_kyrofa: I was trying the configuration hook today with one of my snaps but I somehow miss steps to see if it was actually run or if something has failed15:31
morphis_kyrofa: if a hook prints something to stdout where should this appear?15:31
kyrofamorphis_, I believe it gets swallowed unless the hook encounters an error. Try exiting non-zero15:33
morphis_tried that15:33
morphis_but snap set didn't return with an error15:34
mupPR snapd#2083 opened: cmd/snap: generate account-key revocation requests <Created by cjwatson> <https://github.com/snapcore/snapd/pull/2083>15:34
morphis_kyrofa: did a simple shell script meta/hooks/configuration, added #!/bin/sh exit 1 and made it executable15:34
kyrofamorphis_, sounds like it's not actually running. Is the hook executable?15:34
morphis_+ snapcraft snap . and then tried a snap set test=115:34
kyrofaAh, it's meta/hooks/configure15:34
morphis_kyrofa: ah15:34
morphis_ah, used 'configure'15:35
kyrofamorphis_, some docs here: https://github.com/snapcore/snapd/blob/master/docs/hooks.md15:35
morphis_yeah saw those already15:35
morphis_kyrofa: all I get from snapd is:15:36
morphis_Okt 04 17:35:51 nirvana /usr/lib/snapd/snapd[31630]: daemon.go:174: DEBUG: uid=0;@ PUT /v2/snaps/wifi-ap/conf 14.253704ms 20215:36
morphis_Okt 04 17:35:51 nirvana /usr/lib/snapd/snapd[31630]: taskrunner.go:293: DEBUG: Running task 340 on Do: Run apply-config hook for wifi-ap15:36
morphis_Okt 04 17:35:51 nirvana /usr/lib/snapd/snapd[31630]: daemon.go:174: DEBUG: uid=0;@ GET /v2/snaps 1.030001ms 20015:36
morphis_ah wait15:36
kyrofamorphis_, uh oh15:36
kyrofamorphis_, that looks old15:36
kyrofaThe hook was renamed recently from apply-config15:36
morphis_2.15.2ubuntu115:37
morphis_hm, let me try recent edge then15:37
zygalool: get most recent release (e.g. from ppa)15:37
kyrofamorphis_, yeah, might be a re-exec issue15:37
zygajoc: I crashed again (ENOSLEEP), looking at that now15:38
zygajoc: did you talk to pitti by any chance?15:38
loolzyga: I was asking why master is not CI-ed into the PPA like for snapd15:38
loolI dont actually need it15:38
joczyga: not about that15:38
loolseemed like logic thing to have them both15:38
zygalool: no reason really, it was not changing much earlier, I did a one-off upload but there's currently no setup that would put master into the PPA15:39
loolzyga: right, I was thinking we should copy the snapd one, but up to you15:39
zygalool: there's some work towards that end, we want to run snapd tests with master version of snap-confine15:39
zygalool: which versions are we talking about?15:40
loolzyga: versions?!15:40
zygalool: sorry, perhaps my sleepiness is making me ask the wrong question, can we start over?15:40
loolzyga: right now ubuntu-core snap is built from "edge" and "image" PPAs and pushed to edge channel then promoted15:40
loolzyga: edge PPA has snapd from master; image PPA has snapd and snap-confine from releases; would you want to include snap-confine from master in edge PPA as well?15:41
zyganot at this time15:42
loolok15:42
zygalool: I'm very careful with releases (famous last words) and there's nothing significant there, the version 42.1 that is in the image PPA is the right one to use15:42
zygalool: after I release .43 I will put it to the image PPA agai15:42
jdstrandzyga: .43 will have s/ubuntu-core/core/ support?15:43
zygajdstrand: .42.1 does15:43
zygait's released already (to yakkety though)15:44
zygaas it is more of a OMG images thing15:44
jdstrandzyga: sure, but I was more thinking about when that would be supported in xenial15:44
mvojdstrand, ogra_: we will have a new "core" snap in parallel to ubuntu-core soon, could we make it so that the snapcraft LP builds and the reviewsers scripts consider them the same and not complain? that would be really nice15:44
zygajdstrand: with .43 I think, in one or two days15:44
jdstrandmvo: that landed already15:44
zygajdstrand: I'll try to release it before flying to seoul15:44
mvojdstrand: woah - you rock!15:44
jdstrandmvo: well, the review tools bits15:44
zygajdstrand: unless mvo says we need it now and then we'll upgrade the SRU to .42.115:45
zygajdstrand: ara is working on SRU validation15:45
morphis_kyrofa: ok, nevermind, go it working now15:45
jdstrandmvo: Chipaca had some uploads of 'core' last friday, so I fixed the tools and roadmr got them live this week15:45
kyrofamorphis_, excellent!15:45
morphis_kyrofa: also any ETA on getting hook support into snapcraft?15:45
kyrofamorphis_, I believe it's a sprint topic15:46
jdstrandzyga: ack. fyi, I'm just curious. I much prefer how snapd behaves with just 'core' and want to remove 'ubuntu-core'15:46
zygajdstrand: hmmm15:47
jdstrandcause, with the snapd from the image ppa and both core and ubuntu-core installed, you get all the interfaces twice (one set for core:* the other for :*)15:47
zygajdstrand: well, it's tagged and works OK, if ara verifies that one we can bump the SRU to .42.115:47
morphis_kyrofa: so actually no snapcraft support real soon (1-2 weeks)?15:47
jdstrandzyga: to be clear, I'm not trying to create work. just curious15:48
zygajdstrand: FYI, I'm off on thursday/friday (flying)15:48
zygajdstrand: no worries :)15:48
kyrofamorphis_, that's my impression, yes. There's some disagreement about how it should be implemented15:48
jdstrandzyga: home?15:48
zygajdstrand: roscon15:48
jdstrandah15:48
morphis_kyrofa: I see15:48
mupPR snapd#2081 closed: snap: relax auto-import test <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/2081>15:49
zygajdstrand: and I'll be off on the following week15:49
zygajdstrand: I should send out an email perhaps ...15:49
jdstrandyeah, probably15:50
ogra_mvo, not sure we can do that, but i can set up a parallel build with different name16:05
ogra_mvo, you need to grant me access to it in the store though16:06
mvoogra_: hats good enough16:06
mvoogra_: sure, adding you16:06
ogra_cool16:06
ogra_i assume we can stop ubuntu-core eventually ?16:06
ogra_and just build core16:07
ogra_mvo, how will snapd handle the cross-grade for already installed ubuntu-core ? ubuntu-core is at revision 800+ ... wont that become a prob if core starts at 1 ?16:08
mvoogra_: not at all right now16:10
ogra_ah, k16:10
mvoogra_: we keep the two for a bit16:10
ogra_well, i understood your long-term plan was to simply switch people over16:11
ogra_i was just wondering how you handle the revision bits in such a case16:11
mvoogra_: this is still the case but we have not figured the details out yet16:11
ogra_ah, k16:11
mupPR snapd#2070 closed: all: move from ubuntu-core to core by default, but still support ubuntu-core as core <Created by chipaca> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2070>16:16
zygajdstrand, ara, mvo: https://bugs.launchpad.net/snap-confine/+bug/1628612 updated with SRU meta-data16:19
mupBug #1628612: prefer "core" rather than "ubuntu-core" if installed <Snappy Launcher:Fix Released by zyga> <https://launchpad.net/bugs/1628612>16:19
jdstrandmvo: hey, so I currently have both ubuntu-core and core installed. I have snapd and snap-confine from the image ppa. how can I remove ubuntu-core?16:20
zygajdstrand: you don't have to, we're going to work on making that happen later16:22
zygajdstrand: but with the current arrangement there's no rush16:22
jdstrandwell, snap interfaces is annoying me16:22
pmcgowanfor every new snap I install I get this when I run it execv failed: No such file or directory16:23
pmcgowanthe snaps I installed a while back work fine16:23
pmcgowansure would like to fix that16:23
zygajdstrand: ah16:25
zygajdstrand: oh, you are seeing double?16:25
zygajdstrand: I didn't try that16:25
zygaI wonder what's the plan for that (when oyu have both, do we move the connections over?)16:26
zygajoc: can you please take https://github.com/snapcore/snapd/pull/2065 over16:30
mupPR snapd#2065: interfaces/builtin: use udev to export GPIOs to userspace <Reviewed> <Created by zyga> <https://github.com/snapcore/snapd/pull/2065>16:30
zygajoc: just change that to what you suggested and if it works get it merged16:30
zygajoc: I need to take a nap16:30
zygaI feel so tired today16:30
zygajoc: I'll try to stay away from my computer for some time, be back when I wake up16:31
joczyga: ok, feel better16:31
jdstrandzyga: I am seeing double. I don't know what the plan for that is. I don't know how that affects connections16:31
zygajdstrand: ask chipaca16:32
zygajdstrand: he's been making that work for the past few days16:32
jdstrandI see16:34
pmcgowanjdstrand, zyga sorry to bother you but any idea why this happens http://pastebin.ubuntu.com/23275810/16:36
mupPR snapcraft#849 opened: Bugfixes for gated and validate commands <Created by ralsina> <https://github.com/snapcore/snapcraft/pull/849>16:47
jdstrandpmcgowan: I don't see anything wrt security policy in there. can you paste the output of 'snap list ; snap interfaces'16:50
jdstrandpmcgowan: also, what version of snapd and snap-confine are you using?16:51
pmcgowanhttp://paste.ubuntu.com/23275869/16:52
pmcgowan 2.15.2ubuntu1 and 1.0.38-0ubuntu0.16.04.1016:53
jdstrandpmcgowan: your ubuntu-core is ancient16:54
pmcgowanoh?16:54
jdstrandI don't know why that is16:54
jdstrandtry: sudo snap refresh ubuntu-core16:54
jdstrandpmcgowan: yeah, it should be revision 42316:55
pmcgowanjdstrand, do I need to periodically refresh it ?16:56
pmcgowanits updating now16:56
jdstrandyou're not supposed to have to16:56
jdstrandpmcgowan: you said that new snaps are failing. I suggest removing those failing snaps and installing again16:57
pmcgowanlets see once its updated16:58
* jdstrand nods16:58
pmcgowanI cant afford the bandwidth :)16:58
pmcgowanjdstrand, Connection to the daemon failed: Get http://127.0.0.1/enable: dial unix /var/snap/canonical-livepatch/6/livepatchd.sock: connect: no such file or directory16:59
pmcgowanother snaps are running now17:00
jdstrandpmcgowan: you uninstalled canonical-livepatch?17:00
jdstrandthen reinstalled?17:00
pmcgowannot yet17:00
jdstrandyou might be able to stop the service and restart it, but I'd prefer uninstall/reinstall17:01
pmcgowanjdstrand, working now, thanks17:01
jdstrandcool, np17:01
pmcgowanjdstrand, what about the snap not updating?17:02
pmcgowananything to look for there17:02
jdstrandpmcgowan: I don't know-- that might be a question for mvo or another on the snappy team17:02
pmcgowanjdstrand, is it due to the switch from ubuntu-core to core? should I remove one and get the other?17:12
ogra_mvo, https://code.launchpad.net/~snappy-dev/+snap/core ... i'll set up daily cronned builds tomorrow morning17:13
pmcgowanogra_, quassel-kalikiana in the stable channel :)17:13
ogra_pmcgowan, heh, right, he talked about it (wont make me change to quassel though :) )17:13
ogra_thanks !17:13
ogra_jdstrand, hmm, doesnt seem like the tooools changes for "core" are live yet ...17:14
ogra_https://myapps.developer.ubuntu.com/dev/click-apps/6021/rev/9/17:15
ogra_WOAH !17:20
mupPR snapcraft#850 opened: catkin plugin: add `rosdistro` to help documentation <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/850>17:20
ogra_and it seems htis one fail blocka *all* subsequent uploads of that snap17:20
jdstrandoh I missed one. shoot17:20
ogra_*blocks17:20
ogra_thats new17:20
ogra_rev 10-13 all show "Revision waiting for review of a previous upload."17:20
jdstrandyeah17:21
jdstrandthat's new17:21
ogra_given they are all for different arches thats rather bad17:21
ogra_it should take that into account somehow17:22
ogra_not just blindly block because a former revision failed17:22
jdstrandI don't know the implementation. the idea is to easy reviews. eg, say you upload a snap that uses a special interface. the tools flag for manual review. the reviewer says it is ok and that future uploads are allowed with this interface (ie, snap declaration is used). then approve and then all the other go through without being flagged cause they use that snap decl17:25
jdstrandease*17:25
ogra_well, what if an upload for ppc64el fails for some esoteric reason ...17:25
ogra_that will block all amd64 uploads afterwards17:25
jdstrandthat is an unusual case that I think would need more information to be smarter17:27
jdstrandnote, I didn't implement this17:27
jdstrandbut I agree with the concept17:27
ogra_thats totally not unsual ... regarding the design of store uploads17:27
ogra_if i build in LP for all arches s390x or ppc64el are usually the first to be done and the first to be uploaded17:28
ogra_if there is some ppc specific interface that i enable it will then block everything17:28
ogra_so i have to say i disagree with the concept :)17:29
ogra_thats like a lottery now ...17:31
jdstrandroadmr: sorry, I missed a check cor 'ubuntu-core' -> 'core'. can you pull r755?17:32
ogra_jdstrand, if your check was an arch specific oversight, dont forget s390x too ... (we dont have it yet for that snap since i need to apply for it at the LP team first)17:34
jdstrandogra_: it was not17:35
ogra_ah, k17:35
ogra_(i thought, because the other arches seem to have gone through before)17:35
jdstrandthough there is an arch check too17:35
jdstrandthis is the ubuntu-core rename to core17:35
ogra_(and looking closer i see they havent at all :P ... all manually approved before)17:36
jdstrandbut s390x might be an issue too17:36
ogra_well, it wasnt for ubuntu-core17:36
jdstrandogra_: I just checkd: s390x is considered valid by the tools. no worries there17:37
jdstrandogra_: ok, they are all approved. I see you are publishing them17:37
ogra_heh ... it is funny how the revision history disagrees about the uploader with the "submitted by" field above17:38
ogra_according to revision history it is the shared account ... according to "submitted by" it is me17:38
jdstrandogra_: fyi, you seem to have missed this one https://myapps.developer.ubuntu.com/dev/click-apps/6021/rev/9/17:39
ogra_(and after all it is LP under the snappy-dev account ... it just happened to be me who set it up)17:39
mupPR snapd#2084 opened: snap/squashfs: add -all-root flag to mksquashfs <Created by fgimenez> <https://github.com/snapcore/snapd/pull/2084>17:39
ogra_published rev917:39
ogra_roadmr, let me know when jdstrand's change was merged, then i can enable cronned daily builds for that snap17:40
ogra_(still missing a few bits on my side anyway, so no hurry)17:40
jdstrandroadmr: again, I apologize. I did the hard change last week, but the easy one I missed :\17:40
ogra_(this above was just the first manuall test build)17:41
mupPR snapd#2084 closed: snap/squashfs: add -all-root flag to mksquashfs <Created by fgimenez> <Closed by fgimenez> <https://github.com/snapcore/snapd/pull/2084>17:50
mupPR snapd#2085 opened: many: check installation of slots and plugs against declarations <Created by pedronis> <Conflict> <https://github.com/snapcore/snapd/pull/2085>17:55
roadmrjdstrand: no problem, I'll pull that now18:07
mupPR snapd#2086 opened: client, cmd: prompt for password when buying <Created by pete-woods> <Conflict> <https://github.com/snapcore/snapd/pull/2086>18:09
ralsinasergiusens, elopio: in a PR for snapcraft I am getting  "Coverage decreased (-0.006%)" which is funny, but also, we are checking coverage inside the tests, which makes no sense, right?18:09
om26erogra_, curious, whats the difference with ubuntu-core and core snap ? new name ?18:11
ogra_om26er, yep18:11
ogra_eventually we'll drop ubuntu-core18:12
om26erogra_, I assume its to make snap based systems to not be always tagged 'ubuntu' ?18:13
ogra_right ...18:13
ogra_the all-snap images will go on being called ubuntu-* ... but there is no need to call the core snap itsellf "ubuntu-"18:13
ogra_it is simply the core execution environment for snaps ...18:14
om26erunderstood, makes sense.18:14
mupPR snapd#2075 closed: cmd/snap: add version command, same as --version <Created by niemeyer> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/2075>18:46
mupPR snapcraft#850 closed: catkin plugin: add `rosdistro` to help documentation <Created by kyrofa> <Merged by kyrofa> <https://github.com/snapcore/snapcraft/pull/850>18:47
mupPR snapcraft#851 opened: catkin plugin: nicely handle an invalid rosdistro <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/851>19:08
mupPR snapd#2076 closed: asserts: limit model to only lowercase until we can implement case insensitive/preserving headers <Critical> <Created by pedronis> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2076>19:11
jdstrandroadmr: thanks19:35
mupPR snapd#2066 closed: many: abbreviated forms of disconnect <Created by zyga> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2066>19:36
mupPR snapd#2074 closed: client,daemon,overlord,cmd: add /v2/users and create-user on auto-import <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2074>19:48
mupBug #1629081 changed: "git clone" fails with "source-type: git" and "source-tag: x.y.z" in "snapcraft.yml" <snapcraft> <Snappy:Invalid> <https://launchpad.net/bugs/1629081>20:10
mwhudsongood morning20:12
mupPR snapd#2087 opened: snap, daemon, store: fake the channel map in the REST API <Created by chipaca> <https://github.com/snapcore/snapd/pull/2087>20:25
mupPR snapd#2079 closed: tests: use test snap from the store in the content interface test <Critical> <Created by fgimenez> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/2079>20:34
mupPR snapd#2088 opened: tests: fix snap-disconnect tests after core rename <Created by niemeyer> <https://github.com/snapcore/snapd/pull/2088>21:27
mupPR snapd#2089 opened: store: do not set store auth for local users <Created by matiasb> <https://github.com/snapcore/snapd/pull/2089>21:46
mupPR snapd#2090 opened: wip: initial cleaning up of no arg AutoConnect related bits <Blocked> <Created by pedronis> <https://github.com/snapcore/snapd/pull/2090>21:52
zygare22:25
mupPR snapd#2091 opened: interfaces: tweak wording and comment <Created by zyga> <https://github.com/snapcore/snapd/pull/2091>22:34
mupPR snapd#2088 closed: tests: fix snap-disconnect tests after core rename <Created by niemeyer> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2088>23:02

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