/srv/irclogs.ubuntu.com/2018/08/31/#snappy.txt

sergiusensjdstrand, kyrofa, popey Wimpress https://github.com/snapcore/snapcraft/pull/223401:41
mborzeckimorning05:03
zygaGood morning05:55
zygaSummer feels over now05:55
=== chihchun_afk is now known as chihchun
zygagood morning06:19
zygaagain06:20
zygathis time from a computer06:20
zygaman, Monday will be brutal :)06:20
mvozyga: good morning06:22
mvozyga: monday is back to school?06:22
zygayes06:23
zygamvo: reviewed systemd env generator PR06:27
mvozyga: nice06:27
mborzeckimvo: zyga: hey06:32
mborzeckiyeah, i'll likely be starting late on monday (or have a break sometime around morning) with the school and all06:33
zygamborzecki: same here06:36
zygatwo kids to sort out :)06:36
mvomborzecki: good morning06:36
mvomborzecki: no worries06:36
mborzeckiand back to the 'breakfast, work, kids from school, help with home homework, <do one other thing you may have planned>, dinner, sleep, goto 1' routine06:44
mvozyga: curious, why a man-page for the generator? this thing is pretty well hidden06:44
zygaAll generators have them06:47
mvozyga: aha, ok06:47
zygaIt is good courtesy to users since this is an opaque compiled program06:48
mborzeckisystemd-environment-d-generator has it too, although it's not very useful06:49
zygaI’m moving outdoors06:53
zygaNetwork at home sucks06:53
zygaAnd I need to do a trial school run to see how bus traffic looks like06:53
=== pstolowski|afk is now known as pstolowski
pstolowskimorning07:10
mborzeckipstolowski: heyah07:24
zygare07:31
mvomborzecki: the arch failures on 2.35 look real07:40
mborzeckimvo: that's unfortunate, do you have link to the logs?07:41
mvomborzecki: I mean, they are the same as yesterfday, any idea ? and if this does not affect master we probably just need to find the right commit that fixed it in master07:41
mvomborzecki: https://api.travis-ci.org/v3/job/422560948/log.txt07:41
mborzeckimvo: ssh keygen?07:43
mvomborzecki: ohhh07:43
mvomborzecki: yeah, let me find htis one and cherry-pick it07:43
mvomborzecki: thats definitely a good candidate07:43
mvomborzecki: cherry-picked and pushed, lets see if that helps07:46
mvomborzecki: thank you!07:46
mborzeckimvo: hopefully it'll be green now :)07:48
zygabreaking compat one by one :)07:49
=== chihchun is now known as chihchun_afk
Chipacamoin moin08:02
Chipacaany idea why #5744 still shows up in github as 'in progress' (and no coverage report) when the tests passed?08:03
* zyga works on trespassing unit tests08:03
zygaworking from suse :)08:04
zygaI think we have enough people with ubuntu08:04
zyga(in the team, go ubuntu :)08:04
mborzeckihaha08:06
popeyhmm08:10
popeywhen snapd dies, "snap info" lies08:11
popeyit says "no snap found" when it should really say that it couldn't contact the store08:11
popeyhttps://www.irccloud.com/pastebin/MErmQt88/08:11
Chipacapopey: it didn't say there weren't any, it said it didn't find any08:11
Chipacapopey: like when I tell my boys to go get something08:11
popey"Man looking"08:11
Chipacapopey: "I LOOKED NOWHERE AND I CAN'T FIND IT"08:12
popey:D08:12
popeyShould it not also warn that it can't get to snapd?08:12
popeythe "I looked nowhere" bit08:13
Chipacamaybe snap should learn to pick things up and look under things, especially if it just put that thing down08:13
popeyIt should at least pick up the boxes and give them a bit of a shake.08:14
Chipacathat's like level 308:14
Chipacano chance08:14
popeysnapd needs to lean the Jan Hankel Flank Pat system for finding things08:15
ChipacaI see what's going on, I'll push a pr to address it in a bit08:15
popeyhttps://www.youtube.com/watch?v=DY-Zdgo0OXo08:15
popeyThank you!08:15
Chipacapopey: the first time I saw that vid I didn't notice how old it was08:16
Chipacamaybe it was a long time ago08:16
Chipacahm08:16
popeyYeah, Mitchell & Webb Look aired 10 years ago!08:18
mvomborzecki: aarch is greeeeeeeeeeen08:19
mborzeckiyay, rejoice :P08:19
mvomborzecki: merged into 2.35 thanks again08:19
Chipacapopey: if you did 'snap info foo bar' in that situation, what would you expect to see?08:19
popeyWe'll. Shut down snapd and run snap list. You get a completely different experience08:24
popeyI would prefer snap list had a nicer error too08:24
popeyWhen snapd is down08:24
popey(in this case snapd was down due to lack of disk space)08:25
=== chihchun_afk is now known as chihchun
Chipacapopey: the problem is 'snap info' does a lot more stuff than 'snap list'08:30
Chipacapopey: so each query to  'snap info' can produce up to 2 errors during normal operation08:30
Chipacapopey: and up to 3 errors when snapd is down08:31
Chipacapopey: note info _can_ produce useful output with no snapd08:32
Chipacapopey: e.g. 'snap info /var/lib/snapd/snaps/*'08:32
popey(I never knew that)08:34
popey(I bet nobody else does either)08:34
Chipacapopey: 'snap info /snap/core/current/' also tries to be informative08:34
Chipacapopey: or 'snap info ./prime'08:35
ChipacaI need to make the errors from client a little more structured before I can hope to make this work08:43
Chipacabut we've been wanting this for a while now08:43
Chipacaso, yay08:43
popey\o/08:44
Chipacayeap. Just, not a quick fix08:44
popeyno rush08:44
popeyis it done yet?08:44
Chipacapopey: https://www.youtube.com/watch?v=RYVlJKSEsyM08:48
popeyawwww <3 that video08:49
popeyThose were happy times08:49
Chipacapopey: back when we were part of ESA you mean?08:52
mborzeckizyga: niemeyer: https://github.com/snapcore/snapd/pull/574708:52
zyga+108:55
mvozyga: is there a way to make apparmor refuse to stat/etc/lsb-base-logging.sh  ?08:57
mvozyga: context is https://bugs.launchpad.net/snapd/+bug/177941608:57
popeymain.go:192: cannot change mount namespace of snap "mattermost-desktop" according to change mount (/snap/gtk-common-themes/319/share/sounds/communitheme /snap/mattermost-desktop/78/usr/share/sounds/communitheme none bind,ro 0 0): cannot create writable mimic over "/snap/mattermost-desktop/78/usr/share/": permission denied09:03
popey^ anyone seen that before? What does it mean?09:03
Chipacazyga: ^09:03
popeyI am launching mattermost-desktop from edge, and have the communitheme install09:03
zygapopey: hey09:07
zygapopey: known issue, please show me the snapcraft yaml, I will show you how to work around it09:07
zygait's a trivial trick09:07
popeyzyga: https://github.com/snapcrafters/mattermost-desktop/blob/master/snap/snapcraft.yaml09:22
zygalooking09:23
zygaright09:23
zygaif the snap ships $SNAP/usr/share/{icons,themes,sounds} as empty directories09:24
zygathis bug goes away09:24
popeyhah!09:24
diddledanooh, new mycroft09:25
* diddledan gets compiling09:25
zygapopey: the issue is that the rules for poking holes are super precise09:25
zygaand they should be more generic in cases like this where there are many more directories to create09:25
zygapopey: I'm working on fixing that (2.36)09:26
popeyok, thanks.09:26
diddledanoh, looks like it's not rolled-out yet (mycroft 18.08)09:28
Son_Gokuzyga, snapd 2.35 rolled out to fc2809:29
Son_Gokustill waiting for karma for fc2709:29
zygaSon_Goku: Thank You!09:29
zygaSon_Goku: I can give it a spin today09:29
zygaI'm trying to get out of a very long unit test change09:29
zygafor a .1 release09:29
zygaSon_Goku: btw, I want to participate in the golang sig09:30
zygaI have my own proposal to contribute09:30
zygaI was working on it after hours recently09:30
Son_Gokucool09:30
zygasomething that fits upstream and downstream at the same time09:30
zygais easy to use for regular go developres09:30
zygaand does all that distros need09:30
zyga(magic, right?)09:30
zygaI will have it ready in a few days09:30
zygaI think there's a real chance to unify fedora and suse helpers09:31
Son_Gokuheh09:31
mborzeckizyga: no more shell wrappers for each go command?09:33
zygamborzecki: exactly09:33
zygaanyway, back to work, I'll demo it next week09:34
mborzeckiheh going through opensuse wrappers was interesting, but to be fair OE also has some hacks spefifically for go packaging09:35
Son_GokuFedora's wrappers are pretty thin09:36
Son_Gokuthe macro directly calls the go command09:36
mborzeckiSon_Goku: well, suse has an elaborate shell script(s) :/09:43
Son_Gokuit used to be ruby09:43
Son_Gokuso be happy it's not that09:43
mborzeckihehehe09:43
mborzeckiSon_Goku: but ruby is like haiku09:43
mborzeckiSon_Goku: or better, the code reads like plain english09:44
Son_Gokususe's golang-packaging being rewritten from ruby to shell was the result of an ugly dispute with the original creator of suse's golang packaging standard09:44
mborzeckiSon_Goku: otoh the languages bundling their dependencies do not really fit that well with the packaging model used by some distros, i can't imaging splitting dependencies of a trivial node app09:46
Son_Gokuyou don't have to imagine09:46
Son_Gokuit happened for a while in Fedora09:46
Son_Gokuand it still does in Debian09:46
popeyyeah, the number of ITP mails which are "please package this node module" is too damn high09:47
niemeyermborzecki: Thank you! And sorry for the late reply09:58
niemeyerAnd good morning :)09:58
Chipacaniemeyer: you got to say 'good morning' by two minutes :-)10:03
Chipacaniemeyer: good afternoon sir10:03
niemeyerAftnoon!10:03
Chipaca:-)10:04
mborzeckiniemeyer: sure, thanks for the review10:05
Chipacazyga: pstolowski: if you have a window for an easy review, #5744 is kinda blocking me (I could merge it and then let git sort it out, and I'll do that if it's not reviewed soonish, but i'd rather avoid it)10:05
zygalooking10:06
Chipacawoo10:06
Chipacathanks10:06
Chipacazyga: tests are green despite github not knowing it10:06
pstolowskikk. and what happend to mup btw, is it gone again?10:06
zygamvo: my dragon apparently has android on emmc10:06
Chipacapstolowski: ^ zyga's on it, thanks10:06
mborzeckiniemeyer: pushed the changes to https://github.com/snapcore/snapd/pull/5713 too (hopefully got all of it now)10:06
Chipacaalso, how can I merge a PR if travis didn't let github know it was green?10:07
Chipaca:-(10:07
zygaChipaca: ask mvo10:07
zygaChipaca: is bytes.Bytes expensive?10:08
zygaer10:08
zygabuffer.Bytes10:08
Chipacazyga: only a little bit more than doing the maf ourselves10:08
Chipacazyga: memorywise it's about the same10:08
Chipaca(especially as it's already used elsewhere)10:08
mvoChipaca: I can10:09
zygawhy did you move w.partial = nil; to be a w.partial.Reset() a few lines below (after } )10:09
mvoChipaca: which one do you need10:09
zyga571310:09
zygaer10:09
zyganot10:09
zyga574410:09
Chipacamvo: 5744 (but it doesn't have zyga's second +1 yet)10:09
zygaChipaca: https://github.com/snapcore/snapd/pull/5744/files#diff-7c96d8cb550f40c45f4ded10cc30f5eeL54 about this line10:10
Chipacazyga: yep ye10:10
zygait is clear that it is just a p.Reset but why did you move it?10:10
zygais it because of https://github.com/snapcore/snapd/pull/5744/files#diff-7c96d8cb550f40c45f4ded10cc30f5eeL5810:10
zygawhich is not 1-to-1 (write is append, = was reset + write)10:10
Chipacazyga: because I'd be doing it in the other paths as well10:10
zygaok10:10
Chipacazyga: that is: w.partial = p -> w.partial.Reset(); w.partial.Write(p)10:10
zyga+110:11
Chipacamvo: ok now yes please10:12
zygadone10:12
zygamvo: 3/4 flashed10:12
Chipacamvo: https://github.com/snapcore/snapd/pull/5744 is green and has two +1's10:13
* Chipaca ~> break10:13
mvoChipaca: done thank you10:14
Chipacamvo: whee, thanks10:14
Chipacawait i was supposed to be on a break10:14
* Chipaca hides10:14
popeyhm, is there any way to see which snap pulled in a core18 ?10:14
Chipacawe should include that in the error (there's a // TODO)10:15
Chipaca(in the "snap remove core18" error i mean)10:15
pedronispopey: snap info --verbose should show the base of a snap10:15
zygayep, it does10:15
Chipacamaybe snap list should include it as well10:16
mvoor snap info core18 could list the rdepends10:16
Chipacamvo: oooh10:16
Chipacamvo: I'll add that10:16
popeythat would be nice10:16
Chipacacould also look at slots/plugs10:17
popeydiddledan: you moved to core18 on gimp? :)10:18
diddledanyes. I've got the snap building on launchpad for now10:18
mborzeckizyga: where was the mapping thing that changed system -> core and back?10:18
diddledanjust this second told it to do a new build for libx11-related security patches overnight10:19
zygamborzecki: in overlord/ifacestate10:19
zygaman, SD cards are slow10:19
mborzeckizyga: i'm thinking if we could have something similar instead of snap.Info.ExpandSnapVariables(), like a mapped that does this in 'inside' snap mount ns or 'outside' (cc niemeyer)10:20
mborzeckis/mapped/mapper/10:20
zygahmmm10:20
* zyga thinks10:20
mborzeckizyga: then for eg. snap.MountNSView().ExpandVariables(info)10:21
niemeyermborzecki: The problem tastes a bit simple for that10:22
niemeyermborzecki: What changes between the two cases, and how often we need either?10:22
popeydiddledan: that'll be why core18 number of installs rocketed up over the last few days then :)10:23
mborzeckiniemeyer: yeah, the only use cases now are 'inside' mount ns, so we could go yagni about that just have one function for this (as it is right now)10:23
niemeyermborzecki: !10:23
diddledanrocketed?10:23
diddledanone snap. yeesh :-p10:23
popeyone - popular - snap :)10:24
diddledanaha10:24
diddledanI see what you mean10:24
diddledanpics or it didn't happen!10:24
* niemeyer wants pictures too10:24
popeyhttps://snapcraft.io/core1810:25
popeylook at the distro graph10:25
diddledanI don't see a timeline :-(10:26
MattJOn a tangent, but the map just reminded me of https://xkcd.com/1138/10:27
zygamvo: it's booting10:28
popeyMattJ: hah! :D10:28
pstolowskiniemeyer: hey! will you find a moment today for a quick HO re udev & stop channel?10:28
zygamvo: same as last time10:28
zygamvo: let me look at logs10:28
niemeyerpstolowski: Yeah, definitely10:29
zygamvo: same == "snap command not found"10:29
zygaI'll enable journal and see what we get10:29
mvoxnox: did you get my message about https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1778936 ?10:32
mvoxnox: i.e. can/should I sru this or will you do a sru with more changes anyway where this could be included?10:32
zygamvo: I reflashed and created /var/log/journal10:32
mvozyga: meh, that sucks10:32
mvozyga: thanks10:32
zygait failed10:32
zygalet me recover the journal after a second, I want to make sure this is written to the card10:32
mvozyga: ta10:32
mvozyga: the error you got a couple of days ago should be fixed so I wonder what is going on10:33
zygamvo: I'll bring it for the sprint10:33
zygasie 31 12:32:18 localhost run-snapd-from-snap[738]: stateengine.go:102: state ensure error: devicemgr: cannot use gadget snap because its base "" is different from model base "core18"10:34
zygamvo: :D10:34
zygalet me know if you want another test run10:34
zygamvo: I streamlined my setup so I can test it very quickly10:35
zygathere are more errors, I'll give you the full og10:35
zyga*log10:35
zygamvo: boot log https://paste.ubuntu.com/p/pkhHBrzFNm/10:37
zygaback to coding10:37
zygaping me for another run10:37
mvozyga: thanks, fixing10:38
Chipacaouch, a panic from prepare-image10:59
* zyga is hungry10:59
mborzeckiChipaca: bt?10:59
mvoChipaca: yeah, I saw this a couple of time in the lsat day11:00
mvoChipaca: I pushed a fix for the panic11:01
mvoChipaca: its an open pr, that hopefully gives us clues what is going on11:01
mvoChipaca: 5743 if you look for a simple review ,)11:01
Chipacamborzecki: https://pastebin.ubuntu.com/p/2ZTtNDngWp/11:01
Chipacamvo: nice11:02
mvoChipaca: the question is really why we get into this situation, its impossible :) (well, clearly not but)11:02
Chipacamvo: because it's only added to the map if typ == snap.TypeKernel || local.Name(snapName) == baseName {11:04
Chipacamvo: and then it looks in that map for 'snapweb'11:05
mvoChipaca: is it snapweb its crashing for?11:05
mvoChipaca: I thought that was invisilble without the other PR11:05
mvo(but maybe I hvae not looked hard enough)11:05
Chipacamvo: it prints "fetching %s" before looking at the map?11:05
Chipacabah, i don't realluy know this code11:06
Chipacaso i could have it backwards11:06
=== pstolowski is now known as pstolowski|lunch
Chipacahmm hmm11:07
Chipacamvo: so, with your change, the test will still fail at this point, right?11:07
mvoChipaca: yes11:08
mvoChipaca: it will fail but report on what snap it failed11:08
mvoChipaca: I can add some more code to diagnose this I think11:09
mvoChipaca: heh, you suggested this too :) let me add it11:10
Chipacamvo: +1'ed, but yeah11:10
mvoChipaca: ta11:11
zygare11:23
* Chipaca ~> lunch11:34
=== chihchun is now known as chihchun_afk
mborzeckiniemeyer: https://github.com/snapcore/snapd/pull/574911:42
jdstrandzyga: hey, I'm not really here. I didn't get to the /mnt PR yesterday but will try today12:02
zygajdstrand: hey12:03
zygajdstrand: mvo and I decided to merge it (ahead of an important release) but if you find anything we will do a point release12:03
zygasorry about that, I hope that's okay12:03
zygajdstrand: so take your holidays for real and let's do this next week12:03
xnoxmvo, there is an sru outstanding to do, yes.12:11
xnoxmvo, however, it appears that snapd is failing autopkgtests now that said patch is in, in cosmic-proposed.12:11
xnoxhttps://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-cosmic/cosmic/amd64/s/snapd/20180830_184959_a6800@/log.gz12:12
xnox2018-08-30 18:49:42 Failed tasks: 112:12
xnox    - autopkgtest:ubuntu-18.10-amd64:tests/main/dirs-not-shared-with-host:alternatives12:12
xnoxmvo, as if i should back that change out.... if it is related.12:12
zygamborzecki: less than 20 errors left, brace yourself for that long code review :D12:16
mvojdstrand: who else can approve snaps? I have some gadgets with "base: core18" that neeed approval12:21
mvoxnox: its not related I will push a fix for this12:21
mvoxnox: its a core snap change12:22
xnoxmvo, oh, ok!12:22
xnoxmvo, that's good =) cause it means i need not to back out things in systemd package then!12:22
mvoxnox: yeah, I'm very happy if the fix is included too :)12:22
=== pstolowski|lunch is now known as pstolowski
diddledanwheee12:26
diddledanall those failed-to-release: https://www.youtube.com/watch?v=0BHQT3Omqtw12:27
diddledanerr12:27
diddledanwrong link12:27
diddledanfile:///home/dllewellyn/Pictures/Screenshot_20180831_132719.png12:27
diddledanbah12:27
diddledanTHIS TIME! https://usercontent.irccloud-cdn.com/file/hKp3lTzL/Screenshot_20180831_132719.png12:27
mvodiddledan: i had some store upload rejections as well, mine were some new fields from snapcraft iirc12:29
zygamborzecki: it's green :)12:30
zygamborzecki: I'll chop it12:30
diddledanhttps://www.youtube.com/watch?v=TkZFuKHXa7w12:30
diddledan^^ without comment12:34
cachiomvo, hey12:39
cachiohttps://paste.ubuntu.com/p/Gr835VXzcF/12:39
cachiodid yo see that error on master?12:39
mvocachio: yes12:41
mvoI pushed a PR to make it more obvious what is going on12:41
cachiomvo, great, thanks12:41
mvocachio: 5743 - its ready12:42
mvocachio: that hopefully gives us a clue12:42
mvocachio: what is going on there12:42
mvocachio: I saw it a couple of times today and yesterday12:42
mvodiddledan: let me try to find out what is going on with the rejections - do you also get it for nknown keys in snap/manifest.yaml: snapcraft-os-release-id12:43
diddledanno idea why it's failed to release12:45
diddledaneverything appears normal12:45
diddledanfor example, micropolis failed to release, but: https://usercontent.irccloud-cdn.com/file/eZSJeaNF/Screenshot_20180831_134635.png12:46
cachiomvo, yes it started yesterday suddenly, some branches without any change started failing12:47
diddledangimp just got rejected with "unknown keys in snap/manifest.yaml: snapcraft-os-release-id,snapcraft-os-release-version-id,snapcraft-version lint-snap-v2_snap_manifest"12:48
zygabrb, see you at the call12:54
mvodiddledan: yeah, I had the same12:58
diddledanapparently I singlehandedly skyrocketed the number of core18 installations (according to popey )13:00
diddledanI call shenanigans13:00
diddledanas if my gimp is really _That_ popular!?!13:01
popey"skyrocketed" :D13:01
diddledanoh, right, so in the last 3 days 77 thousand people have updated to core18ified gimp...13:02
diddledanhtf have I managed to become influential to 100k people?!13:03
popeyTurns out people like gimp. who knew? :)13:04
diddledannuts13:04
diddledanleft with no comment: https://usercontent.irccloud-cdn.com/file/gOjXXEh5/Screenshot_20180831_141513.png13:15
diddledanactually, question regarding that, is nil gonna be treated as string 'nil' or as no-value?13:16
diddledanI wanted it to be no-value13:17
diddledani.e. don't you dare think of giving me a base damn you!13:17
* diddledan hacks all the things13:19
Son_Gokumvo, zyga: https://github.com/snapcore/snapd/pull/575013:22
Son_Gokumvo, also, your script for generating changelogs is broken13:22
Son_GokuI had to add the "- New upstream release <version>" line for the last few changelog entries13:23
zygaSon_Goku: shall we sort the changelogs like I did in suse?13:23
zygaI need to update my script13:23
Son_Gokusort them?13:23
zygathe dates are broken13:23
zygaat least in suse they were, due to various historic mistakes13:24
Son_Gokuyeah13:24
Son_Gokuthe dates and authors need to be fixed13:24
zygahttps://gitlab.com/zygoon/rpmtools13:24
Son_GokuI thought about fixing it the last time I went in and whacked the changelogs13:24
=== zarcade_droid is now known as ^arcade_droid
Son_Gokuerr fixed the suse packaging13:24
zygajust need to handle embedded changelogs13:24
Son_Gokuchanges format changed last week13:24
Son_Gokuzyga: spot the difference: https://build.opensuse.org/package/view_file/system:packagemanager:dnf/dnf/dnf.changes?expand=113:25
mvoSon_Goku: thanks, in a meeting right now, but thanks for the PR - and sorry for the broken script I have a look and fix it13:25
zygaSon_Goku: in a meeting, but I will look after13:26
Son_Gokumvo, zyga, clearly you're both in the same meeting :P13:26
zygayep13:26
zygastandup13:26
Son_Gokuzyga, also: https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/7-beta/html-single/7.6_release_notes/#BZ1461652 & https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/7-beta/html-single/7.6_release_notes/#BZ156477513:29
niemeyercachio: uploaded13:39
cachioniemeyer, tx13:39
zygaSon_Goku: how do the two rhel notices you linked to affect us?13:40
Son_Gokuzyga, well, the first one means that the code I'm writing for integration in fedora infra will work for producing centos7 base snaps eventually13:41
Son_Gokuas for the second one, I'm not sure, it sounds like something we might want?13:41
Son_Gokuat least with the mmap() support, that means that the policy will compile again on el713:42
Son_Gokuzyga, it seemed like the socket stuff is something along the lines of what the socket mediation stuff in apparmor does13:42
zygaSon_Goku: apparmor does packet inspection (so to speak)13:43
zygamore like a firewall than a traditional "you can or cannot use that socket" control13:43
Son_Gokuah13:43
zygabut I understand what you meant now13:43
mvodiddledan: I got feedback about the store rejections, they are working on it and it should be back to normal soon. out of curiosity, is your gimp snap core18 based?13:49
diddledanyes, it is13:50
mvodiddledan: thanks for confirming. it looks like only the 18.04 build env is currently in unhappy land13:50
diddledanhence 77k new core18 downloads this week :-p13:50
* mvo hugs diddledan 13:50
mvodiddledan: thats cool13:51
diddledanooh cuddles!13:51
diddledanI wanna see the graph of core18 to see what impact that's had overall13:52
mvodiddledan: its quite dramatic13:53
diddledanI unleashed gimp core18 about 3 days ago IIRC13:53
mvodiddledan: yeah and the graph is exploding since :)13:54
zygaindeed :)13:54
mvodiddledan: nice job13:54
diddledannice13:54
mvoChipaca: 5730 might be interessting for you13:54
zygamvo: beta testers ;)13:54
mvoChipaca: super easy13:54
diddledanhaha. I hope you were ready ;-p13:54
mvozyga: yeah but to be fair, core18 as a base for normal snaps should be fine13:54
diddledanseems stable for gimp's usage13:55
mvodiddledan: he is joking, as a base its good13:55
zygayes, I was somewhat kidding13:55
mvodiddledan: as a booting image we have some work (but even there its pretty good by now :)13:55
diddledan"should"13:55
diddledanyey13:55
* diddledan waits for his pi to automatically upgrade13:55
diddledanI'm guessing moving over to the core18 boot image is gonna be a case of many spinning plates13:56
zygaflashing bits13:56
zygayes13:56
* diddledan flashes his bits13:56
zygabrb, I need some water, it's getting warm on this side of the house13:56
mvo5743 is trivial and needs a second review13:56
diddledanyeah, the sun's just got around to my main window now, too13:56
Chipacamvo: in a mo13:57
zygamvo: shall I squash it13:57
zygaor just merge it13:57
popeyjdstrand: did the snapcraft build get reverted back in launchpad builders?13:57
popeyjdstrand: we are seeing developers complaining that their snaps are not updating13:57
mvozyga: which one?13:57
mvozyga: just merge unless it has the label13:58
zyga574313:58
cjwatsonpopey: for xenial builds, yes14:02
cachioniemeyer, still failing spread14:02
popeycjwatson: ok, thanks14:02
cjwatsonpopey: unless I'm confused about what you mean by "reverted back".  xenial builds are currently running with (effectively) 2.42.114:02
popeywell, a working version of snapcraft, which doesn't have the extra metadata which the store rejects14:03
popeyso <2.4314:03
cjwatsonindeed14:03
cachioniemeyer, I found the problem14:06
cachioit works now14:06
niemeyerAh, nice14:08
zygamvo: will you have another dragon image to test today?14:15
zygaor can I put it back into the drawer14:15
mvozyga: I will ask the store guys if they can approve my dragon gadget14:26
Chipacamvo: 5730 is about talking to a store proxy through an http proxy?14:29
mvoChipaca: correct14:30
Chipacamvo: why tho14:30
Chipaca:-)14:30
mvoChipaca: also about locking14:30
mvoChipaca: oh, if thats not a support configuration I can close the PR14:31
Chipacamvo: I don't know. Maybe sparkiegeek does (but he's away i think)14:31
Chipacamvo: I'll ask14:31
Chipacamvo: seems sane though14:31
ChipacaI mean, why would you do it, but if you need to do it, this is what we need to do to let you do it14:32
pedronisChipaca: should also be normal store, no?14:32
Chipacapedronis: 5730 is only about the proxy store14:32
Chipacapedronis: it changes 'newEnoughProxy'14:32
Chipacaoh! also getSerial14:33
Chipacadarn14:33
Chipacaheh14:33
Chipacamy bad14:33
Chipacapedronis: thanks14:33
pedronisah, I thought I was confused14:33
pedronisit seems consistent at least14:33
pedronisand the bit in getSerial might be useful14:33
Chipacayup14:33
mvothanks guys!14:34
zygamvo: did you ask about /etc/lsb something and apparmor?14:37
mvozyga: I did but i figure out I can't deny stat with apparmor14:38
mvozyga: which is a) surprising b) annoying14:38
zygaYes14:39
zygaYes14:40
zyga:-)14:40
Chipacamvo: is the prepare-image pr merged?14:40
mvoChipaca: someone merge it I think14:41
Chipacamvo: nice14:41
Chipacamvo: and you merged it into 5730?14:41
mvoChipaca: yes, I think I did14:42
sergiusensjdstrand: did you get a chance to look at https://github.com/snapcore/snapcraft/pull/2234/files ? roadmr too maybe14:43
jocmvo: Chipaca: could you clear up something for me - i'm wondering what is the mechanism that distinguishes a series 16 Ubuntu Core image from a series 18 Ubuntu Core image - i.e. both could have the core snap from the other installed, but there is something that indicates where to run init + snapd from?14:44
joc(also trying to get my terminology correct)14:44
sergiusensjoc: isn't that described in the model/gadget?14:45
roadmrsergiusens: oh so we weren't pushing snaps with a manifest before?14:45
roadmrthis looks good14:45
sergiusensroadmr: we always were, just that the review tools are a bit overzealous about new attributes.14:46
roadmryes, that they are.14:46
mvojoc: not sure I get the question, but could you check /etc/os-release?14:46
roadmrsergiusens: remember jdstrand is off today and Monday14:46
jocsergiusens: possibly, but i was looking around the gadget of an image i'm testing and could see an obvious key that says the is a core18 system14:47
sergiusensroadmr: which I think we agreed would be removed. I am not sure all the corner cases for all possible entries for the manifest are covered. I do agree that the review-tools should be strict on what they already know about to be able to generate proper USNs and other tooling14:47
Chipacamvo: I think he's asking, on a core device with core16 and core18 installed, how do we know which one to boot14:47
jocyeah that basically :)14:47
Chipacajoc: it depends on how good the random number generator of the board is14:47
* Chipaca runs for the hills14:47
joclol14:48
Chipacajoc: the model says which one is the blessed one14:48
Chipacajoc: https://forum.snapcraft.io/t/model-assertions-for-core18/687014:49
sergiusensroadmr: the irony in this case is that the review tools are rejecting the keywords Jamie asked me to add :-)14:50
mvopedronis: thanks again for the review for 5583, I think you are right and we need to periodically check if snapd can go into socket activated mode, maybe every 5min?14:50
sergiusenswell, were, as the review tools are up to date already14:51
mvoChipaca, joc yeah, the model it is14:51
mvozyga: new test dragonboard image is uploading, I let you know once its there14:52
jocand that then informs the variables used at boot time to identify which snap to mount?14:52
Chipacajoc: yep14:54
roadmrsergiusens: it's because they're old tools that don't know those keywords :) haha we've seen this, we can't start sending snaps with new fields until the tools support them :/ makes for some interesting interlock14:54
jocChipaca: ok, thanks, that clears thing a bit for me14:55
zygamvo: cool14:58
sergiusensroadmr: the tools already do support them ;-)14:58
zygamvo: did you really compress it this time? :D14:59
roadmrsergiusens: aha, but not the version that's in the store :/ that was added recently AIUI (r1122/1123 while the store has 1121)14:59
roadmrsergiusens: we've discussed using the review-tools snap so it might be easier to update the tools15:00
roadmrsergiusens: haha yes, we haven't embraced the snap :) we had one huge hurdle which was the store being on trusty, that's not true anymore so the road is now open but fraught with peril15:00
mvozyga: new version is up15:01
mvozyga: I did not bother this time15:01
sergiusensroadmr: I do not know what rXXXX are, all I see is revision 553 (version 0.48+git) :-)15:01
zygamvo: it's a big difference on both upload and download15:02
zygaETA 52min15:02
zyga(on my capped link)15:02
Chipacayou know that feeling when you're writing long, hard to read integration-type tests and suddenly realise you could instead just write a unit test and let the rest sort itself out? yeah, that15:04
zygaswitched to my laptop LTE, it's going to be fast now15:04
Chipacai blame friday15:04
zyga3 minutes15:04
Chipacazyga: is this the same lte that you ate 1TB off of?15:04
pedronismvo: as I said I'm not sure it's a serious problem15:04
zygaChipaca: no, I have ... many15:04
zygaChipaca: the 1TB empty one is slow now15:04
pedronismvo: but wanted to point it out15:04
zygaChipaca: my laptop has 100GB of private plan15:04
zygaChipaca: and my phone has 65GB15:05
zygaChipaca: I also have a pre-paid plan on a backup router with 60GB (pre-paid so I won't use it this way)15:05
zygaChipaca: the polish phone has ... some amount but it's running ubuntu so I never use it15:05
roadmrI have 1 GB allowance :~(15:05
zygaroadmr: yes comrade ;-)15:06
zygaroadmr: I know, US&Canada suck wrt network15:06
Chipacazyga: is your whole house glowing at night15:06
roadmrtotally15:06
zygaroadmr: it's crazy how much network you get here15:06
roadmrthere's slightly more competition in the US but Canada is horrible15:06
roadmrand I live in one on the best provinces for that, others are worse15:06
zygaChipaca: plus each phone (e.g. kid phones) have 10GB at least (probably more, never ran out)15:06
zygaroadmr: I wonder if you could get a better deal by roaming from EU sim15:07
zygaroadmr: up until recently I had free roaming in US (I didn't check if that's still valid)15:07
zygaso my 65GB data, yep, same as back home15:08
zygaroadmr: my rural network link I used while on holidays has 100GB of data for 12 euro / month15:09
zygaroadmr: and it came with this big ass outdoor modem / antenna + indoor repeater wifi15:09
roadmrthat's ridiculously cheap hehe15:10
zygaroadmr: it also allows you to make wifi-based phone calls so you don't need coverage in your handset15:10
roadmroh I have that as well, never used it though15:10
zygaroadmr: works great in the forest15:10
zygaI'm very happy with the network here, it's never been this available and cheap15:10
roadmrcool :) somewhat envious :)15:11
zygathe rural aspect is interesting, is it a side product of covering everything else?15:11
zyga(get the last 10% customers?)15:11
roadmrsure, and also of the territory being smaller15:13
roadmrparts of rural and northern Canada has no coverage, because it' sjust huge15:13
jocmvo: i refreshed snapd to edge as the extrausers PR had landed then rebooted - it appears to have fixed those issues i was having - thank you!15:39
mvojoc: cool, thanks for confirming15:39
mvosil2100: we may need your console-conf expertise soon again :) zyga has a test image for the dragonboard and it looks like console-conf is segfaulting there, might be related to not having network yet. we are investigating, just a heads up15:42
sil2100mvo: hm hm! Symptoms sound familiar to what I was seeing on my pi315:42
mvosil2100: it looks like its segfaulting15:43
zygait is definitely signal 1116:00
zygait goes all the way up to 11 in a bad way16:00
zygasil2100: let me know if you want any debugging done16:01
Chipacazyga: where was the thing that compared directories?16:09
zygasyncdir?16:09
zygasyncdir.go16:09
zygaosutils16:09
Chipacazyga: thanks16:09
* zyga hugs Chipaca 16:10
zyga:)16:10
Chipacahmmmm16:10
Chipacazyga: didn't we have a thing where we gave it to dirs and it told us what was different?16:10
zygammmmmmmmm16:10
Chipacazyga: looks like no16:11
zygano, I don't think we had _that_16:11
Chipacazyga: because I see pack_test just doing diff16:11
Chipacawhich is fine, i'll do that as well16:11
zygawe have this thing that compares mount profiles16:11
zygabut that's not that16:11
Chipacazyga: diff is _fine_ :-)16:12
zygawoot16:26
zygaI found the bug :)16:26
zygaman :P16:26
zygarefactoring16:26
zygaold old old refactoring16:26
zygaI used an interface16:26
zygaand review asked to change that16:26
zygaso the only place that cared didn't get updated16:26
zygafixed now!16:26
zygawoooot16:26
Chipacapedronis: if overlord.Settle(t) takes more than t, what do i need to poke?16:28
zygaah maciej is off already16:33
zygawell, fair enough :)16:33
zygaI'll prepare this for chopping16:33
* Chipaca ~> afk (gym! but tests are running)16:37
cmarsquestion about branches & auto upgrades. if i have a snap installed from a branch of edge, and that branch expires, what does auto-upgrade do?16:39
cmarswould it install whatever's on edge automatically?16:39
zygaChipaca: ^16:39
zygaah16:39
zygacmars: I think it's going to behave as if it was on edge16:40
cmarszyga: then i notice the branch expired and i push back to the branch.. would the clients that had this branch installed switch back? or would i have to snap refresh on all the machines/devices to get them back?16:41
cmarsbasically, i needed these devices to stay on the branch, but forgot to keep the branch alive and there was a window where it was gone16:42
cmarswondering what kind of mess this might have made..16:42
Chipacacmars: the client is tracking the branch; when it re-opens, it'll get the new thing16:43
Chipacacmars: you can check this: in "snap info", see what "tracking" says16:43
cmarsChipaca: ok. that's encouraging.. what if i had an older rev on the edge branch than edge, and when i restore the branch, it's still older than edge, but the branch is there again16:44
cmarsChipaca: will the client still switch back to the branch?16:44
Chipacacmars: there is no "older"; there's jsut "is this the current tip"16:44
cmarsor, would it have autoupgraded to edge in the first place?16:44
=== pstolowski is now known as pstolowski|afk
zygawoooooooooooooooooooooooooot16:45
zyga :D16:45
zygalayouts no longer leak16:45
Chipacacmars: if it refreshed while the branch was closed, it would have gone to whatever was current on edge16:45
zygawooot wooot woot16:45
Chipacacmars: also why do you have long-lived branches and not tracks16:46
Chipaca:-)16:46
* zyga is super happy :D16:46
* Chipaca hugs zyga 16:46
Chipacazyga: EOW right now, on this feeling16:47
cmarsChipaca: i don't think tracks existed when we needed different builds. what's a track?16:47
zygaI need to chop the code a little but basically https://github.com/snapcore/snapd/compare/master...zyga:fix/trespassing-v2?expand=1 does the right thing16:47
Chipacacmars: https://forum.snapcraft.io/t/using-tracks/623016:47
* Chipaca needs to run16:48
zygaI'll go for a bike ride with my kids16:48
zygaand then polish the diff16:48
Chipacacmars: i'll be back in ~1.5h if you have more qs16:48
cmarsChipaca: ok, thanks16:48
* zyga EODs16:50
pedronisChipaca: usually something is not right17:21
pedronisChipaca: it's more a matter of understanding what is not happening that should or we think should17:22
pedronisChipaca: going over tasks and changes and look at their state, clean state after the timeout might be a good idea18:16
Chipacapedronis: fair enough. I'll take a look at it Monday morning18:21
Chipacanow I think I'll just call it a wrap18:21
diddledanhere's a good one for snapcraft devs to be thinking "wtf is he doing?!" about: `shutil.Error: [('/root/build_gnome-calculator-gentoo/parts/gnome-calculator/src/var/spool/nullmailer/trigger', '/root/build_gnome-calculator-gentoo/parts/gnome-calculator/build/var/spool/nullmailer/trigger', '`/root/build_gnome-calculator-gentoo/parts/gnome-calculator/src/var/spool/nullmailer/trigger` is a named pipe')]`19:49
diddledannote that it took several hours to get to that point19:51

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