[00:47] PR snapcraft#1533 closed: demos: update the name of the remote mqtt part [01:44] PR snapcraft#1534 closed: jhbuild plugin: fix UnboundLocalError for chmod_path === chihchun_afk is now known as chihchun [03:37] PR snapd#3872 opened: preserve TMPDIR and HOSTALIASES across snap-confine invocation (LP: #1682308) === chihchun is now known as chihchun_afk [05:54] PR core#56 opened: Remove ~ubuntu* instead of cutting after the first "~" [05:55] PR core#54 closed: Use io.snapcraft.Launcher instead of com.canonical.SafeLauncher [05:57] good morning === chihchun_afk is now known as chihchun [06:09] PR snapd#3873 opened: interfaces: add udev netlink support to hardware-observe [06:10] PR snapd#3874 opened: interfaces: add netlink kobject uevent to hardware observe (2.28) [06:11] PR snapd#3875 opened: interfaces: add udev netlink support to hardware-observe (2.27) [06:12] hey zyga-suse, good morning [06:12] PR snapd#3873 closed: interfaces: add udev netlink support to hardware-observe [06:13] mvo: good morning, thank you for the fixes! [06:13] mvo: I edited the description on github so that it shows up in merge commits [06:13] mvo: today is the only day my kids go to school at 8:00 :/ [06:13] PR snapd#3854 closed: corecfg: mock "systemctl" in all corecfg tests [06:13] mvo: I don't suppose there is any chance of getting the last polkit PR into the 2.28 release? [06:14] jamesh: let me check [06:14] jamesh: has that landed in master already? [06:14] mvo: the only open issue from niemeyer's review was scope of the polkit action ID [06:14] mvo: no [06:15] here it is, for reference: https://github.com/snapcore/snapd/pull/3797 [06:15] PR #3797: daemon: allow polkit authorisation to install/remove snaps [06:15] jamesh: yeah, I think if that makes it before the weekend there is a chance, it seems like its very low risk and the benefit is (imo) huge [06:16] jamesh: could you please nudge niemeyer to do a second pass on 3797? and if it goes in ideally it should be squash-merged in the commit so that it can easily be cherry-picked for 2.28 [06:16] mvo: how is the 2.27.6 release looking? [06:17] mvo: on the forum, niemeyer said he wouldn't be around for the next few days, so I don't know how that affects things [06:18] zyga-suse: waiting for tests right now, i.e. the PR for hardware-observe [06:18] zyga-suse: once that is green I will continue with that [06:18] mvo: do you want to wait for anything else or just release ASAP [06:18] jamesh: meh, indeed, thats slightly sad [06:18] PR snapd#3858 closed: snap-confine,snap-update-ns: add -no-pie to fix FTBFS on ppc64el [06:18] zyga-suse: I am not aware of anything else - are you? [06:18] no, just asking if there's more [06:19] * zyga-suse has bad debian day [06:19] gpg: /home/zyga/snapd_2.27.5-1.dsc: error 58: Invocation of gpgme_op_verify [06:19] what? [06:19] well, there are updates to gpg, let's see [06:21] zyga-suse: yeah, it should be just those two PRs (one from me, one from jamie) [06:21] PR snapd#3773 closed: snap-repair: execute the repair and capture logs/status [06:39] PR snapd#3870 closed: interfaces/network-{control,observe}: allow receiving kobject_uevent() messages for 2.27 [06:43] PR snapd#3857 closed: tests: fix lxd test for external backend [07:05] ikey: hey, I noticed that you ship 2.27.0, do you plan on updating snapd to 2.27.5 or .6 later today? [07:20] FAIL: cmd_watch_test.go:44: SnapSuite.TestCmdWatch [07:20] cmd_watch_test.go:84: [07:20] c.Check(string(buf), testutil.Contains, "\rmy-snap 50.00 KB / 100.00 KB") [07:20] ... container string = "\rmy-snap 0 B / 100.00 KB 0.00%\r\x1b[K" [07:20] ... elem string = "\rmy-snap 50.00 KB / 100.00 KB [07:21] I have seen a couple of times [07:22] PR snapd#3856 closed: overlord: introduce Mock which enables to use Overlord.Settle for settle in many more places [07:24] mo'in [07:24] pedronis: I have as well, that looks like ansi escape [07:25] Chipaca: hey, thank you for the epic feedback :D [07:25] what does? [07:25] zyga-suse: epic feedback? [07:25] launchpad must adopt not only markdown [07:25] but greek statue images [07:25] hehe [07:25] (or was it roman?) [07:25] zyga-suse: https://en.wikipedia.org/wiki/Facepalm [07:26] was going to point you to the image directly, but this way is funnier [07:26] ogra_: you around already? [07:28] zyga-suse: so to answer your actual question, french (turn-of-the-(20th)-century) [07:32] mvo: after I dput and get the ACCEPTED email (I did :), how can I track the package further? [07:39] mvo: remind me, when is the cut-off for 2.29? [07:49] Chipaca: after the rally afaict [07:50] that's nearly two months after 2.28's cut-off [07:50] (granted we weren't very good at cutting off at that particular cut-off) [07:50] ? [07:51] 2.28 was Sept 4 [07:51] oh wait, got my dates wrong [07:51] i thought i read aug 11 [07:51] but it's sep 11 [07:51] that's fine then [07:52] (i'd understood 2.28 to be already forked) [07:52] pedronis: Sep 4? github has it wrong then [07:54] * zyga-suse reads https://github.com/snapcore/snapd/pull/3852/ [07:54] PR #3852: hooks: commands for controlling own services from snapctl [07:56] * zyga-suse looks at https://ci.debian.net/packages/s/snapd/unstable/amd64/ and :/ === LarreaMikel1 is now known as LarreaMikel [08:02] zyga-suse: https://packages.qa.debian.org/s/snapd.html might be a good place [08:02] Chipaca: in ~3 weeks time 2017-10-02 [08:02] mvo: right, I managed to find that now, I wish it was more real-time tho :-) [08:03] mvo: a'ight [08:03] mvo: and 2.28 is already cut, right? [08:03] (just checking) [08:03] Chipaca: correct, fixes and low-risk things only [08:03] zyga-suse: yeah, there is a bit of latency in the debian archive [08:04] * zyga-suse looks at https://bugs.debian.org/cgi-bin/pkgreport.cgi?src=snapd&archive=no&pend-exc=pending-fixed&pend-exc=fixed&pend-exc=done&sev-inc=important&sev-inc=normal&repeatmerged=no [08:04] mvo: s/and low-risk things // [08:04] mvo: :) [08:04] Chipaca: *cough* yes! === LarreaMikel1 is now known as LarreaMikel [08:04] I need to recall how to reply to those bugs [08:04] zyga-suse: email? [08:05] I guess I need to stand on my right leg, curl up my right feet behind my back and pick my nose while typing with my ear [08:05] zyga-suse: or do you mean "lis'n 'ere mate"? [08:05] Chipaca: right but don't forget plain text and the magic first-few-line headers [08:05] zyga-suse: how could I forget [08:05] zyga-suse: pretty much https://wiki.debian.org/HowtoUseBTS [08:05] something that I never knew [08:05] :-D [08:05] * zyga-suse hugs mvo [08:06] * Chipaca actually might have known once, but has forgotten not only the knowledge, but also the knowledge of knowing [08:06] in alternate universe debian has a web-based bug tracker with a "reply" button not using mailto:// an ubuntu is geeky and built from source [08:07] zyga-suse: and we're all using something written by djb as pid 1 [08:07] * zyga-suse stops looking at the BTS and then goes back to stuffing sharp items under his fingernails [08:12] PR snapd#3875 closed: interfaces: add udev netlink support to hardware-observe (2.27) [08:13] https://forum.snapcraft.io/t/introducing-overlord-mock/2028 [08:18] Chipaca: your PR had the funniest of runs, everything failed _except_ xenial-ppc64el [08:18] pedronis: :-D [08:18] ikr [08:18] pedronis: i've got a typo (catalog -> catalogue, apparently) [08:19] but I disagree [08:19] I don't know why that's considered a typo [08:19] it's US american vs UK [08:19] nope [08:19] “In both the US and Canada, both catalog and catalogue are used, with catalogue commonly used in government and traditional institutions and catalog commonly used in informal, business, retail, and computing contexts.” [08:19] it's pretentious [08:20] PR snapd#3876 opened: debian: update trusted account-keys check on 14.04 packaging [08:21] Chipaca: that sentence doesn't say anything about the UK [08:21] PR snapd#3877 opened: debian: update trusted account-keys check on 14.04 packaging [08:21] pedronis: ah, i read you backwards [08:21] why are we checking our spelling against british english agian? [08:21] again?* [08:21] oxford says catalague is UK spelling [08:21] *catalogue [08:22] * Chipaca goes with catalgae [08:22] Chipaca, i am now [08:22] next it's going to object to Color vs Colour [08:22] ogra_: /var/cache/snapd, how much work is it to make it happen? [08:22] ogra_: good morning! had your coffee yet? here have some writable-paths nightmares instead [08:23] Chipaca, one deb upload and an edge rebuild [08:23] ogra_: deb upload + sru, or ppa? [08:23] PPA [08:23] ogra_: any objection to doing it? [08:24] none at all ... apart from "whatch it closely after the change and before committing to beta/candidate" [08:24] -h [08:24] pedronis: if we're using british spelling, then I'm fine with it (my local tools will be happier) [08:24] pedronis: if it's using US spelling, then it's just being pretentious [08:24] ogra_: could you do it, please? [08:25] ogra_: (not urgently) [08:25] Chipaca: I have no clue [08:25] pedronis: check for footprints [08:25] Chipaca, mind if i do it in https://github.com/snapcore/core-build/pull/17 (same change other dir) ? [08:25] PR core-build#17: switch /etc/systemd/system to "synced" mode [08:25] is there an official choice for ubuntu? [08:25] * Chipaca wonders what was in his coffee [08:26] (would you approve that) [08:26] Chipaca: I just discovered I left extra lines in my just merged branch, this by looking at the diff I posted to the forum :/ [08:27] Chhmm, are you sure its /var/cache, not /var/lib ? (i thought you are talking about #3807 ) [08:27] ogra_: I don't mind! and should I approve it before or after the change [08:27] PR #3807: cmd/snap-confine,packaging: import snapd-generated policy [08:27] after [08:27] Is it possible to make a branch from edge? And if so, why? :-) [08:27] pedronis: fun! [08:27] Chipaca: there's always the next PR [08:28] ogra_: /var/cache/snapd, for #3866 [08:28] PR #3866: many: implement fetching sections and package names periodically [08:28] Chipaca: we are using github.com/client9/misspell/cmd/misspell don't know if there's a way to enforce a dictionary [08:28] Chipaca, then i'll do cache and lib in that PR as well, that will help zyga-suse's issue [08:28] at least to avoid getting UK one place and US another [08:29] pedronis: https://github.com/client9/misspell/#locale [08:29] :-) [08:29] Chipaca, i assume you want "synced" as well ? [08:30] ogra_: well, it could be 'temporary' [08:30] (or is "copy readonlöy once and never again" enough ?) [08:30] Chipaca: ok, now pick your poison and change run-checks [08:30] ogra_: but if temporary means in-memory, then maybe something else [08:31] (one of the files that'll land there might be a couple of megs) [08:31] Chipaca, i have two ooptions, one copies the content from the readonly dir once on first mount (transition), the other constantly updates if new stuff shows up in the readonly partition (synced) [08:31] ogra_: you have three :-) what does temporary do? [08:32] well temporary won't survive reboots [08:32] one would think [08:32] Chipaca, its like /run ... [08:32] right, what pedronis said [08:33] it'll be recreated on boot (if there's network), but ok, sure, umm... 'synced'! [08:33] and we could ship a default sections file :-) [08:33] Chipaca, it will be created on boot in any case, not just with network ... but it will come up empty [08:33] ogra_: i meant the files [08:34] ah [08:34] snapd creates them on startup [08:34] ok [08:34] but, make it synced [08:34] will do [08:34] i don't want these on tmpfs on little devices [08:34] and if we feel like it we can ship a pre-loaded cache [08:34] sounds like a win [08:37] Chipaca, ogra_: please no changes to the ppa right now, we need to push a new 2.27.6 [08:37] later is fine, but not just now [08:37] mvo, only doing the PR, i'll wait fo you PR approval there ;) [08:37] ogra_: ta [08:38] Chipaca, hmm, the dir doesnt exist by default, does it ? [08:38] ogra_: currently it doesn't [08:38] ogra_: is that a problem? do we need to change the snapd packaging first? [08:38] right, then i need to create it too (no mountpoint, no bind mount) [08:38] right [08:41] mmg, no change was fine, copy-pasting from terminal issue [08:42] gah, why is spellcheck _still_ not part of the static checks :-( [08:42] zyga-suse: congrats on your upload :) [08:42] shellcheck* [08:42] (also, i should run the static checks locally more often, tut tut) [08:43] meh, gccgo build failure on xenial/powerpc with master: https://launchpadlibrarian.net/336142278/buildlog_ubuntu-xenial-powerpc.snapd_2.27.5+git360.71a180c~ubuntu16.04.1_BUILDING.txt.gz - looks conistent [08:44] Chipaca: should that dir comes from snapd.dir ? [08:44] *dirs [08:45] mvo: yay! [08:45] pedronis: which dir? [08:45] /var/cache/snapd ? [08:45] pedronis: /var/lib/snapd? [08:45] pedronis: i think so yes [08:45] mwhudson: if you have any ideas, I would be very happy. fwiw, I worked around the ppc64el issue succesfully [08:45] mwhudson: thanks for adding the ACLs! [08:45] i can created it from core-config as well if you want ... your choice [08:45] *ceate [08:45] mwhudson: I think there are some bugs to address, some low hanging fruit there [08:46] mwhudson: man pages and such [08:46] mvo: how did you manage that -extldflags -no-pie? [08:46] mvo: well the test is hanging [08:46] dammit, i'd _fixed_ this GOPATH issue :-( [08:46] Chipaca: we need on classic and we need it also on core reexec fwiw [08:47] pedronis: yes [08:47] mwhudson: https://github.com/snapcore/snapd/pull/3858/files is what I did for -no-pie [08:47] PR #3858: snap-confine,snap-update-ns: add -no-pie to fix FTBFS on ppc64el [08:47] stating the obvious just in case [08:47] yeah, then creating it from snapd is the better move [08:47] pedronis: not sure where core reexec comes into it [08:47] so you also need a mkdir [08:47] mwhudson: yeah, it is hanging but apparently only on powerpc (gccgo), I have a look in a bit [08:47] or debian/dirs [08:47] ogra_: both \o/ [08:47] :) [08:47] Chipaca: even if it's dirs or core , you still need to create it on classic because your code might be running before you got the new deb [08:48] mvo: ah i guess that works [08:48] pedronis: ah, yes [08:48] i'll be tweaking that code [08:48] mvo: you can also pass -ldflags=-extldflags=-no-pie to go install [08:48] mvo: my enthusiasm for debugging gccgo issues is, erm, low [08:48] want to have it in dirs, created on boot via initramfs-tools, created on ensure, and skipping the download entirely if unable to create [08:48] i think that covers it enough [08:49] but also, who reverted the GOPATH%%:* change :-( [08:49] * Chipaca curses [08:49] PR snapd#3878 opened: tests: disable opensuse 2.28 as their infrastructure is having issues [08:49] mwhudson: aha, thanks for this hint! [08:49] mwhudson: gccgo> yeah, its pita that we still need to support powerpc [08:50] mvo: i guess i should really SRU the fix i linked into zesty but my enthusiasm for that is not very high either :) [08:50] Chipaca: anyway double checked, seems indeed packages tend to put their /var/cach/pkg dir into .dirs [08:51] PR snapd#3878 closed: tests: disable opensuse 2.28 as their infrastructure is having issues [08:51] pedronis: awesome, thank you for checking that [08:52] pedronis: i think that makes dpkg clean it up on purge [08:52] mwhudson: no worries [08:52] pedronis: https://www.debian.org/doc/manuals/maint-guide/dother.en.html#dirs [08:52] mwhudson: the workaround is ok for now [08:52] PR core#56 closed: Remove ~ubuntu* instead of cutting after the first "~" [08:53] mvo: i don't glean anything very much from that traceback of the hanging test [08:53] it looks like it's just sitting there doing nothing [08:53] but it's hard to be sure of course [08:53] Chipaca: that seems what we want, no? remove on purge? [08:53] pedronis: yes [08:54] so... which english do we use? [08:54] mvo? [08:54] US or UK? is behaviour wrong, or is catalog wrong? :-) [08:59] Chipaca: UK is closer to me [09:00] Chipaca: what is complaining ? spellcheck? [09:00] mvo: misspell, but yes [09:00] for some reason i thought we were using US [09:01] but we've managed to be neutral so far :-) [09:01] except for one word in CONTRIBUTING.md [09:01] :-) [09:01] mwhudson: nevermind, I will dive into it [09:01] I'll change it to Catalogue then [09:02] Chipaca: lets stick with uk and solve the problem via po/en_US.po [09:02] mvo: it's a variable name [09:02] :-) [09:03] *pffff* ok [09:04] interestingly, without telling misspell which locale to use, it seems to use a mixture? [09:04] * Chipaca puzzled [09:05] * Chipaca facepalms [09:06] * Chipaca is dumber by the day [09:06] * Chipaca doesn't get any better by night either [09:07] it wasn't catalogue vs catalog [09:07] it was "cataloge" [09:07] * Chipaca <- dumbarse [09:08] * mvo hugs Chipaca === JoshStrobl is now known as JoshStrobl|zzz [09:09] can we add shellcheck from zesty to our ppa? [09:12] why to the PPA ? [09:19] ogra_: how else do you get shellcehck from zesty everywhere? [09:20] i mean everywhere xenialish [09:20] although i guess we could just make sure we run the static checks on zesty [09:20] or artful even [09:22] Chipaca, where exactly do you want to run shellcheck ? [09:23] ogra_: if shellcheck is installed, the static checks use it [09:23] (do we have anything that has the PPA enabled where we use it ?) [09:23] ogra_: but maybe running it on artful is good enough [09:24] what i mean is that typically our travis tests run shellcheck already ... before a package hits a PPA [09:26] ogra_: our travis checks don't run shellcheck [09:27] ogra_: if they did, they'd be red [09:27] oh [09:27] mine all do :) === JanC is now known as Guest49554 === JanC_ is now known as JanC [09:42] ogra_: I left one comment on the core PR [09:43] zyga-suse, already answred ;) [09:43] thanks! [09:45] Chipaca: are you looking into classic revert now ? [09:45] pedronis: yes [09:46] Chipaca: did you see #3852 ? [09:46] PR #3852: hooks: commands for controlling own services from snapctl [09:47] pedronis: uh, yes [09:47] didn't i submit that review? [09:47] i'll look again in a sec [09:47] you commented but I don't see a code review there [09:48] from you [09:49] mvo: did you see the comment from jdstran-d about adding bind to harward-observe ? [09:52] pedronis: yes, I think I did that, let me double check [09:52] you did [09:52] github confused me [09:52] pedronis: my bad, I force pushed [09:52] pedronis: to make the cherry-pick easier [09:53] pedronis: sorry for that, it was an exception [09:53] yea, now I noticed [09:54] mvo: any reason not to merge #3864 ? [09:54] PR #3864: interfaces: add udev netlink support to hardware-observe [09:55] pedronis: thanks, merged now [09:55] PR snapd#3864 closed: interfaces: add udev netlink support to hardware-observe [09:57] PR snapd#3867 closed: interfaces/network-{control,observe}: allow receiving kobject_uevent() messages [09:58] PR snapd#3879 opened: release: merge release 2.27.6 [09:59] mvo: now that we run unit tess on more places, SnapSuite.TestCmdWatch seems to fail randomly but often enough to be annoying, haven't looked at it yet [09:59] *tests [10:01] pedronis: thanks, I have no looked either yet, there are some build issues in the edge ppa that I wanted to look at next, once these are under control I can have a look (until you beat me to it of course :) [10:01] s/until/unless/ [10:02] * zyga-suse needs to pick up his daughter in 10 minutes [10:02] why does it have to rain all week :/ [10:03] PR snapd#3880 opened: tests: add trivial canonical-livepatch test to ensure we do not break it [10:03] mvo: I looked at it briefly now, I don't understand the failure mode at all [10:04] might also be different versions of pb ? [10:04] pedronis: could be, we recently changed the import [10:05] debian-unstable seems one place where it fails [10:05]