/srv/irclogs.ubuntu.com/2018/11/07/#snappy.txt

zygagood morning07:11
mvodoko: fyi - all adt tests on snapd are green except s390x, I take care of this one today07:12
mvohey zyga, good morning07:12
zyga:)07:50
pstolowskimorning08:09
Chipacamo'in09:33
Chipacamvo: EHLO09:33
mupPR snapcraft#2394 opened: build providers: fix osx non base and injection <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2394>09:33
Chipacamvo: I haven't merged #6093 because I hear it's wanted in 2.36 and don't  know if we still need to squash-merge things that need that09:34
mupPR #6093: daemon: spool sideloaded snap into blob dir <Created by chipaca> <https://github.com/snapcore/snapd/pull/6093>09:34
Chipacamvo: or should I write a separate PR and target it at the release branch?09:34
mvoChipaca: thats fine, pedronis wanted to have a quick look at 6093 before it is merged09:41
mvoChipaca: and once its in (ideally via squash merge) we can cherry pick09:41
Chipacaok09:42
pstolowskiChipaca: hey! one question to your userid&snapshots fix, ready to re-approve once clarified09:44
pedronismvo: Chipaca: looking at it now09:47
mupPR snapcraft#2393 closed: lifecycle: make snapcraft init template use > not | <Created by sparkiegeek> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2393>09:51
mupPR snapd#6102 closed: overlord/snapshots: survive an unknown user <Created by chipaca> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6102>10:00
pedronismvo: Chipaca: commented. mostly nitpicks that could go in a follow up but a bit worried at the cost vs frequency of cleanupOldTemps10:02
Chipacapedronis: ah, I'll add a time check thing10:04
Chipacapedronis: i got confused because i have a different pr that has this already :-)10:04
Chipacanot pr, a commit10:04
Chipacaa stash even10:04
Chipacathe one with warnings for devmode things :-)10:04
mborzeckimorning10:11
pstolowskihey mborzecki10:15
zygahey mborzecki10:17
mborzeckistuck in a slightly boring talk, doing reviews :)10:18
zygahaha10:19
pedronismvo: Chipaca:  https://github.com/snapcore/snapd/pull/6101 is green but doesn't have a description/motivation10:19
mupPR #6101: switch travis unit tests to xenial <Created by chipaca> <https://github.com/snapcore/snapd/pull/6101>10:19
mborzeckibtw. there should be some video streams at http://codedive.pl/ in case anyone is interested10:19
diddledandesktop helpers part for the post-remote-parts world https://www.irccloud.com/pastebin/fwLMLqKc/10:26
mupPR snapcraft#2395 opened: Fixed Errors of macOS environment <Created by hsbt> <https://github.com/snapcore/snapcraft/pull/2395>10:33
Chipacapedronis: i'll add motivation10:40
Chipacapedronis: it's not a strong one right now :-) but will be someday10:40
Chipacasoon i hope10:40
pedronisChipaca: thx10:43
pedronisChipaca: ah, dist: xenial wasn't a thing until now10:57
pedronis?10:57
Chipacapedronis: it's not a thing until tomorrow, officially10:57
pedronisheh10:57
pedronisok10:57
pedronisChipaca: so we should not merge it until tomorrow?10:57
Chipacapedronis: up to us :-)11:00
Chipacapedronis: it's no big secret, tomorrow is when it's supported officially but we're sitting next to them today and tomorrow :-)11:00
Chipacapedronis: we can also say "nah not worth it" and close the pr11:00
pedronisChipaca: no, it's fine, just trying to understand11:01
mupPR snapcraft#2394 closed: build providers: fix osx non base and injection <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2394>11:33
zygamvo: https://forum.snapcraft.io/t/lack-of-compiled-locales-breaks-gettext-based-localisation/3758/1611:36
zygalocale strikes back11:36
zygaI would love for us to include locale in core instead of having layers of hacks like that11:37
zygabr11:40
zygabrb11:40
mupPR snapcraft#2395 closed: Fixed Errors of macOS environment <Created by hsbt> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2395>11:48
pedronisChipaca: I'm going to review your epochs PR, but if I understand correctly is not the full story, we should chat about epochs next week11:51
cachiomvo, hey, I see this often on core 1811:53
cachiohttps://paste.ubuntu.com/p/G6GFxSphzs/11:53
Chipacapedronis: +111:58
Chipacapedronis: AIUI the bit missing is the refresh revving, but I might've missed something else11:58
mupPR snapd#6104 opened: snapctl: add "services" <Created by chipaca> <https://github.com/snapcore/snapd/pull/6104>11:58
Chipacapstolowski: ^^^ might be of interest to you.11:58
Chipacaalso, hm, wut, missing services in -h11:59
* Chipaca investigates12:00
Chipacaoh d'oh12:01
Chipacaalso, also, d'oh² broke unit tests12:01
pstolowskiChipaca: thanks, will take a look12:05
mvocachio: when you see this, what is the status of "systemctl status snapd"?12:05
mvocachio: did you managed to reproduce it?12:05
cachiomvo, no12:05
cachioI cant reproduce it12:05
mvo:/12:06
mvook12:06
cachioI just see the error on travis12:06
cachioI can create a PR to force it12:06
mvoI need to look at this then and try to understand how it can happen, if its frequent it might be worthwhile to add debug: | for that12:06
cachiomvo, which debug info do yo uneed12:06
cachiojust status for snapd¡12:07
mvocachio: systemctl status snapd would be good for a start12:07
mvocachio: maybe journalctl -u snapd as well just to be on the safe side12:07
mvocachio: it might be that its just running too early and no snapd is around at all yet12:07
mvocachio: in which case its probably harmless but let me look at the test again after lunch with fresh eyes12:07
cachiook, I'll create a PR to for that12:07
cachiosure12:08
mupPR snapd#6105 opened: NOT REVIEW: New task to force error on degraded test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6105>12:19
oSoMoNzyga, on fedora with core18, xdg-open doesn't work, is that a known issue?12:28
oSoMoN"/usr/bin/xdg-open: 2: exec: snapctl: not found"12:28
zygaoSoMoN: no, I don't believe this is know12:28
zygawhich snapd version?12:28
zygathere is an update: https://bodhi.fedoraproject.org/updates/FEDORA-2018-48cc10ba1d12:29
zygacould you please test it12:29
zygaand report on the update page12:29
oSoMoNzyga, ack, will do just after lunch and will keep you posted12:29
zygathank you12:29
pedronismvo: I added some comments to the originally dotfiles PR, about the name and the base declaration bits12:32
mupPR snapd#6106 opened: overlord/ifacestate: Handler for hotplug-update-slot tasks <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6106>12:34
oSoMoNzyga, the update fixes my issue with xdg-open \o/13:22
oSoMoNI'll report on the test page13:22
zygawhee, that's great13:22
zygathank you13:22
mvopedronis: thank you!13:28
zygaflock unit test randomly failing  https://www.irccloud.com/pastebin/eL5WynJ7/13:41
andyrockdo snaps have access to /run ?13:45
zygaandyrock: to some parts of run, yes13:47
zygait depends on specific path13:47
andyrockzyga: e.g. if there is a snap called "random-snap" can I access /run/random-snap ?13:49
andyrockotherwise how does it work?13:49
zygaandyrock: are you asking if some snap can access /run/random-snap?13:51
zygain general the answer is no13:51
zygain specific case the answer may be yes13:51
zygait is defined by the snap interface system13:51
zygaby the set of plugs, slots and the connections between them13:51
seb128zyga, the real question is "what would it take to let the livepatch snap write it's status file in /run (so it's auto-cleaned on reboot)" I think13:52
seb128andyrock, ^ right?13:52
seb128jdstrand, ^13:52
andyrockzyga: yes, livepatch snap write a status file to communicate with update-notifier13:53
zygaseb128, andyrock a trivial tweak to the interface I suspect13:54
* zyga looks13:54
jdstrandandyrock: the snap already has access to XDG_RUNTIME_DIR, if that is helpful:13:54
jdstrand$ sudo hello-world.env |grep XDG_RUNTIME_DIR13:54
jdstrandXDG_RUNTIME_DIR=/run/user/0/snap.hello-world13:54
andyrockright now that file is in /var/snapd/canonical-livepatch/current/status13:54
zygaah, there's no live patch interface13:54
* jdstrand is assuming it is the daemon that needs write access to it13:55
andyrockbut considering that files applies to a boot session would be nice to have it under a "run" directory13:55
jdstrandXDG_RUNTIME_DIR still isn't created by anything, byt you can mkdir it and write it there13:55
andyrock*considering that the status file13:56
jdstrandandyrock: I suggest looking at XDG_RUNTIME_DIR. you don't need any changes to snapd for that13:56
andyrockjdstrand: is that directory accessible by the system?14:00
jdstrandandyrock: yes14:00
andyrockjdstrand: kk. It looks like it's what we need14:01
andyrockthx14:01
jdstrandoh hrmm, /run/user/0 doesn't exist and the snap can't create it. zyga, that really needs to be fixed somewhere14:03
jdstrandandyrock: ^14:03
mupPR snapd#6107 opened: cmd/snap, snapctl: add column selectors to services <Created by chipaca> <https://github.com/snapcore/snapd/pull/6107>14:04
zyga(standup)14:04
jdstrandzyga: I've always argued that the session manager (ie, systemd) should do it, but it isn't14:04
zygajdstrand: I think we can do it now14:04
jdstrandfor non-root it all works. we should probably have snapd just create it14:04
jdstrandor snap-confine, which is where I think you are going14:05
zygayes14:05
zygasnap-update-ns specifically14:05
jdstrandandyrock: you will need some snapd support after all14:05
zygaanyway, I will think about it14:05
zygaandyrock, seb128: can you please file a bug on this14:05
zygaOR find one that exists14:05
zygaabout XDG_RUNTIME_DIR for root14:05
zygaI'll assign that to myself14:05
jdstrandzyga: note, it needs to preserve the property that the host can see it14:05
zygaright14:05
=== matteo| is now known as matteo
jdstrandzyga: thanks!14:05
seb128jdstrand, zyga, those are created/destroyed with the corresponding user session, if root has no login/session that's why it doesn't exist14:07
seb128it would make more sense to have a /run/livepatch-status14:08
seb128but that might require a custom interface for livepath then...14:09
seb128unless there is a new rule to allow using "/run/<snap_name>"14:09
zygaseb128: I think we could do a custom interface as well14:17
zygajdstrand: ^14:18
zygaseb128: there is no such rule today14:18
seb128right14:18
zygaseb128: perhaps we could add /run/snap.$SNAP_NAME14:18
seb128well either way it seems it needs changes in snapd14:18
zygathat might be a nice default14:18
seb128right14:18
zygaseb128: yes, for sure14:18
seb128I think it would be useful14:18
seb128for others snaps as well14:18
zygaI agree14:18
zygaand would be easier than a new interface14:18
seb128indeed14:18
zygasomething for 2.36.1 perhaps14:18
seb128I like it14:18
seb128where should that wishlist be reported? github? forum?14:19
zygaseb128: I actually prefer a bug report but it can also be a forum14:19
zygaseb128: bug is something I can assign to myself14:19
zygaseb128: forum is nicer for discussion14:19
seb128k, so basically you want both :)14:19
seb128andyrock, want me to open those?14:20
andyrockseb128: as you prefer, otherwise I'll do that once I finish what I'm currently working on14:20
seb128andyrock, I'm doing it14:20
zygathank you!14:22
seb128zyga, https://forum.snapcraft.io/t/snaps-can-not-write-to-run-by-default/8367 and https://bugs.launchpad.net/snapd/+bug/180211214:33
mupBug #1802112: Snaps can't write to /run by default <snapd:New> <https://launchpad.net/bugs/1802112>14:33
zygathank you14:33
zygapedronis: ^ can you look at this please - if there's agreement to allow snaps to write to /run/snap.$SNAP_NAME then it could be a trivial one liner change for 2.36.1 - the use case is live patch dropping a status file that some part of classic world can look at14:34
pedroniszyga: it does sound reasonable and aligned to other things we do, we do need jdstrand input on it tough15:16
popeyjdstrand: is there a way in which a snap can set network configuration? Specifically setting a static IP address? On core?15:19
pedroniszyga: I answered in the bug15:21
zygathank you15:22
* pedronis break15:22
zygajdstrand: can you please look at https://bugs.launchpad.net/snapd/+bug/180211215:23
mupBug #1802112: Snaps can't write to /run by default <snapd:New for zyga> <https://launchpad.net/bugs/1802112>15:23
jdstrandpopey: network-control gives all the raw lowlevel stuff (eg, use of 'ip')15:25
popeyjdstrand: even on core?15:26
jdstrandpopey: yes15:26
popeyawesome15:26
jdstrandpopey: if you want netplan configuration, then look at network-setup-control15:26
jdstrandpopey: or if they have network-manager installed, plugs network-manager15:27
popeyjdstrand: thanks, this is a 'legacy'15:27
jdstrandpopey: there may be some interplay with netplan that they'll need to work through when using network-control, but network-control will definitely allow them to do stuff. ogra might have tips15:28
popeyok, ta15:29
ograpopey, https://github.com/ogra1/dashkiosk-image-config netplan-import might be interesting15:29
popeythe problem is this snap needs to run on core and non-core15:30
popeyand could run on non-ubuntu15:30
popeyso needs to special case all the random nonsense ways people set IP addresses etc15:30
ogra(copies any "netplan.yaml" file from a USB device you plug into the device and reboots with new network config)15:30
ograoh, ok15:30
ograpopey, in any case network-setup-control is enough to replace the config in /etc/netplan ... you want the shutdown interface too and trigger a reboot for it to take effect (netplan generate/apply are not allowed atm)15:31
pedronisChipaca: is #6107 something that was discussed at the summit?15:32
ograto check for core just look for snap_core= in /proc/cmdline (shouldnt be restricted)15:32
mupPR #6107: cmd/snap, snapctl: add column selectors to services <Created by chipaca> <https://github.com/snapcore/snapd/pull/6107>15:32
Chipacapedronis: yes15:32
Chipacapedronis: gustavo's suggestion actually15:33
Chipacapedronis: reasoning  being that the table output isn't too script-friendly15:33
pedronisChipaca: I see, that's a bit true of a lot of our commands tough15:35
pedronisChipaca: is this one particurly egregious?15:35
pedronisChipaca: I mean, do we need a choose your column general strategy instead?15:37
Chipacapedronis: this one is the only one in snapctl afaik15:37
Chipacapedronis: I asked the same thing though,  --column=foo15:37
Chipacapedronis: but, translations15:37
pedronisChipaca: the only one, doesn't sound a statement that will be true forever15:38
* Chipaca nods15:38
pedronisChipaca: I understand that we tried something for snap list and it was a quagmire15:39
Chipacapedronis: with --format you mean? yes15:39
pedronisyes15:39
pedronisChipaca: I understand I'm not being super constructive, but I'm trying to understand if it's the start of a slippery slope15:41
Chipacapedronis: I understand :-)15:41
Chipacazyga: ping15:45
zygayes sir15:45
Chipacazyga: https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/180212415:45
mupBug #1802124: Error installing chromium-mir-kiosk on RPi3/edge <snapd (Ubuntu):New> <https://launchpad.net/bugs/1802124>15:45
Chipacazyga: does this mean we need to move those to per-arch code?15:46
zygayes15:46
Chipacazyga: note the lookup table thing is already per-arch15:46
zygaI though that was exactly what you did15:46
zygaaaah15:46
Chipacazyga: ah indeed15:46
zygaI thought it was all per-arch15:46
zygayes, I'm afraid that's needed15:46
zygaChipaca: while I have you15:46
* Chipaca is had by very few people15:46
Croephadoes the snap environment use the core system's ld-linux-x86-64.so.2 or can you ship your own? having glibc version issues15:47
zygaquick question about this particular piece of code https://github.com/snapcore/snapd/blob/master/run-checks#L18015:47
Chipacazyga: shoot15:47
ChipacaCroepha: why are you having glibc version issues15:48
zygaChipaca: is it legitimately complaining about this usage: https://github.com/snapcore/snapd/pull/6095/files#diff-36a23153f5bbd7bffd11c24597ac50fdR23815:48
mupPR #6095: packaging/opensuse: stop using golang-packaging <Created by zyga> <https://github.com/snapcore/snapd/pull/6095>15:48
ChipacaCroepha: is your snap classic, or strict?15:48
CroephaChipaca: im shipping a newer version of libpython and it requires a newer version of glibc than what is in core, I wasn't using strict, I was using try, right now im just using a chroot directory in my home while I do some development iterations15:51
ChipacaCroepha: you get glibc mismatches because you build against a different glibc than the one in the core you specify15:52
ChipacaCroepha: I seriously doubt libpython requires a  particular glibc version, although it's quite likely that a particular version of libpython is only available in a distribution with a newer glibc15:53
ChipacaCroepha: you can probably build the newer libpython against the older glibc15:53
ChipacaCroepha: or you can use a newer base15:53
ChipacaCroepha: or you can tell snapcraft to ship your own libc in the snap15:54
ChipacaCroepha: in general the last one is not a good idea15:54
Chipacabut, it's there if you need it15:54
pedronismvo: question, do we check when installing if bases have actually type: base,  I looked around and didn't find such a check15:54
Chipaca(you probably don't need it)15:54
Croephaok, thanks Chipaca15:54
Chipacazyga: thinking about it15:54
zygayes?15:54
Chipacazyga: i mean, the usage is ok, although I don't know what an empty entry in GOPATH does15:55
zygain this specific case it's never empty15:55
zygasuse is a bit looney that it sets GO{ARCH,OS,PATH,ROOT} for all users15:55
zygathere's a /etc/profile.d/go.sh15:55
zygathat contains values matching some compiler (via alternatives)15:56
pedroniszyga: are these layout docs:  https://forum.snapcraft.io/t/snap-layouts/7207 ? does it need to mention that the experimental flag is no more now15:56
mvopedronis: not sure, I need to look. will do so after the meeting15:56
pedronismvo: thank you15:56
zygapedronis: that's correct,15:56
zygapedronis: I will amend it to say that since snapcraft 3.0 that uses bases and since snapd 2.36 it is not experimental anymore15:56
pedronisthanks15:57
Chipacazyga: so, i've got a fix for you15:59
zygapedronis, degville: I edited https://forum.snapcraft.io/t/snap-layouts/7207 to account for the new reality (enabled since 2.36 and since snapcraft 3.0 with bases)16:00
Chipacazyga: http://paste.ubuntu.com/p/zJPFQnRgX8/16:00
Chipacazyga: bundle that in if you can16:00
zygathank you!16:00
zygayep, I will16:00
degvillezyga: thanks for the heads-up!16:01
pedroniszyga: degville: should layout be a top level topic in the docs? either way I think snap format doc should mention them and link to this16:01
pedronisI'm talking about  https://docs.snapcraft.io/the-snap-format/698 -> layout docs16:02
Chipacazyga: the hint was in what exactly was getting highlighted in the output :-)16:02
zygapedronis: I think it should be referenced from the format spec16:03
zyganot otherwise top-level IMO16:03
pedronisok16:03
degvillepedronis, zyga : yep, I agree.16:04
pedroniszyga: degville: I left a note for you in the forum topic about this16:05
degvillethanks!16:05
pedronisthank you16:05
roadmrHello folks; we're currently investigating a snap store outage, snap requests may be slow or fail intermittently.16:10
zygaoh, thank you for the note roadmr16:10
=== sparkieg` is now known as sparkiegeek
* zyga needs coffee16:41
cachioroadmr, thanks16:43
roadmrsnap store should be back to normal now, folks. Thanks for your patience16:48
pstolowskiChipaca: re 6104, making it 2 separate PRs (1 PR to just move the old code around) would make review easier16:55
cachiomvo, https://travis-ci.org/snapcore/snapd/jobs/451975712#L123417:10
cachiothis is the error on degraded with extended debug info17:10
cachiomvo, I see some retries to get the catalod17:12
dokomvo: snapd failed to build on amd6417:13
cachiomvo, but nothins weird apart fo that17:13
zygare17:14
zygaroadmr: thank you!17:14
* zyga got involved in some homework17:14
=== pstolowski is now known as pstolowski|afk
mvodoko: looking17:28
mvocachio: if you are on the system, can you please paste the output of "journalctl -u snapd.seeded.service" and "systemctl status snapd.seeded" ?17:29
Chipacapstolowski|afk: noted17:32
cachiocachio, I am not but I'll trigger it again17:34
cachiozyga, updating scripts to create tumbleweed image17:38
Chipacapstolowski|afk: not sure i'll be able to do it tonight though but i'll try17:39
cachiozyga, it should be ready soon17:39
Chipacayesterday i fell asleep on the train and stuff17:39
* Chipaca really tired by all this weird "human" interaction17:39
mupPR snapd#6108 opened: many: apparmor support for rpm distros <Created by jdstrand> <https://github.com/snapcore/snapd/pull/6108>17:39
Chipaca:)17:39
zygacachio: super, thanks17:40
=== shaner is now known as shaner2_
=== shaner2_ is now known as shaner
mupPR snapd#6108 closed: many: apparmor support for non-AA rpm distros <Created by jdstrand> <Closed by jdstrand> <https://github.com/snapcore/snapd/pull/6108>19:10
zygajdstrand: FYI https://github.com/snapcore/snapd/pull/6108#issuecomment-43675036619:36
mupPR #6108: many: apparmor support for non-AA rpm distros <Created by jdstrand> <Closed by jdstrand> <https://github.com/snapcore/snapd/pull/6108>19:36
zygajdstrand: sorry,  if I knew you were interested in packaging I would have said something19:36
zygaI'm working on packaging for a few days now19:36
zygajdstrand: I will propose the stuff I have as soon as https://github.com/snapcore/snapd/pull/6095 lands19:37
mupPR #6095: packaging/opensuse: stop using golang-packaging <Created by zyga> <https://github.com/snapcore/snapd/pull/6095>19:37
mupPR snapd#6095 closed: packaging/opensuse: stop using golang-packaging <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6095>19:53
mupPR snapd#6108 opened: many: apparmor support for non-AA rpm distros <Created by jdstrand> <https://github.com/snapcore/snapd/pull/6108>19:59
cachiozyga, finally the image ready20:02
cachiozyga, and I could create a script to automate the process20:02
zygacachio: super20:02
zygaI can run a tumbleweed test quickly20:03
cachioyes20:03
cachiojust use image: opensuse-tumbleweed-6420:03
zygathank you, I will report back20:04
cachiozyga, great, thanks20:04
=== chrisccoulson_ is now known as chrisccoulson
rogpeppewhat's the story about running external executables inside a snap? I'd like to publish a couple of snaps that work on Go source code, and these days the only way to do that is by invoking the go command. Should I package up the go compiler into my snaps too?23:37

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