| mup | PR snapd#3556 opened: Support snap license field (SPDX expression) <Created by robert-ancell> <https://github.com/snapcore/snapd/pull/3556> | 02:22 |
|---|---|---|
| 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:19 |
| 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> | 04:28 |
| === chihchun_afk is now known as chihchun | ||
| mup | PR snapd#3557 opened: systemd: add snapd.core-fixup.service unit <Created by mvo5> <https://github.com/snapcore/snapd/pull/3557> | 05:22 |
| === chihchun is now known as chihchun_afk | ||
| zyga | oh linode why do you hate me | 06:56 |
| zyga | and qemu, why do you need all the network to run tests :/ | 06:56 |
| mvo | hey zyga, good morning | 06:57 |
| mvo | zyga: could you please push your branch that backports the internaltool use for 2.26? | 06:57 |
| 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:58 |
| 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 | 06:59 |
| zyga | mvo: please ack, I'm not even sure IRC works (I always see lag: $LOTS) in irssi | 07:00 |
| mvo | zyga: ack | 07:03 |
| mvo | zyga: great, I can run tests here | 07:03 |
| zyga | thanks! | 07:03 |
| 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:05 |
| 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> | 07:10 |
| === JanC is now known as Guest1067 | ||
| === JanC_ is now known as JanC | ||
| mvo | zyga: I like the idea, however its out already so kind of difficult to change now | 07:36 |
| mvo | zyga: nevermind about https://github.com/zyga/snapd/tree/backport/2.26/detect-re-exec - I think we merged this already | 07:37 |
| zyga | mvo: +1 to keep as-is | 07:43 |
| mvo | zyga: ta | 07:46 |
| zyga | brb | 07:59 |
| pstolowski | hey guys, any improvement on linode side, or no change? | 08:41 |
| zyga | pstolowski: I tried an hour ago, all failed | 08:42 |
| pstolowski | zyga, ack, thanks | 08:42 |
| mvo | zyga: I pushed a fix for the failing test, I think we are good now | 08:45 |
| mup | PR snapd#3558 opened: cmd: backport fix for internal tool path <Created by mvo5> <https://github.com/snapcore/snapd/pull/3558> | 08:47 |
| zyga | mvo: thank you! let's see | 08:51 |
| 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:52 |
| zyga | ah, great | 08:53 |
| zyga | mvo: ok, so that's all good for 2.26 | 08:53 |
| mvo | zyga: yeah, I run tests locally right now and if things are good will release | 08:57 |
| zyga | sounds good, thank you! | 08:57 |
| mvo | zyga: yes | 08:59 |
| pstolowski | oh noes, 3 pages of PRs.. 56 | 09:14 |
| mvo | pstolowski: well, linode broken | 09:15 |
| mvo | zyga: hrm, hrm, for some reason I cannot push to your branch zyga:backport/2.26/fix-internal-tool-path | 09:21 |
| pstolowski | yeah i know | 09:23 |
| mup | PR snapd#3559 opened: Backport/2.26/fix internal tool path <Created by mvo5> <https://github.com/snapcore/snapd/pull/3559> | 09:24 |
| 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:29 |
| mvo | zyga: couldn't push to your branch, I used my own now | 09:30 |
| zyga | mvo: ah, odd | 09:30 |
| * Chipaca suddenly finds himself recompiling ncurses, and takes a step back | 10:01 | |
| pstolowski | Chipaca, I'll start to get worried when you fork ncurses | 10:05 |
| * Chipaca puts away his cutlery | 10:05 | |
| pstolowski | just don't curse and you'll be fine | 10:07 |
| 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:08 |
| pstolowski | Chipaca, is it important for us? | 10:19 |
| Chipaca | pstolowski: no :-) | 10:19 |
| Chipaca | that's why i stopped | 10:19 |
| pstolowski | :) | 10:19 |
| Chipaca | pstolowski: also, dangit, the 2*len(args)+4 was correct before i split -ojson -nn into -o json -n n | 10:28 |
| Chipaca | needs to be +6 now :-( | 10:29 |
| Chipaca | i shall fix it, and add a comment, at some point before landing this | 10:29 |
| zyga | re :-( | 10:42 |
| * zyga break | 11:11 | |
| 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:18 |
| pstolowski | mvo, are you sure? looking at retry.go we do retry everything >= 500 | 11:21 |
| pstolowski | perhaps we exhausted all retries? | 11:22 |
| zyga | re | 11:24 |
| 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:25 |
| pstolowski | zyga, don't tell me. I feel stupid still fighting json issue :} | 11:26 |
| 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:27 |
| zyga | fgimenez: any luck in that home test? | 11:28 |
| Chipaca | pstolowski: step away from ncurses tho | 11:32 |
| zyga | mvo: anything I can help with to have a change of focus? | 11:32 |
| pstolowski | Chipaca, ah, ok, I misunderstood then | 11:34 |
| mvo | zyga: not right now, running the tests, then I push a new release | 11:35 |
| zyga | ok | 11:35 |
| 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:38 |
| niemeyer | Good morning | 11:39 |
| niemeyer | News from Linode: | 11:40 |
| 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:41 |
| 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:42 |
| 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:43 |
| mvo | niemeyer: \o/ | 11:50 |
| 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:51 |
| niemeyer | Looks promising: https://travis-ci.org/snapcore/snapd/builds/250270516 | 11:55 |
| * mvo crosses fingers | 11:59 | |
| niemeyer | Still going strong | 12:00 |
| 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:28 |
| 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:31 |
| Chipaca | niemeyer: mvo: i'm skipping the standup today -- i need to go to the boys' school | 12:40 |
| 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 :-) | 12:41 |
| === Son_Goku is now known as Conan_Kudo | ||
| === Conan_Kudo is now known as Son_Goku | ||
| 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> | 12:55 |
| * zyga has network issues, trying to reconnect | 13:08 | |
| zyga | mvo: can you see my messages now? | 13:09 |
| zyga | ok, back | 13:09 |
| 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:29 |
| zyga | pedronis: can you repeat what you said on the call? | 13:35 |
| 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:36 |
| 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:37 |
| mvo | fgimenez: almost, one sec | 13:38 |
| mvo | fgimenez: now it is up-to-date | 13:38 |
| fgimenez | mvo: cool thx! | 13:38 |
| 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:39 |
| 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:43 |
| mvo | fgimenez: the output is correct, we skipped 2.26.7 because it was ready before the core-fixup script came up | 13:45 |
| 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:48 |
| 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:50 |
| 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:53 |
| niemeyer | pstolowski: and tagged it for you | 13:54 |
| pstolowski | niemeyer, cool, thanks | 13:54 |
| niemeyer | pstolowski: Thanks for looking into this | 13:54 |
| pstolowski | niemeyer, btw did you get the calendar invitiaion for interfaces stuff? | 13:55 |
| niemeyer | Yep, looks good, thanks | 13:57 |
| 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:12 |
| niemeyer | coreycb: It's just waiting for a second +1 I think.. will look | 14:25 |
| coreycb | niemeyer: ok thanks! | 14:25 |
| * zyga dinner | 14:28 | |
| 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:39 |
| 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:42 |
| zyga | jdstrand: o/ | 14:43 |
| icey | No time flexiondotorg | 14:45 |
| flexiondotorg | OK :-( | 14:45 |
| mvo | fgimenez: I just double checked, 2.24.1 was a autopkgtest fix so not relevant for core | 14:54 |
| 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:56 |
| 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:57 |
| 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:58 |
| 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 | 14:59 |
| kyrofa | niemeyer, any chance you have some time available today to discuss the two snapcraft items from the sprint agenda? | 15:06 |
| niemeyer | kyrofa: Yeah, feel free to pick any open slot in the agenda | 15:07 |
| kyrofa | niemeyer, agenda = your calendar? :P | 15:09 |
| ogra_ | heh | 15:10 |
| cholcombe | flexiondotorg: no i haven't made progress on that | 15:11 |
| flexiondotorg | cholcombe Thanks for the feedback. | 15:11 |
| === cachio is now known as cachio_afk | ||
| niemeyer | kyrofa: Sorry, yeah | 15:15 |
| pstolowski | bah, got the json numbers working | 15:27 |
| 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:28 |
| fgimenez | mvo: great, thanks! | 15:29 |
| Chipaca | is ubuntu-16.04-32:tests/main/create-key being a problem, or have i been unlucky? | 15:32 |
| 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:49 |
| 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:50 |
| 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:51 |
| zyga | mvo: looking | 15:52 |
| Chipaca | mvo: 3557, or 8? | 15:53 |
| mvo | Chipaca: the systemd unit fix thing, 3557 I think | 15:53 |
| mvo | Chipaca: 3558 already got cherry picked when tests were still unhappy | 15:54 |
| Chipaca | ok | 15:54 |
| zyga | mvo: done | 15:56 |
| Chipaca | all the lgtms! | 15:57 |
| Chipaca | :-) | 15:57 |
| 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:05 |
| 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:07 |
| 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:17 |
| 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:22 |
| 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:23 |
| mvo | cachio_afk: if you could address the feedback in 3409, that would be appreciated, then this branch can go in | 16:25 |
| 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:53 |
| mvo | pstolowski: 3451 is dir abstractions - is this the one you have in mind? | 16:55 |
| pstolowski | mvo, sorry, 3541 | 16:56 |
| mvo | pstolowski: unfortunately too late, 2.26.8 is out, so unless this is critical it will go into 2.27 | 16:57 |
| pstolowski | mvo, i see. i don't think it's critical | 16:58 |
| === cachio_afk is now known as cachio | ||
| 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:43 |
| 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 | 17:44 |
| 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:09 |
| coreycb | jdstrand: i've just released that. thanks! | 18:11 |
| jdstrand | cool np | 18:15 |
| 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:02 |
| 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:03 |
| 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:04 |
| 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:08 |
| mvo | zyga: #3518 needs a test update, the failure look real | 19:10 |
| * zyga nods | 19:29 | |
| 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> | 19:34 |
| 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:22 |
| zyga | jdstrand: can you please have a 2nd look at ^ | 20:23 |
| === benedikt_ is now known as Guest21032 | ||
| jdstrand | zyga: ack | 20:38 |
| zyga | jdstrand: thank you | 21:07 |
| jdstrand | zyga: yw | 21:14 |
| 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:09 |
| mup | PR snapcraft#1392 opened: catkin plugin: extract rosdep into new package <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1392> | 23:56 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!