[02:22] <mup> PR snapd#3556 opened: Support snap license field (SPDX expression) <Created by robert-ancell> <https://github.com/snapcore/snapd/pull/3556>
[04:19] <mup> Bug #1563358 changed: snappy can only handle one bootloader (if grub is installed, uboot is completely ignored) <Snappy:Expired> <https://launchpad.net/bugs/1563358>
[04:28] <mup> PR snapcraft#1386 closed: tests: workaround issue that causes failures to download core <Created by elopio> <Merged by elopio> <https://github.com/snapcore/snapcraft/pull/1386>
=== chihchun_afk is now known as chihchun
[05:22] <mup> PR snapd#3557 opened: systemd: add snapd.core-fixup.service unit <Created by mvo5> <https://github.com/snapcore/snapd/pull/3557>
=== chihchun is now known as chihchun_afk
[06:56] <zyga> oh linode why do you hate me
[06:56] <zyga> and qemu, why do you need all the network to run tests :/
[06:57] <mvo> hey zyga, good morning
[06:57] <mvo> zyga: could you please push your branch that backports the internaltool use for 2.26?
[06:58] <zyga> yes
[06:58] <zyga> pushed
[06:58] <zyga> there are three commits
[06:58] <zyga> one is the squash of your branch
[06:58] <zyga> and there are two more that are small backports to make things work
[06:59] <zyga> backport/2.26/fix-internal-tool-path
[06:59] <zyga> I didn't manage to run tests yesterday
[06:59] <zyga> my network at home is super unreliable
[06:59] <zyga> and capping makes things fail all the time
[06:59] <zyga> if you have better connectivity please pull and run locally
[06:59] <zyga> I think one test failure was genuine, it is just measuring a stale reexec message
[07:00] <zyga> mvo: please ack, I'm not even sure IRC works (I always see lag: $LOTS) in irssi
[07:03] <mvo> zyga: ack
[07:03] <mvo> zyga: great, I can run tests here
[07:03] <zyga> thanks!
[07:05] <mvo> zyga: do I need detect-re-exec as well?
[07:05]  * mvo needs to take a short break, doorbell
[07:05] <zyga> mvo: which branch is that?
[07:10] <zyga> mvo: as for https://github.com/snapcore/snapd/pull/3557/files -- how about naming those as fixup-1-ownership-change.after so that we can just have a namespace for the files
[07:10] <mup> PR snapd#3557: systemd: add snapd.core-fixup.service unit <Created by mvo5> <https://github.com/snapcore/snapd/pull/3557>
=== JanC is now known as Guest1067
=== JanC_ is now known as JanC
[07:36] <mvo> zyga: I like the idea, however its out already so kind of difficult to change now
[07:37] <mvo> zyga: nevermind about https://github.com/zyga/snapd/tree/backport/2.26/detect-re-exec - I think we merged this already
[07:43] <zyga> mvo: +1 to keep as-is
[07:46] <mvo> zyga: ta
[07:59] <zyga> brb
[08:41] <pstolowski> hey guys, any improvement on linode side, or no change?
[08:42] <zyga> pstolowski: I tried an hour ago, all failed
[08:42] <pstolowski> zyga, ack, thanks
[08:45] <mvo> zyga: I pushed a fix for the failing test, I think we are good now
[08:47] <mup> PR snapd#3558 opened: cmd: backport fix for internal tool path <Created by mvo5> <https://github.com/snapcore/snapd/pull/3558>
[08:51] <zyga> mvo: thank you! let's see
[08:52] <zyga> mvo: which test was failingt?
[08:52] <mvo> zyga: I run the tests now against the full 2.26
[08:52] <mvo> zyga: it was re-exec and the string of the log messgae that it did not restart has changed
[08:53] <zyga> ah, great
[08:53] <zyga> mvo: ok, so that's all good for 2.26
[08:57] <mvo> zyga: yeah, I run tests locally right now and if things are good will release
[08:57] <zyga> sounds good, thank you!
[08:59] <mvo> zyga: yes
[09:14] <pstolowski> oh noes, 3 pages of PRs.. 56
[09:15] <mvo> pstolowski: well, linode broken
[09:21] <mvo> zyga: hrm, hrm, for some reason I cannot push to your branch zyga:backport/2.26/fix-internal-tool-path
[09:23] <pstolowski> yeah i know
[09:24] <mup> PR snapd#3559 opened: Backport/2.26/fix internal tool path <Created by mvo5> <https://github.com/snapcore/snapd/pull/3559>
[09:29] <zyga> mvo: why two PRS?
[09:29] <mup> PR snapd#3558 closed: cmd: backport fix for internal tool path <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/3558>
[09:30] <mvo> zyga: couldn't push to your branch, I used my own now
[09:30] <zyga> mvo: ah, odd
[10:01]  * Chipaca suddenly finds himself recompiling ncurses, and takes a step back
[10:05] <pstolowski> Chipaca, I'll start to get worried when you fork ncurses
[10:05]  * Chipaca puts away his cutlery
[10:07] <pstolowski> just don't curse and you'll be fine
[10:08] <Chipaca> ncurses on debian has been waiting for a soname change to add support for mouse wheel events for 12 years now
[10:08] <Chipaca> I wonder if at any point it becomes "enough's enough"
[10:19] <pstolowski> Chipaca, is it important for us?
[10:19] <Chipaca> pstolowski: no :-)
[10:19] <Chipaca> that's why i stopped
[10:19] <pstolowski> :)
[10:28] <Chipaca> pstolowski: also, dangit, the 2*len(args)+4 was correct before i split -ojson -nn into -o json -n n
[10:29] <Chipaca> needs to be +6 now :-(
[10:29] <Chipaca> i shall fix it, and add a comment, at some point before landing this
[10:42] <zyga> re :-(
[11:11]  * zyga break
[11:18] <mvo> pstolowski, Chipaca: I just got a 504 from the store - we do not retry those currently, should we? if so, I can prepare a PR
[11:21] <pstolowski> mvo, are you sure? looking at retry.go we do retry everything >= 500
[11:22] <pstolowski> perhaps we exhausted all retries?
[11:24] <zyga> re
[11:25] <zyga> this is a very unproductive and annoying day :/
[11:25] <mvo> pstolowski: aha, maybe thisis just in the 2.26 branch (which I'm testing right now)
[11:25] <pstolowski> mvo, ah, could be, the >= 500 condition was introduced recently. before that we only retried select codes
[11:26] <pstolowski> zyga, don't tell me. I feel stupid still fighting json issue :}
[11:27] <pstolowski> maybe I need to take a step back and look at ncurses too ;)
[11:27] <zyga> pstolowski: I'm fighting weather and network while making no progress on apparmor profile load
[11:28] <zyga> fgimenez: any luck in that home test?
[11:32] <Chipaca> pstolowski: step away from ncurses tho
[11:32] <zyga> mvo: anything I can help with to have a change of focus?
[11:34] <pstolowski> Chipaca, ah, ok, I misunderstood then
[11:35] <mvo> zyga: not right now, running the tests, then I push a new release
[11:35] <zyga> ok
[11:38] <mvo> sorry
[11:38] <mvo> zyga: I guess the work for base-snaps on core (in snap-confine) is already done, right? that might be a nice project
[11:39] <niemeyer> Good morning
[11:40] <niemeyer> News from Linode:
[11:41] <niemeyer> Actually, I won't quote him directly as he was using language for a private conversation
[11:41] <niemeyer> But the point is, apparently their backend corrupted one of our images
[11:42] <niemeyer> Restore underway.. we can try it in a moment
[11:42] <zyga> niemeyer: do you know which one?
[11:42] <pstolowski> hey niemeyer!
[11:43] <pstolowski> great they found out finally
[11:43] <niemeyer> zyga: At least ubuntu-16.04-64 I suppose, which is the one presenting issues all the time
[11:43] <niemeyer> pstolowski: Fingers crossed for now
[11:43] <niemeyer> I don't know what I'll do when this one guy leaves Linode :)
[11:43] <zyga> hehe, I was about to say, murhpy picked the "best" image to corrupt
[11:50] <mvo> niemeyer: \o/
[11:51] <fgimenez> zyga: nope, i've kept retrying but no trace of it, i start from the stable image, sudo snap refresh, reboot and then run the tests, last week the same actions led to the error after about 15 runs, now i can't reproduce it
[11:51] <fgimenez> hey niemeyer, great!
[11:51] <zyga> fgimenez: ok, thank you!
[11:55] <niemeyer> Looks promising: https://travis-ci.org/snapcore/snapd/builds/250270516
[11:59]  * mvo crosses fingers
[12:00] <niemeyer> Still going strong
[12:28] <mup> PR snapcraft#1391 closed: tests: reduce the amount of test code in test_meta <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1391>
[12:31] <mup> PR snapcraft#1389 closed: Use newer distro module with no tuple <Created by kalikiana> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1389>
[12:40] <Chipaca> niemeyer: mvo: i'm skipping the standup today -- i need to go to the boys' school
[12:41] <Chipaca> the only thing different on my front is that i had some fun snapping things and then realising the bugs i was seeing was because of a bug in the distro itself, not in the snap :-)
=== Son_Goku is now known as Conan_Kudo
=== Conan_Kudo is now known as Son_Goku
[12:55] <mup> PR snapd#3560 opened: cmd/snap-repair: implement most logic to get the next repair to run/retry in a brand sequence <Created by pedronis> <https://github.com/snapcore/snapd/pull/3560>
[13:08]  * zyga has network issues, trying to reconnect
[13:09] <zyga> mvo: can you see my messages now?
[13:09] <zyga> ok, back
[13:29] <fgimenez> mvo: i have the beta images already built, just to confirm release/2.26 is up to date for running the tests from it, correct?
[13:35] <zyga> pedronis: can you repeat what you said on the call?
[13:36] <pedronis> zyga: do we know if there was a self-built snapd involved?
[13:36] <pedronis> I know he had a very old one on his path at some point by mistake
[13:37] <pedronis> anyway not sure snapd itself would affect this unless it was a full deb
[13:37] <pedronis> and not just the daemon
[13:37] <zyga> pedronis: good question, I'll ask the reporter
[13:38] <mvo> fgimenez: almost, one sec
[13:38] <mvo> fgimenez: now it is up-to-date
[13:38] <fgimenez> mvo: cool thx!
[13:39] <niemeyer> Chipaca: Thanks for the note
[13:39] <niemeyer> Our Linode pool is clean and seems to be churning at an appropriate pace now
[13:43] <fgimenez> mvo: i see 2.26.8 in snap version's output, that is fine right? iirc we omited the last digit in previous releases
[13:45] <mvo> fgimenez: the output is correct, we skipped 2.26.7 because it was ready before the core-fixup script came up
[13:48] <fgimenez> mvo: ok thanks, but did we always include the 3 digits in the version? we have a 2.24.1 entry in the changelog but the current stable shows 2.24 in snap version
[13:50] <pstolowski> niemeyer, did you mean this topic https://forum.snapcraft.io/t/how-to-snap-get-root-document/522 ?
[13:50] <niemeyer> pstolowski: yes, that looks like the one
[13:50] <fgimenez> mvo: i'm in validation mode now, pls forgive my pickiness :)
[13:53] <mvo> fgimenez: we included the digests before, at least in theory, I need to figure out why changelog/core disagree
[13:53] <mvo> fgimenez: it might be that 2.24.1 was never uploaded as a core snap becasue it contained something that was only relevant on classic (adt fix or something)
[13:53] <fgimenez> mvo: ok great, don't spend too much time on it, just curious
[13:53] <niemeyer> pstolowski: I've added another note there as well on a closely related need
[13:54] <niemeyer> pstolowski: and tagged it for you
[13:54] <pstolowski> niemeyer, cool, thanks
[13:54] <niemeyer> pstolowski: Thanks for looking into this
[13:55] <pstolowski> niemeyer, btw did you get the calendar invitiaion for interfaces stuff?
[13:57] <niemeyer> Yep, looks good, thanks
[14:12] <coreycb> hi all, what's the next step for getting auto-aliases setup?  i have a security ack on the request: https://forum.snapcraft.io/t/auto-aliases-for-openstack-base-snaps/1146
[14:25] <niemeyer> coreycb: It's just waiting for a second +1 I think.. will look
[14:25] <coreycb> niemeyer: ok thanks!
[14:28]  * zyga dinner
[14:39] <mup> PR snapd#3539 closed: tests: fix timeout issue for test refresh core with hanging … <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3539>
[14:42] <jdstrand> coreycb: I'll circle back around in a bit. there is a voting process. I'll gide it
[14:42] <jdstrand> guide*
[14:42] <coreycb> jdstrand: ok thanks
[14:42] <flexiondotorg> icey A while back you mentioned that you a cholcombe were working on a rustup snap. Any progress on that?
[14:43] <zyga> jdstrand: o/
[14:45] <icey> No time flexiondotorg
[14:45] <flexiondotorg> OK :-(
[14:54] <mvo> fgimenez: I just double checked, 2.24.1 was a autopkgtest fix so not relevant for core
[14:56] <fgimenez> mvo: great thanks a lot, btw i just had an error on amd64 in the refresh from stable scenario http://paste.ubuntu.com/25025482/ i have the session open in case you want to have a look
[14:57] <fgimenez> mvo: core went from 1441 (this comes from the stable image) -> 2312 (after the first refresh, still stable) -> 2329 (beta)
[14:57] <mvo> fgimenez: yes, would love to have a look, do you have a ssh session for me?
[14:58] <mvo> fgimenez: if so, please /msg or tg
[14:58] <mvo> fgimenez: looks like snap list --all is affected
[14:58] <fgimenez> mvo: sure 1sec, fwiw the same scenario just finished successfully on i386
[14:58] <mvo> fgimenez: the mount unit for 1441 would be interessting
[14:59] <mvo> fgimenez: I suspect the snap is there but for some reason it is not mounted
[14:59] <fgimenez> mvo: indeed, it shows up in /var/lib/snapd/snaps
[15:06] <kyrofa> niemeyer, any chance you have some time available today to discuss the two snapcraft items from the sprint agenda?
[15:07] <niemeyer> kyrofa: Yeah, feel free to pick any open slot in the agenda
[15:09] <kyrofa> niemeyer, agenda = your calendar? :P
[15:10] <ogra_> heh
[15:11] <cholcombe> flexiondotorg: no i haven't made progress on that
[15:11] <flexiondotorg> cholcombe Thanks for the feedback.
=== cachio is now known as cachio_afk
[15:15] <niemeyer> kyrofa: Sorry, yeah
[15:27] <pstolowski> bah, got the json numbers working
[15:28] <pstolowski> now, i have so many changes in the tree... need to find out which ones are irrelevant to these fixes ;)
[15:28] <mup> PR snapd#3561 opened: tests: store /etc/systemd/system/snap-*core*.mount in snapd-state.tar.gz <Created by mvo5> <https://github.com/snapcore/snapd/pull/3561>
[15:28] <mvo> fgimenez: this is the PR we just talked about -^
[15:29] <fgimenez> mvo: great, thanks!
[15:32] <Chipaca> is ubuntu-16.04-32:tests/main/create-key being a problem, or have i been unlucky?
[15:49] <mup> PR snapd#3512 closed: cmd: avoid using current symlink in InternalToolPath <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3512>
[15:50] <mup> PR snapd#3559 closed:  cmd: backport fix for internal tool path #3558  <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/3559>
[15:51] <mvo> if someone could review 3557 that would be great
[15:51] <mvo> zyga: maybe -^
[15:51] <mvo> then we can restart edge builds today
[15:52] <zyga> mvo: looking
[15:53] <Chipaca> mvo: 3557, or 8?
[15:53] <mvo> Chipaca: the systemd unit fix thing, 3557 I think
[15:54] <mvo> Chipaca: 3558 already got cherry picked when tests were still unhappy
[15:54] <Chipaca> ok
[15:56] <zyga> mvo: done
[15:57] <Chipaca> all the lgtms!
[15:57] <Chipaca> :-)
[16:05] <mup> PR snapd#3557 closed: systemd: add snapd.core-fixup.service unit <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3557>
[16:07] <mup> PR snapd#3562 opened: systemd: add explicit sync to snapd.core-fixup.sh <Created by mvo5> <https://github.com/snapcore/snapd/pull/3562>
[16:17] <mup> PR snapd#3563 opened: release: merge release/2.26 branch back into master <Created by mvo5> <https://github.com/snapcore/snapd/pull/3563>
[16:22] <mup> PR snapd#3546 closed: tests: fix for rng-tools service not restarting <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3546>
[16:23] <mup> PR snapd#3541 closed: snapd: fix for snapctl get panic on null config values <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3541>
[16:25] <mvo> cachio_afk: if you could address the feedback in 3409, that would be appreciated, then this branch can go in
[16:53] <pstolowski> mvo, shall I propose a cherry-picked PR 3451 for 2.26?
[16:53] <mup> PR snapd#3510 closed: tests: shellcheck improvements for nightly upgrade and regressions tests <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3510>
[16:55] <mvo> pstolowski: 3451 is dir abstractions - is this the one you have in mind?
[16:56] <pstolowski> mvo, sorry, 3541
[16:57] <mvo> pstolowski: unfortunately too late, 2.26.8 is out, so unless this is critical it will go into 2.27
[16:58] <pstolowski> mvo, i see. i don't think it's critical
=== cachio_afk is now known as cachio
[17:43] <coreycb> jdstrand: any chance you can publish this for me? https://dashboard.snapcraft.io/dev/snaps/6421/rev/78/   it looks to be blocked by a previous upload that used  kernel-module-control, but has since been dropped.
[17:44] <jdstrand> coreycb: sure thing (fyi, was off yesterday and monday so queue is a little behind (though it was caught up on friday)
[17:44] <jdstrand> )
[17:44] <coreycb> jdstrand: thanks, and no problem at all
[18:09] <jdstrand> coreycb: oh, that's done now. the newer ones are approved but you need to release them. did you want me to release the latest (r78) to edge for you?
[18:11] <coreycb> jdstrand: i've just released that. thanks!
[18:15] <jdstrand> cool np
[19:02] <mup> PR snapd#3538 closed: tests: create ramdisk if it's not present <Created by fgimenez> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3538>
[19:03] <mup> PR snapd#3508 closed: tests: shellcheck improvements for tests/lib scripts <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3508>
[19:04] <mup> PR snapd#3492 closed: cmd/snap: `--last` for abort and watch, and aliases (search→find, change→tasks) <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3492>
[19:08] <mup> PR snapd#3534 closed: tests: shellcheck improvements for tests/main tasks - first set of tests <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3534>
[19:10] <mvo> zyga: #3518 needs a test update, the failure look real
[19:29]  * zyga nods
[19:34] <mup> PR snapd#3518 closed: cmd/snap-confine: various small fixes and tweaks to seccomp support code <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/3518>
[20:22] <mup> PR snapd#3518 opened: cmd/snap-confine: various small fixes and tweaks to seccomp support code <Created by zyga> <https://github.com/snapcore/snapd/pull/3518>
[20:23] <zyga> jdstrand: can you please have a 2nd look at ^
=== benedikt_ is now known as Guest21032
[20:38] <jdstrand> zyga: ack
[21:07] <zyga> jdstrand: thank you
[21:14] <jdstrand> zyga: yw
[23:09] <mup> PR snapd#3550 closed: update firewall-control to allow {arp,ip,ip6}tables to control bridged vlan/ppoe-tagged traffic <Created by coreycb> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/3550>
[23:56] <mup> PR snapcraft#1392 opened: catkin plugin: extract rosdep into new package <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1392>