/srv/irclogs.ubuntu.com/2017/07/28/#snappy.txt

janisozaurhi07:00
janisozaurhas anyone been successful in establishing opengl context from a snap on Arch Linux?07:01
janisozauryesterday I tried and nothing really worked, not even glxinfo07:01
janisozaurit either crashed (on a host with nvidia + binary drivers) or reported an error (host with Intel GPU)07:02
janisozaurthe very same snap worked, however, in a ubuntu artful VM07:02
janisozaurI tried specifying LIBGL_DRIVERS_PATH to the correct ($SNAP/usr/lib/x86_64-linux-gnu/dri) directory, but that didn't help07:05
janisozaurhas anyone been able to run e.g. glxgears on Arch? How can I help debug?07:17
janisozaurhmm, perhaps my Arch system did not offer OpenGL slot properly?07:35
=== mcphail_ is now known as mcphail
Chipacajanisozaur: I think the guy that knows (or remembers) most about that is zyga08:10
Chipacas/guy/person/08:10
janisozauryeah, i saw his comments on issue trackers08:11
janisozaurcan he be met round these parts?08:12
zyga-ubunture!08:13
zyga-ubuntuman, I thought it is Saturday08:13
janisozaurhi zyga-ubuntu08:13
zyga-ubuntuhey08:13
janisozaurI'm having issues running opengl snaps on my Arch systems08:14
zyga-ubuntuusing nvidia proprietary drivers or anything else?08:14
janisozaurone machine with Intel reports errors in glxinfo, the other with nvidia just segfaults08:15
zyga-ubuntucan you please describe this in detail on forum.snapcraft.io08:15
zyga-ubuntuthe nvidia issue is probably more well known08:15
zyga-ubuntuthe intel issue new to me08:15
janisozaurI'm not around these machines right now, but I'm available for debugging in the evening08:15
zyga-ubuntuthere are some people working on improving opengl snap support (alberto) but it's not even a PR yet08:15
zyga-ubuntujanisozaur: if you can stick around for some time (days) and help us understand the issue I'll make sure the solution is working and tested on arch08:16
zyga-ubuntuunfortunately many opengl problems are hard to test for us (lots of distros and virtualization complexity)08:16
zyga-ubuntualso the design we have now is pretty old and we learned about some of the issues08:16
zyga-ubuntuso we really need people with hands-on access to the given distribution and hardware08:17
zyga-ubuntuyour help can be really invaluable to us08:17
janisozaursure, i'll try helping with that08:17
janisozauris there any debugging FAQ for snaps?08:18
janisozaurI only started using them yesterday08:18
zyga-ubuntusome but some issues are more complex than others08:18
zyga-ubuntuas a few hints:08:18
zyga-ubuntuyou can look at the output of snapd systemd unit08:19
zyga-ubuntu(journalctl -u snapd)08:19
zyga-ubuntuthat tells a lot about what is going on in the background08:19
zyga-ubuntuyou can also set SNAP_CONFINE_DEBUG=yes to see what's going on at application startup08:19
zyga-ubuntulastly snap version is the key thing many people will ask about08:19
zyga-ubuntuthe arch maintainer is, I believe, not updating the package lately,08:19
zyga-ubuntuwe don't have automatic regression tests for arch yet so we don't know if it's working really08:20
zyga-ubuntuif you want to help in any of that it's all something I can guide you with08:20
zyga-ubuntuwe have automatic regression tests for debian, ubuntu, fedora and opensuse now08:20
zyga-ubuntuand arch is not far away08:20
janisozaursounds like something i could maybe look into08:20
zyga-ubuntubtw, what is the current version of snapd on arch?08:20
zyga-ubuntuI have an arch VM and I can dual-boot on one of my laptops08:21
zyga-ubuntuand I was working on small updates to the package08:21
janisozaurthe snap i was creating worked fine in ubuntu artful VM08:21
zyga-ubuntunow with 2.26.14 released and upcoming 2.27 I think it would make sense to update everything08:21
janisozaurit probably was using swrast, though08:21
zyga-ubuntuyes, I suspect so08:21
janisozaurhttps://www.archlinux.org/packages/community/x86_64/snapd/08:21
janisozaur2.26.108:22
zyga-ubuntunot too far off08:22
zyga-ubuntubut the package can use some love08:22
zyga-ubuntuas we added some new features lately08:22
zyga-ubuntuand last time I looked they were not supported in arch yet08:22
zyga-ubuntuhow about this, let's talk later today or over weekend08:22
zyga-ubuntulet's start a thread in the forum08:22
zyga-ubuntuon arch package update08:22
zyga-ubuntuand on opengl08:22
zyga-ubuntuI'll let you start the one on opengl and I'll quickly start the one on arch package update08:23
zyga-ubuntuand we can discuss there08:23
zyga-ubuntuthe forum is really nice, works paritally like a mailing list and partially like IRC (being in real time) and it scales better than IRC (timezones/offline problems)08:23
janisozaursure, i should be available at ~1730 CEST08:23
zyga-ubuntuhow does that sound?08:23
zyga-ubuntuperfect08:23
zyga-ubuntuI should be available all day (I'm in europe too), just may be offline for some moments08:24
janisozaurgreat!08:24
zyga-ubuntujanisozaur: https://forum.snapcraft.io/t/updates-to-snapd-package-on-arch/146708:28
zyga-ubuntujanisozaur: feel free to comment there!08:28
zyga-ubuntunote that I used the "snapd" category as this involves snapd itself08:29
zyga-ubuntuand "arch" tag (along with some other tags I use to organize my tasks)08:29
zyga-ubuntuso if you create a post about opengl you can also use the "arch" tag to group posts this way08:29
jameshIs there any way to get the CI tests to rerun for my pull request?  I got a failure onlinode:ubuntu-14.04-64:tests/main/econnreset and it isn't clear how my changes could have caused that08:30
zyga-ubuntujamesh: I think it's not your fault, I think that test is flaky08:31
zyga-ubuntujamesh: but to answer your question, if you look for that id (ubuntu-14.04-64....) in the log you will see a *fold* that has the details of the error08:31
zyga-ubuntujamesh: and just below it another fold that has lots of debug information08:32
jameshzyga-ubuntu: right.  I read the CI log, and couldn't see how my changes impacted it08:32
zyga-ubuntujamesh: I think though, as I said, that that test is racy and sometimes fails incorrectly08:32
zyga-ubuntujamesh: if you re-run it it will likely pass08:32
zyga-ubuntuthough today master feels broken (on autopkgtests) as something odd has regressed in one of the upgrade tests after the 2.26.14 stable release08:33
zyga-ubuntuI've been investigating that last night but no conclusions yet08:33
jameshzyga-ubuntu: do I need to make a trivial commit to get it to rerun, or is there a button to trigger it again?08:33
zyga-ubuntujamesh: what's the PR URL?08:33
zyga-ubuntujamesh: you can re-trigger from the UI08:33
Chipacajamesh: you don't have a "restart" at the top of the travis output?08:33
jameshzyga-ubuntu: https://github.com/snapcore/snapd/pull/358108:33
mupPR snapd#3581: Add polkit authorization support to /v2/login <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/3581>08:33
zyga-ubuntu(or maybe you cannot but I can help)08:33
zyga-ubuntujamesh: all tests passed there08:34
Chipacaum08:34
Chipacayeah08:34
jameshChipaca: no.08:34
Chipacajamesh: tests are all green08:34
jameshdoh.  Hadn't reloaded the page.08:35
jameshso many other things seem to auto-update on github, but that didn't.08:35
jameshsorry for the noise08:36
Chipacajamesh: the auto-update breaks if the network hiccups, apparently08:36
jameshChipaca: I guess the restart button only shows up for core developers?08:37
Chipacajamesh: maybe you need to log into travis?08:37
jameshChipaca: I am logged in08:38
Chipacaah08:38
Chipacathen maybe08:38
jameshIt's not a big deal08:38
* zyga-ubuntu hits the road, will be off IRC today often08:52
zyga-ubuntuChipaca: hey09:08
Chipacazyga-ubuntu: wut09:08
Chipacazyga-ubuntu: hey :-)09:08
zyga-ubuntuChipaca: I plan to spend the day 1/3 on reviews 1/3 on master regression and 1/3 on mvo's SPDX parser09:08
zyga-ubuntudo you need my help on anything?09:08
Chipacazyga-ubuntu: how do you divide 8 hours into thirds09:10
zyga-ubuntuChipaca: easy, just start with 12 hours09:10
zyga-ubuntueverything is easy to divide then D:09:10
Chipacazyga-ubuntu: aw :-(09:10
zyga-ubuntuI like languages with rational numbers09:11
Chipacazyga-ubuntu: the right answer is that it divides into 3 2-hour chunks, and a spanish lunch break09:11
zyga-ubuntu8/3 is such a nice construct09:11
Chipacazyga-ubuntu: other than a review, which is already on your list, i don't think i need anything09:11
zyga-ubuntuChipaca: the logs PR? or something else as well?09:12
Chipacazyga-ubuntu: sorry about my commit messages falling in quality. The first pass at this work I took the effort of doing them properly.09:12
zyga-ubuntuChipaca: I know that's extreme but I really like LKLM style commits09:12
Chipacathe second pass i also did a fairly good job at keeping the commits tight and the messages clean09:12
* Chipaca looks for a link to show he isn't making it up (maybe)09:13
Chipacagah09:14
Chipacai might've been making that up after all09:14
Chipacahttps://github.com/snapcore/snapd/pull/3549/commits09:14
mupPR snapd#3549: many: expose services commands for snap services <Created by chipaca> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/3549>09:14
Chipacaanyway09:16
Chipacazyga-ubuntu: i'm aware of making them better, and I'll continue improving them09:16
Chipacasome days it's harder than others09:16
zyga-ubuntuChipaca: thanks, that's all I ask for :)09:16
mupPR snapd#3631 closed: tests: apply underscore convention for SNAPMOUNTDIR variable <Created by sergiocazzolato> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3631>09:16
Chipaca:-)09:16
zyga-ubuntuChipaca: I was just wondering if we can land "snap logs" today09:16
zyga-ubuntuthat shiny green button there09:16
Chipacazyga-ubuntu: just a simple matter of reviews and addressing issues found therein09:17
zyga-ubuntuI mean do we have enough +1 votes with mvo and gustavo being off09:17
zyga-ubuntuChipaca: on that topic, I have one monster PR https://github.com/snapcore/snapd/pull/361909:18
mupPR snapd#3619: interfaces/builtin: reduce duplication and remove cruft in Sanitize{Plug,Slot} <Created by zyga> <https://github.com/snapcore/snapd/pull/3619>09:18
Chipacaah yes09:20
Chipacai was 3/8 through that the other day :-p09:21
zyga-ubuntuthat's a monster PR, sorry09:24
zyga-ubuntubut you can go patch-by-patch09:24
zyga-ubuntuit's actually easier to follow this way09:25
Chipacazyga-ubuntu: you mean commit-by-commit?09:25
zyga-ubuntuChipaca: yes09:26
Chipacazyga-ubuntu: you're aware it's 12 commits?09:27
Chipacaif I +1 a commit, is github smart enough or does it think I +1'd the PR?09:28
zyga-ubuntuChipaca: GH doesn't support that (not gerrit) *but* you can open 12 tabs and just read one by one09:30
zyga-ubuntuand only give +1 if you found nothing wrong09:30
Chipacayeah, doing that09:31
zyga-ubuntubut you can easily comment on each patch and GH handles that nicely09:31
zyga-ubuntuthanks, I'm reading logs now09:31
Chipacazyga-ubuntu: is there a reason the repo itself doens't do the ensureSlotIfaceMatch(iface, slot) before calling SanitizeSlot?09:32
zyga-ubuntuChipaca: yes, read on09:32
Chipacaok09:32
zyga-ubuntuChipaca: it's rolled into {plug,slot}.Sanitize09:32
zyga-ubuntuand also somewhat silly since repo does plug.Interface lookup09:33
zyga-ubuntuso it's a no-op test if the compiler is smart enough09:33
pedroniszyga-ubuntu: which btw reminds, checking if a slot is for a given snap types is also done by base declaration, so maybe is something else that can go away, but that I think you should discuss with jdstrand09:33
zyga-ubuntupedronis: I think it's fine to keep as is, we don't check base declaration for sideloaded snaps, do we?09:33
zyga-ubuntupedronis: and the check is really really small now, just two lines in all of the tree09:34
pedroniswe don't indeed09:34
zyga-ubuntupedronis: and I'm very happy I took your suggestion: so much code got removed09:35
zyga-ubuntupedronis: vast majority of sanitize functions are out09:35
zyga-ubuntuI think pawel will be happy as there will be less things for him to edit09:37
zyga-ubuntu(really, going through 190 files to do one change is heart wrenching)09:38
pedroniszyga-ubuntu: why did you change the errors to say operating system ? instead of core ?09:39
zyga-ubuntuoh, not sure, I think I just unified it (it said various things)09:39
pedronismmh09:39
zyga-ubuntubut we can now tweak it either way easily09:39
pedronisI think  we need to say either "core snap"  or operating system (without snap)09:40
pedronisbut I fear that becomes a Gustavo's question09:40
zyga-ubuntuaha, indeed09:40
zyga-ubuntuI'm fine with core snap for now and I'll ask gustavo next week09:40
Chipacazyga-ubuntu: +1'ed with a suggestion09:40
zyga-ubuntuI'll finish the review for chipaca and then I'll update09:40
zyga-ubuntuChipaca: woot, thank you!09:41
zyga-ubuntu(that was fast!)09:41
Chipacazyga-ubuntu: reading the commits indeed made it fast09:41
Chipacaa lot of it was repepepetititve09:41
Chipacazyga-ubuntu: and i know logs would've been easier to read in (at least) three separate commits :-(09:41
zyga-ubuntuand all hand made, I tried to automate it but then just decided not to as the result wasn't perfect09:42
Chipacazyga-ubuntu: what are the autopackage failures on your pr about?09:42
pedroniszyga-ubuntu: it's also interesting  that some had the comment "s allowed only by a gadget or os snap" but then allowed only os09:45
pedroniszyga-ubuntu: maybe it would be good to cross check with the base decl?09:45
pedronisactually there should be a way to write a generic test helper like that09:46
pedronismaybe09:46
zyga-ubuntuChipaca: no idea, I'm checking that09:47
zyga-ubuntusomething broke with hooks09:47
zyga-ubuntunot sure yet, I cannot reproduce, I plan to check if hook handler names changed lately09:47
zyga-ubuntupedronis: yes, but I choose to keep the semantics of the code now, I think we can review who can do what09:48
zyga-ubuntupedronis: good idea about cross checking with base declaration09:48
pedroniszyga-ubuntu: actually I alread had a test like that09:53
pedronis:)09:53
pedroniszyga-ubuntu:  func (s *baseDeclSuite) TestSlotInstallation logic about compareWithSanitize seems to do that09:54
pedronisor at leat a part of that09:54
pedronisnow that I read better09:54
pedronisanyway seems something could be done there09:55
pedronisor based on that09:56
pedroniszyga-ubuntu: btw I just noticed that the various definer interfaces in the backends have the same problem Chipaca mentioned09:59
zyga-ubu1tupedronis: aha, thank you, I will check it out09:59
* zyga-ubu1tu looks at Chipaca's review10:00
zyga-ubu1tupedronis: I started with public interfaces but Gustavo at the time wanted private ones10:00
pedroniswe have a bunch of essentially secret interfaces atm10:00
zyga-ubu1tupedronis: if they were public I'd have an easier life in some tests10:00
zyga-ubu1tupedronis: I can make a RFC PR with just the private -> Public switch for htem10:01
pedronisI'm not sure what the reasoning was10:01
zyga-ubu1tuI don't remember anymore10:01
pedronisusually secret interfaces are for tightly coupled internal helpers10:01
zyga-ubu1tuI think it was not perceived as necessary as they were only used by the backends10:01
pedronisand usually then the methods are also private10:01
zyga-ubu1tuI see10:01
pedroniswell, but the interfaces need to implement them10:01
pedronisand the doc don't have a signature of what they need to implement10:02
pedronisI find it strange as Chipaca did here10:02
zyga-ubu1turight, I think they could be public10:02
zyga-ubu1tueh, IRC, the protocol for wired mainframes10:03
=== zyga-ubu1tu is now known as zyga-ubuntu
pedroniszyga-ubuntu: at least  the various methods would need to mention them in the docs10:05
pedronisalso the docs look strange10:06
pedronisin the backends, like copy-paste strange10:06
zyga-ubuntuyes, many of the interfaces have copy pasted stuff10:06
zyga-ubuntujust little by little :)10:06
pedronislike mount-specific  in apparmor?10:06
zyga-ubuntuI'd happily remove useless docs10:06
zyga-ubuntuI think I even noticed that a few months ago when doing a pass10:07
zyga-ubuntujust forgot to correct it at the time10:07
pedroniszyga-ubuntu: anyway +1 apart "core snap" and agreeing with Chipaca10:10
zyga-ubuntuack, thank you10:11
popeyogra_: finaly got my serial cable hooked up to my nano pi. it only sees eth0 and not the wireless interface :(10:17
popeyogra_: any ideas how I can get in and get it on the wifi, as I'm stuck in subiquity right now10:17
popeylornajane: heya, couchdb landed in the snap store last night :)10:19
lornajanepopey: so twitter tells me :)  But I tried to install it and now I have questions10:19
lornajanewhen I install a snap, how can I tell whether it has created me some commands, and what those are/do?10:20
lornajaneif it starts daemon things, how can I tell what those are and what services I can now start/stop?10:21
zyga-ubuntulornajane: Chipaca can help you!10:21
* Chipaca hides10:21
ogra_popey, stop the boot at the uboot countdown10:22
lornajaneawesome :)  Long-time ubuntu user, but new to snaps and not really a sysadmin so I think my questions are newbie ones - but I promise to blog what I learn so hopefully will head off future newbie questions10:22
zyga-ubuntulornajane: \o/10:22
Chipacalornajane: _right_ now, there's not a super simple way. For services, systemctl status snap.<tab> should help.10:22
zyga-ubuntulornajane: also we have a very nice forum on forum.snapcraft.io10:22
ogra_popey, then: setenv mmcargs 'setenv bootargs "console=ttyS0,115200 root=${mmcroot} init=/bin/bash"'10:23
ogra_popey, then: saveenv; reset10:23
Chipacalornajane: really soon (1 or at most 2 releases away) there'll be a simplified interface so you don't need to look at systemctl unless you want to10:23
ogra_popey, that should give you a prompt after boot10:23
Chipacabut, today, that's what there is10:23
lornajaneI can manage systemctl, and I did realise there was a process running10:23
lornajaneit's not attached to the port I would expect though, and I don't know how to help it10:24
Chipacalornajane: and once you know the unit name, journalctl -u <that name> will show you logs10:24
Chipacalornajane: ah, that's down to the individual snap i'm afraid10:24
Chipacasome'll let you configure it, some won't10:24
lornajaneoh poor thing is stuck in a loop10:24
Chipacaaw10:25
lornajaneor maybe looped a bit when it started?  Hmm.  I notice that I have a couchdb command now but that is also unhappy10:26
popeyogra_: ok, will play with taht thanks!10:31
lornajaneOK.  I'm not sure what to do next but I think it's couchdb-specific so I'll wander back over there and see if they can assist.  I'm so pleased though to see snappy stuff taking over the world :)10:32
popeyogra_: yay, got a console prompt, now what? :)10:33
ogra_popey, cat /proc/net/dev ... do you see a wlan device there ?10:40
popeyogra_: no10:42
ogra_popey, modprobe brcmfmac10:43
ogra_and check in dmesg10:43
zyga-ubuntuChipaca: reviewed10:47
Chipacazyga-ubuntu: int((^uint(0)) >> 1) is only math.MaxInt64 on 64 bits10:54
Chipacazyga-ubuntu: there isn't a MaxInt10:54
Chipaca:-(10:54
Chipacaanyway, i'll comment on the PR when i get back from a run10:55
* Chipaca getting ready to go afk for a bit10:55
zyga-ubuntuah, Isee10:55
zyga-ubuntuenjoy!10:55
zyga-ubuntuI'll get to your feedback now10:55
Chipacauh, that println was debugging i needed to nuke10:56
* Chipaca goes afk10:56
popeyogra_: [ 1247.038733] usbcore: registered new interface driver brcmfmac10:56
* zyga-ubuntu needs wording help11:00
zyga-ubuntupedronis:11:00
zyga-ubuntu+// PlugSanitizer describes an interface that can sanitize plug instances.11:00
zyga-ubuntu+type PlugSanitizer interface {11:00
zyga-ubuntuWDYT?11:00
zyga-ubuntuor just "that can sanitize plugs"11:00
ogra_popey, any change in /proc/net/dev ? (or ifconfig)11:00
morphis_zyga-ubuntu: there seems to be a problem on debian with latest snap-confine from 2.26.1411:08
morphis_zyga-ubuntu: see https://github.com/anbox/anbox/issues/38611:08
morphis_"snap-confine has elevated permissions and is not confined but should be. Refusing to continue to avoid permission escalation attacks"11:08
pedroniszyga-ubuntu: PlugSanitizer can be implemented by Interfaces that have a need to sanitize their plugs  ?11:09
pedronisor "have reasons"11:10
zyga-ubuntumorphis_: that machine must have apparmor enabled11:10
zyga-ubuntupedronis: nice, thank you!11:10
zyga-ubuntumorphis_: (via boot command line argument)11:11
zyga-ubuntumorphis_: but I'm aware of "something being off" on debian since we published the last stable snap, I intend to keep digging today, I cannot reproduce the problem yet (which drives me nuts)11:11
zyga-ubuntumorphis_: we also know that once you are in the wedged state you are also affected by lack of reverse reexec11:12
morphis_zyga-ubuntu: afaik we do --disable-apparmor on debian11:12
zyga-ubuntumorphis_: no, that's not even it11:13
zyga-ubuntumorphis_: the thing is that that message is *only* shown when the kernel has apparmor enabled (not just built but actually active) *and* snap-confine is not confined11:13
zyga-ubuntumorphis_: may be a misalignment with snapd's appamror profile generation for core's snap-confine11:13
morphis_zyga-ubuntu: but actually if we build snap-confine on debian with --disable-apparmor should that be respected when we re-exec?11:14
zyga-ubuntumorphis_: but I haven't seen it before11:14
zyga-ubuntumorphis_: no, remember that when we re-exec we use the one from core11:14
morphis_right11:14
zyga-ubuntumorphis_: so the compile time build option is not relevant11:14
morphis_but it was the distro decision to disable AppArmor11:14
zyga-ubuntumorphis_: can you reproduce that problem?11:15
morphis_zyga-ubuntu: I can't, it's an anbox bug report from a user11:15
zyga-ubuntumorphis_: is /sys/kernel/security/apparmor around?11:15
zyga-ubuntuaha11:15
zyga-ubuntuwell, must be that then11:15
morphis_feel free to comment on the report11:15
morphis_it's just an hour old11:15
zyga-ubuntudoing that now11:15
morphis_thanks!11:15
pedroniszyga-ubuntu: still lgtm,  don't know if Chipaca wants to chime in on the wording.11:16
zyga-ubuntumorphis_: done11:18
zyga-ubuntupedronis: updated to what you suggested11:18
zyga-ubuntupedronis: I think chipaca might be back by the time tests finish11:18
morphis_zyga-ubuntu: thanks11:21
popeyogra_: nothing11:43
ogra_popey, then i fear there is something missing in the devicetree file, the nanopi-air should use an ampak AP6212 http://linux-sunxi.org/Wifi AFAIK ... i guess the HW isnt properly initialized ... i'll have to dig a bit11:45
popeyogra_: ok. lemme know if there's anything I can get off the device :)11:47
ogra_popey, theer we go https://github.com/armbian/build/blob/master/patch/kernel/sun8i-dev/add-nanopi-neoair.patch11:47
popeyogra_: so you need to make a new image with that patch?11:48
ogra_popey, re-building my hackish allwinner kernel will take a while though, it is very fiddly ... but that seems to be the right patch top enable the AP621211:48
popeyok, great11:48
ogra_s/top/to/11:48
ogra_yeah, i need to make a new kernel11:48
popeyok, I'l put him back in his box for now then :)11:49
popeygimme a ping when you need me to test again :)11:49
ogra_yeah, i'll try to get this done on the weekend11:49
popeynice, thanks11:49
popeybeer++11:49
ogra_:)11:49
Chipacapedronis: zyga-ubuntu: wording of what?11:49
* Chipaca still not fully back (mostly thinking about lunch right now)11:50
pedronisChipaca: zyga's branch,  doc comments for the new public interfaces11:50
Chipacaah, i thought that was to be separate because it needed discussion11:50
pedronisChipaca: ?11:55
Chipacapedronis: i thought as gustavo had originally said no to the public interfaces, zyga was going to make a separate pr to discuss it11:58
pedronisChipaca: in a different context11:58
pedronisanyway pawel will need to redo this anyway11:58
pedronisso another chance to argue one way or another11:59
zyga-ubunture12:01
* zyga-ubuntu writes user docs12:01
zyga-ubuntuChipaca: just this https://github.com/snapcore/snapd/pull/3619/commits/f6fb74af3fcbcea7caa72ee645c169f9f33f962d12:04
mupPR snapd#3619: interfaces/builtin: reduce duplication and remove cruft in Sanitize{Plug,Slot} <Created by zyga> <https://github.com/snapcore/snapd/pull/3619>12:04
Chipacazyga-ubuntu: hmmm12:04
Chipacazyga-ubuntu: i guess there's no way around interfaces vs interfaces :-)12:05
Chipacazyga-ubuntu: +112:05
pedronisyes, that is our self impose loss in the terminology battle12:06
zyga-ubuntuhehe12:07
zyga-ubuntulet's also add Channel so that we can have more confusion12:07
* zyga-ubuntu resumes writing docs12:08
zyga-ubuntuwriting docs is *fun*12:08
Chipacazyga-ubuntu: next feature we add is being able to expose only a subset of the full store to a device, something like a gadget-set filter12:08
Chipacazyga-ubuntu: we'll call that a "slice"12:08
* Chipaca runs for the hills12:09
zyga-ubuntuChipaca: slice and dice12:09
pedronisinterface, channel, slice       , map next?12:12
pedroniswe'll we dare to go where nobody has gone, and have features called "integers" and "bools"12:12
zyga-ubuntuLOL :D12:13
zyga-ubuntumap:\n\tbool:\n\t512:13
Chipacaregisters and pointers, that's what's next12:15
* zyga-ubuntu writes stuff at https://forum.snapcraft.io/t/feature-snap-layouts/1471/212:26
zyga-ubuntuactually, I think this is a very good moment to take a break and eat lunch12:27
=== matteo` is now known as matteo
mupPR snapcraft#1426 opened: tests: adapt opencv's test expectations <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1426>12:58
* Chipaca hunts for tea12:58
zyga-ubuntuare we doing the standup today?13:01
Chipacayes13:02
Chipacaomw13:02
Chipacabah, i think we are :-)13:02
Chipacapedronis: standup?13:05
cachiopedronis, joining standup?13:05
Chipacapedronis: https://www.youtube.com/watch?v=pZG7IK99OvI !13:05
zyga-ubuntucachio: I'm reading your pastebin13:13
cachiozyga-ubuntu, the failover tests are the one to see13:14
fgimenez_cachio: about the test error in https://paste.ubuntu.com/25188524/ you should remove auto-refresh from the job definitions for the dragonboard executions on the lab, the post has the details13:15
fgimenez_https://forum.snapcraft.io/t/core-snap-validation-process-at-beta-channel/145413:15
zyga-ubuntuerror: cannot copy request into temporary file: write13:15
zyga-ubuntu       /tmp/snapd-sideload-pkg-627586801: no space left on device13:15
zyga-ubuntuThis doens't look like that error13:15
zyga-ubuntulooks like you ran out of space13:15
ogra_not really out of space but out of ram13:16
cachiofgimenez_, ok, the failover tests could be failed because of that?13:16
ogra_since /tmp is a tmpfs13:16
ogra_use TMPDIR and point to some place on disk13:17
fgimenez_cachio: not sure, i ended up removing it because the whole suite started to fail after the auto-refresh error13:18
zyga-ubuntuogra_: indeed13:19
Chipacazyga-ubuntu: what would you call the const for the journal timestamp key?13:20
zyga-ubuntuconst magic = "__STUFF"13:21
zyga-ubuntuas long as it is used >113:21
zyga-ubuntuthe name isn't as relevant if it can stay private13:21
Chipacazyga-ubuntu: only other use is in the tests13:24
cachiofgimenez_, zyga-ubuntu  I am running again the suite without the auto-refresh test13:29
cachioi'll keep you updated13:29
kjackal_hi everyone, how do I specify daemon dependencies? I want mysnap daemon to start after an optional service eg lxd.service13:29
ogra_i dont think thats possible yet13:29
ogra_(ICBW though)13:29
fgimenez_cachio: great, by default tpr splits the suite in 4 parts, you need only to modify the part that includes auto-refresh (removing it) the rest of executions should be fine with the generated testflinger job definitions13:30
cachiofgimenez_, yes, I forgot that :(13:30
kjackal_ogra_: is there a way to manualy edit the systemd unit file?13:31
ogra_nope13:31
ogra_but you could use a wrapper script13:31
ogra_i.e. have a single systemd unit that invokes the wrapper ... and have the wrapper start the daemons in the right order13:31
ogra_... as a workaround until such a feature is provided by snapd13:32
fgimenez_cachio: np, the good thing of having the suite splitted in four is that you can run the dragonboard tests in parallel, so that it doesn't take >6h, each part takes 1-2h13:32
kjackal_thanks ogra_13:34
fgimenez_cachio: and because those are executed in the lab you can run other kvm and board tests locally13:35
cachioyes13:35
cachiofgimenez_, starting now with the pi's13:35
kjackal_ogra_: is there a way to say i want my-snapped-service to start after all other services? like the Type=idle on systemd?13:37
fgimenez_cachio: cool! let me know how it goes13:37
ogra_kjackal_, not sure ... perhaps Chipaca knows, he played with systemd stuff a lot recently13:38
ogra_but i think the bits we can define for systemd units are still pretty limited13:38
Chipacakjackal_: no, there isn't ordering yet13:39
ogra_again though ... you can have a wrapper, use the system-observe interface to watch if something you wait for runs and sleep until it gets up13:39
Chipacakjackal_: and no support for Type=idle right now (although it _might_ work if you're sideloading)13:40
ogra_(though that means manually connecting the interface)13:40
Chipaca(I'd have to check)13:40
Chipacakjackal_: confirmed daemon: idle does not work13:44
kjackal_Chipaca: ogra_: Just for some context, the questions I am asking have to do with this issue: https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/357 Where the kubernetes bundle will not work after rebooting the machine hosting a deployment of the bundle on lxd13:46
kjackal_seems snap services should start after the lxd.service (if present) but they do not13:47
Chipacakjackal_: do you need a fix for the general problem, or for the particular instance?13:47
Chipacakjackal_: because you could drop a unit snipped in there13:47
ogra_kjackal_, bug 1687079 ??13:48
mupBug #1687079: cannot change profile for the next exec call: No such file or directory <Snappy:Confirmed> <https://launchpad.net/bugs/1687079>13:48
* zyga-ubuntu solicits feedback on https://forum.snapcraft.io/t/feature-snap-layouts/1471/213:49
kjackal_ogra_: the bug is most probably related to what I see13:50
ogra_yep13:50
ogra_thats why i posted it :)13:50
kjackal_Chipaca: since there are a bunch of services affected I would rather go for a generic solution (it affects practicaly all services)13:51
Chipacakjackal_: the lxd service the snap services are running are not part of the same snap?13:52
kjackal_no, as I understand it, you have an lxd container that runs its services (including and lxd.service) and inside that container you want to deploy a snapped service eg etcd13:54
Chipacakjackal_: so you have lxd, and inside lxd you have snapd, and install a snap with it?13:58
Chipacathat should just work, i don't see how it's a dependency13:58
kjackal_Chipaca: it is not the install that fails. The failure comes after rebooting the host. After the reboot the mysnapped-daemon comes before the lxd.service (both running inside the lxd container)14:00
Chipacakjackal_: how can the mysnapped-daemon come up before the lxd service? the mysnapped-daemon is inside a container started by lxd14:02
kjackal_Chipaca: wait, I am not explaining this correctly. Inside the lxd container there is an lxd service running.14:07
kjackal_Chipaca: have alook here: http://pastebin.ubuntu.com/25190796/14:07
kjackal_I ssh on a machine on AWS14:07
mupPR snapcraft#1426 closed: tests: adapt opencv's test expectations <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1426>14:07
kjackal_and then I juju ssh on an etcd/0 and ask for the lxd.service14:07
Chipacawith you so far14:09
cachiofgimenez_, the failover tests still failing after remobe the auto-refresh test14:40
cachiofgimenez_, I'll dig into those14:40
cachioniemeyer, there?14:45
fgimenez_cachio: maybe something has changed in the lab, if they are the only failing tests you can run them locally on your db14:45
zyga-ubuntuwat14:51
zyga-ubuntuwhy is master not broken anymore14:51
zyga-ubuntuWTF is going on :/14:51
kjackal_Chipaca: ogra_: if I set the restart option to always is there a way to also set the delay between restarts (RestartSec=??? on systemd)14:51
* zyga-ubuntu merged interface sanitize change, cannot wait to patch other PRs now14:51
kjackal_?14:51
ogra_zyga-ubuntu, want me to break it so you feel better ?14:52
zyga-ubuntuogra_: if you can break it the same way it used to be broken a moment ago? please !14:52
Chipacakjackal_: no14:52
mupPR snapd#3619 closed: interfaces/builtin: reduce duplication and remove cruft in Sanitize{Plug,Slot} <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3619>14:52
ogra_zyga-ubuntu, nah, thats not very creative ...14:52
Chipacakjackal_: you kinda wandered off mid-explanation back there14:52
* ogra_ would only break it in new and unexpected ways14:53
kjackal_Chipaca: sorry about that there was the daily standup14:53
zyga-ubuntuogra_: indeed, it would be exactly very repetetive14:53
ogra_cachio, fginther just run them with TMPDIR=/var/tmp ...14:53
* zyga-ubuntu looks at ondra's PR14:53
ogra_err14:53
ogra_s/fginther/fgimenez/14:54
kjackal_Chipaca: So we were at the point where, inside an AWS machine I have an lxd container. Inside the container I have a snaped etcd running next to lxd.service14:55
Chipacakjackal_: i think we didn't get that far14:55
Chipacakjackal_: you ssh'd to a host, juju-ssh'd to another, and showed me the lxd.service14:55
Chipacai asusme juju-ssh is to a container14:56
kjackal_Chipaca: Yes exactly (http://pastebin.ubuntu.com/25190796/)14:56
Chipacayou also showed me that there wasn't an lxd.service in the "outside" host14:57
kjackal_Chipaca: yes14:57
Chipacakjackal_: and it's in the "juju-ssh" container that you snap install something?14:57
kjackal_Chipaca: I suspect the lxd on the host is packaged with the conjure-up snap14:58
kjackal_Chipaca: yes14:58
Chipacakjackal_: and that thing you snap install depend son the lxd.service that runs on that same container?14:59
kjackal_yes14:59
Chipacakjackal_: so you you "systemctl stop lxd.service", it dies?14:59
kjackal_Chipaca: depends in the sence of a service/daemon14:59
Chipacait == the thing in the snap14:59
Chipacakjackal_: so, to be clear, I doubt very much there's going to be a way to be able to express this dependency in snapd any time soon15:00
kjackal_Chipaca: I see so... what else can we do?15:02
kjackal_Chipaca: thowing ideas, "Type=idle", "RestartSec=X"?15:03
zyga-ubuntuondra: hey15:03
ogra_wrappers ;)15:03
zyga-ubuntuondra: I pushed to https://github.com/snapcore/snapd/pull/362415:03
mupPR snapd#3624: rework for avahi interface <Created by kubiko> <https://github.com/snapcore/snapd/pull/3624>15:03
Chipacakjackal_: does Type=idle fix the thing for you?15:03
ondrazyga-ubuntu hey :)15:03
zyga-ubuntuondra: please look at my feedback and ping me if you want to talk15:03
ondrazyga-ubuntu sorry I was off yesterday15:03
Chipacakjackal_: that might be the easiest15:03
kjackal_Chipaca: it is a workaround15:03
Chipacaclearly15:03
zyga-ubuntuondra: use go fmt, I changed 2/3 of your files after just saving them in vim :)15:03
Chipacabut then, what you're wanting to do isn't supported15:03
ondrazyga-ubuntu yeah I read then all good15:03
Chipaca¯\_(ツ)_/¯15:04
zyga-ubuntuondra: I resolved conflict after landing a small API change in master15:04
zyga-ubuntuondra: so you should be all good now15:04
ondrazyga-ubuntu I have alerdady changes even15:04
ondrazyga-ubuntu ahh you did it as well?15:04
Chipacakjackal_: but my question is whether it's a workaround that _works_ :-)15:04
zyga-ubuntuondra: well, I just said so on the PR :)15:04
zyga-ubuntuondra: pull, you should be fine15:04
Chipacaif it's a workaround that leads to a race condition, then it's no good either15:04
kjackal_Chipaca: tested it with etcd service and it works, yes15:04
zyga-ubuntu(or rebase on top)15:04
Chipacakjackal_: but you said you had "a bunch"15:05
Chipacakjackal_: what happens if the whole bunch are all idle?15:05
zyga-ubuntuondra: feel free to iterate or stop as you see fit, I cna iterate on that next week as well15:06
ondrazyga-ubuntu let's have a look next week15:06
kjackal_Chipaca: If we identify this as a potential solution then I will need to test that before we commit to a fix/feature implementation15:06
ondrazyga-ubuntu I did changes as per your comments, but then two of my tests were paninckin15:07
ondrazyga-ubuntu and I was not able to figure out why15:07
zyga-ubuntuondra: ok, put a comment on the PR or here if you want some insta-hints15:07
kjackal_Chipaca: can I ping you on monday with what I have? I am almost at EOD here15:07
Chipacakjackal_: so, the Type=idle thing is easy to implement at our end, but that's still a month away from you being able to use it15:08
zyga-ubuntuChipaca: unless edge15:08
zyga-ubuntuChipaca: then it's tomorrow :)15:08
Chipacakjackal_: so I think we can talk about whether allowing snaps to specify arbitrary After= lines is reasonable15:08
Chipacakjackal_: that's a day of discussion probably, but over a week away because our architect is on holiday15:09
ChipacaI don't think it'd break anything conceptually to allow snaps to say After=bananas15:10
Chipacait's not a dependency, it doesn't pull in anything, it's just ordering15:10
ogra_as long as the service is included in your snap at least15:10
Chipacaalthough it can break a system :-(15:10
Chipacagah15:10
Chipacait can break a system by creating a loop15:10
ondrazyga-ubuntu so those changes you pushed, are those just formatting changes?15:11
kjackal_Chipaca: ogra_: After=?? would also need for Wants=???15:11
Chipacakjackal_: why? all you want is After=15:11
Chipacakjackal_: lxd is already being started, you just want to tweak the order15:11
Chipacaanyway, it's for discussion with our architect, not for irc15:11
Chipaca(and it's not super-obvious that it'll work)15:11
zyga-ubuntuondra: no, I resolved conflicts, fixed one more non-compiling-after-merge conflict (that git wasn't complaining about), re-formatted, dropped the Sanitize methods (empty, no longer needed)15:11
ChipacaWants= is super obvious to me that it's wrong for snaps15:12
zyga-ubuntuondra: that's all15:12
=== cachio is now known as cachio_lunch
zyga-ubuntumorphis: I'm going to push some changes to https://github.com/snapcore/snapd/pull15:12
morphiszyga-ubuntu: which pull request?15:12
ondrazyga-ubuntu so shall I rebased on your commit and to changes based on comments on top of that?15:12
kjackal_Chipaca: sounds good Chipaca should we talk again towards the end of next week to setup a meeting?15:13
zyga-ubuntuoh15:13
zyga-ubuntuman15:13
zyga-ubuntuhttps://github.com/snapcore/snapd/pull/362315:13
mupPR snapd#3623: interfaces/builtin: implement broadcom-asic-control interface <Created by morphis> <https://github.com/snapcore/snapd/pull/3623>15:13
zyga-ubuntuhow did that happen15:13
Chipacakjackal_: sounds good15:13
zyga-ubuntuondra: yep, that sounds perfect15:13
morphiszyga-ubuntu: what kind of changes?15:13
kjackal_Thank you both Chipaca and ogra_15:13
ogra_well, i was only shouting from the side-line ...15:14
zyga-ubuntumorphis: I'll resolve conflicts and do small tweaks in line with recently landed huge interface cleanup PR15:14
morphisok15:14
Chipacaogra_: take your internet thank yous! they're tradeable for beer!!15:14
morphiszyga-ubuntu: thanks!15:14
ogra_Chipaca, noted ...15:14
* ogra_ puts in his bag15:14
zyga-ubuntujust so that you and others don't have to wonder what to do after their branches break15:14
zyga-ubuntumorphis: and rename the files, man, nobody noticed that !15:15
morphiswhich files?15:15
ogra_the wrongly named ones15:15
ogra_:P15:15
zyga-ubuntumorphis: dashes instead of underlines in filenames15:16
morphisdid that change recently?15:16
zyga-ubuntumorphis: no, I think we never used dashes in filenames15:18
zyga-ubuntumorphis: pushed15:22
* zyga-ubuntu moves to another PR15:26
* zyga-ubuntu moves to a coffee break15:30
* genii cartwheels by without spilling his coffee15:30
=== grumble is now known as grumble2
=== rumble is now known as grumble
=== cachio_lunch is now known as cachio
ogra_fgimenez, the pi2-kernel in edge is behind the rest once again, are you fie with me bumping it to the version of beta/candidate ?15:58
fgimenezogra_: sure, we are currently not doing any testing of pi2 in edge, cachio, ok to have them in sync?15:59
janisozaurzyga-ubuntu: jestem16:01
mupPR snapd#3632 opened: asserts,overlord/assertsstate: auto refresh account-keys <Created by pedronis> <https://github.com/snapcore/snapd/pull/3632>16:11
* zyga-ubuntu EODs16:49
zyga-ubuntuttyl16:49
zyga-ubuntuo/16:49
mupPR snapd#3633 opened: overlord/devicstate: fix, don't assume that the serial is backed by a 1-key chain <Created by pedronis> <https://github.com/snapcore/snapd/pull/3633>16:58
=== alan_g is now known as alan_g|EOW
=== JanC is now known as Guest48595
mupPR snapcraft#1427 opened: Add cx_Freeze options targeting bin/snapcraft <Created by alextnewman> <https://github.com/snapcore/snapcraft/pull/1427>18:49
mupPR snapcraft#1428 opened: core: reexec as root for `os` snaps if necessary <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1428>19:28
=== jaggz_ is now known as jaggz
mupPR snapd#3634 opened: interfaces/many, cmd/snap-confine: miscellaneous policy updates <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3634>21:27
mupPR snapcraft#1429 opened: cli: properly handle exceptions in lifecycle <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1429>22:59

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