/srv/irclogs.ubuntu.com/2017/08/15/#snappy.txt

=== abner is now known as Guest68496
=== Abhishek_ is now known as Guest42362
=== ikey is now known as ikey|zzz
=== JoshStrobl is now known as JoshStrobl|zzz
mupPR snapd#3734 opened: add packaging for debian-unstable <Created by mwhudson> <https://github.com/snapcore/snapd/pull/3734>02:27
chad_youngI am looking for help with snapcraft (on a working snap), anyone here good at it?03:26
=== ubott2 is now known as ubottu
mwhudsonwhoa the failure in https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-artful-snappy-dev-image/artful/amd64/s/snapd/20170815_034238_adfc1@/log.gz is funky04:44
mwhudson(and surely not related to the change)04:44
=== mardy_ is now known as mardy
mupPR snapd#3727 closed: daemon, snapstate: move ensureCore from daemon/api.go into snapstate.go <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/3727>07:38
mupPR snapd#3726 closed: interfaces: backport broadcom-asic-control interface (2.27) <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3726>08:09
zyga-susemvo: hello08:12
zyga-susemvo: I see you are busy already :)08:12
zyga-susemvo: I'm off today, just preping the leave the house08:12
=== ShalokShalom_ is now known as ShalokShalom
mvozyga-suse: good morning! yes :)08:12
mvozyga-suse: uploading a 2.27.2~rc1 to artful to get the "real" autopkgtest running on it08:13
mvozyga-suse: to validate the the i386 416 issue is really fixed08:13
zyga-susegreat08:14
zyga-susemvo: if you make a tarball and give me the URL I'd love to check suse later08:15
zyga-susere08:16
zyga-susesorry I got disconnected08:16
zyga-susewhat is the 416 issue?08:16
mvozyga-suse: no worries. there is an autopkgtest failure in artful (and other) on i386 where the server replies 41608:18
mvozyga-suse: turns out (apparently) we already have the full snap and send a GET anyway and the content-rage is data>size which the server cannot honor08:19
mvozyga-suse: welcome back, did you see anything I wrote before you disconnected?08:21
zyga-suseah, I see, I _found_ that problem :D08:26
zyga-susezyga-suse: to validate the the i386 416 issue is really fixed08:26
zyga-susethis is the last message I saw08:26
zyga-susedo you have the RC tarball already08:26
zyga-suseand will you change it again to remove the rc wording?08:27
zyga-susemvo: to watch later, https://www.youtube.com/watch?v=SPr--u4n8Xo08:28
mvozyga-suse: I will change the rc wording hopefully this evening, unfortunately the publishing/autopkgtest runs takes a litte while :(08:29
mvozyga-suse: but it will be quicker than a 2.27.3 if this is not actuallly the issue :)08:29
* popey hugs diddledan 08:29
* zyga-suse merges big PR to help gary-wzl 08:31
zyga-susegary-wzl: can you please merge master into your udev branch now08:31
zyga-susegary-wzl: I think everything has landed so we should now see the interesting parts in that diff08:31
mupPR snapd#3714 closed: interfaces: a bunch of interfaces test improvement <Created by adglkh> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3714>08:32
gary-wzlzyga-suse: Oh nice. working on it now08:32
zyga-susethank you!08:32
gary-wzlI just happened to ping you as there's only one(bit) PR left to merge. You did it08:33
gary-wzlThanks!08:33
zyga-susemvo: do watch that, it's interesting08:37
mvozyga-suse: you remember that error when snap/snapd versions got out of sync? iirc you had that at some point08:37
zyga-suseyes08:37
mvozyga-suse: I wrote a testcase that tries to reproduce it but for me on core install undo things get restarted correctly :/ you don't happen to have more details/memory about the failure?08:38
zyga-suse"crazy insecure lunacy of appimage"08:38
zyga-suseoh boy, shots fired08:38
zyga-susemvo: mmm, is it the case where snapd is new and snap is old?08:39
mvozyga-suse: correct08:39
zyga-susewhile I may be missing the details I think one thing that is sufficient is a refresh failing so if you hack the script (like we did in one other test) to always fail (the configure script) I think you can then refresh to that snap08:40
zyga-suseand then I think it will happen08:40
zyga-suseI know we allow the configure to fail08:40
zyga-susebut there must be a real way to make this happen as it runs in the wild08:40
mvozyga-suse: yeah, I did pretty much that and it restarts back to the right version for me08:40
zyga-susewas it also related to changelog and stale version number?08:40
zyga-suseoh? it restarts?08:40
mvozyga-suse: yeah, it restarts into the new version but on undo it also restarts into the old version - this is the puzzling part08:41
zyga-susemmm08:41
zyga-suseindeed08:41
zyga-suseis that also the case that gustavo hit?08:41
zyga-suseI mean, for him it did _not_ restart08:41
mvozyga-suse: I will push the test soon, there must be something I am missing, as gustavo hit it too08:41
zyga-susewhat happened there?08:41
mvozyga-suse: correct08:41
mvozyga-suse: I don't know, I keep digging, for gustavo it was "out-of-diskspace", anyway, just wanted to check if there is anything you remember that might help me :)08:42
zyga-suseI see, sorry I'm not much of a help08:47
mupBug #1627643 changed: oops on pi3 (seemingly wlan related) <Snappy:New> <linux-raspi2 (Ubuntu):Fix Committed by p-pisati> <https://launchpad.net/bugs/1627643>08:48
mupBug #1678076 changed: console-conf crashes with eth0 and wlan0 on Pi 3 <Snappy:New> <subiquity (Ubuntu):Fix Committed> <https://launchpad.net/bugs/1678076>08:48
mvozyga-suse: no worries, I keep digging08:48
mupBug #1627643 opened: oops on pi3 (seemingly wlan related) <Snappy:New> <linux-raspi2 (Ubuntu):Fix Committed by p-pisati> <https://launchpad.net/bugs/1627643>08:51
* zyga-suse goes to the park with his family09:03
zyga-susegood luck mvo, please PM me on telegram with the 2.27.2 URL09:03
zyga-suseI'll release suse then09:03
Chipacapedronis: aw! that PR was all green, and now i've got to push to it. You know it's not going to be all green next time :-(09:16
Chipacapedronis: ^^^ that's a joke by the way09:17
* Chipaca has had too little sleep09:17
=== chihchun_afk is now known as chihchun
ograoh. lovely !09:21
ograhttps://askubuntu.com/questions/938606/dwarf-fortress-starting-during-apt-get-upgrade09:21
ogra(nearly as good as "libreoffice only prints on thursdays" )09:21
pedronisChipaca: heh09:23
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
pedronisChipaca: mmh, now that I think, is this stuff tested by spread?  or only the global setup?10:54
=== ikey|zzz is now known as ikey
Chipacapedronis: good point, i could add a spread test variant that doesn't source complete.sh10:55
Chipacapedronis: but10:55
Chipacapedronis: in both cases it'll now be testing the snippets :-)10:55
pedronisChipaca: so sourcing it and having the snippet at the same time doesn't confuse things?10:56
Chipacanope10:57
pedronisgood10:57
Chipacawell10:57
Chipacano, but now that i think of it, if you source it it doesn't hit the snippets10:57
Chipacaso we want that spread variant10:57
Chipaca… which is a lot more work than just adding a variant because it doesn't multiply on its own :-/10:57
Chipacaeh. i could just try the snippets, and test the snippetless in one case10:58
Chipacaanyway, i'll get back to it and add something10:58
Chipacaright now i have panics to sort out :-)10:58
Chipacasilly managers_test10:58
mvoChipaca, pedronis, zyga: have you ever seen something like this runtime: pointer 0x11673605 to unused region of span (http://paste.ubuntu.com/25318446/ ) - this is an autopkgtest failure in artful on i386 that is blocking us currently. I will write a forum topic too and investigate further just wanted to hear if you guys have seens this before11:00
mvo(extra fun, the autopkgtest log is 20mb big and makes poor old firefox very unhappy)11:01
Chipacamvo: niiiice11:03
Chipacamvo: https://github.com/CanonicalLtd/dqlite/issues/711:03
mvoChipaca: extra points because I can *not* reproduce it in my own autopkgtest setup when I run it inside qemu :(11:04
Chipacamvo: artful is 1.8?11:07
mvoChipaca: yes11:07
Chipaca“In Go 1.8 the garbage collector has changed so that when you change the value of a pointer field, the old pointer is "shaded". In this case the C code has modified that old pointer so that it is no longer a valid Go pointer, since Go pointers, unlike C pointers, are not permitted to point past the end of an object. The garbage collector is now detecting that fact, and is complaining.”11:07
Chipacamvo: i'm terrible at understanding what in that stacktrace is the code _doing_ that, however :-/11:08
mvoChipaca: hmmmmm, interessting. I would assume this only affect cgo and we have elimiated all that code, no?11:08
mvoChipaca: yeah, the trace is hard to read11:08
Chipacamvo: I'm pretty sure we have cgo again11:08
mvoChipaca: hm, maybe that was a bad idea then :)11:09
mvoChipaca: also funny, only happening on i386, not on amd6411:09
Chipacamvo: would I be shocked if it were a bug in os/user.a or net.a in go itself?11:10
Chipacamvo: your autopkgtest setup that you run in qemu, is it also artful?11:10
mvoChipaca: net.a sounds plausible indeed11:11
mvoChipaca: I will run again, maybe I used the wrong image, i did not sleep well last night :(11:12
Chipacamvo: my next question was whether your qemu was multi-cpu, and whether the "real" autopkgtest was also mutli-cpu11:12
mvoChipaca: aha, thats a good one, I need to check. If I can reproduce, I can try a cgo-less built and see if it goes away (if we can still do it)11:13
Chipacamvo: probably not, but maybe11:13
Chipacamvo: i _think_ there's a replacement pure-go user.Current() (which 1.6 didn't have, so it would panic)11:13
Chipacamvo: but i'm not 100%11:13
mvoChipaca: it looks like we still have the test for it11:14
Chipacamvo: yes, but it's a runtime panic, not a build failure11:14
mvoChipaca: tests/main/static11:14
mvoChipaca: sure, what I mean is it ought to still be possible to built it without cgo11:14
mvoChipaca: thanks a lot for your ideas so far, I will poke at it a bit harder11:15
Chipacamvo: yes! yes. What I meant is that with 1.6 you can build it but user.Current() throws an error11:15
mvoChipaca: aha, ok11:15
mvoChipaca: sorry, misunderstood.11:15
Chipacamvo: because os/user/lookup_stubs.go's Current(), which is the one that loads without cgo, just returns 'not implemented'11:16
Chipacabut that's 1.611:16
mvofun11:16
Chipacaah11:16
Chipaca1.8's stub is much smarter11:16
Chipacahttps://golang.org/src/os/user/lookup_stubs.go11:16
mvoChipaca: aha, nice11:19
mvoChipaca: I keep you updated, thanks again for your help11:19
* mvo gets lunch11:19
Chipacalunch! a radical concept11:20
Chipacaniemeyer: i think this is because i had a slightly older spread, but: http://pastebin.ubuntu.com/25318689/12:05
niemeyerChipaca: Looks like a Linode issue ("something went wrong")12:31
niemeyerChipaca: I don't recall spread being broken in that way12:31
zyga-susemvo: any idea what that crash may be?12:34
zyga-suseanything happening at that time12:34
Chipacazyga-suse: the one with the weird backtrace?12:34
zyga-suseyes12:35
Chipacazyga-suse: cgo fighting with the garbage collector12:35
zyga-susewith gc crashing12:35
zyga-suseaha12:35
zyga-susecgo?12:35
Chipacacgo12:35
zyga-susebecause syscall?12:35
zyga-suseor because of something else?12:35
Chipacazyga-suse: syscall <=!=> cgo12:35
zyga-suseok12:35
Chipacazyga-suse: probably net, but could be os/user12:35
Chipacaonly bits of cgo we have in snapd12:36
mvozyga-suse: I'm writing a forum post now, I can reproduce in an artful i386 vm12:36
Chipacahuzzah!12:36
zyga-suseluck in bad luck12:37
mvoreproduce != fix :(12:37
mvobut yeah, progress!12:37
zyga-susemvo: we are experts, we can do _anything_12:37
mvo(looks thorny though)12:37
zyga-suseincluding fixing compiler and GC bugs12:37
* mvo conjures up a unicorn and a cup of tea12:38
mvozyga-suse: indeed we can :)12:38
* zyga-suse was partially joking about 7 perpendicular lines12:38
pedronisChipaca: I would hope net is more exercized than os/user12:45
pedronisotoh 386 , might a be a size issue12:45
Chipacamvo: to be clear, if this is what we think it is, it's going to be a slow one to resolve12:47
Chipacaas in, quickest involves a ticket with upstream go, and if they agree on our analysis, distropatching12:48
pedronisalso do we know if it fails with 18.3 ?  seems that's in artful proposed12:49
Chipaca1.8.3?12:49
pedronisyes, sorry12:49
Chipacamvo: could you try with golang from artful proposed?12:49
mvoChipaca: yeah, trying some things now. I was not able to reproduce it with git master, maybe related to the packaging in some way12:52
Chipacamvo: ours, or theirs, git master?12:52
mvoour git master12:52
mvoof snapd12:52
mvo(sorry)12:52
pedronisthat sounds weird12:52
Chipacaapology accepted12:52
* Chipaca goes for mate12:52
pedronisotoh not unhread of that moving stuff in memory12:53
pedronisfixes this kind of stuff12:53
cachioniemeyer, I-ll be few minutes late12:59
popeyikey: just installed a snap locally on up to date solus, and it hasn't connected any of the interfaces automatically.13:00
popeyikey: seen that before?13:00
ikeydon't know enough about them at this point13:00
ikeyi know the gnome ones do that13:00
ikeybut thats a known issue on the forums13:00
niemeyercachio: Ack, thanks for the note13:00
ikeyopengl/home/etc/ all work here13:01
popeylemme try another one, maybe this snap at fault13:01
ikeyok13:01
popeynone of them connected here, with a snap locally installed - not from the store13:01
ikeyoh locally13:01
ikeyok that ive not tried13:01
popeyok, worked with this second snap, so must be my snap at fault :S13:02
popeysorry for the noise :)13:02
ikeyno np at all - i like that its getting testing from folks more qualified than me :))13:02
popeySteady :)13:02
popeyI'm not _that_ qualified :)13:03
ikeywell im certainly not :p i mashed bits together until they fit13:03
ikeylol13:03
ikeysolus in a nut shell really..13:03
ikeyxD13:03
popey:)13:03
ikeygotta ask ...13:03
ikeywhich version of solus did you get?13:03
popeyoh my, I'm not sure I want to admit the level of fail I have made13:03
ikeys/version/edition/13:04
popeylsb_release says 313:04
ikeygnome/mate/budgie?13:04
popeyoh, i have all three :D13:04
ikeyoh wow ok lol13:04
popeyALL THE VMS13:04
ikeyflexiondotorg casually ignores the others like they dont exist and only sees MATE :p13:04
popeyI'm currently in the budgie one because that's the OG one IMO13:04
ikeyhah ty13:05
flexiondotorgmate mate mate mate mate13:05
ikeyXD13:05
popey(I missed the word "plugs:" in my yaml)13:05
* ikey just sees the finding nemo seagulls13:05
zyga-suseo/13:05
popeyThis is surprisingly not detected by snapcraft!13:05
* zyga-suse sends regards from his free day 13:05
ikey\o13:05
* ikey is playing it a bit fast and loose with version numbers lately13:08
ikeyBudgie 10.3.1 -> Budgie 10.4 = change basically everything13:08
ikeythis isnt how minor releases work..13:09
ikeythen solus 2017.04.18.0 -> 313:09
ikeyXD13:09
* ikey will pretend that the versions were stored in mariadb with VARCHAR(1) hence the increment13:09
zyga-susejjohansen: hey13:12
zyga-susejjohansen: can you please tell me quickly what is missing in 4.14 wrt apparmor, AFAIR you said you will not get one patch in, can you shed some light on that?13:12
tyhickszyga-suse: it'll be a little while before he starts his day but I can help13:15
zyga-susetyhicks: oh, thank you, if you know about this I'm all ears13:15
tyhickszyga-suse: the mediation feature that won't make it in is UNIX domain socket mediation13:15
tyhickszyga-suse: that kernel feature translates to policy rules such as "unix (getattr, shutdown) addr=none,"13:16
tyhickszyga-suse: signals, mount, and AF_SOCKET mediation will make it in13:17
zyga-susetyhicks: thank you!13:18
tyhickszyga-suse: unfortunately, dbus mediation depends on the UNIX domain socket mediation so "unix" and "dbus" rules will be the only two remaining rules that aren't supported in 14.1413:18
tyhickss/14.14/4.14/13:18
zyga-suseI was wondering if we could do a policy for 4.14 vanilla without this patch around13:19
zyga-suseor if this patch is huge13:19
* zyga-suse looks at http://kernel.ubuntu.com/git/ubuntu/ubuntu-artful.git/log/security/apparmor13:19
zyga-suseaha13:19
zyga-susetyhicks: do you know if this is still being discussed upstream or is it just because of merge window timings?13:19
tyhickszyga-suse: merge window timings - jj had some reworking to do on the UNIX mediation implementation that is taking a little longer than what will be possible to land in 4.1413:20
zyga-suseack13:23
sergiusens_mvo: hey, are you going to be fixing the test errors on https://github.com/snapcore/snapcraft/pull/1419 ?13:40
mupPR snapcraft#1419: meta: add `base` as a type and top level property <Created by mvo5> <https://github.com/snapcore/snapcraft/pull/1419>13:40
sergiusens_or should I bump that to the next release? We are cutting it today13:40
=== JoshStrobl|zzz is now known as JoshStrobl
=== sergiusens_ is now known as sergiusens
pedronisniemeyer: btw the first PR for repairs is this one: snapd#356013:53
mupPR snapd#3560: cmd/snap-repair: implement most logic to get the next repair to run/retry in a brand sequence <Critical> <Created by pedronis> <https://github.com/snapcore/snapd/pull/3560>13:53
niemeyerpedronis: Thanks!13:55
niemeyerpedronis: Is the numeric order the best one to do reviews also?13:55
Chipaca8-|13:56
pedronismostly yes13:56
Chipacawhy is the managers_test hitting the actual real store13:56
* Chipaca takes a break before coming back to dig13:56
pedronisChipaca: it usually shouldn't, but maybe somethign has changed13:56
Chipacapedronis: well, I changed something :-D13:57
pedronisChipaca: it runs the real Ensure13:57
Chipacabut … bah. need to dig.13:57
pedronisfwiw13:57
pedronisso unless you are careful it will tend to talk to the real store13:57
Chipacapedronis: yeah but as there's a fake store in there i assumed it was hitting that13:57
pedronisfor some stuff13:57
pedronisit depends exactly what you do13:58
pedronisI know I had to tweak again in one of my PRs for example13:58
* Chipaca goes to take that break13:58
Chipacapedronis: i'll come back with a shovel and a mug of tea13:58
pedronisniemeyer: yes about number for repairs,  I have some other PRs around about generic and stuff, of those only snapd#3710 would be nice to land sooner14:01
mupPR snapd#3710: many: allow and support serials signed by the 'generic' authority instead of the brand <Created by pedronis> <https://github.com/snapcore/snapd/pull/3710>14:01
niemeyerpedronis: Ok, I'll start from there then14:09
Chipacapedronis: my ensureMisc thing just needed to check CanAutoRefresh != nil :-D14:12
* Chipaca is Very Good With Names14:16
diddledanpopey, I've submitted the gradio PR: https://github.com/haecker-felix/gradio/pull/19714:34
mupPR haecker-felix/gradio#197: add support for snap package building <Created by diddledan> <https://github.com/haecker-felix/gradio/pull/197>14:34
popeydiddledan: yay!14:34
popeythanks diddledan14:34
pedronisChipaca: we don't set a Timeout on the client for downloads because if don't know how long it will take, but there's Transport.ResponseHeaderTimeout , do you think it would make sense to set that?14:37
pedroniss/if don't/we don't/14:38
Chipacapedronis: what is it by default?14:38
Chipacaand is it there in 1.6? i thought finer timeout control was post-1.614:38
pedronisit's in 1.614:38
Chipacacool14:38
pedronisand default is as usual, no timeout14:38
pedronisI mean also in DefaultTransport14:41
Chipacapedronis: no timeout means OS timeout means 30 seconds i think14:41
pedronisso not worth for download14:41
pedroniswe just take that14:41
popeysgclark: heya https://forum.snapcraft.io/t/change-owner-of-some-kde-apps/1682/2 - could you confirm the account to migrate to please? :)14:42
Chipacapedronis: i think so14:42
Chipacapedronis: for other things i'd love to make it much smaller, but people will get upset14:42
pedronisok, asked because I need to twek httputil.ClientOpts to control some tls stuff14:43
Chipacapedronis: there were some notes by netflix (or was it backblaze?) about tweaking these timeouts for production, did you see that?14:43
Chipacait's about 6 months old i think14:43
* sgclark looks14:43
pedronisdon't remember14:43
Chipacaah, https://blog.cloudflare.com/the-complete-guide-to-golang-net-http-timeouts/14:44
Chipacapedronis: ^14:44
sgclarkpopey: done :)14:45
* popey hugs sgclark 14:46
popeythanks14:46
Chipacapedronis, niemeyer: if I write a single everything-panics fakeStore, should I put it in store/fake and call it Store?14:46
pedronisChipaca: we tend to put that sort of stuff in X/Xtest  like snap/snaptest   or asserts/assertstest14:47
niemeyerChipaca: It's a bit unclear what the use is for something that only panics... not sure if that deserves to be on its own package?14:47
Chipacastoretest.FakeStore is at least 20% less cute than fake.Store though :-)14:47
Chipacaniemeyer: we have 4 implementations of it already14:48
Chipacaniemeyer: :-)14:48
niemeyerChipaca: I said it was unclear to me, not that that it wasn't being used :)14:48
Chipacaniemeyer: :-)14:48
niemeyerChipaca: Why do we need those?14:48
Chipacaniemeyer: everything that needs a fake store stubs out some of them, and implements the ones it cares about14:48
niemeyerChipaca: I see14:49
niemeyerChipaca: Yeah, what pedronis said then.. sounds like storetest14:49
Chipacaniemeyer: overlord/devicestate/devicestate_test.go vs overlord/assertstate/assertstate_test.go vs daemon/api_test.go vs overlord/snapstate/backend_test.go14:49
pedronisChipaca: storetest.Mock  ?14:49
niemeyerChipaca: The issue with fake is that we cannot have fake.Assert, fake.Store, fake.Foo14:49
niemeyerChipaca: Because they all conflict14:50
Chipacaniemeyer: unless we had testutil/fake/<stuff>14:50
niemeyerChipaca: Yeah, but then you have a massive package which . it.. you get14:50
niemeyer*... you get it14:50
pedronisChipaca: this comes from the stdlib,    http/httptest etc14:50
Chipacaok14:50
ikeyTFW you dont make it into this week in snapcraft.. :p14:50
Chipacaso what's the thing itself called?14:50
* Chipaca hugs ikey 14:51
ikeyxD14:51
pedronisChipaca: I don't think we have uppercase Fake stuff14:51
pedronisexcept in tests/lib/ maybe14:51
niemeyerChipaca: storetest.BrokenStore? :P14:51
ChipacaPanicingStore14:51
Chipacastoretest.LetsGoShopping14:51
niemeyerstoretest.Store seems fine :)14:52
Chipacaokey doke14:52
* Chipaca runs with it14:52
=== chihchun is now known as chihchun_afk
mvosergiusens: i missed the errors, sorry, on it now14:58
* ogra sees https://forum.snapcraft.io/t/snap-dependencies/1699/1 and runs away screaming14:58
ikeybuild + runtime dependencies..15:00
ikeyhm what else can do that..15:00
ograno, this is about actual snap dependencies (things you want pre-installed before a snap can run)15:02
ograpretty much going back to dpkg15:02
ogra(instead of bundling)15:02
ikeyya swhat i said :p15:05
ograheh, sorry ...15:05
ikeyoh don't be - i'm surprised ive gotten this far in life with anyone understanding me xD15:05
mupPR snapd#3735 opened: many tests: move all panicing fake store methods to a common place <Created by chipaca> <https://github.com/snapcore/snapd/pull/3735>15:16
popeysgclark: cprov has moved the snaps over, over to you! :D15:19
niemeyerpedronis: #3710 just reviewed.. will have lunch and continue soon15:20
sgclarkpopey: thanks so much for your help :)15:22
ograhey sgclark good to see you around :)15:22
sergiusenssgclark: hey, are you also taking care of konversation (the snap is my current irc app)?15:23
sergiusenspopey:  does the lxd snap work on solus?15:23
sgclarkhi ogra! yeah I am happily on the neon team and we are snappifying kde apps15:23
ograyay15:23
sgclarksergiusens: hi! yes we will be moving to snaps for all apps, in time15:24
sergiusensoh, nice!15:24
sgclarkmuch work to be done :)15:24
popeysergiusens: no15:25
ograoh ?15:25
ograwhy not ?15:25
popeyhttp://paste.ubuntu.com/2531948115:26
sergiusenssgclark: great to hear, the contact info in the snap tells me to go to kde.org but I was wondering if there is a better place to report issues?15:26
sergiusensmy current one now is that opening links is quite slow and I have to choose the handler every time15:26
ograpopey, hmm ... ouch ... sounds like LD_LIBRABRY_PATH bits missing ?15:27
sergiusenspopey: that's unfortunate, it would of made me switch for a while ;-)15:27
ogra*LIBRARY15:27
sergiusensogra: but it works on ubuntu and fedora15:28
sgclarksergiusens: doesn't look like we have konversation, let find out who did it15:28
sgclarkme*15:28
sergiusenshah15:28
ograsergiusens, yeah, must be something missing in snap-configne for classic snaps on solus ... i bet liblxc.so.1 is somewhere undr /snap/lxd/current/15:28
ogra*confine ...15:29
sergiusensogra: lxd is not classic15:29
* ogra cant type anymore 15:29
ograoh15:29
ograindeed15:29
ogralxd-support interface ?15:29
sergiusenspopey: does the client side work? `lxc list`15:29
sergiusensyeah, maybe the interface is not connected, but it should15:29
ogra(does solus have that yet)15:29
popeyit complains that lxd isnt setup15:30
ikeysolus doesn't have lxd, because, well, it wasn't really in demand..15:30
ikeyand we kinda have docker, etc..15:30
ograikey, well, its a snap :) ... just needs the interface enabled15:30
sergiusensikey: there is a snap, just figuring out why it doesn't work15:30
ikeycould try LD_DEBUG=files in the environment before running lxc/lxd15:31
ikeyand find out where the linker is looking15:31
ikeywhere it shouldnt15:31
ikeytis possible the snap is making some multiarch assumption15:31
sgclarksergiusens: ok looks like sitter did it under KDE infra, please report on bugs.kde.org15:32
sergiusensok, sounds good, but sgclark are you taking that under the neon umbrella soon?15:33
popeyhttp://paste.ubuntu.com/2531951015:33
sgclarkneon is under kde umbrella :) so will remain the same15:33
sgclarksergiusens: ^15:33
sergiusensgot it15:33
sergiusenspopey: I guess I'll be installing solus and figuring it out unles I accidentally ping stgraber and he fixes it before I can blink ;-)15:35
popeyhehe15:37
pedronisniemeyer: answered15:44
=== chihchun_afk is now known as chihchun
ograjdstrand, hmm http://paste.ubuntu.com/25319575/ ... shouldnt that suggest the kernel-module-control interface instead of talking about env vars ?15:49
ogra( i wonder what any of the vars would help here )15:50
niemeyerpedronis: certified is such a strange name for this idea16:13
niemeyerpedronis: Too late I guess16:13
niemeyerIt really means "verified"16:13
pedronisyes16:16
pedronisniemeyer: I think we use it so little that we could change it, but it's a bit of a separate issue16:16
niemeyerYeah16:17
=== ikey is now known as world
=== world is now known as ikey
jdstrandogra: yes, it should. note that snappy-debug doesn't have AI so it has to be taught this stuff ;)16:19
jdstrandogra: I'll take care of that16:19
ograthanks :)16:19
pedronisniemeyer: anything else? as I wrote  the other changes around assertstest, NewStoreStack I would prefer to do them in a follow up16:22
niemeyerpedronis: No, per note there LGTM once you're happy16:22
niemeyerpedronis: The core change is sensible16:23
pedronisok, I fixed the error now, then I'll merge, and rework NewStoreStack in a different PR16:23
diddledanjust waiting on the review to add this to the store: https://github.com/diddledan/gnome-twitch-snap16:31
mupPR snapd#3568 closed: snapctl: add new `snapctl internal configure-core`  <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3568>16:32
mupPR snapd#3594 closed: corecfg: add proxy configuration via `snap set core proxy.{http,https,ftp}=...` <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3594>16:32
diddledanneeds someone to approve my coopting of the name :-)16:32
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
diddledanpopey: flexiondotorg: I've just imported gimp into snapcrafters org and set the autobuilder running17:01
popeydiddledan: nice!17:13
diddledanI don't believe I can change the ownership in the store, though, so I'll have to leave that up to one of you, popey17:19
mupPR snapd#3710 closed: many: allow and support serials signed by the 'generic' authority instead of the brand <Reviewed> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/3710>17:19
zyga-susere17:21
* zyga-suse is home17:21
ograshouldnt you then be zyga-home ?17:21
ogra:P17:21
zyga-suse:-)17:22
zyga-suseI was in a tank today17:22
zyga-suseI should have joined as `zyga-tank`17:22
popeydiddledan: who currently owns it?17:22
popeydiddledan: you can make a request in the store category and likely cprov will action it :)17:22
popey(on the forum)17:22
diddledanI own the name in the store. so it just needs moving to snapcrafters. the git repo is moved now as I do have access to do that bit :-)17:23
popeykk, so yeah, make the request and it'll get done17:23
diddledannew snap :-) https://forum.snapcraft.io/t/call-for-testing-gnome-twitch/170117:26
popey\o/17:26
popeydiddledan: need to add "snap install gnome-3-24" to your instructions17:28
diddledanwell caught. that's fixed now17:29
popeyooh, the page just dynamically changed when you edited it17:29
diddledanit's a cool forum!17:29
popeygonna post on the snapcraft socials ;)17:30
diddledanok, I've hit a snag - it doesn't work strict with dbus issue.. let me see if I can fix that17:32
popeyok, will delay the post, just let me know when you want me to release it17:32
diddledanbuilding now17:33
* diddledan twiddles thumbs while compilation churns17:34
popeyhttp://imgur.com/a/8gLad wanna embed that into your post to make it pretty? :)17:34
popeyif you download it and just drag it into the edit window for the post it should work17:35
diddledandanke17:35
diddledandone17:35
diddledanthe image is done, I mean17:35
popeynp17:35
diddledanstill building the fixed snap17:36
mupPR snapcraft#1419 closed: meta: add `base` as a type and top level property <Created by mvo5> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1419>17:36
* diddledan sits patiently: ┏(-_-)┛┗(-_-)┓┗(-_-)┛┏(-_-)┓17:37
popeyThrow some coal in the launchpad builders17:37
diddledanhttps://imgs.xkcd.com/comics/compiling.png17:38
mupPR snapd#3736 opened: apparmor: pass --quiet to parser on load unless SNAPD_DEBUG is set <Created by jdstrand> <https://github.com/snapcore/snapd/pull/3736>17:38
jdstrandstgraber: fyi ^17:40
zyga-susejdstrand: +117:41
mvosergiusens: thank you so much17:43
diddledan"human review required due to 'deny-connection' constraint for 'slot-attributes' from base declaration declaration-snap-v2_slots_deny-connection (dbus-gnome-twitch, dbus) "17:43
pedronisChipaca: I see, I thought a couple of times that snapstate.Store(), StorService etc should move to a something smaller, more specific under overlord  but never acted on it so farbut17:44
pedronisChipaca: all places you change import snapstate anyway17:48
cprovdiddledan, popey: gimp name transferred to snapcrafters.17:48
Chipacapedronis: yes, but what if snapstate wants to import it17:48
diddledanthanks cprov17:48
pedronisChipaca: ?17:48
pedronisChipaca: snapstate_test you mean ?17:49
Chipacaah17:49
pedronisit probably does but is not snapstate17:49
Chipaca:-)17:49
Chipacafair 'nuf17:49
Chipacai'll fix it (later)17:49
Chipacaright now i'm EODing with extreme prejudice17:49
Chipaca:-)17:49
diddledanjust waiting on a manual review, popey. I've tested it locally and strict works, but it won't publish in the store until someone OKs the dbus socket17:54
jhobbscan someone look at https://bugs.launchpad.net/snapd/+bug/1710933 please? it seems like 2.26.10 has a regression preventing snapd from working in lxd containers17:58
mupBug #1710933: 2.26.10: snapd doesn't start in a container <cdo-qa> <cdo-qa-blocker> <snapd:New> <https://launchpad.net/bugs/1710933>17:58
pedronismvo: fyi, my first 'generic' PR is in now, so nothing needs reverting for 2.2817:59
niemeyerGoing outside for a bit.. back soon18:05
mupPR snapcraft#1483 closed: lxd: path cannot have extra forward slashes <bug> <Created by kalikiana> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1483>18:09
popeyniemeyer: liked the presentation of your ideas as a video. Do you have a green screen? Seemed pretty professionally done!18:12
mupPR snapcraft#1479 closed: rosdep: add support for multiple dependency types <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1479>18:12
popeyjdstrand: i just updated the slot definition for gnome-twitch for diddledan - could you verify I did the right thing? (Not done slots before)18:15
popeyjdstrand: https://dashboard.snapcraft.io/dev/snaps/8171/18:15
jjohansenzyga-suse: the unix patch (in ubuntu) isn't small, but mostly self contained. However the version going upstream depends on the typesplitting work which is large.18:53
zyga-susehey :)18:53
zyga-susejjohansen: do you have any patches for refrence? I would like to encourage some distributions to ship that on top of 4.1418:53
zyga-susecan you tell me more about typesplitting?18:54
jjohansenzyga-suse: hey, sorry I just haven't been able to finish this stuff up18:54
jjohansenzyga-suse: the patches are a wip atm, I am hoping to have something to RFC by next week, that way I can get the upstream discussion started early18:54
zyga-susejjohansen: no worries, I understand18:55
jjohansenzyga-suse: we need to be caching the type on objects, not just the domain label18:55
zyga-suseI see18:55
jjohansencurrently (in ubuntu) we get around this problem by caching the unix socket path (a second time, ugly hack) and revalidating18:56
jjohansenthat isn't going to work for other sockets, and isn't going to fly upstream18:56
jjohansenit also has a few artifacts in some corner cases18:57
jjohansenbasically if you think of apparmor in terms of TE (selinux), we use the domain (task) label, and look up a set of conditionals (path, ...) to find permissions18:59
jjohansenthat set of conditionals can resolved down to a type (in selinux terms). And we can store that on the object so that when we hit a revalidation case, where some of our conditionals are no longer available we can find the correct permissions18:59
jjohansenbasically the LSM is more structured to selinux and we need to do somethings to make apparmor more amenable to that upstream structure19:00
zyga-suseI see19:02
zyga-suseinteresting explanation, I don't know the kernel parts that well but I'm eager to lern19:02
zyga-suselearn*19:03
niemeyerpopey: Thanks! I made a green screen at home for about 3USD :)19:06
popeyhah, excellent19:07
popeyOBS?19:07
niemeyerpopey: Three layers of nonwoven fabric on an old whiteboard frame19:07
niemeyerpopey: https://goo.gl/photos/RWEU7jCzBa7Cdnp7819:08
popeynice19:09
popeytotally doing this19:09
diddledanwhat are you two scheming?!19:13
popeyhiding the crap behind my head when doing videos :D19:15
niemeyer:)19:15
niemeyerNot only that, but putting more interesting crap there! :P19:16
popeytrue19:17
jdstrandpopey: looking. note that for snap declarations I recommend you come to the security team. they can be tricky to get right (though the one you did isn't) and our reviewers tooling doesn't help you (yet)19:25
jdstrandpopey: it looks fine. at this point you should scroll to the bottom of review page, add a comment like "Granting use of the well-known dbus name 'com.vinszent.GnomeTwitch' to the snap", then click 'Run the automated review again"19:28
popeythanks jdstrand19:29
popeynoted for next time :)19:29
jdstrandpopey: I didn't do the comment or button pushing for this one19:30
jdstrandpopey: I see a bunch of snaps that came in today. I'll take a look. I think the are all dbus related19:30
popeyexcellent19:30
popeyi hit the button...19:30
jdstrandcool19:31
diddledan\o/19:31
* diddledan releases to beta19:32
diddledandone19:32
popeywant me to release a social post?19:32
diddledanok, you can fire the social now :-)19:32
popey\o/19:32
* popey gets the coal19:32
jdstrandpopey: I want to ask oSoMoN a question about chromium when he comes online, so am leaving that19:32
popeykk19:32
popeydiddledan: sent19:33
diddledanI see it19:33
ikeyadwaita19:37
* ikey shudders19:37
popeyYou liked my choice of desktop to use in the screenshot? :)19:37
ikeyforgive me for saying but is that solus 3? :P19:38
popeyya :)19:38
ikeylooks like solus 2017.04.18.019:38
popeyfix ur version numbers19:38
ikeyor did you rejigger it?19:38
ikeyXD19:38
popey#blamepopey19:38
ikeybut yeah nice use of budgie :]19:38
popeyno, it's whatever I instaleld and updated19:38
popeyThis is one thing I love about posting on the snapcraft social stuff, we get to use whatever desktop is fun today :)19:38
ikeySolus 3 https://solus-project.com/imgs/posts/2017/08/Budgie.jpg19:38
ikeyxD19:39
popeyodd19:39
popeyi only installed it last week or so19:39
popeyand updated as per the software doohdah19:39
ikeyyeah it was released around 4 this morning19:39
popeyhahah19:39
popeyoookay19:39
ikeyits ok you'll still be updated on "Solus 3"19:39
popeygotcha19:39
ikeywe used a new branding pkg in that one19:39
ikeyplus this is cool because it means you're testing linux-lts (4.9.43)19:39
ikeyso we get both kernels tested now19:40
ikey:]19:40
jdstrandpopey: fyi, I'm leaving https://dashboard.snapcraft.io/dev/snaps/8122/rev/8/ for the advocacy team19:41
jdstrandI guess could leave a comment to ask to post something to the forum...19:41
popeyI'll add a card for it19:42
popeywill look in the morning19:42
popeyikey: the downside is.. people expect it to be ubuntu... https://www.facebook.com/snapcraftio/photos/a.127475154349370.1073741828.106271556469730/350162465413970/?type=3&permPage=119:48
popey(hope that works) - person asking how you get the launcher in the top of the screen :)19:49
popeyoops19:49
popeyI'll throw them a solus link :)19:49
ikeyhah20:01
ikeywell it just so happens that budgie 10.4 was made to be backportable20:01
ikeyso i mean straight up its looking like it'll be in 17.10 anyway but could easily end up in older ones with PPAs20:02
ikeysupports 3.18 stack..20:02
* zyga-suse pulls in budgie desktop 20:09
zyga-susewell, solus 3 that is20:09
zyga-suseikey: any useful link on "my first solus package" or "solus for developers"?20:10
ikeyerm20:10
ikeycontext?20:10
ikeylike you need build tools right?20:10
ikeyi.e build-essential equivalent?20:10
zyga-suseyes20:10
zyga-suseand packaging primer20:10
ikeysudo eopkg it -c system.devel20:10
ikeyhttps://solus-project.com/articles/packaging/20:11
ikeynote build tools are largely mutually exclusive of packaging20:11
ikeyas we mandate everyone builds packages using solbuild20:11
ikeywhich is like basically if docker and schroot had a baby, and it wasn't an abomination20:11
JoshStroblAlso: https://www.youtube.com/watch?v=rlPnHjUBpJ8 https://www.youtube.com/watch?v=xK-1726MWqQ https://www.youtube.com/watch?v=c_vIg3ep3YE20:11
* JoshStrobl goes back to watching Logan20:11
ikeywhere did you com..20:11
ikeylogan is *win* btw, enjoy20:11
zyga-suse:)20:12
ikeythis is the single most arch-linux-thing ive ever seen written https://twitter.com/fivelek/status/89754418116555981320:12
zyga-susethanks, let me look at those20:12
ikeyit served zero purpose other than telling everyone that they use arch linux xD20:12
zyga-suseikey: no, it served a purpose20:15
zyga-suseikey: I got to tell them snaps work on arch :D20:15
ikeyoh man he got reverse-preached20:15
* ikey approves20:15
* zyga-suse notices this channels has 284 people 20:25
=== ejat_ is now known as ejat
pedronisniemeyer: thanks for the repair review, I answered some things, I'll work through the feedback in my morning21:21
mwhudsonwth22:01
mwhudsoncan someone tell me how to check if apparmor confinement is working?22:02
mwhudsoni thought it was broken in debian but actually it seems to be working now, very confused22:02
mwhudsonjdstrand: ^ -- is there maybe a test snap that tries to violate its confinement?22:03
* mwhudson forums22:11
tyhicksmwhudson: #13 of https://wiki.ubuntu.com/Process/Merges/TestPlans/AppArmor#Desktop_only would do22:21
iatrouFor snaps with classic confinement, like conjure-up, is there a workaround to make them work when $HOME is on NFS?22:45
mwhudsoniatrou: with classic confinement, no workaround should be needed...22:50
mwhudsontyhicks: yeah, confinement is not working :(22:52
iatroumwhudson: and yet, running on snapd 2.25 conjure-up 2.2.2 it complaints: cannot create user data directory: /home/qz40qq/snap/conjure-up/549: Permission denied22:54
mwhudsoniatrou: huh :(22:55
zyga-suseiatrou: any denials?23:07
zyga-suseiatrou: I think classic confinement _still_ applies apparmor so there might be something missing23:08
zyga-susebut ... what are the facts?23:08
iatroumwhudson: zyga-suse http://paste.ubuntu.com/25322186/23:36
zyga-suseaha!23:37
zyga-suseinteresting23:37
zyga-suseso yes23:37
zyga-susesnap-confine itself is affected23:37
zyga-suseinterersting bug23:37
zyga-suseiatrou: can you please open a forum topic on forum.snapcraft.io23:37
zyga-suseI believe we can get that fixed23:37
iatrouzyga-suse: sure thing, thank you!23:38
=== ikey is now known as ikey|zzz
iatroumwhudson: zyga-suse https://forum.snapcraft.io/t/snaps-with-classic-confinement-and-nfs-home/170623:47
zyga-susethank you23:47
mwhudsonzyga-suse: go to bed :)23:57
zyga-suseI fell asleep earlier and now I cannot sleep23:58

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