/srv/irclogs.ubuntu.com/2019/03/15/#snappy.txt

kenvandinepedronis: thanks01:17
mupPR snapd#6604 opened: tests: tests add check to detect a broken snap on reset <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6604>01:52
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== cpaelzer__ is now known as cpaelzer
zygaGood morning06:03
zygastgraber: ouch, thanks for sharing!06:03
=== chihchun_afk is now known as chihchun
mborzeckimorning06:22
zygaHey hey06:26
zygaMore rain today06:27
zygaI’ll be home soon, just finishing morning walk06:27
zygaback in the office now06:42
zygamborzecki: hey, what is https://github.com/snapcore/snapd/pull/6329 blocked on?06:56
mupPR #6329: cmd/snap-confine, packaging: support SELinux <SELinux> <â›” Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6329>06:56
zygaor blocked by06:56
mborzeckiah, it's not now07:03
mborzeckizyga: forgot to take the label off07:03
zyga:-)07:03
zygala-la-land-it07:03
mborzeckioh, and it's green too07:04
zygaquick, before it turns sour ;)07:04
mborzeckihaha07:04
mborzeckipedronis: do you want to take a look at #6329? it's snap-confine07:04
mupPR #6329: cmd/snap-confine, packaging: support SELinux <SELinux> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6329>07:04
zygahttps://github.com/snapcore/snapd/pull/6601 is a low hanging fruit07:05
mupPR #6601: errortracker: fix panic in Report if db cannot be opened <Simple 😃> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6601>07:05
zygapstolowski: https://github.com/snapcore/snapd/pull/6601 is green and +2 :)07:06
mupPR #6601: errortracker: fix panic in Report if db cannot be opened <Simple 😃> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6601>07:06
* zyga breakfast07:06
mborzeckiwow, emacs crashed07:08
zygatoo many open tabs?07:10
mborzeckizyga: backtrace leads back to imagemagick07:19
zygaI...07:19
zygawell07:19
mborzeckimust have been that treemacs thing i tried07:19
zygaunusual but what can I say07:19
mborzeckihmm https://paste.ubuntu.com/p/h7MtPn3cYr/07:20
pedronismborzecki: do we need jdstrand reviews for 6485 and 6592, I thought he +1 the original long branch? did things change a lot from it?07:35
pedronissorry07:35
pedronismborzecki: 6592 and 659307:35
pedroniswe can still ask him to (re)review the final bit07:36
mborzeckipedronis: only significant change in 6592 is the test to check whether we accounted for all syscalls known to seccomp07:37
mborzecki6593 has a little api tweak, but that's all07:37
pedronismborzecki: yea, I think if they have two review they can land without jdstrand07:38
mborzeckipedronis: ack07:38
mupPR snapd#6593 closed: sandbox/seccomp: a helper package wrapping calls to snap-seccomp <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6593>07:39
mupBug #1819629 changed: no default locale in core18 causes pam errors <snap-core18:Fix Released by sil2100> <Snappy:Fix Released by sil2100> <https://launchpad.net/bugs/1819629>07:41
mupPR snapd#6601 closed: errortracker: fix panic in Report if db cannot be opened <Simple 😃> <⚠ Critical> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6601>07:49
mvosil2100: hey, hope I don't get on your nerves already - if you could upload the bionic version of systemd from -proposed to ppa:canonical-foundations/ubuntu-image that would be awesome, then I can trigger a core18 build and add UC18 to the regression test for the systemctl hang07:53
mvosil2100: (I would love to do that myself and safe you trouble but I don't have the right permissions)07:53
zygahello mvo :)07:58
pedronismvo: hello07:58
pedronismvo: sil2100:  ogra last night was reporting that snapfuse is not available with core1807:58
mvozyga: good morning!07:59
mvopedronis: hm,hm, thats a tricky one, it comes from the snapd binary and is in /usr/bin/snapfuse, we should probably move it to /usr/lib/snapd/snapfuse and make /usr/bin/snapfuse a symlink08:01
pstolowskimornings!08:01
mvohey pstolowski08:01
mvosil2100: I'm looking at the xenial systemd ADT issues right now and I see enough failure to think we should pull the update for now :(08:07
mvosil2100: at least the xenial version, looking at bionic next08:07
sil2100mvo: I thought I copied systemd to ubuntu-image yesterday08:08
* sil2100 looks08:08
sil2100mvo: yeah, systemd is there since yesterday systemd 237-3ubuntu10.1608:09
sil2100(binary copy, identical to the one in -proposed)08:09
mvosil2100: oh, maybe I looked at the wrong thing, sorry08:09
mvosil2100: we need to pull the xenial verison :( I just asked in #ubuntu-release08:09
* zyga just updated to .4 via SRU08:17
zygathanks everyone!08:17
zygamvo: what's wrong with the systemd update?08:17
mvosil2100: and one last question (promised!) - did steve had any idea why trusty snapd was in dep-wait state?08:20
sil2100mvo: no, I guess we'll have to dig into that today again, he just said demoting snapd to universe is not an option, but in the end he didn't say if our predictions about the reason for the failure are correct08:22
sil2100mvo: btw. so hmmm, even without us copying systemd to ubuntu-image I guess should have the new systemd?08:23
sil2100mvo: I mean, aren't we getting systemd from ubuntu-base?08:23
mvozyga: some failures like this http://autopkgtest.ubuntu.com/packages/p/python-systemd/xenial/i38608:23
mvosil2100: we get it from ubuntu-base - is that building from -proposed?08:24
sil2100mvo: yes, which now actually scares me a bit08:24
mvozyga: https://paste.ubuntu.com/p/8hF3mPxKTR/08:24
mvosil2100: uhhhh08:25
mvosil2100: yes, me too08:25
sil2100mvo: since it means in the core18 snap we're actually using packages from -proposed everytime ;/08:25
mvosil2100: I was not aware of this08:25
mvosil2100: yeah, also people who build other things on ubuntu-core may not be aware of this08:25
mvosil2100: is this a conscious decision and if so, whats the reasoning behing it?08:25
sil2100mvo: yeah, so by default in Ubuntu when a stable lts series is released but still gets its daily-builds, they're by default built using -proposed to ease testing of proposed packages IIRC08:26
sil2100Anyway, this needs to be fixed08:27
sil2100Somehow08:27
sil2100Either by revisiting this policy or by building ubuntu-base without -proposed somewhere for our purposes08:27
* sil2100 gets the shivers now08:28
mvosil2100: thank you!08:28
* zyga hugs mvo08:37
zygathank you for being on the front line of systemd issues08:37
mborzeckitests unhappy again or something really broke? Remove data for snap "test-snapd-tools" (x1) (failed to remove snap "test-snapd-tools" base directory: remove /var/snap/test-snapd-tools: directory not empty)08:38
pedronismborzecki: there was a change in that area:  273d4853d08:41
zygamborzecki: it sounds like a known issue08:41
zygamborzecki: chipaca and I discussed it about two weeks ago08:41
zygamborzecki: rm -rf vs rmdir08:42
Chipacaso, turns out that having a snap confuse snapd with the assertions in a refresh is pretty bad09:18
Chipacai now have a broken snap i can't remove :-|09:18
* Chipaca encourages snapd to remove it09:22
* pstolowski runs a quick errand09:48
ograsil2100, the enw gadget is fine ... the user with the issue also confirmed he sees the missing interfaces now09:58
sil2100ogra: \o/09:58
ogras/enw/new/ ... (tsk ... *glares at his fingers* )09:59
sil2100ogra: hm, crap, forgot to ask CE for validation, anyway, I'll make it move forward today09:59
ograi guess the user is fine using edge for the moment, so i doubt there is any pressure09:59
mupPR snapd#6592 closed: cmd/snap-seccomp: version-info subcommand <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6592>10:05
ogramvo, as a reminder ... https://bugs.launchpad.net/snapd/+bug/182024210:17
mupBug #1820242: snapfuse missing in core18 breaks snap installation in lxd containers <snapd:New> <https://launchpad.net/bugs/1820242>10:17
pstolowskire10:23
mupPR snapd#6605 opened: cmd/libsnap,osutil: fix parsing of mountinfo <Created by zyga> <https://github.com/snapcore/snapd/pull/6605>10:49
zygabrb, coffee11:01
* zyga really goes to grab that coffee now11:23
cachioniemeyer, hey, I yesterday pushed this one https://github.com/snapcore/spread/pull/7511:23
mupPR spread#75: Make spread tests for spread project run on google backend <Created by sergiocazzolato> <https://github.com/snapcore/spread/pull/75>11:23
mborzeckii've updated #6485, anyone who was involved in the review, please take a look11:30
mupPR #6485: interfaces/seccomp: regenerate changed profiles only <â›” Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6485>11:30
niemeyercachio: Reviewed11:37
cachioniemeyer, thanks11:44
=== ricab is now known as ricab|lunch
=== chihchun is now known as chihchun_afk
SaviqoSoMoN: hey, I've been trying to update the subsurface snap for new snapcraft (no remote bases) https://github.com/Subsurface-divelog/subsurface/compare/master...Saviq:update-snap-packaging - but the launcher doesn't seem to be setting the QML env... you seem to be somewhat active in that repo ;), any idea?12:28
Saviq`$ snap run --shell subsurface -c '$SNAP/bin/desktop-launch env' | grep QML` turns up empty12:28
Saviqah I think I see the problem12:29
mupPR snapd#6606 opened: selinux, systemd: support mount contexts for snap images <SELinux> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6606>12:37
mborzeckianother selinux piece ^12:37
mupPR snapd#6607 opened: cmd: typedef mountinfo structures <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6607>12:54
* zyga has a *wild* idea13:02
zyga**wild**13:02
=== ricab|lunch is now known as ricab
jdstrandmborzecki: I know you're working on selinux support in snapd. I'm not sure of your grand plans but note there are *many* limitations to ultimately have an selinux backend that is equivalent to apparmor and it really isn't feasible. lsm stacking is the answer for strict snaps on selinux systems13:24
jdstrandmborzecki: that's been discussed elsewhere and a long time ago and not sure you were aware13:25
jdstrandthat isn't to say it is impossible. but there would be a high up front and persistent maintenance cost13:30
jdstrandwith stacking, that's all avoided (again, I'm talking about strict mode snaps, not targetted policy for snapd to run them (which, aiui, is what you're working on and good))13:32
* jdstrand might be optimistic on 'not impossible'. there would need to be a lot of investigation13:33
cachiomvo, pedronis hey, do you know is anyone is looking at the error: task.go:337: DEBUG: 2019-03-15T10:29:19Z ERROR failed to remove snap "test-snapd-tools" base directory: remove /var/snap/test-snapd-tools: directory not empty?13:34
pedronisChipaca: hi, zyga said you were looking into that area at some point ^13:35
pedronisalso it's close to were you landed something recently13:35
Chipacayes, i landed a fix for a cleanup that wasn't happening that would cause that13:36
Chipacacachio: you'll have some random revision in /var/snap/test-snapd-tools, rm -r that revision, try the remove again, should work13:36
cachioChipaca, it is failing on the master13:37
cachioon the travis executions13:37
Chipacaevery time?13:37
cachioI saw many testa failing because of that13:37
Chipacahmm13:37
Chipacadunno then13:37
cachioChipaca, https://api.travis-ci.org/v3/job/506691205/log.txt13:38
Chipacamore cleanup needed?13:38
Chipacawhoa13:38
Chipacacachio: could the tar be wrong in some way?13:38
Chipacaanyway, I'll look13:39
cachioChipaca, , not sure, It didnt change13:39
cachioI though it was related to the clean up changes13:39
cachioChipaca, thanks13:39
Chipacacachio: the cleanup changes should've made it not happen, not happen more :-)13:41
cachioChipaca, :)13:42
oSoMoNSaviq, problem solved, then? (just back from lunch and seeing your messages)13:44
SaviqoSoMoN: yeah, would be solved even more if https://github.com/ubuntu/snapcraft-desktop-helpers/pull/177 was merged ;) can you review in helpers?13:48
mupPR ubuntu/snapcraft-desktop-helpers#177: [qt] fix prepend_dir usage <Created by Saviq> <https://github.com/ubuntu/snapcraft-desktop-helpers/pull/177>13:48
oSoMoNlooking13:49
oSoMoNSaviq, that looks fine, I guess the gtk part of the helpers gets much more testing than its qt counterpart, which would explain why this slipped in13:52
Saviqindeed13:54
mvocachio: I saw that error in some tests but it does not ring any bell13:56
mborzeckijdstrand: no worries, right now it's merely accounting for what snapd, s-c, s-u-n and s-d-n do so that they do not cause unnecessary selinux denials13:56
mvocachio: we did get a new systemd recently though13:57
mborzeckijdstrand: similar thing for snaps with sevices, make sure that it's possible to start them without causing additional denials13:57
mupPR snapd#6608 opened: cmd/snap-confine: umount scratch dir using UMOUNT_NOFOLLOW <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6608>14:01
jdstrandmborzecki: yep, that's all good and thanks for doing that :) just wanted to make sure you got the memo on the other bits14:04
dot-tobiasI'd like to put configuration in a gadget.yaml for a snap that I have not yet released on the store. I have its snap ID, but putting that in gadget.yaml seems to have no effect. The snap is included in ubuntu-image via --extra-snaps and installs fine, just no configuration applied. Also, connecting another snap to it does not do anything (nothing in snap tasks #id), so I suspect it's something with the ID. An idea, anyone?14:15
mupPR snapd#6609 opened: snap/gadget: introduce volume update info <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6609>14:19
Chipacadot-tobias: do you have the right id?14:28
zygajdstrand: suse review is going forward, they will be looking at apparmor and snapd proper as well14:51
zygajdstrand: I must say I'm very pleased with the outcome so far and with the interaction in particular14:52
jdstrandzyga: curious what they are looking at with apparmor; they have had it for years. I guess the policy14:52
jdstrandbut regardless, cool and yes14:53
zygajdstrand: yes, they are reviewing the policy15:08
Chipacaniemeyer: if you could find the time to read https://forum.snapcraft.io/t/use-of-markdown-in-snap-metadata-summary-description/2128/40 and comment there I'd appreciate it15:11
niemeyerChipaca: Done15:16
Chipacaniemeyer: thank you15:16
Chipacapedronis: http://paste.ubuntu.com/p/qKBpN4Zsjz/ fwiw15:23
pedronisChipaca: it looks kind of what I would have expected15:28
Chipacapedronis: I maintain that it works by magic and happy accident15:28
Chipacabut if you're happy with it until we refactor, ok15:28
pedronisChipaca: we need store changes too, though?15:28
Chipacapedronis: for the more limited retry strategy, yes?15:29
pedronisyes15:29
Chipacapedronis: yes15:29
Chipacabut those don't scare me :-)15:29
pedronisok15:29
Chipacatrying to repro https://api.travis-ci.org/v3/job/506691205/log.txt now though15:29
* cachio lunch15:40
=== chihchun_afk is now known as chihchun
mupPR # closed: snapcraft#1649, snapcraft#1875, snapcraft#2020, snapcraft#2135, snapcraft#2176, snapcraft#2229, snapcraft#2239, snapcraft#2398, snapcraft#2413, snapcraft#2433,16:47
mupsnapcraft#2444, snapcraft#2445, snapcraft#2463, snapcraft#2470, snapcraft#2473, snapcraft#2493, snapcraft#2495, snapcraft#2500, snapcraft#250216:47
mupPR # opened: snapcraft#1649, snapcraft#1875, snapcraft#2020, snapcraft#2135, snapcraft#2176, snapcraft#2229, snapcraft#2239, snapcraft#2398, snapcraft#2413, snapcraft#2433,16:50
mupsnapcraft#2444, snapcraft#2445, snapcraft#2463, snapcraft#2470, snapcraft#2473, snapcraft#2493, snapcraft#2495, snapcraft#2500, snapcraft#250216:50
Chipacaoff to the gym for me16:57
Chipacahappy EOW all (i'll bbl to see how spread is doing wrt repro'ing the thing)16:58
zygajdstrand: super-low-hanging-fruit for EOW: https://github.com/snapcore/snapd/pull/660717:00
mupPR #6607: cmd: typedef mountinfo structures <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6607>17:00
mupPR snapd#6610 opened: interfaces/builtin: add add exec "/" to docker-support <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/6610>17:04
sil2100mvo: hey! I'd need some super-powers to the cm3 and pi2 old core16 gadget snap github repos if you have a moment ;)17:13
cjwatsonmvo: Hi - did you notice my comments on https://code.launchpad.net/~mvo/launchpad/add-cnf-metadata-to-release-file/+merge/343161 ?  They should be easy, and then I can land it17:15
mvocjwatson: I missed those, looking now, thank you17:17
mvosil2100: what do I need to do?17:18
sil2100mvo: we'd need foundations having super-power powers at https://github.com/snapcore/cm3-gadget and https://github.com/snapcore/pi2-gadget17:21
mvosil2100: sure, on it17:21
mvosil2100: please try now - ubuntu-foundations is now marked "admin" in both17:22
sil2100\o/17:22
sil2100I have power!17:23
=== pstolowski is now known as pstolowski|afk
mupPR snapd#6600 closed: timings: add new helpers, Measurer interface and DurationThreshold <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6600>17:42
cachioniemeyer, the PR https://github.com/snapcore/spread/pull/75 has been updated17:56
mupPR spread#75: Make spread tests for spread project run on google backend <Created by sergiocazzolato> <https://github.com/snapcore/spread/pull/75>17:56
cachioniemeyer, thanks for reviewing17:56
cjwatsonmvo: Cheers.  Landing now18:11
niemeyercachio: Responded18:34
zygacachio: question about tumblweed, what happened so that that branch enabling it finally landed?18:36
mvocjwatson: thank you18:40
cachiozyga, tumbleweed is being used currently18:40
cachiozyga, why?18:40
cachioniemeyer, thanks18:40
zygacachio: yes, I noticed18:41
zygacachio: the PR enabling it used to fail18:41
zygacachio: so I wonder what changed18:41
cachiozyga, there was a problem with a dependency18:44
cachiozyga, which was fixed in the opensuse repo, so then I updated the image and everything worked again18:45
zyganice18:45
zygathank you18:45
cachiozyga, np18:45
mvocjwatson: just saw the test failure in my LP branch, fixing it now19:01
cachioniemeyer, hey, about the change I did for the residue test19:02
cachioniemeyer, do you think it is better to add it in a different PR?19:02
cachioniemeyer, I added that to have 100% test passing19:03
cachiootherwise we will see 1 error19:03
niemeyercachio: More generically, it's never a good idea to pack side changes that are completely unrelated to the stated purposed for the PR19:03
niemeyercachio: Ah, interesting19:03
niemeyercachio: So it is related to making tests pass.. it doesn't look like so19:04
niemeyercachio: Why is the change necessary?19:04
cachioniemeyer, when I added the repeat option for spread I broke this test19:04
niemeyercachio: When you added that option where?19:04
cachioniemeyer, so, residue is not working in case hte test fails19:04
cachioniemeyer, in spread code19:05
cachioniemeyer, long time ago19:05
niemeyercachio: Sorry, I don't get19:05
mvocjwatson: https://code.launchpad.net/~mvo/launchpad/add-cnf-metadata-to-release-file/+merge/364599 <- sorry again for not spotting this earlier19:05
cachioniemeyer, I added a options to be able to do > spread -repeat 1019:05
niemeyercachio: Yeah, ok19:05
niemeyercachio: And..?19:06
cachioniemeyer, so the residue test it is broken since that time19:06
niemeyercachio: The residue test always fails then?19:07
cachioniemeyer, yes19:07
cachioniemeyer, this change fixes the problem19:07
niemeyercachio: I see.. do we have a test exercising residue with repeat at the same time?19:09
cachioniemeyer, no19:09
cachioniemeyer, the residue test checks spread leave residue in 2 scenarios19:10
cachioniemeyer, 1 when the test pass19:10
cachioniemeyer, 2. when the test fails19:10
cachioniemeyer, so the residue test is failing in the second scenario/part19:11
niemeyercachio: Cool, thanks for the explanation.. sounds good19:11
cachioniemeyer, nice19:11
cachioniemeyer, perfect I'll retest the cache system and leave a message in the PR19:12
cachiothanks for reviewing19:12
niemeyercachio: Thank you!19:13
yotuxCan someone explain why an app has multiple loops directorys19:13
cjwatsonmvo: Heh, wonder why I missed that.  Landing, thanks19:44
yotuxI found the answer to my question sorry its like pacman has 3 versions19:53
mupPR snapcraft#2473 closed: plugins: improve python and go schema validation <Created by cmatsuoka> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2473>20:05
kjackalHey snappy people, there is this TGI Kubernetes right now https://www.youtube.com/watch?v=3Px-kbcIaVE . They are going to deploy MicroK8s snap on arch, centos etc. It might be good feedback for snaps20:20
kjackalplus you may be able to help in case some distros do not work20:21
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
Chipacakjackal: thanks for the heads up21:18
Chipacait's being fun21:18
kjackalthanks people21:23
cjwatsonmvo: Don't worry about the next failure - transient glitch, I'll retry until it works21:30
Chipacaok, have great weekends people21:32
* Chipaca EOWs21:32
mvocjwatson: thank you!21:41
* mvo calls it a day21:42
mupPR snapcraft#2503 opened: cli: disable raven if not running from package <Created by cmatsuoka> <https://github.com/snapcore/snapcraft/pull/2503>21:44
mupPR snapcraft#2502 closed: meta: fix management of snap/local <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2502>22:29

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