/srv/irclogs.ubuntu.com/2017/09/19/#snappy.txt

=== ikey is now known as ikey|zzz
=== JoshStrobl is now known as JoshStrobl|zzz
zyga-ubuntuo/05:21
zyga-ubuntumvo: hey, how is 2.28 looking06:13
mupBug #1718026 opened: Applications from installed snaps don't appear in activities overview <Snappy:Fix Committed> <snapd (Ubuntu):New> <https://launchpad.net/bugs/1718026>06:14
mvozyga-ubuntu: looking good overall looks like some things need to go into it (3937)06:14
mvozyga-ubuntu: did you figure out if it is this rule that triggered the issue?06:14
zyga-ubuntumvo: no, I had to leave last night06:14
zyga-ubuntumvo: jamie prepared a PR and some rationale06:14
zyga-ubuntumvo: it looks good but fails testing06:14
zyga-ubuntumvo: I will investigate that soon06:14
mvozyga-ubuntu: testing is currently failing left and right but I have not looked into the why yet too06:15
zyga-ubuntumvo: I wanted to highligh one bug https://bugs.launchpad.net/snappy/+bug/1718026 - this is fixed in master, I haven't checked if it is in 2.28 reease branch06:15
mupBug #1718026: Applications from installed snaps don't appear in activities overview <Snappy:Fix Committed> <snapd (Ubuntu):New> <https://launchpad.net/bugs/1718026>06:15
zyga-ubuntumvo: if we do another respin it would be worth to pick it up06:15
zyga-ubuntumvo: specifically this patch should be in 2f55619677810a4872145b8ccbbe2bdc1ba364fa06:16
mvozyga-ubuntu: aha, build fails in udev-support.c in a bunch of places - setup_dvices_cgroup06:16
zyga-ubuntumvo: build fails? that's odd - in jamie's branch tests failed but the build was (supposedly) ok06:17
* zyga-ubuntu looks again06:17
mvozyga-ubuntu: this is the opengl change06:18
* mvo goes top-to-bottom06:18
zyga-ubuntuaha, I was looking another pr06:19
mvozyga-ubuntu: no worries, I fixed the issue in jamies06:19
zyga-ubuntuthank you06:20
* zyga-ubuntu walks kids to school06:22
niemeyerEarly hellos06:23
mvoniemeyer: good morning, early is an understatement here :)06:24
niemeyermvo: :P06:24
mvozyga-ubuntu: btw what happend to the debian man-page bug, did it fix itself magically?06:24
niemeyermvo: Just felt like one of those days..06:24
mvoniemeyer: heh :) take it easy06:25
zyga-ubuntumvo: probably06:26
zyga-ubuntumvo: it broke a lot of sid :)06:26
niemeyermvo: We had some issues with Debian preparing yesterday.. have you seen that?  Just wondering if it went away already..06:28
niemeyermvo: Felt like an unrelated packaging bug06:29
mvoniemeyer: yeah, we ran into it yesterday, I'm just looking, its debian bug #876128, zyga-ubuntu has a workaround PR, I check the bugstatus and then we can apply the workaround if needed06:34
mupBug #876128: Window too tall <amd64> <apport-bug> <oneiric> <running-unity> <thunderbird (Ubuntu):New> <https://launchpad.net/bugs/876128>06:34
* mvo wonders if we should test against testing instead of unstable - this would prevent issues like this one06:36
niemeyermvo: Yeah, that might be a good idea06:39
niemeyermvo: We'd still pick issues earlier but not be so close to the edge to face those unrelated bugs that will likely be fixed there06:40
mvoindeed06:40
niemeyermvo: What are most devs using?06:40
mvoniemeyer: unstable I would say06:40
niemeyermvo: Well.. then there's not much of a choice06:40
mvoniemeyer: we can pick 3932 with the workaround06:40
niemeyermvo: We do want to know if people have a broken setup06:41
mvoniemeyer: right06:41
niemeyermvo: 20h ago.. was it broken for other reasons?06:42
ackkniemeyer, thanks for the review06:44
niemeyerackk: Heya06:45
mvoniemeyer: tsee the comment in the PR, the PR itself is fine, I think it should include a link to the debian bug so that we know when we can remove this again. I added that now06:45
niemeyerackk: No problem, thanks for the PR.. hopefully the notes make sense?06:45
niemeyermvo: Good idea06:46
ackkniemeyer, sure. fwiw the main reason I went for a string for the socket-mode is that using an int makes snacfraft write it out in decimal notation in the snap.yaml. I guess that can be somehow overridden (although even leaving as decimal works of course)06:46
niemeyerackk: Yeah, and I think it's easy to fix that as well06:51
niemeyerackk: I mean, forcing the notation on ints06:51
niemeyerackk: Look at the second answer here:06:53
niemeyerackk: https://stackoverflow.com/questions/18666816/using-python-to-dump-hexidecimals-into-yaml06:53
niemeyerHey.. I know this guy... :P06:53
ackkheh06:57
ackkniemeyer, thanks for the pointers06:57
mvozyga-ubuntu: do you know more about this udev regression that jamie talks about in the forum? I just tried to reproduce but no luck06:57
niemeyerackk: The question about the actual structure of the syntax and its equivalence in the systems is more of an issue though, but we should discuss this in the forum07:00
zyga-ubunture07:07
zyga-ubuntumvo: the udev issue can be reproduced on a artful (probably zesty as well) machine runnin wayland07:07
zyga-ubuntumvo: for me it was sufficient to install gimp and then hit alt-f4 (note, just alt-f4, nothing more)07:07
zyga-ubuntumvo: if you switch to VT4 you are affected07:08
zyga-ubuntumvo: and subsequent ctrl-c will kill gnome-shell07:08
mvozyga-ubuntu: sorry, I mean the other issue that jamie reported, https://forum.snapcraft.io/t/2-28-release-cycle-started/2026/1307:08
zyga-ubuntumvo: I wasn't aware of this, lookng07:08
zyga-ubuntumvo: it looks like a busy day ahead, sorry for starting late btw, I wanted to have a walk as I'm mostly indoors lately07:10
mvozyga-ubuntu: no worries07:10
mvozyga-ubuntu: and +1 for walking, its good for productivity :)07:10
mvo(seriously!)07:10
zyga-ubuntumvo: so first thing I checked if jamie's test snap has any apps in it07:11
zyga-ubuntubut it does...07:11
zyga-ubuntu(pretty fancy snap actually)07:11
zyga-ubuntumvo: let me try to reproduce on master07:12
zyga-ubuntumvo: so far I cannot think of anything07:12
zyga-ubuntumvo: well, :)07:13
zyga-ubuntumvo: maybe one (silly) explanation would be that jamie tried to disable the udev backend while exploring the wayland issue07:13
zyga-ubuntumvo: and then didn't notice this when testing the other issue07:14
zyga-ubuntumvo: also the udev code has not changed lately07:16
zyga-ubuntumvo: last change is in March 201707:16
zyga-ubuntumvo: so +1 on doubt it's real07:16
mupPR snapd#3777 closed: snap-repair: implement basic `snap-repair {list,show}`  <Created by mvo5> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/3777>07:32
mvoniemeyer: I was just looking over the spread PRs (got a ping about #30) and it seems like e.g. #33 is uncontroversial (maybe tweaking the error a bit)07:37
mupPR #33: small daemon tests cleanup <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/33>07:37
mupPR snapd#3932 closed: spread: work around temporary packaging issue in debian sid <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3932>07:38
niemeyermvo: Yeah, the queue needs some love.. let me check that one07:39
zyga-ubuntumvo: btw, I found the debian bug tracker entry for manpages: 87612807:40
zyga-ubuntumvo: shall I add it?07:40
niemeyerzyga-ubuntu: mvo already did it07:41
zyga-ubuntuah, thank you mvo :)07:41
mvozyga-ubuntu: I added it already and merge the PR, sorry that I wasn't more explicit about it07:41
zyga-ubuntuI didn't notice07:41
niemeyerAnd merged it07:41
niemeyerzyga-ubuntu: He also fixed the apparmor summary/level PR07:42
zyga-ubuntuniemeyer: yes, we discussed that earlier so I knew about it07:42
niemeyerzyga-ubuntu: You'll need to get layouts working to compensate him ;)07:43
zyga-ubuntuworking on that :)07:44
pedronismvo: hi, where should we create the lock file for snap-repair run ?07:45
niemeyerzyga-ubuntu: That last comment from jdstrand is a bit eye opening.. we need to address review items made, or respond to the points when that's the case07:45
niemeyerzyga-ubuntu: But definitely not ignore.. as a rule of thumb, always respond to all review points in a review, even if it's a thumbs up07:46
zyga-ubuntuniemeyer: I discussed each point jamie made07:47
niemeyerzyga-ubuntu: So why is his review reading like that?07:47
zyga-ubuntuniemeyer: I didn't understand that he essentially said that is all mandatory07:48
zyga-ubuntuniemeyer: when you are subtle on the internet...07:48
niemeyerzyga-ubuntu: "You did not address the off-by-one in read_cmdline() yet"07:48
zyga-ubuntuniemeyer: read the full discussion, I disagree about that item (it's not a bug, it's an optimization)07:49
zyga-ubuntuniemeyer: I'll address each element jamie made, it's just not easy to keep track of everything in that PR and, as I said, I was confused by his earlier statements about what is needed and what is optional and nice07:49
mvopedronis: I have not looked at your latest branch yet but maybe in main.go when you start the runner. I will soon look at your latest PRs maybe I have a better idea then07:50
zyga-ubuntu(some of the things I pushed into the PR were in the "nice" category according to my earlier understanding)07:50
oSoMoNzyga-ubuntu, hey, out of curiosity, could you please point me to the commit in master that fixes bug #1718026 ?07:50
mupBug #1718026: Applications from installed snaps don't appear in activities overview <Snappy:Fix Committed> <snapd (Ubuntu):New> <https://launchpad.net/bugs/1718026>07:50
pedronismvo: no, I mean on disk?07:50
zyga-ubuntuoSoMoN: yes, one sec07:50
zyga-ubuntuoSoMoN: 2f55619677810a4872145b8ccbbe2bdc1ba364fa07:50
pedronismvo: sorry, I wasn't clear, I mean where on disk07:50
zyga-ubuntuoSoMoN: I verified this by installing the updated file on 17.10 running wayladn07:51
oSoMoNzyga-ubuntu, thanks07:51
mvopedronis: oh, sorry :) /var/lib/snapd/repair/lock maybe?07:51
pedronisok07:51
pedronisthat seems reasonable07:51
pedronisand what I was thinking as well07:51
pedronisah07:52
pedronismmh07:52
pedronismvo: no reason to creat it in /run/snapd/repair/lock ?  (I see we create some other locks under /run)07:52
niemeyerzyga-ubuntu: His comment seems spot on regarding off by one..07:52
niemeyerzyga-ubuntu: and he's asking you to take it into account07:52
mvopedronis: oh, that is a good one too, the advantage is of course that its slightly cleaner)07:52
mvopedronis: less clutter on the real disk, so that works for me07:53
niemeyerzyga-ubuntu: His code is both safer and more technically correct as it handles the length of the string properly for the exact fitting case07:53
zyga-ubuntuniemeyer: yes, I don't disagree about it but it's not a blocker in any sense of the word07:54
niemeyerzyga-ubuntu: just n > m on itself would already be worth fixing07:54
pedronismvo: thanks07:54
niemeyerzyga-ubuntu: Instead of an exact == match07:54
niemeyerzyga-ubuntu: Please take Jamie's review more carefully..07:54
pedronismvo: I'm reviewing your PRs btw07:55
mvopedronis: thanks for that!07:58
mvopedronis: I will address and look at your PRs too. fwiw, if you merge master into your PRs tests should be happy again (kudos to zyga-ubuntu for the workaround)08:01
pedronisah, good08:02
mwhudsonwhat does "Ubuntu Artful, for Ubuntu Core 16" mean in a snap build on launchpad?08:32
mvomwhudson: my guess is that the snap was build with an artful chroot but is targeted towards ubuntu core 16 (which AFAIK is the only target we have currently :)08:36
mwhudsonah so build-packages etc will come from artful08:36
mwhudsonthat might actually be useful for me08:36
mwhudsoncan i get cleanbuild to use an artful lxd?08:36
pedronismvo:  I added the flock code to the loop PR08:37
mvomwhudson: I think you can but I'm not familiar with the syntax unfortunately08:38
mvopedronis: thank you08:38
mwhudsonmvo: "        self._image = 'ubuntu:xenial/{}'.format(deb_arch)" :(08:40
mvomwhudson: *cough*08:40
ogra_urgh ... i hope we dont allow that08:40
mvomwhudson: sounds like a PR waiting to happen then08:40
ogra_nne of the deps will be coorrrect08:40
ogra_*none08:40
ogra_afaik launchpad always calls cleanbuild regardless ... so the host might be artful but the build container will still be xenial08:42
pedronisthat message seems mostly confusing then (the host shouldnd't play a big role)08:43
mvoindeed, if thats the case its pretty useless to even offer builds on anything but xenial08:44
ogra_well, and we shuldnt08:45
pedronisyes, until we have base-18 (or however we will call it)08:46
ogra_buillding on anythhing but xenial for core 16 willl only result in dependency mess for stage-packages, you really dont want that08:46
ogra_ight08:46
ogra_once we have another base/core we should have choices, but not atm08:46
mwhudsonogra_: no, that's definitely not what is happening08:47
ogra_got a link to the log ?08:47
mwhudsonogra_: https://launchpadlibrarian.net/337523100/buildlog_snap_ubuntu_artful_amd64_subiquity-artful_BUILDING.txt.gz08:47
mvopedronis: should we restrict summary further or is checking for \n enough in your estimate?08:48
ogra_yeah, that uses atful all the way through, thats definitely wrong and dangerous08:48
mwhudsoni assume lp didn't just add this by accident...08:49
ogra_mwhudson, talk to wgrant or cjwatson08:49
ogra_such builds shouldnt be provided08:49
cjwatsonwat08:50
ogra_(imagine yoou wouldnt use the python plugin (which pulls in the interpreter) but just dump to copy sme py modules in place ... how woould you knw they are even remotely compatible with the intepreter shhipped in core (which we'd fall back to in this case)08:50
mvohow would people get e.g. stage-package for artful then?08:50
cjwatsonsorry, how is this our problem?08:50
ogra_cjwatson, why doesnt it call cleanbuild ?08:51
cjwatsonLP has never used cleanbuild08:51
cjwatsonit uses containers based on its own chroots08:51
ogra_still08:51
pedronismvo: ScanLine uses the equivalent of `\r?\n`08:51
cjwatsonif it's using artful it's because the person who set it up explicitly asked for it to use artful08:51
pedronis*ScanLines08:51
cjwatsoncleanbuild is beside the point08:52
ogra_another example ... pulse has 100% incompatible API changes in libpulse between xenial and all the following releases ... as soon as your app uses sound you are screwed ...08:52
mvopedronis: aha, nice. I will use a similar pattern than, but essentially a blacklist \n\r? is ok, we don't want a fancy whitelist "[a-zA-Z,.-?...] (?)08:52
cjwatsonright, so don't do that, but it still needs to be possible to run test builds where appropriate08:52
ogra_builds should only target xenial08:52
cjwatsonso we won't withdraw the facility08:52
pedronismvo: don't think we need a fancy whitelist, no08:52
mvopedronis: thank you08:53
pedronisit's mostly us for now that will make them08:53
mvo+108:53
pedronismostly keeping us honest, not confuse that code08:53
ogra_cjwatson, well, as long as you cant release the result to the store i guess it is fine for test building ... to me it looked like mwhudson was hitting a default though08:53
cjwatsonyou can, but it's up to the builder to ensure that the result works, like anything08:54
cjwatsonI mean the person operating the recipe08:54
cjwatsonartful is not the default08:54
mwhudsoni selected this option because i _want_ all my stage-packages etc to come from artful08:54
mwhudsonat least for a trial08:55
ogra_mwhudson, right, just dont release it ... (and i think there shoudl be at least a warning when you select that option)08:55
cjwatsonthe default is xenial08:55
mwhudsonogra_: this is subiquity, it is a normal snap in no ways at all08:55
mwhudsonfor another questoin08:56
ogra_mwhudson, well, what goes to the store should always target xenial ... if you do test builds it doesnt matter indeed (not sure about the benefit of it ... but well)08:56
wgrantSomeone can produce something that doesn't work by asking snapcraft to download a tarball with the wrong ABI.08:56
ogra_sure08:56
wgrantI don't see why we should prevent them from shooting themselves in the foot if they don't know what they're doing by selecting a series that won't work for them.08:57
mwhudsonfor another question08:57
mwhudsonhm no08:57
ogra_but thats a pretty explicit action ... i'd expect you to knwo what yuo do any why in that case08:57
ogra_wgrant, the point is that it wont work for the ednuser ... it might just build fine for the developer08:57
mwhudsonoh heh pushing my change to github and then immediatly building from the import branch on lp, not so useful08:57
ogra_mwhudson, fsvo "immediately" :)08:58
mwhudsonogra_: i generally don't wait for 3 or so hours by mistake08:59
ogra_(i have never the luck to hit the import schedule with my commits ... it usually just ran or says "will run in 20min" or soo)08:59
ogra_you can just click the import button08:59
mwhudsonyes i know09:00
mwhudsoni implemented the import button :)09:00
ogra_heh09:00
mwhudsona very very long time ago now09:01
pstolowskizyga-ubuntu, is the bug re manpages (that you mentioned above) concerning snapctl (discussed yesterday)?09:07
zyga-ubuntupstolowski: there are two bus, one affecting all of debian sid (packaging bug where file clashes) and one affecting snapd (snactl does not have a man page but is in path)09:08
mupPR snapd#3939 opened: core/repo: Connection type <Created by stolowski> <https://github.com/snapcore/snapd/pull/3939>09:11
pstolowskiniemeyer, pedronis, zyga-ubuntu : quick first feedback appreciated before I dive in with more significant changes: https://github.com/snapcore/snapd/pull/393909:11
mupPR #3939: core/repo: Connection type <Created by stolowski> <https://github.com/snapcore/snapd/pull/3939>09:11
pstolowskizyga-ubuntu, i see, ack. ok, I'm going to work on snapctl man page now09:12
zyga-ubuntupstolowski: thanks, it's not a high priority but I think one will be useful for developers and users alike09:17
niemeyerIs it time for another review sprint yet, or too soon?09:35
niemeyermvo: How do you feel about our queue atm?09:35
mvoniemeyer: its too big, but I hope before EOD we can close a good bunch, some are very close but tests issues have slowed progress a bit09:35
ackkniemeyer, wrt the SocketInfo struct, you're suggesting to add Name as an attribute and have App.Sockets as a []SocketInfo rather than a map, right?09:36
ackkniemeyer, that would also allow having socket.File() as you suggested09:36
niemeyerackk: Let's please discuss this in that topic in the forum09:36
niemeyerOh, wait09:36
niemeyerackk: Sorry, separeate issue09:36
niemeyerackk: No09:36
ackkyeah, that's about implementation details09:36
ackk:)09:36
niemeyerackk: The idea was still having it as a map09:37
niemeyerackk: Having a Name attribute isn't something I thought of, but sounds like a good idea09:37
niemeyerackk: Even if redundant with the map key09:37
ackkniemeyer, yeah I was trying to avoid duplication09:37
niemeyerackk: It's good duplication in this case..09:38
ackkniemeyer, FWIW I'm not sure we even a map at that point09:38
niemeyerackk: Means we can use the SocketInfo by itself09:38
ackkright, I do like the idea of socket.Name09:38
niemeyerackk: What's the issue iwth the map?09:38
ackkniemeyer, I don't think there's really one, but if we don't need to look up by name (which we probably don't), we could just keep a list09:39
ackkI'm talking just about App.Sockets, not about the Yaml format, to be clear09:39
ackkanyway, duplication is not a big deal in this case, agreed09:40
ackkif you prefer to keep the map, LGTM09:40
niemeyerackk: Yeah, let's keep it at least for now, as it reflects the structure in the yaml.. but there's still that more fundamental underlying question about whether the whole thing is workable or not09:40
ackkniemeyer, what's the issue?09:41
niemeyermvo: Ack.. let's wait til the EOD and reevaluate tomorrow09:41
niemeyerackk: It's in the review09:41
ackkniemeyer, ah yeah, I did answer that in the review09:42
mvoniemeyer: +109:44
ackkxnox, hi, can you define more than one Listen* in a .socket file?09:46
pstolowskizyga-ubuntu, do we still want bugs concerning snap-confine to be reported against https://bugs.launchpad.net/snap-confine/+filebug ? or is it the main snapd bug tracker now?09:53
zyga-ubuntupstolowski: I don't have a strong opinion on that09:54
zyga-ubuntupstolowski: it can be moved to snapd09:54
pstolowskizyga-ubuntu, i see, just asking, wasn't sure if snap-confine bugtracker isn't dead or something09:55
zyga-ubuntupstolowski: I'm not sure either, we have snapd, snappy, snap-confine + distro versions of those09:56
zyga-ubuntuso it's all a bit all over the place09:56
pstolowskizyga-ubuntu, ok, I was wondering about updating snap-confine manpage while on it, but it seems there is no obvious answer, that's fine09:58
xnoxackk, hi, you can use $ systemd-analyze verify FILE for systemd to check syntax of a unit file.10:03
zyga-ubuntupstolowski: go ahead and do it if you want to10:05
zyga-ubuntupstolowski: one less bug tracker to worry about (eventually)10:05
pstolowskiok10:05
pedronismvo: niemeyer: I have an errand this afternoon, if there are no delays I should be back for the standup, but I might not10:06
mvota10:07
niemeyerpedronis: Thanks for the note10:08
niemeyermvo: #3934 is reviewed10:09
mupPR #3934: snap-repair: implement `snap-repair {list,show}` <Created by mvo5> <https://github.com/snapcore/snapd/pull/3934>10:09
niemeyermvo: Have you seen the description on #3929?10:09
mupPR #3929: Correct typo in test <Created by gliptak> <https://github.com/snapcore/snapd/pull/3929>10:09
niemeyermvo: I'm assuming the answer is "no" :D10:10
niemeyermvo: I mean, to the questions in the description10:10
mvoniemeyer: thanks for the review, i have a look10:11
mupPR snapd#3940 opened: dirs: fix classic support detection <Created by chipaca> <https://github.com/snapcore/snapd/pull/3940>10:11
mvoniemeyer: I had hoped we get an answer for the question.10:12
mupPR snapd#3929 closed: snap: correct typo in test name <Created by gliptak> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3929>10:12
niemeyermvo: Well, getting "No" as a description wouldn't be much better I'm afraid.. LOL10:12
mvoniemeyer: heh :)10:13
* zyga-ubuntu runs tests on updated update-ns PR and takes a break10:17
mupPR snapd#3941 opened: overlord/snapstate: prefer a smaller corner case for doing the wrong thing <Created by chipaca> <https://github.com/snapcore/snapd/pull/3941>10:21
* zyga-ubuntu hugs chipaca10:22
* Chipaca hugs back10:23
mwhudsonsnapping snapcraft is quite a slow process, probably not the right way to go for testing local hacks :)10:26
* Chipaca hugs the snaps he's written before snapcraft was a thing10:28
zyga-ubuntumwhudson: does snap try on snapcraft works?10:28
niemeyerI still sort of want "snap pack" or something similar..10:29
mwhudsonzyga-ubuntu: huh i didn't know about snap try10:29
Chipacamwhudson: dude :-)10:30
Chipacamwhudson: as long as you're not modifying the snap metadata, with snap try you can edit stuff and test them without touching anything else10:30
mwhudsonsounds exciting :)10:32
mvoniemeyer: "snap pack" would be like "snapcraft snap" ?10:33
niemeyermvo: I'm not sure, which is perhaps another reason why I'd like to have it :)10:35
niemeyermvo: snapcraft snap used to mean something else10:35
niemeyermvo: Its help text says:10:36
niemeyer  If you want to snap a directory, you should use the snap-dir command10:36
niemeyer  instead.10:36
niemeyerExcept, snap-dir doesn't exist?10:36
niemeyersergiusens: ^10:36
niemeyermvo: I think "snapcraft" and "snapcraft snap" are the same thing10:37
niemeyermvo: Anyway, yeah.. something dumb that just packs a snap dir..10:37
niemeyerpstolowski: #3939 LGTM10:38
mupPR #3939: interfaces: add Connection type <Created by stolowski> <https://github.com/snapcore/snapd/pull/3939>10:38
Chipacaniemeyer: snapcraft snap <directory> just mksquashfs's10:39
* Chipaca hasn't read all the context10:39
niemeyerChipaca: I don't think so..10:39
Chipacaniemeyer: you think wrong10:40
Chipaca:-)10:40
pstolowskiniemeyer, great, thank you. i'll shortly outline what's next on the standup (or after it)10:40
Chipacaniemeyer: but yes the docs for snap are wrong and/or misleading10:42
mwhudsondid i imagine something about using a persistent container for building snaps?10:44
niemeyerChipaca: Well, if it is a dumb pack, then it's broken in terms of UX too10:44
Chipacaniemeyer: no no no10:44
niemeyerChipaca: As "snapcraft snap" and "snapcraft snap foo/" would be completely different10:44
Chipacaniemeyer: no adding ux requirements mid-rant10:44
Chipaca:-)10:45
niemeyerLOL :)10:45
niemeyerChipaca: The former reads snapcraft.yaml and builds the whole snap10:45
Chipacaniemeyer: yes they are completely different, and surprising, and being in the current directory and doing one instead of the other is a pain10:45
Chipacawhich suggests that the idea is to move it to snap-dir? dunno10:46
Chipacabut today, it is like it is10:46
Chipaca(i'm just telling you how it is)10:46
Chipaca(in particular, i'm not saying it's the way it should be)10:46
pedronismvo: btw, should we build snap-repair for 14.04 at all?  we don't have 14.04 core systems10:47
niemeyerChipaca: I honestly don't know either.. but "snap pack" will solve it all.. :)10:48
Chipacaniemeyer: we used to have that10:48
pedroniswe still do internally (for tests)10:49
Chipacaniemeyer: we got rid of it to only have the mksquashfs call in one place (lots of little flags that need to be in sync)10:49
pedronisI mean, we didn't quite manage to get rid of the code10:49
pedronisanyway10:49
Chipacai know i know10:49
Chipacaalso, more and more snapcraft calls snap for things, so maybe we made the wrong call10:49
Chipacaand moving to snap pack is the right thing10:49
Chipacabut, there is that DRY thing i wanted to point out10:50
niemeyerChipaca: Yeah.. we should probably seed the idea with a hidden command and see what happens next10:50
sergiusensogra_ mwhudson wrt launchpad, it creates its own container and runs plain old snapcraft (without cleanbuild10:50
niemeyersergiusens: What's the deal with snap vs. snap-dir10:50
niemeyer?10:50
niemeyersergiusens: The snapcraft commands, I mean10:50
sergiusenswrt "mvo: "        self._image = 'ubuntu:xenial/{}'.format(deb_arch)" :(" as soon as base support comes in, we will map this to proper bases10:51
sergiusensbuild.snapcraft.io does not allow you to pick artful, if using launchpad directly you have a bit more "advanced" flexibility10:51
Chipacahow is there a travis build running for 3h+10:52
niemeyerChipaca: If seen a bug in travis happening recently where they concatenate two independent runs and sum up the logs and the times10:54
niemeyerChipaca:  Have a look in the middle of the logs.. look for a big jump in the timestamp10:54
niemeyersergiusens: Parlez-vous anglais?10:56
* zyga-ubuntu pushes lots of changes to https://github.com/snapcore/snapd/pull/3621 and crosses his fingers10:59
mupPR #3621: cmd/snap-{confine,update-ns}: apply mount profiles using snap-update-ns <Created by zyga> <https://github.com/snapcore/snapd/pull/3621>10:59
sergiusensniemeyer we have a bug for that, should be fixed soon (wrt snap-dir)11:09
niemeyersergiusens: Ack, thanks11:09
niemeyersergiusens: Which direction will the fix go?11:09
sergiusensand the issue was oversight11:09
sergiusensniemeyer we need to implement snap-dir and deprecate using `snap <directory>`11:10
niemeyersergiusens: Nice, sounds like the best option11:10
niemeyersergiusens: Might be worth renaming snap-dir, though, to make the delta more clear11:10
sergiusensniemeyer suggestions?11:11
niemeyersergiusens: snapcraft pack11:11
* sergiusens is open for them early in the morning after staying up too late working on integration tests11:11
niemeyersergiusens: Hopefully one day this will simply call "snap pack"11:11
* zyga-ubuntu does school run11:11
sergiusensniemeyer rings nice as I say it out loud11:12
niemeyersergiusens: Yeah, it's dumb-sounding11:12
pedronisChipaca: what's the status of #3835 ?11:12
mupPR #3835: packaging: bring down the delta between 14.04 and 16.04 <Created by chipaca> <https://github.com/snapcore/snapd/pull/3835>11:12
niemeyer"Just pack it up..."11:12
Chipacapedronis: needing a second review11:12
zyga-ubuntuChipaca: it's also red11:13
ogra_sergiusens, yeah, i learned that above, i always though it uses cleanbuild :)11:13
mvopedronis: good point, we don't need snap-reparir for 14.04 at all11:14
Chipacapedronis: "red"?11:14
zyga-ubuntuChipaca: tests are failing11:15
Chipacathe last spread test was green11:15
pedronisthey are being re-run11:15
Chipacawhich tests were failing?11:15
zyga-ubuntuautopackage11:16
pedronisChipaca: afaict spread tests are running atm because mvo just pushed master into it11:16
zyga-ubuntumaybe harmless, just observing11:16
Chipacayes11:16
Chipacabut if you click the previous commit's red X, you'll see the green spread tests11:16
mvoniemeyer: I would love to bring "snap pack <dir>" back, this would allow us to stop building the snapbuild helper for our tests. if there is a +1 from you I make this happen, should be reasonable quick11:16
mvoChipaca: there wre conflicts11:16
niemeyermvo: +1!11:16
pedronismvo: let's close a bit more PRs first though :)11:17
mvoChipaca: this is why I merged master/pushed (in debian/snapd.dirs)11:17
mvopedronis: *pfff*11:17
Chipacamvo: thank you11:17
mvopedronis: ;)11:17
Chipacato be expected when it's just sat there for nearly three weeks11:17
mvoChipaca: yeah, it has my +111:17
Chipacamvo: yes11:17
niemeyerpedronis: Man, we're in a roll today! ;)11:18
niemeyerI should not sleep more often.. feels so much more productive11:18
niemeyerLOL11:18
Chipacaniemeyer: _on_ a roll, hopefully11:18
niemeyerChipaca: Both!11:18
Chipacaheh11:19
pedronismvo: what's the status of #3901 ?  issues with 14.04 ?11:20
mupPR #3901: snap-seccomp: run secondary-arch tests via gcc-multilib <Created by mvo5> <https://github.com/snapcore/snapd/pull/3901>11:20
pedroniscan't we skip it there for now? and see about that in a follow up?11:20
mvopedronis: the last run died because of the debian issue, I haven't yet the status since (I merged master this morning)11:22
niemeyerpstolowski: #3918 reviewed too11:23
mupPR #3918: hooks: substitute env vars when executing hooks <Created by stolowski> <https://github.com/snapcore/snapd/pull/3918>11:23
niemeyerpstolowski: LGTM, but let's just move the test logic into the existing test that is already supposed to check basics of snap run for hooks11:23
sergiusensogra_ they should produce similar environments in the end11:25
ogra_sergiusens, sure ... unless the wrong release is used11:25
pstolowskiniemeyer, i can, np.. it is sometimes nice though to have even basic tests separated, as a failure gives a very quick hint about what went wrong11:25
ogra_(which was my point)11:25
sergiusensogra_ oh, not sure we plan to support anything outside of bases for that11:26
ogra_sergiusens, well, we do support building a snap completely on artful atm ... that was what made me curious11:26
ogra_sergiusens, but it is an explicit action you have to select (despite likely producing something useless)11:26
sergiusensogra_ well, yes, given we release snapcraft to artful means we have to make sure it passes tests11:27
ogra_sergiusens, this is about the option to pick artful as a target for your snap ... i know snapcraft runs its autopkgtests11:27
* Chipaca ~> lunch11:31
niemeyerpstolowski: We don't really want to split out functional tests every few lines because they are testing a slightly different aspect of the same logic11:35
pstolowskiniemeyer, fair enough11:35
niemeyerpstolowski: The test is literally called "snap-run-hook" already..11:35
niemeyerpstolowski: I'd even more the env into the snap used in that test11:36
* Chipaca ~> really lunch11:47
ogra_mvo, uhmmmm ... your change to xdg-open cant really work ... we have no dbus-send inside core ...11:47
ogra_hmm, ignore that, the old code also used dbus-send ... how the heck did that work without a dbus-send binary11:49
ackkxnox thanks11:55
mupPR snapd#3887 closed: snapstate: auto-install missing base snaps <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/3887>11:58
=== ShalokShalom_ is now known as ShalokShalom
niemeyerpstolowski: #3852 just has the rename mentioned in the first comment pending12:04
mupPR #3852: hooks: commands for controlling own services from snapctl <Created by stolowski> <https://github.com/snapcore/snapd/pull/3852>12:04
niemeyerpstolowski: Should be something like servicestate.Change, similar to configstate.Configure, cmdstate.Exec, snapstate.Install, etc.12:05
pstolowskiniemeyer, indeed, but I thought it was ok to postpone this, per your second comment?12:06
niemeyerpstolowski: The second comment is talking about the return value12:07
niemeyerpstolowski: Unrelated to the rename12:07
pstolowskiniemeyer, I see. ok. misunderstood12:07
niemeyernp12:07
ackkniemeyer, posted the question related to the yaml format on https://forum.snapcraft.io/t/socket-activation-support/2050/1212:08
mupPR snapd#3942 opened: doc: snapctl man page <Created by stolowski> <https://github.com/snapcore/snapd/pull/3942>12:10
pstolowskizyga-ubuntu, ^12:10
niemeyerackk: Thanks12:13
niemeyerackk: Also reading your feedback on the PR12:13
ackkniemeyer, I just pushed the other changes you requested12:13
=== chihchun is now known as chihchun_afk
zyga-ubunture12:21
zyga-ubuntupstolowski: thank you12:21
zyga-ubuntupstolowski: reviewed12:25
zyga-ubuntujdstrand: hey, around?12:28
=== JoshStrobl|zzz is now known as JoshStrobl
pedronismvo: there's a error difference in artful that makes some unit tests fail here on artful: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-artful-snappy-dev-image/artful/amd64/s/snapd/20170919_102916_63405@/log.gz12:46
mvopedronis: yeah, I saw that too, I will update my tests12:49
jdstrandmvo: hey, regarding cgroups-- I was doing this on xenial classic distro, using a dpkg-buildpackage built deb from master. what were you doing? (I'm going to try again)12:52
mupPR snapcraft#1554 opened: store: handle revoked developers <enhancement> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1554>12:52
mvojdstrand: I was doing it on zesty running snapd from git12:53
mvojdstrand: I can try your setup next12:53
jdstrandmvo: 'snapd from git' means dpkg-buildpackage?12:53
mvojdstrand: it means "go build && sudo ./snapd" from cmd/snapd in the source tree12:55
jdstrandmvo: so, I was using dpkg-buildpackcage because of all the different binaries involved12:56
jdstrandsnapd, snap, snap-confine, snap-seccomp, snap-update-ns, snap-discard-ns, snap-exec, etc12:56
jdstrandI have a script to go build everything, copy over, etc12:57
jdstrandbut I was trying to do what you typically do :P12:57
jdstrand(we talked about this I think last week)12:57
jdstrandmvo: I'm going to retest on classic xenial, zesty and artful. maybe I messed something up in the vm12:58
jdstrandmvo: does core in edge have master code? I'd like to retry there too12:58
mvojdstrand: yeah, core should be up-to-date13:00
mvojdstrand: I can check after the standup13:00
niemeyerJust grabbing a cup'a and will be in the standup13:01
roadmrmvo hi13:08
roadmrmvo: question about base snaps: is it possible for a snap to have more than one base snap? (i.e. can it have "no" base snap or "one" base snap but not more?)13:09
roadmrmvo: and I guess follow-up: is it planned for multi-base snaps to be a possibility in the future? or not at all?13:09
roadmr(I guess not because that'd reintroduce dependency hell, right?)13:09
ogra_yeah, very unlikely13:09
ackkniemeyer, I'm not sure I follow the point about systemd generating the .socket files. where do you put the [Socket] definition itself?13:21
niemeyerackk: In a configuration file for systemd?  :)13:22
niemeyerackk: Ah, no.. not those socket files13:22
niemeyerackk: The *real* socket files13:22
ackkniemeyer, for unix sockets?13:23
zyga-ubunturoadmr: not right now because a base snap is the rootfs13:23
* ackk is a bit lost :)13:23
zyga-ubunturoadmr: so we'd have to understand what you mean by having more than one first13:23
roadmrzyga-ubuntu: I mean exactly that :P13:23
zyga-ubunturoadmr: and what would that do? there can be only one rootfs in the traditional meaning of the word13:24
roadmrzyga-ubuntu: I don't know, that's why I'm asking :)13:24
zyga-ubunturoadmr: why are you asking? do you have a workflow on your mind?13:25
roadmrzyga-ubuntu: it's in connection with https://bugs.launchpad.net/snapstore/+bug/1715824. We'll work on implementing that but we'd like to know if we need to design for 0-N bases or only for 0,1 bases13:25
mupBug #1715824: Please provide "base" snap data from snaps <Snap Store:Confirmed> <https://launchpad.net/bugs/1715824>13:25
zyga-ubunturoadmr: for that you can safely assume each snap has exactly one base snap (with a default, so it's never empty)13:26
zyga-ubunturoadmr: technically it's a unset -> default or explicitly set but I don't know if you need to have that detail at the API level13:26
roadmrzyga-ubuntu: nice, thanks!13:27
niemeyerackk: Yeah, the unix socket files.. sorry, in the standup so a bit slow13:27
ogra_zyga-ubuntu, base-busybox + base-rootfs-core + base-fobbar-service13:27
zyga-ubuntuogra_: what is the composition operator for a filesystem? union?13:28
* ogra_ hopes thats a nono13:28
ogra_zyga-ubuntu, well, it is all squashfses .... i think you can merge them on mount ...13:28
ogra_(would still be awful though)13:28
zyga-ubuntuogra_: we cannot currently merge them on mount13:29
ackkniemeyer, but that's what is defined by ListenStream, no?13:29
ogra_(i just meant to outline the possibility of multiple base snaps)13:29
niemeyerackk: Yes, exactly13:29
ackkniemeyer, so I don't get the comment about FileDescriptorName (which TIL)13:29
zyga-ubuntuogra_: since we never discussed merging multiple base snaps I'll skip that for now, once overlayfs becomes LSM-aware and apparmor is overlayfs aware we can explore this with layouts13:29
ogra_zyga-ubuntu, uh, i hope we never have to ... :)13:30
zyga-ubuntuogra_: as a layout element it would be powerful capability, it's just we cannot do it yet13:30
ogra_one bae snap is enough for everyone !!!13:30
niemeyerackk: Those two things are unrelated.. do you get the first one, before we move to something else?13:31
ackkniemeyer, about sanitizing the keys for the sockets: entries? yes13:31
niemeyerackk: That's a third thing..13:31
* ogra_ wonders if ackk's choice of nick is because you always have the feeling of agreement if people talk to you 13:32
ackkniemeyer, sorry, not sanitizing, ensuring the generated names are safe13:32
ackkogra_, lol, I had this since since forever (actually just "ack" but that's taken here)13:32
ogra_:)13:33
ogra_it surely gives a positive feeling :)13:33
niemeyerackk: Ok, cool.. yeah, not quite clear how we'll do that13:33
ackkniemeyer, isn't validating the key like for snap names not enough?13:34
ackk(as you mentioned)13:34
niemeyerackk: No.. as I said, that's a separate problem and a separate solution13:34
niemeyerackk: The file location safety is about confinement13:35
ackkniemeyer, right. for that I guess we could change it so that listen-stream in the case of an actual socket file would be prefixed with the snap dir ($SNAP_DATA IIRC?)13:36
=== chihchun_afk is now known as chihchun
mvoroadmr: no plans to have multiple bases currently. you can have "no" base by using the "bare" bases which is literally an empty base snap, nothing inside it (not even libc)14:02
roadmrmvo: nice. Hey, we put that (and a few other) question(s) in the bug, would you mind answering there? it helps with keeping things documented :)14:03
roadmrmvo: (thanks in advance !)14:03
=== ikey|zzz is now known as ikey
mvoroadmr: sounds good, what was the bugnumber again ?14:13
roadmrhttps://bugs.launchpad.net/snapstore/+bug/171582414:14
mupBug #1715824: Please provide "base" snap data from snaps <Snap Store:Incomplete> <https://launchpad.net/bugs/1715824>14:14
roadmrmvo: ^^14:14
mvota14:14
zyga-ubuntumvo: question about bases that's probably related: can we agree that base snaps cannot have any apps14:16
zyga-ubuntuactually, no because core config14:16
zyga-ubuntusorry14:16
zyga-ubunturoadmr: I think mvo is right that base snaps don't have a base14:16
zyga-ubunturoadmr: because they are their own base14:17
ogra_zyga-ubuntu, i was about to ask you to explain how we boot core then ;)14:17
ogra_(if a base cant have any apps)14:17
mvozyga-ubuntu: we could, but I think morphis has some interessting ideas for base snaps with apps14:17
zyga-ubuntuogra_: apps != programs14:17
ogra_?14:18
ogra_what are they then ?14:18
zyga-ubuntumvo: I think I wanted to say that base is its own base snap14:18
zyga-ubuntumvo: then the propery I cared about holds regardless of apps/hooks14:18
mvozyga-ubuntu: yes14:18
mvozyga-ubuntu: I mean, yes, base is its own base snap14:18
zyga-ubuntumvo: or that the "base" property applies to app snaps and gadget snaps but not to base snaps :)14:18
zyga-ubuntuyep14:18
ogra_why would it apply to gadget snaps ?14:19
ogra_(a gadget should pretty much operate standalone without any dependency on a base ... the model assertion should then define the base)14:20
zyga-ubuntuogra_: because those will have hooks as far as I konw14:20
zyga-ubuntu*know14:20
zyga-ubuntuand any executable process needs a well-defined base14:21
ogra_they *can* have hooks (note we dont have a single one that does atm)14:21
ogra_currently our gadgets only have bootloader binaries and a snap.yaml with interface definitions14:22
zyga-ubuntuogra_: s/will have/may have/ and I agree14:22
niemeyerpedronis, zyga-ubuntu: Any objections to LivePlug instead of ConnectedPlug?14:22
pedronisnot from me14:22
Pharaoh_Atemoh god, I'm so confused now14:23
niemeyerJust want to avoid typing 24 characters every single time on every single interface14:23
Pharaoh_Atemniemeyer: tab completion ;)14:23
ogra_zyga-ubuntu, we should make that optional so you can use gadgets without hooks still with multiple bases via models ... (so that a fedora image could be built using the existing bootloader for a board)14:23
ogra_(snapcraft should handle that imho ... if there is a hook, fail the build if there isnt a base defined ... without hooks ... dont care)14:24
niemeyerpedronis, zyga-ubuntu: We can even dial them back to interfaces.Plug/Slot again after it's all clean and we're used to the new world14:26
ogra_Pharaoh_Atem, +100014:27
zyga-ubuntuniemeyer: hmmm14:27
ogra_(you cant even easily copy/paste them because of the colon currently)14:27
zyga-ubuntuniemeyer: LivePlug doesn't feel very clear, is there a DeadPlug or is it in some form "ready" (as in live explosives)14:27
zyga-ubuntuniemeyer: what made you change your mind about connected plug?14:27
zyga-ubuntuniemeyer: +1 on a temporary name though14:28
niemeyerzyga-ubuntu: I don't care right now.. will name them as ConnectedPlug as we agreed14:28
zyga-ubuntuniemeyer: I don't mind that, it also has a nice "unclashing" property of assumptions14:28
zyga-ubuntu(no prior semantics so nobody can get confused about what it used to do)14:28
mvozyga-ubuntu: slightly bad news, bug 1667479 becomes an issue when playing with bases, i.e. when testing and installing bases then the things that consume bases hit this bug, I think morphis just got hit by this14:31
mupBug #1667479: mount namespace is not discarded when core snap changes revision <snapd:In Progress by zyga> <https://launchpad.net/bugs/1667479>14:31
Pharaoh_Atemwhat am I supposed to do anymore?14:31
zyga-ubuntumvo: let's sync on this first thing tomorrow, I think we can solve it much how like I initially attempted,14:33
zyga-ubuntumvo: with a way to work around any kernel issues14:34
mvozyga-ubuntu: neat14:34
zyga-ubuntumvo: for today it's too much stack, I think14:34
mvozyga-ubuntu: yeah, I have some more things I need to finish as well14:37
pstolowskiback15:05
pstolowskiniemeyer, pedronis, zyga-ubuntu can we resume the discussion (or are you still in the HO ;) ?15:05
niemeyerackk: $SNAP_DATA is probably too restrictive.. software will often want its socket in a predictable place such as /run.. that's the challenge15:06
niemeyerpstolowski: Just sent a summary to the forum15:06
mvoroadmr: I replied, please let me know if there is aynthing unclear15:06
roadmrthanks mvo15:06
niemeyerStepping out for lunch15:07
pedronispstolowski: https://forum.snapcraft.io/t/preparing-the-interfaces-logic-for-connection-hooks/218415:08
pstolowskiniemeyer, thanks, reading15:10
=== ikey is now known as ikey|work
=== chihchun is now known as chihchun_afk
ackkniemeyer, yeah, for instance LXD expects it as $LXD_DIR/unix.socket15:14
pstolowskiniemeyer, great summary, looks like a sensible plan. thanks for detailing that!15:29
mupPR snapd#3937 closed: interfaces/udev: only 'trigger --action=change', not 'control --reload-rules' <Created by jdstrand> <Closed by jdstrand> <https://github.com/snapcore/snapd/pull/3937>15:32
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
cachiomvo, about the service snapd.autoimport.service16:09
cachioit is in state Active: inactive (dead) since Thu 2017-09-14 14:33:18 UTC; 4 days ago16:09
cachioit is making the tests fail on db16:10
cachiomvo, should be enough with status=0/SUCCESS?16:10
mzanettimvo, ondra: hi there. The "After=network-online.target" doesn't seem to help16:11
ondramvo some other ideas what to try?16:18
mupBug #1710637 changed: Input falls through to gdm3 and terminates the session on Ctrl+C after udevadm trigger is executed under wayland <amd64> <apport-bug> <artful> <gnome-17.10> <rls-aa-incoming> <wayland-session> <Snappy:Won't Fix> <console-setup (Ubuntu):Fix Released by cyphermox> <gdm316:18
mup(Ubuntu):Invalid> <gnome-shell (Ubuntu):Invalid> <mutter (Ubuntu):Invalid> <systemd (Ubuntu):Invalid> <https://launchpad.net/bugs/1710637>16:18
=== chihchun_afk is now known as chihchun
ppisatiogra_: what did you have to do to get the bluetooth to work on the dragonboard? i'm using ubuntu classic ATM16:26
mupPR snapd#3943 opened: tests: fix ubuntu core services  <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/3943>16:27
ogra_ppisati, https://bugs.launchpad.net/snappy/+bug/167450916:30
mupBug #1674509: Unable to find bluetooth device on RPi3 running Ubuntu Core 16 <snapd-interface> <Snappy:Confirmed> <linux-raspi2 (Ubuntu):Invalid by p-pisati> <https://launchpad.net/bugs/1674509>16:30
ogra_has the full set of steps i used16:31
ogra_ppisati, mainly the hciattach line and you need to power on the device with bluetoothctl16:31
ppisatiogra_: i guess you have this stuff in some systemd service on ubuntu core16:32
ogra_we dont yet16:32
ogra_the bug is still open ... but it describes what is needed16:33
ppisatiogra_: do you have this lp bug handy?16:34
ogra_ppisati, you mean beyond the one i pasted 110 lines ago ? :)16:35
ogra_*1016:35
ppisatiogra_: but that is for the rpi316:35
ogra_uh, oh16:35
ogra_sorry, i totally mis-read16:35
ogra_the dragonboard should be the same but without the hciattach call16:35
ogra_if not, then we are missing something16:36
ogra_bluetoothctl should just work afaik16:36
ppisatiogra_: it's an old ubuntu classic image, so i'm probably missing something16:36
ppisatiogra_: hcitool doesn't show anything, and just invoking 'bluetoothctl' hangs the tty16:36
ogra_yeah ... you probably use the old bluez stack16:36
ppisatiogra_: i'm on xenial here16:37
ogra_well, i'm not sure, does xenial have bluez5 at all ?16:37
ppisatime checks16:38
ppisati$ dpkg -l | grep -i blue16:38
ppisatiii  bluez                               5.37-0ubuntu5.1                       arm64        Bluetooth tools and daemons16:38
ogra_the snap is at 5.4416:39
ogra_but i guess it shoudl still just work16:39
ppisatii think i'm gonna try with ubuntu core, and see if i can deduct what's going on16:39
ogra_ppisati, https://forum.snapcraft.io/t/wifi-and-bluetooth-on-snappy-ubuntu-on-a-dragonboard/1297/9?u=ogra16:40
ogra_so it just works here ...16:40
ogra_your bluetoothctl call should actually give you a shell, it definitely does here16:41
ppisatiogra_: ok, i'll give it a try with a more recent bluez stack, let's see16:43
ogra_if that doesnt work either i'D guess there is firmware missing16:44
ogra_assuming the kernels are actually identical16:45
ppisatiogra_: yep, same kernel, and i installed the linux-firmware-snapdragon package - so that should be fine, i don't see any 'missing fw' msg in dmesg16:45
ogra_your wlan works fine ?16:46
ppisatiogra_: let me try ubuntu core, and see if i can grasp what's going on there16:46
ppisatiogra_: i can see the wlan0, but i didn't attach to my wifi network16:46
ogra_(we did have to jump through some hoops to fix the MAC addresxs handling of that weirdly broken driver)16:46
ogra_i wonder if that would influence BT behaviour16:46
Chipacaniemeyer: 0*: bad epoch, or equivalent to 0?16:55
Chipacaniemeyer: yes i will transcribe this to the forum16:55
mvocachio: re snapd.autoimport.service> yes, the SUCCESS is enough17:00
mvoif we can get a second review for 3835 it can go in17:02
mvoalso 380717:04
mupPR snapd#3814 closed: interfaces: enable partial apparmor support <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3814>17:05
cachiomvo, this is the log17:05
cachiohttps://paste.ubuntu.com/25573688/17:05
mzanettiondra: also, makes me think... even if the network-online.target would work for me, Simon has no ipv6 setup except the zigbee interface in our device... so in his case ipv6 only DNS resolving would never work.17:07
mupPR snapd#3889 closed: interfaces: mount host system fonts in desktop interface <Created by jhenstridge> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3889>17:10
mvocachio: oh, " Main PID: 1301 (code=exited, status=1/FAILURE)" woudl be really good to figure out why that failed, any hints in /var/log/syslog or anything?17:10
mzanettialtough, manually restarting snapd at a later point makes it work... so... dunno, really17:11
cachiomvo, no yes, checking17:17
cachiono yet17:17
mupPR snapd#3927 closed: tests: only run tests/regression/nmcli on amd64 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3927>17:23
cachiomvo, not able to get any other log17:24
cachiomvo, this error is from the nightly execution17:24
cachioi am gonna run in a vm here now to see the log17:25
mvocachio: thank you17:26
mvocachio: I call it a day, but I'm keen to hear tomorrow what you figured out17:26
cachiomvo, SURE17:26
=== chihchun is now known as chihchun_afk
Chipacaniemeyer: i have some questions: https://play.golang.org/p/AA24O9k_oo17:56
niemeyerChipaca: I'm almost alive again.. will just grab a mate and we can talk if you still have some minutes18:07
Chipacaniemeyer: heh. i didn't expect an answer until tomorrow (but i am still here for a bit)18:09
niemeyerChipaca: Okay, checking it out18:20
niemeyerChipaca: For 0*, I think that's an error18:22
Chipacaok18:23
Chipacaniemeyer: the others are questions about the behaviour of go-yaml :-)18:23
niemeyerChipaca: We have no -1 epoch of course, so it would be a clear exception and perhaps an indication that there's a misunderstanding about how it works18:23
niemeyerChipaca: Vaguely, what should I be looking for there?18:25
niemeyerChipaca: The fact strings always unmarshal?18:25
Chipacaniemeyer: if you run that pastebin (sadly it won't run directly), you'll notice that the custom unmarshaller isn't called in a couple of cases18:25
Chipacain particular, for an empty string, the yaml.Unmarshal call returns an error, but the unmarshaller never got a say18:26
Chipacaalso for the null value it isn't called, but at least no error is returned18:26
Chipacait just assumes that null value means empty value, which isn't true for us18:27
Chipaca(to be clear, these won't block my work, but they were weird enough that i thought i'd check with you if it's working as expected)18:28
niemeyerChipaca: Weird.. for the null case I sort of expected it to not be called18:28
niemeyerChipaca: But I cannot see the behavior on the empty string18:28
Chipacaniemeyer: "cannot see" in the sense of "cannot understand"?18:29
niemeyerChipaca: No.. I get no errors see18:29
niemeyerArgh18:29
niemeyerChipaca: I get no errors here18:29
Chipacaniemeyer: running that code?18:29
niemeyer--- `a: ""`:18:29
niemeyerstruct { A main.Foo }{A:main.Foo{X:"", Y:true}}18:29
Chipacawat18:29
Chipaca--- `a: ""`:18:30
Chipacayaml: unmarshal errors:18:30
Chipaca  line 1: cannot unmarshal !!str `` into main.Foo18:30
niemeyerChipaca: Let me update my package18:30
Chipacaniemeyer: go 1.6?18:30
niemeyerChipaca: There might be changes in tip of go-yaml itself that I don't have18:30
Chipacaand i'm on tip?18:30
Chipaca(i'm on whatever get-deps got)18:31
Chipacaniemeyer: i thought you were the go-yaml guy?18:31
niemeyerChipaca: Nope, still runs fine18:31
niemeyerChipaca: Thankfully I have other people maintaining it by now18:31
Chipacaniemeyer: what version of go?18:31
niemeyerRoger Peppe specifically18:32
niemeyerChipaca: tip18:32
Chipacaniemeyer: can you try with 1.6?18:32
niemeyerChipaca: But I don't expect that to count.. yeah, sure.. let me find it :)18:32
Chipaca:-)18:32
niemeyerChipaca: Still works18:34
Chipacaok...18:34
Chipaca18:34
niemeyerChipaca: It's probably the proximity with me.. it knows that if it breaks in front of me I will fix it18:34
Chipacaniemeyer: so, i just go got it, and it works18:35
Chipacaso whatever go get gets, works18:35
Chipacawhich is good18:35
Chipacabut whatever we have in snappy doesn't?18:35
niemeyerChipaca: Ah, okay.. so probably something that was fixed back then18:35
Chipacaniemeyer: and whatever we have in vendor is also ok18:37
Chipacabecause get-deps is no longer updating outside of vendor18:37
Chipacaso i still have leftover crud18:37
Chipaca:-(18:37
Chipacaniemeyer: thank you :-)18:37
* Chipaca wipes it all clean18:38
niemeyerChipaca: np, and quite curious indeed18:38
Chipacaok, i'm off to walk the dog and make dinner (or viceversa)18:40
Chipacaniemeyer: take care, and get more sleep18:40
Chipacaniemeyer: sleep is good for you, keeps the cray-cray away18:40
niemeyerChipaca: Thanks, enjoy your evening and hope the dog tastes good18:41
niemeyerI should probably not make that sort of joke on such a diverse community :)18:42
Chipacaniemeyer: 🌭18:44
niemeyerChipaca: One of our favorite bad jokes was translating that to Spanish literally18:45
Chipacaniemeyer: there was a skit we'd do where we spoke in Bad Translator Accent, and talk about carros and perros calientes18:48
Chipacawe were all sorts of fun18:48
* Chipaca ~> really off now18:48
mupPR snapcraft#1555 opened: store: switch to new endpoints <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1555>19:02
diddledanogra_: I got ring working19:20
diddledanogra_: this is the yaml which worked for me:  https://www.irccloud.com/pastebin/JFeIwqQr/19:22
diddledanI'll pop it on github in a sec19:23
diddledanthere ya go: https://github.com/diddledan/ring-snap19:24
mupIssue snapcraft#1556 opened: build-snaps recording <design-required> <Created by sergiusens> <https://github.com/snapcore/snapcraft/issue/1556>19:26
elopio[m]Hello from riot.im !19:56
sergiusenselopio[m] inovative ;-)19:56
elopio[m]I'm from the future.19:56
=== elopio[m] is now known as elopio
sergiusensdiddledan do you have an example on how to build a nice project using jhbuild and snapcraft?20:19
diddledansergiusens: older builds of corebird used jhbuild: snapcraft.yaml: https://go.bwlh.at/2xkN0ga corebird.modules file in parent directory: https://go.bwlh.at/2w5UUHs full repo at latest commit which uses jhbuild: https://go.bwlh.at/2xv37bA20:36
diddledanI specifically used jhbuild because a lot of gnome stuff it required were newer than 16.04 provided, but I've since moved to using the gnome-3-24 platform snap and ppa20:38
diddledanjhbuild was the easiest way I could find to reliably build the gnome deps20:38
sergiusensdiddledan boo, is `jhbuild` now abandonware? :-)20:43
diddledanI think it's still usefull, but people keep moaning at me for large download sizes :-(20:50
diddledanof snaps*20:51
diddledanI particularly like it for the ability to build on other people's work because you can import other *.modules files from gnome for example which are prepared with all the relevant dependencies and build info for stuff you might rely on20:54
fmultihi, im behind a corporate proxy and cant get snap to  install anything due to the x509 certificate checking20:55
fmultisame issue as https://forum.snapcraft.io/t/certificate-substitution-and-snaps/1077 but cant find any resolution20:55
elopiosergiusens: I would remove this sentence from the release notes: (for the case of today of only supporting one base)21:07
elopiosergiusens: the build-snaps section has a sentence that's not finished: "This release exposes the feature through"21:12
=== elopio is now known as elopio2
=== elopio2 is now known as elopio
mwhudsoni can't cleanbuild snapcraft master22:17
mwhudsonfails with "mkdir: cannot create directory ‘build’: File exists"22:18
mwhudsonduring the build of apt it seems22:18
mwhudsonwell ok this is my hacked up version, not master but my changes really shouldn't have caused this :)22:18
kyrofamwhudson, I get that if I try to build snapcraft from snapcraft src22:19
mwhudsonyes, that's what i mean22:19
kyrofamwhudson, my solution was to build it using the deb or snap instead22:19
mwhudsonoh22:19
kyrofaThere's _something_ weird there22:20
mwhudsonyeah ok i have the snap i built from master earlier installed22:20
mwhudsonso if i refresh that back to candidate it'll work?22:20
mwhudsonbah22:21
mwhudsonreverting from a local snap to the store should be easier than this i think22:21
kyrofamwhudson, I don't think you should be able to install local over the store or vice-versa. They could be completely different snaps, but now you're sharing data22:24
kyrofaIf it's not associated with the store, it could be named anything22:24
mwhudsoni guess22:25
Hilikushello. i'm trying to understand when do i need to create a custom gadget snap. can someone explain this to me. i see for example a pi2 and pi3 image, but there's also a i386 and amd64 image. is the gadget snap per architecture? how come the pi image is not ARMv7 image?22:50
kyrofaHilikus, normally I'd refer you to ogra_, but it's pretty late for him22:54
kyrofaHilikus, so I'll do my best :)22:54
Hilikusawesome :)22:54
kyrofaGadgets have a number of responsibilities. They contain the bootloader, and they also are responsible for exposing the hardware of the board to the rest of the system22:55
kyrofaHilikus, so yes, they are not only arch specific, but hardware specific22:55
kyrofaReference platforms, such as the rpi2/3, amd64, etc. have a gadget already created for them22:56
kyrofaBut if you're using other hardware, you may have to write one22:56
kyrofaSo there's one reason to write one. Another reason is that perhaps the hardware isn't being exposed quite how you're like using the official gadget. An example of this is serial ports: gadgets are the only place you can specify udev-like rules to get serial ports consistently mounted and provided to snaps in a confined manner22:57
kyrofaAs for the pi image arch, I'm afraid I can't field that one22:58
Hilikuswhat exactly does "other hardware" mean? where i get confused is with the i386 and amd64 images. does that mean that i can buy any embedded computer kind of machine and as long as it's an x86 32 or 64bit then i can use the i386 or amd64 images? if not, for what exact type of i386 and amd64 where those made? would they work in a desktop? server?22:58
Hilikusby having an i386 and amd64 i get the idea that this image will work almost anywhere as long as the arch matches, but based on what i read from the gadget snap that's not the case22:59
kyrofaYeah as a general rule that's not the case, but amd64 and i386 gadgets are a little more general purpose as I don't imagine they expose any specific hardware and use grub22:59
kyrofaYou can use them in virtualbox, for example23:00
kyrofaOr on a NUC23:00
kyrofaKernels are often device-specific as well23:00
Hilikusreally? i thought the kernel snap was only arch dependent23:02
mupPR snapcraft#1557 opened: cleanbuild: add a --image option to build in a different image <Created by mwhudson> <https://github.com/snapcore/snapcraft/pull/1557>23:03
Hilikusok, so if udev is really configured in the gadget snap then i think that's clear to me. i can't for example use the x64 ubuntu core image and configure my bluetooth or NFC adapter in it23:03
Hilikusi would have to create an image specifically for the peripheral I want to control23:04
kyrofaHilikus, well, other interfaces might cover that. I know there are bluez interfaces23:16
kyrofaBut I've not experimented with them23:17

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