/srv/irclogs.ubuntu.com/2018/12/10/#snappy.txt

mborzeckimorning06:04
mupPR snapd#6274 closed: cmd, dirs, interfaces/apparmor: update distro identification to support ID="archlinux" (2.36) <Simple πŸ˜ƒ> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6274>06:05
mupPR snapd#6277 opened: centos: enable SELinux support on CentOS 7 (2.36) <SELinux> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6277>06:09
=== pstolowski|afk is now known as pstolowski
pstolowskimorning!07:59
mborzeckipstolowski: hey08:00
mborzeckimvo: welcome back!08:00
mvohey pstolowski and mborzecki ! thanks, glad to be back :)08:01
mupPR snapd#6272 closed: overlord/snapstate: run 'remove' hook before 'auto-disconnect' <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6272>08:02
pstolowskihey mvo o/, not resting today after travel?08:03
mvopstolowski: I pushed that to the end of the week, I feel I need to take care of things first and see what happend while we were away and all that08:04
mvozyga: also good morning to you :)08:05
zygaGood morning :-)08:08
jameshmvo: hi.  I tried pushing an update to test-snapd-dbus-service last week that seems to have got stuck.  Launchpad built it okay, but it never published.  Is it waiting for "request manual review" by any chance?08:09
zygamvo: how are you feeling? I though you planned on taking a swap day today08:10
mborzeckizyga: hey08:10
zygao/08:10
mvojamesh: good morning (evening in your case I guess) - let me check what is going on there08:11
mvozyga: I feel better, I will swap end-of-week-ish I want to see what I missed and if there is anything I can help unblocking today08:11
jameshmvo: by the way, your original PR was very helpful in putting together mine08:12
mvojamesh: \o/ happy to hear that08:12
mvojamesh: thanks for working on this feature!08:13
jameshmvo: although I've been running into some problems with my current design for activating system services not through systemd units08:13
zygawhat kind of problems?08:14
jameshzyga: when doing non-systemd activation, the dbus-daemon-launch-helper program is spawned to start the daemon08:15
jameshzyga: it uses a simplified version of the config file parser that ignores the <include> directive.  That means it doesn't know about the extra servicedir I configure08:15
jamesheverything works fine for session services and systemd activated services (both system and session)08:16
jameshin particular, this is a problem for 14.04 where we can't do systemd activated system services08:16
mvojamesh: can you give me the url to the snap that was build but is not published please? I will investigate, I don't see anything in the need-review queue right now08:16
jameshmvo: the builds were done with this recipe: https://launchpad.net/~mvo/+snap/test-snapd-dbus-service08:17
jameshwhich says it should publish to edge08:17
mvojamesh: thank you08:18
jameshzyga: one option would be to refactor the branch to comingle the service files in /usr/share/dbus-1/system-services with the rest of the ones from the system -- that might be a bit messier on core systems though08:19
zygamborzecki: shall we merge https://github.com/snapcore/snapd/pull/6111 ?08:20
zyganeal didn't comment further08:20
mupPR #6111: packaging/opensuse: move most logic to snapd.mk <Created by zyga> <https://github.com/snapcore/snapd/pull/6111>08:20
zygamvo: proposal, could we do a 2.36.3 or 2.37 end of this week08:20
mborzeckizyga: works for me, you can try and ping him one last time today08:21
zygait could go to non-stable snap channel and non-ubuntu distributions08:21
zygamborzecki: I'm really happy to iterate on top, just not feel like postpoing helps08:21
mborzeckizyga: ok, so let's land it08:21
mborzeckizyga: maybe ask pstolowski to take a look too08:21
zygathat works :)08:21
zygapstolowski: would you mind having a look at the PR above, last one before landing, it refactors the build system around opensuse08:22
zygaand adds a shared element that can be easily reused for arch and fedora08:22
mvojamesh: might be something silly like a expired macaroon that prevents the published, the last upload of this snap was >1y ago. anyway, I re-authed and triggered a new build that should do an upload then08:22
pstolowskizyga: sure08:23
mvojamesh: I keep you updated08:23
mvozyga: yeah, 2.37++08:23
mvozyga: thanks for raising it08:23
zygamvo: it would be lower risk08:23
mvozyga: unless master is for some reason unfit for this08:23
zygasince not stable08:23
zygamvo: yeah08:23
zygamvo: but we could get some casual testing over xmas period08:23
zygamvo: and a better feel of what 2.37.1 might look like08:23
jameshmvo: awesome.  I've been running spread tests locally modified to use a local copy of the package, but that's only covering a couple of distros.  It'll be good to see results on the others.08:24
mvozyga: yeah, having 2.37 in candidate for christmas would be great08:24
mvojamesh: yeah08:24
mvojamesh: you can always mail/tg me btw in such case, I was at a sprint last week so couldn't be on irc08:24
jameshmvo: do you know how important 14.04 compatibility is for new features?08:28
zyga(on topic): mvo we should ask JamieBennett  or other stakeholders if 14.04 support extends over the extended paid support period08:28
mvojamesh: "Store upload failed:08:29
mvo    Cannot upload new revisions for name=test-snapd-dbus-service08:29
mvo" (<- not very helpful, I will ask in the store channel)08:29
mvojamesh: 14.04 we still need to support until it is EOL if we want support certain desktopish things that is unfortunate but not too terrible, our 14.04 work was always more server/cloud oriented08:30
mvozyga: yeah, we were talking about ESM for snapd08:30
jameshmvo: on the desktop side, my branch is fine.  However I thought it would be worth generalising it to system services too, and that's where the problems are08:31
jameshso that's something that might be interesting to server/cloud08:31
mvozyga: its not clear yet, I am personally not a huge fan because of the support burden (I think if we do it we need another push at systemd and update it to at least 205 there). I think it comes down to how many esm users are interessted in snaps08:31
mvojamesh: oh, ok08:31
zygamvo: I'm in agreement08:32
mvojamesh: thats interessting, what is 14.04 missing here? is systemd too old or something else?08:32
jameshbut at the same time it is a new feature: it's not going to break any existing snaps08:32
mvojamesh: thats good08:32
jameshmvo: as my branch is currently designed, services directly activated by the system bus fail due to not being able to find the .service files08:33
jameshmvo: on 14.04 systems, this means "all system services"08:33
jameshon other distros where the system bus is launched by systemd, we could just tell people to make the service a daemon and be done with it08:34
mvojamesh: aha, I see. so on 14.04 it is a fundamental problem because the system dbus is launched via upstart?08:34
jameshmvo: yes.08:35
* mvo nods08:35
jameshmvo: the other option is to refactor the branch to put service files in /usr/share/dbus-108:35
jameshthat means we can't assume the directory is entirely under our control08:35
jameshand I'm not sure whether that's desirable on an ubuntu core system08:36
mvojamesh: yeah, /usr is not writable on ubuntu-core systems08:37
mvojamesh: does dbus look at any other dirs for this? /etc or /run ?08:37
jameshmvo: the man page seems to indicate <standard_system_servicedirs/> will include /usr/local/share/dbus-1/system-services and /lib/dbus-1/system-services too08:40
jameshalthough both of those could contain other files08:41
mvojamesh: the other files I'm less concerned about but all of those will be read-only on core :/08:41
jameshmvo: my current approach was to configure an extra <servicedir> through a config file in /usr/share/dbus-1/system.d, but the simplified config parser dbus-daemon-launch-helper uses ignores the include directive so doesn't see it08:42
jameshmvo: we're fine for systemd activated services because they don't use launch-helper08:43
mborzeckianyone up for a 2nd review on https://github.com/snapcore/snapd/pull/6277 ? it's a backport to 2.3608:46
mupPR #6277: centos: enable SELinux support on CentOS 7 (2.36) <SELinux> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6277>08:46
zygaperhaps mvo?08:46
zygasince it's a stable backport08:46
mborzeckimvo: ^^ pretty please08:46
mvojamesh: aha, I see. hm, I need to look at the PR I think, I don't have enough state loaded right now about the details08:47
mvozyga: yeah, will do, let me open it08:47
mvojamesh: i386/amd64 of the snap should be available now08:56
jameshmvo: thanks.08:57
mupPR snapd#6277 closed: centos: enable SELinux support on CentOS 7 (2.36) <SELinux> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6277>08:58
Chipacamoin moin moin09:12
zygahello sir09:12
Chipacahow tings?09:12
zygasplendid09:12
Chipacawoo!09:12
* Chipaca runs to the bank with that09:12
mvohey Chipaca ! great to "see" you :)09:31
Chipacamvo: I'd say the same, except you shouldn't be here :-)09:32
mvoChipaca: nah, its fine, I will bugger off at the end of this week instead :)09:34
=== cpaelzer__ is now known as cpaelzer
zygaChipaca: idea, on a small terminal, snap list doesn't show revision and doesn't wrap10:14
zygarevision is really least useful part we show10:14
Chipacazyga: I had adaptive output for things and it got -1'ed10:29
zygaboooo10:30
zygaI'm making a quick PR to look at10:30
zygavery dumb10:30
zygabut maybe enough :)10:30
Chipacazyga: I tried to write it up in the forum, to explore finding a way out of it10:30
=== chihchun_afk is now known as chihchun
cachiomvo, hey10:36
mvocachio: hey, great to see you - how are things?10:36
cachio2.36.2 is ready to stable10:36
mvocachio: \o/ if all is green and the store is happy I would say: lets do it :)10:37
mvocachio: did we get some feedback from the desktop team, did they had a chance to try it as well?10:37
cachiomvo, nice, I'll ping store people10:37
mvocachio: I know we got some +1 from advocay10:37
mupPR snapd#6278 opened: cmd/snap: modularize snap list processing <Simple πŸ˜ƒ> <Created by zyga> <https://github.com/snapcore/snapd/pull/6278>10:37
cachiomvo, I didn't get any feedback10:38
mvocachio: ok, thanks, should be fine, they should run beta anyway :)11:08
mvozyga: do you think you could look at 5845 again? iirc I addressed a lot (most?) of your comments11:09
cachiomvo, I was trying to reproduce the error installing the failing.snap in the failover test11:09
cachiomvo, https://paste.ubuntu.com/p/RF3SSzgNG8/11:10
cachiomvo, it is failing and the test exits11:10
mvocachio: thanks, looking11:11
cachiomvo, no extra info, then the system reboots11:11
mvocachio: hm, this is interessting: "Dec 08 23:42:33 dec082230-921432 snapd[1773]: daemon.go:670: Waiting for system reboot" and then "Dec 08 23:45:07 dec082230-921432 systemd[1]: snapd.service: Watchdog timeout (limit 5min)!"11:12
mvocachio: what is the name of the test again? and does it always fail or is this sometihng new?11:13
cachiofailover11:14
cachiomvo, it was failing since some weeks11:14
cachiosporadically11:15
mupPR snapd#6279 opened: interfaces: tweak deny-auto-connect policy tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/6279>11:15
cachiomvo, 2.36.2 is stable11:16
cachiorunning smoke test now11:17
pstolowskimvo: hey, will you find some time for review 1 or 2 of my hotplug PRs?11:18
mvopstolowski: I hope so11:21
mvopstolowski: there is some more going on (some core18 work and prepareing 2.37) but I will put this on my list11:21
pstolowskimvo: thanks11:30
mupPR # closed: core-build#11, core-build#22, core-build#26, core-build#3711:30
mupPR # opened: core-build#11, core-build#22, core-build#26, core-build#3711:31
zygabrb11:31
pstolowskimvo: fyi, it's #6100 and #611411:31
mupPR #6100: overlord/ifacestate: hotplug-remove-slot task handler <Hotplug πŸ”Œ> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6100>11:31
mupPR #6114: overlord/ifacestate: handler for hotplug-disconnect task <Hotplug πŸ”Œ> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6114>11:31
zygamvo: https://github.com/snapcore/snapd/pull/6279/files is broken11:32
mupPR #6279: interfaces: tweak deny-auto-connect policy tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/6279>11:32
mvozyga: uh, sorry, looking11:32
zygadoes it pass for you locally11:33
zygaunit tests11:33
Wimpressmvo Chipaca We had an adhoc update from zyga last week about the status of the fontconfg cache fixes. What is the ETA for that landing in stable?11:40
mvozyga: silly me, I will fix the issue11:42
mvoWimpress: all validations are done and core is ready, cachio  will do the release once he gets green light from the store11:43
cachiomvo, I already did the release11:43
mvocachio: \o/11:44
mvoWimpress: ^- here is your answer :)11:44
popeydid 2.36.2 *just* hit stable?11:44
cachiopopey, yes11:45
popeyok, I was confused. just updated and looked at wikipedia which said it was released last week11:46
zygamvo: ah, FYI we learned that the fixes don't work on fedora11:48
zygamvo: mborzecki has the details11:48
cachio12:48, 29 November 2018β€Ž Herakleitoszefesu (talk | contribs)β€Ž m . . (6,865 bytes) +316β€Ž . . (Version 2.36.2; infobox formatting improvement) (undo | thank) Tag: 2017 wikitext editor11:48
cachiopopey, I updated the release 2.36.1, then on 11/29 there was another update11:49
popey\o/ I love me some fresh updates11:49
cachiopopey, I don't know who did it11:50
popeyI usually do, glad someone else is :)11:50
mvozyga: uh, that is sad11:50
mvomborzecki: can you fill me in with the details what is missing on fedora?11:50
zygamvo: fedora has cache in /usr/share/something11:51
zygabut I'll let you draw your own conclusions11:51
mborzeckimvo: aah yes, fedora has fc cache in different location, /usr/lib/fontconfig/cache iirc11:51
mvomborzecki: woah, that is a strange location for a cache. very unfortunate as well :(11:51
mborzeckimvo: heh :P11:51
zygamvo: note that the impact on strict and classic snaps is different11:52
zygaand some classic snaps also differ in impact11:52
zygadepending on how they are made11:53
mborzeckimvo: yeah, it's unfortunate, afaik it's only fedora (and rhel probably), arch uses /var/cache, zyga can tell something aboue opensuse, but iirc it's /var/cache too11:53
zygatnere11:53
zygathere's no /usr/lib/fontconfig on opensuse for sure11:54
zygapstolowski: replied to your questions on opensuse PR11:57
pstolowskithx11:57
zygapstolowski: with the binary options we can also estimate the number of configurations that we need to support12:00
mvozyga, mborzecki: I wonder what we can do, could we play tricks in snap-confine  ? i guess that won't help with clasic confned snaps12:00
zygamvo: my opinion is as follows:12:00
mvozyga, mborzecki or will a relocatable format v7+uuid help?12:00
zygamvo: the issue affects graphical snaps made on top of fedora29 base, our fc cache work doesn't help there12:00
zygamvo: strict snaps based on core and core18 are not affected12:00
mvozyga: aha, good point12:01
zygamvo: but our extra cache may be seen as undesired, we could look at hiding it12:01
zygamvo: the issue also affects some classic snaps12:01
zygamvo: classic snaps that link with host fontconfig will not be affected by negative performance as cache will be populated by the host mechanism, they will also use the right cache, wherever that might may be12:01
zygamvo: we MAY consider a system where where the native package with snapd has the equivalent of your cache mechanism12:02
zygathat always links to the native libraries12:02
zygafor the purpose of triggering that cache refresh12:02
zygamvo: classic snaps that link to core snap will remain affected12:02
zygamvo: but they will benefit from your mechanism12:02
zygamvo: to make sure we fully improve the situation I would do the following:12:02
zyga(some of which are probably optional)12:03
zygamvo: 1) we should run different cache mechanism for classic snaps and different for strict snaps12:03
zygamvo: 2) for classic snaps we should run the host provided cache and the snapd provided cache12:03
zygamvo: 3) the snapd provided cache *should* move to the base snap instead12:03
zygamvo: and we should run the one matching the base snap of the classic snap being operated on12:04
zygamvo: this would fix all the issues12:04
zygamvo: let me know what you think12:04
mvozyga: yeah, that sounds sensible12:04
mborzeckizyga: sgtm12:04
zygaI especially think 3 is important12:04
zygasince it means we can deprecate old cache and grow new ones12:04
zygaand solve the hypothetical f29 based apps in the same go12:05
zygaI would say that we may make it generic enough that snapd doesn't know it's fonts12:05
zygamaybe /meta/hooks/snap-installed12:05
zygaand then that does whatever is needed12:05
zygait would be special because base snap hooks may run unconfined12:05
zygabut base snaps are privileged already12:05
zygaso I think this is reasonable, it is the same mechanism you implemented but associated with the correct entity12:06
mupPR snapd#6280 opened: cmd/snap: make 'snap warnings' output yamlish <Created by chipaca> <https://github.com/snapcore/snapd/pull/6280>12:25
Son_Gokuzyga, I felt like garbage all weekend (and Simon Peter did not help...) but I did manage to look over your yak-shaving PR12:41
zygaand?12:43
zygahmm12:46
zygaprepare error on opensuse https://www.irccloud.com/pastebin/crnFHL37/12:46
zygasome SNAFU12:47
mupPR core18#94 closed: hooks: add cloud-init <Created by mvo5> <Merged by sil2100> <https://github.com/snapcore/core18/pull/94>12:54
mborzeckiehh import cycles13:10
mborzeckirefactoring selinux package, got a cycle snap -> dirs -> release -> selinux -> osutil -> dirs13:10
mborzeckioff to pick up the kids13:11
Son_Gokuzyga, I left feedback13:49
zygathanks, I will chec13:49
zygacheck*13:49
Son_Gokuzyga, also, wtf is up with the opensuse golang compiler?13:49
zygano idea13:49
Son_Gokuisn't -std a normal flag?13:49
zyganot broken on opensuse13:49
zygaer13:50
zygaon tumbleweed13:50
zygaat least since I updated this morning13:50
mupPR snapcraft#2419 closed: project: state file path change <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2419>14:17
ackkhi, I'm having an issue with multipass on xenial: https://paste.ubuntu.com/p/7HrZTvBh3k/14:25
ackkit mentions incompatible openssl version14:25
sergiusensackk: I pinged someone who can help you with that14:29
sergiusensmvo: we need to discuss the `.snapcraft` thing. Can you add the reasons for needing it to https://bugs.launchpad.net/snapcraft/+bug/1805219 ?14:29
mupBug #1805219: .snapcraft <19.04> <19.04-external> <Snapcraft:Triaged> <https://launchpad.net/bugs/1805219>14:29
sergiusensmvo: I think I want to go with `snapcraft.yaml` at the root of the project and allowing a `.snapcraft` as the root for assets likes hooks, gui and such14:30
ackksergiusens, thanks14:40
Saviqackk: hey, while our snap is confined "classic" we might stumble upon such problems, but we're moving towards strict confinement very soon so those should go away14:49
ackkSaviq, does multipass currently only work on bionic?14:50
ackk(this is a xenial host)14:50
Saviqackk: in any case, it should work on xenial, can you please file an issue here https://github.com/CanonicalLtd/multipass/issues/new and we'll try and fix asap14:50
ackkSaviq, https://github.com/CanonicalLtd/multipass/issues/51814:53
Saviqackk: thank you!14:53
ackkSaviq, np14:53
mvosergiusens: in a meeting, I opened the bug and have a look15:04
dot-tobiasHi alan_g , short question concerning mir-Kiosk: Iβ€˜m running the chromium-mir-Kiosk snap on mir-Kiosk. After mir-Kiosk gets refreshed, the browser snap does not get restarted automatically. This leaves my device (meant to be always-on) with a black screen until I manually restart the chromium snap. Is this a problem specific to the chromium snap (Xwayland) or in general? And can I do something abo15:04
dot-tobiasut it? Thanks!15:04
* zyga -> quick context switch for taxes and dinner15:09
ackkSaviq, does multipass currently pick proxy options from the hosts' /etc/environment ?15:19
ackkSaviq, from the log I see it can't access cdimage.ubuntu.com, and this host has a proxy set15:20
Saviqackk: it's a systemd service, so no, it doesn't see that environment, you have to `snap set multipass proxy.http …` and similarly for https15:23
ackkSaviq, ah I see, thanks15:23
alan_gdot-tobias, I'd imagine that's a problem with the chromium-mir-kiosk example. You do realise that's an example to work from, not a stable, supported snap?15:23
ackkSaviq, ah, so setting the proxy the daemon runs, but those opensssl errors are stil three15:25
Saviqackk: yeah it won't be able to access any https resources most likely15:26
ackkSaviq, fwiw I have the same errors on my cosmic machine15:27
dot-tobiasalan_g : Yes, I read that in the Ubuntu forums. Sorry to keep pestering you about that, it’s just I do not have a lot of understanding about the graphics stack and thus not the faintest idea where to start improving / integrating this into my snap … and regarding my question today, wanted to make sure it’s not a known issue with mir-Kiosk before I start sinking hours into debugging the chromium sn15:27
dot-tobiasap 😊 again, sorry to keep bothering you.15:27
ackkSaviq, multipass seems to be working though15:27
Saviqackk: as long as it (multipass itself) doesn't need to access any https resources, it should work indeed15:32
mupPR snapd#6281 opened: interfaces/builtin: add Multipass interfaces <Created by gerboland> <https://github.com/snapcore/snapd/pull/6281>16:05
* Chipaca loves that unplugging the headset now disables the trackpad16:28
* Chipaca does not actually love it16:28
* Chipaca needs to reboot16:28
cachiomvo, hey, is it any way to set the StartLimitInterval for a snap service?16:30
okamisHello, I installed slack and ran it as root by mistake, how do I purge the configuration files?16:30
=== jdstrand_ is now known as jdstrand
Saviqokamis: /root/snap/slack should be everything it saved16:39
Saviqor, if ran through sudo, it might be /home/your_user/snap/slack - sudo doesn't change $HOME normally16:40
cachiomborzecki, do you kbiw if it is any way to set the StartLimitInterval for a snap service?16:41
Saviqcachio: not according to https://docs.snapcraft.io/snapcraft-app-and-service-metadata/833516:51
mvocachio: I think not :/16:52
mborzeckicachio: afaik there's no way currently16:52
cachiomvo, ok, so I need to make it manually16:52
cachiomvo, i SEE https://paste.ubuntu.com/p/NdFRpYHHzP/16:53
cachioon boards16:53
cachioat least on pi316:54
mvocachio: what does journalctl -u  test-snapd-daemon-notify.notify say?16:54
mborzeckicachio: the service shouldn't fail, provided the interface is connected at this stage16:54
cachiomvo, I am executing it again16:55
cachiomborzecki, not sure if when it fails the interface is connected16:56
cachiobecause part of this test is with the interface diconnected16:56
mupPR snapcraft#2422 closed: clean: user friendly message when cleaning <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2422>16:56
mborzeckicachio: yup, there should be an echo before, or maybe extend debug to show snap interfaces?16:57
cachiowhen the test fails the interface is disconnected16:57
cachioI already check that16:58
cachiomborzecki, could that be affecting?16:58
sergiusensjoc: hey, what does this mean? https://www.irccloud.com/pastebin/Nh31ajqw/16:59
mborzeckicachio: maybe, do you have a log from spread?17:01
cachioI have a debug session here17:02
cachiomborzecki, I just created it17:02
cachiomborzecki, what info could help?17:02
cachiomborzecki, this is the spread log17:03
cachiohttps://paste.ubuntu.com/p/dq7JgX7RmF/17:03
cachiomvo, external:ubuntu-core-16-arm-32 .../tests/main/interfaces-daemon-notify#  journalctl -u  test-snapd-daemon-notify.notify17:05
cachio-- No entries --17:05
cachiomvo, this could help17:06
cachiohttps://paste.ubuntu.com/p/CcmqqPsx34/17:06
cachiomvo, this shows the start-limit-hit17:06
Chipacacachio: if a test is breaking a service and such, and it doesn't do reset-failed, it'll sometimes not start17:11
Chipacanot sure if that's what you're seeing though17:11
Chipacacachio: but that's what i had to do with the nfs-support test17:11
=== pstolowski is now known as pstolowski|afk
cachioChipaca, the test is disconnecting the interface and trying to start the service17:12
cachioand checking the log contains denials17:12
cachiobut those denials don't appear so the test fail17:12
cachioChipaca, and the problem is that we are hitting the start limit17:13
Chipacacachio: right17:13
cachioand it is just happenning on core for arm17:13
cachioChipaca, what do you mean with "breaking a service"?17:14
Chipacacachio: what's stopping the service, in this case?17:14
mborzeckicachio: about that fedora 29 bug, i think we'll have to remote the kernels manually17:16
cachiomborzecki, ok17:17
cachioI'll do that17:17
mborzeckicachio: something like this:  `dnf autoremove -y $(rpm -qa kernel-core | sort -r | tail -1)`17:18
cachioChipaca, the test stops the service17:18
cachiodisconect -> stop -> loop with start17:19
cachiowe check that snap logs contains denials17:19
Chipacacachio: but17:19
Chipacacachio: it seems the service is exiting with error after getting the permission denied17:20
Chipacaah17:20
Chipacacachio: the snap is buggy17:20
Chipacacachio: the daemon says it's "simple", but it's not a daemon at all17:21
cachioChipaca, this is the snap.yaml https://paste.ubuntu.com/p/NxtnkdYVjy/17:22
Chipacayes i'm looking at it here17:22
cachioChipaca, why it is not a daemon?17:22
Chipacait says "daemon: simple"17:22
Chipacacachio: because it just runs a command and exits?17:22
cachioChipaca, ahh, ok17:22
Chipacacachio: and17:23
Chipacacachio: a shell script exits with the exit status of the last command it executed17:23
Chipacacachio: which means that this thing exits with error17:23
Chipacaso systemd tries to restart it a few times, and then gives up17:23
Chipacaso the test is a race between systemd restating the thing, and spread running the test17:23
Chipacacachio: easiest fix, probably, is to add a "true" after the systemd-notify call in the script :-), and (optionally? but probably best) set "daemon: oneshot"17:25
cachioChipaca, so we should make sure we exit 017:25
cachioI'll try with oneshot17:25
Chipacak17:26
cachioChipaca, thanks17:26
Chipacacachio: let me know how it goes17:26
cachioChipaca, one shot does not work17:44
cachioI think I made it work17:44
cachiooneshot fails to start17:45
cachiobecause the interface is not connected17:45
cachioso it exits with status 117:45
Chipacacachio: you made the script not exit with error first?17:45
cachioChipaca, no17:45
Chipacacachio: you did the optional thing, and not the non-optional thing17:45
Chipaca:-)17:45
cachioI just installed and connected to the interface17:45
cachiotogether17:46
cachiobecause between the snap is installed in prepare and the interface is connected during the execution17:46
cachiosystemd is retrying the service17:46
cachioChipaca, what do you suggest'17:47
cachio?17:47
Chipacacachio: add an '|| true' to the systemd-notify call, or a 'exit 0' after it or something17:48
cachioChipaca, 017:51
cachiook17:51
* Chipaca 117:51
cachioChipaca, well this also works well18:04
cachioI'll create the PR18:04
roadmrsergiusens or kyleN : what's the expected Ubuntu version to host snapcraft development? (kind of a trick question - I tried bionic but some of the deps don't exist as noted in the docs, so I went down to xenial) Is it xenial?18:08
mupPR snapd#6282 opened: tests: force test-snapd-daemon-notify exit 0 when the interface is not connected <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6282>18:10
sergiusensroadmr: it is xenial, given away by the debian/changelog still there and the lack of a base in snapcraft.yaml πŸ˜‰18:33
roadmrsergiusens: nice - so how do you guys get black (which requires python 3.6) to run on xenial (which has python 3.5)?18:34
roadmrI disabled the black portion of ./runtests.sh for now, am just curious, this is not blocking me or anyting18:34
sergiusensroadmr: `snap install black --edge --devmode`18:34
roadmrsergiusens: ah, perfect :) thanks!18:35
roadmrhaving two ways to install stuff is confusing. Hopefully soon there'l be only one :D18:35
sergiusensI intend to make all our devtools a single snap to bootstrap faster, but haven't gotten around to it yet18:35
roadmrsergiusens: it didn't take me a long time to bootstrap in any case, just black was baffling me18:35
sergiusensroadmr: yeah, but imagine not needing to ask me which dev environment is the expected one and have a snap with the interpreter, the required dependencies and tools πŸ˜‰18:37
roadmrsergiusens: \o/ I like that future heh18:37
=== ricab is now known as ricab|leaving
=== Sir_Gallantmon is now known as Son_Goku
mupPR snapcraft#2423 opened: Do not use `bash` package name in staging <Created by jdahlin> <https://github.com/snapcore/snapcraft/pull/2423>22:27
jdahlin^ we found some issues related to the `bash` package name already being registered on staging22:27

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