=== chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun [04:57] kyrofa: I cant find any details on using the environment keyword in the documentation, do you have a link handy, just curious about it [05:12] PR snapd#3692 opened: tests: install most important snaps === JanC_ is now known as JanC [06:31] PR snapd#3685 closed: interfaces: add missing test for optical_drive interface [06:33] PR snapd#3684 closed: interfaces: convert physical_memory_observe to common iface [06:42] * zyga-ubuntu is very grumpy about quite sub-optimal package build/test failure outupt [06:42] output* [06:42] extreme verbosity and no substance [06:43] good morning [07:02] hey zyga-ubuntu! where do you see this output? [07:09] mvo: unit tests [07:09] mvo: first you get a list of all the files, because that's apparently interesting [07:10] mvo: then you get a per-module test result, because we just need to see that too [07:10] zyga-ubuntu: aha, yes, its quite terrible [07:10] mvo: and somewhere in that output there is one test failure [07:10] mvo: obviously you don't know because the output for all the 100s of other modules follows [07:10] mvo: .... [07:10] zyga-ubuntu: yes, I suffered this pain also [07:11] mvo: let me see if I can cure this [07:11] zyga-ubuntu: if you find a switch to make this less insane, I will give you an extra hug [07:11] because day-to-day experience is nothing like that [07:12] mvo: I mean this seems to do the right thing: $ go test github.com/snapcore/snapd/... [07:12] are we getting the painful output from debhelper or is that self-induced-suffering? [07:12] zyga-ubuntu: probably, I guess it sets some verbose flags somewhere [07:13] I'm grumpy enough to find this today [07:13] thank you mvo :) [07:14] PR snapd#3689 closed: Improve the test for physical memory control and observe interface [07:50] PR snapd#3693 opened: snapsate: improve the error message when classic confinement is not supported [07:52] woot [07:52] thank you mvo! [07:52] mvo: 2.27 backport would be lovely [07:52] mvo: I'll add this to arch ASAP :) [07:52] oh the list of files is dh-golang being stupid i think [07:53] DH_VERBOSE=0 will stop that [07:53] mwhudson: thank you :) [07:55] zyga-ubuntu: I hope 2.27 is closed, I need to talk to Sergio but ideally rc8 will be renamed to -final and it goes to candidate. but lets see what he says [07:57] mvo: CE asked to include the broadcom-asic interface there [07:57] mvo: if you still allow PRs I'll make a backport [07:58] zyga-ubuntu: I still need to try implementing this in snap-confine, but I think I've worked out a way to handle the per-user mounts that should work with snap-update-ns [07:58] jamesh: that's great, I saw the messaging about this [07:59] zyga-ubuntu: it's essentially unshare the mount namespace, then remount "/" with MS_REC|MS_SLAVE, then overlay the extra mounts [08:01] I get all the mount changes from the parent without the parent seeing any of child's [08:02] what happens if, say, the parent them mounts something over your changes? [08:06] zyga-ubuntu: are you ok with merging 3604? or do you see any blockers in there? [08:13] zyga-ubuntu: another snapd bug you might be interested in: https://github.com/anbox/anbox/issues/399 [08:18] * zyga-ubuntu looks at both [08:19] mvo: letm's merge it, I'll improve the thing I care about soon [08:20] PR snapd#3604 closed: tests: enable main suite for opensuse [08:35] zyga-ubuntu: thank you [08:58] zyga-ubuntu: o/ [08:59] zyga-ubuntu: snapd#3694 has a commit message i think you'll like [08:59] PR snapd#3694: wrappers, overlord/snapstate/backend: make link-snap clean up on failure [08:59] PR snapd#3694 opened: wrappers, overlord/snapstate/backend: make link-snap clean up on failure [08:59] mup: where's your source? I want to add a "no parroting" feature [08:59] Chipaca: I apologize, but I'm pretty strict about only responding to known commands. [08:59] mup: no you're not, no matter what I type you respond. [08:59] Chipaca: In-com-pre-hen-si-ble-ness. [08:59] mup: see? [08:59] Chipaca: Roses are red, violets are blue, and I don't understand what you just said. [09:01] Chipaca: looking [09:01] Chipaca: that's a world of difference to me, thank you! [09:02] zyga-ubuntu: :-) [09:03] and now i need to go push some papers around. Should be back in ~30 minutes, but who knows. [09:05] o/ [09:06] Chipaca: I like the cleanup approach [09:06] zyga-ubuntu: i've tried to keep it boring :-) [09:07] also i'm not very good at doing the "go push papers around" thing [09:10] * Chipaca tries again [09:11] what kind of go push papers around are you referring to? [09:12] ok, I read the non-test code and it looks good [09:12] looking at tests now [09:12] I wish github had a way to split this [09:12] zyga-ubuntu: need to renew my lease [09:12] ah [09:13] I misunderstood "go" as "golang" [09:13] sign a stack of papers [09:13] * zyga-ubuntu hugs Chipaca after looking at his pile of papers [09:13] * Chipaca hugs zyga-ubuntu back === rumble is now known as grumble [10:44] * zyga-ubuntu -> food [10:50] cachio: hey,good morning! did you find out anything about the - external:ubuntu-core-16-arm-32:tests/main/security-setuid-root failure(s)? [10:50] cachio: if not, let me build an image now and try to reproduce [10:53] niemeyer: poke [11:00] niemeyer: https://forum.snapcraft.io/t/command-line-interface-to-manipulate-services/262/33?u=chipaca [11:00] niemeyer: basically i'm not liking the very different behaviour of "snap restart" vs "snap restart --reload", and would like to make a minor change to it so the problem goes away [11:25] niemeyer: just FYI, I looked into updating the support: field for the core snap but this is not yet supported in the store. bug 1666792 is tracking this work (sort of) [11:25] Bug #1666792: Please add "contact" field to the API [11:32] mvo, perhaps via snapcraft somehow ? [11:32] mvo: well that's part of the larger discussion we had in London [11:33] about what's authoritative [11:34] cachio: you around? [11:38] nm, network slowness [11:48] PR snapd#3688 closed: Improve the test for ppp interface === chihchun is now known as chihchun_afk [11:59] jdstrand: can you have a 2nd look (and perhaps +1) https://github.com/snapcore/snapd/pull/3499 [11:59] PR snapd#3499: interfaces/builtin: add the spi interface [12:02] Chipaca: this is not the first place reporting op conflicts for sure, I suppose the rest are doing 400 or 500 [12:02] PR snapd#3478 closed: tests: extend upower-observe test to cover snaps providing slots [12:10] Bug #1645731 opened: Fail to access the shared content if app starts before connect interface [12:11] Chipaca: Yo [12:11] Chipaca: Is that in the topic? Looking [12:11] q: what is the preferred method for packaging snapd? given the number of golang dependencies [12:11] solus packaging explicitly disables networking during build for security [12:11] mvo: You mean "contact"? [12:12] niemeyer: yes, contact, sorry [12:12] thus is it safe to lift the networking restriction and trust the vendor tool, or should the revdep chain be packaged? [12:12] mvo: I'm hoping to not confuse "support" here, as very often that means something specific and different [12:12] niemeyer: yes, my bad, I wrote "contact" in bugreport and in my other irc, sorry for the confusion [12:13] mvo: Thanks [12:13] mvo: So people are unable to set their own contact field? [12:13] mvo: Or where else the problem lies? [12:14] niemeyer: AIUI the store hardcodes contact (supported-by is iirc the name still in use in the returned json) to the publishers primary email [12:14] well [12:14] the old UI allowed to set it [12:14] mvo: I think I've seen some snaps using a URL [12:14] niemeyer: i.e. no way to change that and no way to set it on a per-snap basis currently [12:14] ogra@pi3:~$ snap info upnp-server|grep contact [12:14] contact: ogra@ubuntu.com?subject=upnp-server [12:14] Ah, perhaps ogra nails it [12:15] niemeyer, ogra_: yes, the old webui had this, it got removed as part of https://bugs.launchpad.net/snapstore/+bug/1672664 (private unfortunately) [12:15] (i used to set the subject line in that mailto link for every snap i have) [12:15] mvo: Okay, that makes it pretty irrelevant.. I don't want my personal email on every snap I publish, for example [12:15] Let's see if we can get that sorted [12:16] niemeyer: I can create a forum topic if you want, I also filed https://bugs.launchpad.net/snapstore/+bug/1709610 just to be sure [12:16] Bug #1709610: Please allow "contact:" override on a per-snap basis [12:16] the backend is definitely capable of doing it ... just the UI gield vanished when we switched to dashboard.s.io [12:16] *field [12:16] mvo: niemeyer: I think contact was added to snapcraft [12:16] ikey: hello! [12:16] ogra_: maybe we still set it in the db in a one-off basis, I don't know, I need to followup [12:16] but the store doesn't know [12:16] ikey: I can help you out [12:16] mvo: Thanks a lot! I'll open the forum topic and point to the ticket [12:16] heya zyga-ubuntu :] [12:17] pedronis: its not there yet afaics but I can trivially add it and push a PR ( doing some snapcraft things for bases anyway) [12:17] mvo, well, my snaps still have the "?subject=" [12:17] ikey: I'm not a solus user or developer but if you want to make a snapd package available I can help you out [12:17] so it seems to be handled when set [12:17] niemeyer: https://bugs.launchpad.net/snapstore/+bug/1666792 and https://bugs.launchpad.net/snapstore/+bug/1709610 should have the relevant info [12:17] Bug #1666792: Please add "contact" field to the API [12:17] Bug #1709610: Please allow "contact:" override on a per-snap basis [12:17] zyga-ubuntu, well its basically a case of looking at how everyone else is packaging it. are they allowing networking during build, have a huge tarball, or package all the reverse golang deps as vendored packages (brittle) ? [12:17] niemeyer: it's also then proabably related to https://forum.snapcraft.io/t/development-sprint-june-26th-2017/415/48 as well [12:17] ogra_: yeah, lets see what the store people tell us, this is definitely encouraging [12:18] ikey: all of those [12:18] mvo, this is definitely a regression :) [12:18] ikey: depending on the distribution [12:18] ok what does debian/ubuntu world do? [12:18] (specifically your second bug) [12:18] because we have networking nuked in builds (solbuild implements a lightweight container) [12:18] ogra_: yeah, that reminds me that I need to write a bugreport about the vanishing architecture in the webui, it used to be there but is no more [12:18] ikey: different things actually, in debian snapd is packaged separately from all dependencies, in ubuntu it is a big tarball [12:18] i can turn it on .. but.. yknow. [12:18] hm ok [12:19] ikey: what is customary for solus? [12:19] (for golang) [12:19] mvo, oh, yes please ... thats super annoying (the uploads overview still has it, just not the details) [12:19] are all golang modules packaged separately? [12:19] well we actually have very few golang things packaged zyga-ubuntu except our own stuff (like solbuild) - but we create proper release tarballs for those with src/vendor submodules setup [12:19] with src/vendor actually being git submodules [12:19] ikey: aha [12:19] and using git-archive-all to create a true dist tarball [12:20] ikey: we have a "fat" tarball that mvo makes for each release I think [12:20] consider my interest peeked [12:20] lol [12:20] ikey: so if you want to get started, start with that please [12:20] sure would love it, ty [12:20] mvo: I'm probably just confused then, anyway the status about all these fields is confusing [12:20] ikey: two requests [12:20] * ikey listens [12:20] ikey: can you please start a thread on forum.snapcraft.io about this, I'm sure many other people would love to know this is happening [12:21] ikey: and as a personal request, can you please add /etc/os-release to github.com/zyga/o-release-zoo, it helps with some tests [12:21] github.com/zyga/os-release-zoo [12:21] pedronis: yeah, its all a bit in flux it seems, sounds like a good opportunity to clarify things (or my git tree is outdated, also quite possible :) [12:21] pedronis: anyway, I get to the bottom of it [12:22] zyga-ubuntu, for the 2nd one, we roll, so VERSION_ID is non static (heads up) - for the 1st one - uhm.. sure...? lol [12:22] seems a bit like self promotion on my part doing so though :p [12:22] sure [12:24] PR sent to the zoo [12:25] ikey: merged [12:25] ta [12:25] ok got a snapcraft.io forum account, what exactly am i posting here? :p [12:26] ikey: just make a post that you're working on a package for solus [12:27] aite [12:27] in snapd tag ? [12:27] * ikey & lost with discourse [12:27] ikey: yes that's fine :) [12:27] ta [12:29] ikey: and please stay in touch, I'd love to help you with the package [12:29] cheers, will do [12:31] PR snapd#3695 opened: interfaces: convert kvm to common iface [12:31] forum post done [12:32] ikey: thanks, looking [12:32] 4 potential users \o/ [12:32] gary-wzl: hey, thank you for all the work :) [12:32] PR snapcraft#1447 opened: add support for the "contact" field in snapcraft [12:32] ogra_, lol [12:32] gary-wzl: can you cross-review https://github.com/snapcore/snapd/pull/3696 please? [12:32] PR snapd#3696: interfaces: covert framebuffer to commonInterface [12:33] PR snapd#3696 opened: interfaces: covert framebuffer to commonInterface [12:33] zyga-ubuntu: sure, here is more to come. :) [12:34] gary-wzl: reviewed kvm [12:40] ikey: replied :) [12:42] ta [12:45] I guess I'll install solus now :-) [12:46] might be worth waiting a few days tbh [12:46] impending release [12:46] and snapd will be more formally supported after that [12:47] ikey: aha, I'll hold off then [12:47] zyga-ubuntu: reviewed and just left a comment. [12:47] zyga-ubuntu: I could fine-tune the framebuffer_test.go in a separated PR as I go. [12:47] cachio, do you happen to know where exactly i can find the code for "failover:emptysystemd" ? [12:47] gary-wzl: I'm re-writing framebuffer tests now [12:47] zyga-ubuntu: Thanks! [12:48] ogra_, yes, [12:48] PR snapd#3697 opened: docs: add PULL_REQUEST_TEMPLATE.md [12:49] ogra_, https://github.com/snapcore/snapd/blob/master/tests/main/failover/task.yaml [12:50] cachio, thanks [12:51] ogra_, I updated this test yesterday [12:51] * ogra_ has been debugging a reboot failure with a customer for days ... it just struck me that this test actually *wants* it to fail :P [12:51] silly thinko [12:51] mvo, resutls updated [12:57] * mvo hugs zyga-ubuntu and zyga-suse [12:58] cachio: things looks mostly good it seems, one strange pi2 issue, I try to reproduce htis now with my pi2 - I had forgotten just how long this takes :( [12:58] cachio: and console-conf seems to be very unahppy, do you know what is going on there? maybe we can talk in the hangout [12:58] mvo, we should really pay him a psychiatrist there is a new personality coming every week now ! [12:58] ETOOMANYZYGAS [12:59] ogra_: he has as many personalities as there are linux distros ;) [12:59] yeah, i noticed :) [12:59] mvo, most of console conf tests are known issues [12:59] mvo, I'll take a look again [12:59] Issue # opened: snapcraft#1448, snapcraft#1449, snapcraft#1450, snapcraft#1451, snapcraft#1452, snapcraft#1453, snapcraft#1454 [13:00] mvo, for the pi2, I added a second link with the tests passing [13:00] cachio: aha, great! [13:00] cachio: yay [13:02] gary-wzl: added [13:02] zyga-ubuntu: done [13:02] ogra_: just wait for more :) [13:02] jdstrand: thank you! [13:02] Issue snapcraft#1455 opened: share cache with local container [13:02] Issue snapcraft#1456 opened: clean up stale containers [13:02] Issue snapcraft#1457 opened: remote per-project container [13:04] thomi: FYI, I just put the Inkscape update into the stable channel. If you want to watch the diffs there. [13:04] thomi: Guessing it'll cause a bunch of traffic. [13:05] Issue # opened: snapcraft#1458, snapcraft#1459, snapcraft#1460, snapcraft#1461, snapcraft#1462 [13:08] Issue snapcraft#1463 opened: core build triggers root [13:11] Issue snapcraft#1464 opened: plugin reorg (rosdep) [13:12] zyga-ubuntu: +1 [13:12] zyga-ubuntu: here is one more :) [13:13] PR snapd#3698 opened: interfaces: convert joystick to common iface [13:17] cachio, oh, looking at your https://github.com/snapcore/snapd/commit/f1cf4755312713f8024a4c29e1b0173dd4a42419 ... [13:18] does that mean there never actually was an issue with resize ? [13:18] or is that unrelated ? [13:19] gary-wzl: lovely, thanks! [13:52] popey: Are you "snapcrafters" ? The discord app won't start because it's out-of-date. [13:54] ogra_, this test was failing because of this [13:54] ok, so not related [13:55] tedg: just bumped it [13:55] building now [13:56] popey: Great, thanks! [13:56] np, thanks for letting us know - https://github.com/snapcrafters/discord/blob/master/snap/snapcraft.yaml fyi [13:57] popey: Cool, is upstream considering taking it on? [13:57] tedg: possibly. [13:59] Issue snapcraft#1465 opened: node classic path issue [bug](https://bugs.launchpad.net/snapcraft/+bug/1706371) [13:59] Issue snapcraft#1466 opened: scriptlets erroring behavior [14:02] tedg: edge has 0.0.2 - fancy testing it? [14:05] Issue snapcraft#1467 opened: rust plugin: new rustup [14:05] Issue snapcraft#1468 opened: override ARCH for kbuild [14:05] Issue snapcraft#1469 opened: rust plugin: cache rust toolchain offline [14:06] zyga-ubuntu: can you ping me once you found out more about the fedora build failure please? [14:09] tedg: nvm, busted here. [14:10] mvo: yes, installing 32 bit build now [14:10] popey: Yeah, seemed to migrate and then stopped. [14:12] tedg: missing opengl plug, added it, and rebuilding [14:21] mvo, I just updated the db results, 32/32 [14:25] ogra_, i found this http://elinux.org/RPi_SD_cards [14:25] ogra_, late [14:25] oh wow [14:26] thats a gigantic table ! [14:26] * ogra_ wonders who tested all these cards [14:26] cachio: heh, I wanted to copy this link into the standup window but not having verified it I didn't want to bother you with unproven stuff [14:26] PR snapd#3695 closed: interfaces: convert kvm to common iface [14:26] mvo: thank you! [14:27] mvo: I'm installing fedora 26 i386 (still) [14:27] mvo: but I'm working on a branch to tweak snap-seccomp builds for everyone to make it easier [14:27] ok [14:27] ogra_, at least there is a name and date in the last column [14:28] yeah [14:28] zyga-ubuntu: fwiw, for 2.27 I would love something *tiny* :) [14:28] mvo: one more data point [14:28] mvo: very tiny [14:28] \o/ [14:28] cachio, so your broken card is actually one marked red in that table ? [14:28] mvo: one line change so far [14:28] well, two [14:28] mvo: if we do monthly releases I can help on xdistro more [14:28] * ogra_ must be extremely lucky that he mostly got good cards then [14:29] zyga-ubuntu: sounds good, I think we need to discuss this, but I think its reasonable [14:29] zyga-ubuntu: moving to this model [14:29] (eh schedule) [14:29] mvo: so far my gut feeling is that +//#cgo CFLAGS: -D_FILE_OFFSET_BITS=64 [14:29] this is *enough* [14:30] but need a sec to test [14:30] ogra_, I think it is in green [14:30] weird [14:30] then it is probably really worn out [14:30] though ... the stable gadget has really old start.elf and friends [14:30] might be that it got fixed later by broadcom but the stable image doesnt have that fix [14:31] (testing edge would tell) [14:31] so zyga-ubuntu i got the initial packaging working, but when i run "/snap/bin/hello", it just seems to time out forever ™ [14:31] ikey: yep, there's a lot of missing bits I suspect [14:31] just using devmode confinement atm [14:31] ogra_, ok [14:32] well, error messages wouldn't hurt :P [14:32] do you have a link? [14:32] ikey: I think it is waiting for the seccomp profile [14:32] ikey: is snapd running? [14:32] yes [14:32] ikey: can you tell me if you patched anything? [14:32] no im using 2.26 [14:32] cachio, for edge images ? [14:32] yes [14:32] ikey: and what kind of installation layout did you pick for solus (e.g. /usr/lib vs /usr/libexec) [14:32] ikey: /snap vs /var/lib/snapd/snap [14:32] cachio, http://people.canonical.com/~ogra/snappy/all-snaps/daily/current/ or http://cdimage.ubuntu.com/ubuntu-core/16/edge/current/ (shoudl be largely identical) [14:32] hm the snap mount dir could be the issue. [14:33] ikey: I *think* you will want to do a few small patches first [14:33] ikey: mark solus as a non-reexec distro for now (cmd/cmd.go) [14:33] cachio, i randomly test the images on the first url (not on a regular schedule or anything though) [14:33] well until i make it do-a-thing i cant know what to patch [14:33] ikey: and use /snap (that's way nicer and simple) [14:33] because i wont know whats broken [14:33] ikey: then look at journal output [14:33] i did [14:33]