| mborzecki | morning | 06:05 |
|---|---|---|
| === jwheare_ is now known as jwheare | ||
| pstolowski | morning | 06:57 |
| zyga | good morning :-) | 06:59 |
| mborzecki | pstolowski: zyga: morning guys | 06:59 |
| zyga | I've spent some time last night booting a raspberry pi without an SD card or any other storage attached | 07:01 |
| zyga | it's actually pretty easy but I need to tweak one thing to move the idea from an older pi I had on hand to a pi4 | 07:02 |
| mborzecki | mvo: morning | 07:15 |
| zyga | hey mvo | 07:17 |
| mvo | hey mborzecki and zyga ! | 07:18 |
| mvo | is it just me or did the "indicator" color change in GH that a PR has new comments or other activity? | 07:18 |
| zyga | there are three new themes | 07:19 |
| mvo | maybe sometihng is wrong on my system but with the default (light) I can't really see which PR has changes anymore in the /pulls overview page | 07:20 |
| mborzecki | mvo: where's that indicator? in notifications? | 07:25 |
| mvo | mborzecki: it used to be a little colored strip/bar next to each PR line, left of the checkbox, iirc some blueish color if there were new comments on the pr | 07:27 |
| mvo | and I don't see this anymore | 07:28 |
| mvo | and wonder what happend | 07:28 |
| mvo | I don't even know what it's called :/ | 07:29 |
| mvo | mborzecki: it seems we can land 10170 once the error is updated, is that right? | 07:31 |
| mborzecki | yes | 07:32 |
| mborzecki | mvo: i'm slowly going through #10164, will update 10170 next | 07:32 |
| mup | Bug #10164: Wrong shortcut path <gnome-panel (Ubuntu):Fix Released by seb128> <https://launchpad.net/bugs/10164> | 07:32 |
| mup | PR #10164: o/servicestate/servicemgr.go: add ensure loop for snap service units <Bug> <Squash-merge> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10164> | 07:32 |
| mvo | mborzecki: \o/ no worries, just wanted to check | 07:32 |
| mup | PR core20#102 opened: hooks: add mDNS hostname resolution support <Created by jhenstridge> <https://github.com/snapcore/core20/pull/102> | 08:10 |
| pstolowski | mvo: thanks for looking at refreh-control; if you have time then https://github.com/snapcore/snapd/pull/10053 needs 2nd review | 09:24 |
| mup | PR #10053: o/snapstate: helper for getting snaps affected by refresh, define new hook <Needs Samuele review> <Refresh control> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10053> | 09:24 |
| mvo | pstolowski: sure, let me look | 09:56 |
| pstolowski | ty | 09:57 |
| mborzecki | mvo: can you land https://github.com/snapcore/snapd/pull/10170 ? | 10:00 |
| mup | PR #10170: gadget: multi line support in gadget's cmdline file <Run nested> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10170> | 10:00 |
| mborzecki | the spread failures are unrelated to the change | 10:00 |
| mvo | mborzecki: sure | 10:11 |
| mup | PR snapd#10170 closed: gadget: multi line support in gadget's cmdline file <Run nested> <Created by bboozzoo> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/10170> | 10:11 |
| mvo | mborzecki: heh, too late :) | 10:11 |
| mborzecki | mvo: pedronis: thanks | 10:11 |
| mup | PR snapd#10176 closed: wrappers: support in EnsureSnapServices a callback to observe changes <Services ⚙️> <Squash-merge> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/10176> | 10:51 |
| mborzecki | quick errand, back in 30 | 11:04 |
| === sil2100_ is now known as sil2100 | ||
| mborzecki | re | 11:28 |
| mborzecki | as a side note, compressing images with xz takes almost 2 minutes, since those are built as sparse files, the actual size is 199MB, but compressed with xz is 195MB :/ | 11:43 |
| mup | PR snapd#10178 opened: tests/lib/nested: do not compress images, return early when restored from pristine image <Run nested> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10178> | 11:56 |
| === JanC is now known as Guest65907 | ||
| === JanC_ is now known as JanC | ||
| mvo | pedronis: fwiw, I addressed feedback for hooks v2 pr in 10168, hope I did not overlook anything there | 12:43 |
| pedronis | mvo: I made more comments | 12:44 |
| pedronis | I think | 12:44 |
| pedronis | I asked also a question in a PM | 12:44 |
| mvo | pedronis: \o/ thanks, looking | 12:46 |
| mvo | pstolowski: 10167 has a gofmt error | 12:47 |
| mvo | pedronis: pm on mattermost or irc? I don't see this one right now | 12:47 |
| pstolowski | mvo: oh, fixing, thanks | 12:47 |
| pedronis | mvo: mm | 12:47 |
| mup | PR snapd#10053 closed: o/snapstate: helper for getting snaps affected by refresh, define new hook <Needs Samuele review> <Refresh control> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/10053> | 13:42 |
| mup | PR snapcraft#3507 opened: storeapi: add classes for validation sets <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3507> | 14:26 |
| cachio__ | mborzecki, hi, could you please take a quick look to #10137 | 14:33 |
| mup | Bug #10137: ted: new changes from Debian require merging <Ubuntu:Fix Released by cjwatson> <https://launchpad.net/bugs/10137> | 14:33 |
| mup | PR #10137: tests: update os.query check to match new bullseye codename used on sid images <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10137> | 14:33 |
| pedronis | pstolowski: I reviewed https://github.com/snapcore/snapd/pull/10169 hope the comments make some sense | 15:09 |
| mup | PR #10169: o/snapstate: helper for creating gate-auto-refresh hooks <Needs Samuele review> <Refresh control> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10169> | 15:09 |
| pstolowski | pedronis: thanks, looking | 15:10 |
| pedronis | mborzecki: pstolowski: https://github.com/snapcore/snapd/pull/10122 needs a 2nd review, the changes are the last 3 commits, they are all test changes | 15:13 |
| mup | PR #10122: daemon: refine access testing <Created by pedronis> <https://github.com/snapcore/snapd/pull/10122> | 15:13 |
| pstolowski | pedronis: register for gate-auto-refresh already landed although it uses the generic handler | 15:15 |
| pedronis | pstolowski: we'll have to change that | 15:15 |
| pedronis | I think | 15:16 |
| pedronis | the spec has precise details what to do on errors | 15:17 |
| pstolowski | pedronis: indeed! i missed that. i'd address this in a followup though | 15:18 |
| pedronis | that's fine | 15:18 |
| mup | PR snapd#10137 closed: tests: update os.query check to match new bullseye codename used on sid images <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10137> | 15:32 |
| pedronis | mborzecki: thanks | 15:34 |
| cachio__ | mborzecki, hey, any idea if it is a problem related to a new version of glibc, https://paste.ubuntu.com/p/mHpktWNVzj/ ? | 15:35 |
| cachio__ | it is happening in the new image of centos8 | 15:36 |
| ijohnson | $ systemctl show --property=KernelTimestamp | 15:39 |
| ijohnson | KernelTimestamp=Thu 1970-01-01 00:00:00 UTC | 15:39 |
| ijohnson | pedronis: so yeah as expected on the pi getting the boot time will not be super useful | 15:39 |
| mup | PR snapd#10174 closed: boot: split out a helper for making recovery system bootable <Run nested> <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10174> | 15:47 |
| ijohnson | hmm though `uptime -s` is returning something sensible, I suppose I can just use that instead | 15:54 |
| * cachio__ lunch | 16:48 | |
| mup | PR snapcraft#3507 closed: storeapi: add classes for validation sets <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3507> | 17:37 |
| ijohnson | cachio__: do you know where the source for test-snapd-curl is ? | 19:32 |
| mup | PR snapcraft#3508 opened: storeapi: add binding for validations-sets <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3508> | 19:47 |
| === ogra_ is now known as Guest66031 | ||
| === ogra_ is now known as Guest75940 | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!