[06:05] <mborzecki> morning
=== jwheare_ is now known as jwheare
[06:57] <pstolowski> morning
[06:59] <zyga> good morning :-)
[06:59] <mborzecki> pstolowski: zyga: morning guys
[07:01] <zyga> I've spent some time last night booting a raspberry pi without an SD card or any other storage attached
[07:02] <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:15] <mborzecki> mvo: morning
[07:17] <zyga> hey mvo
[07:18] <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:19] <zyga> there are three new themes
[07:20] <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:25] <mborzecki> mvo: where's that indicator? in notifications?
[07:27] <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:28] <mvo> and I don't see this anymore
[07:28] <mvo> and wonder what happend
[07:29] <mvo> I don't even know what it's called :/
[07:31] <mvo> mborzecki: it seems we can land 10170 once the error is updated, is that right?
[07:32] <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
[08:10] <mup> PR core20#102 opened: hooks: add mDNS hostname resolution support <Created by jhenstridge> <https://github.com/snapcore/core20/pull/102>
[09:24] <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:56] <mvo> pstolowski: sure, let me look
[09:57] <pstolowski> ty
[10:00] <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:11] <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:51] <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>
[11:04] <mborzecki> quick errand, back in 30
=== sil2100_ is now known as sil2100
[11:28] <mborzecki> re
[11:43] <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:56] <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>
=== JanC is now known as Guest65907
=== JanC_ is now known as JanC
[12:43] <mvo> pedronis: fwiw, I addressed feedback for hooks v2 pr in 10168, hope I did not overlook anything there
[12:44] <pedronis> mvo: I made more comments
[12:44] <pedronis> I think
[12:44] <pedronis> I asked also a question in a PM
[12:46] <mvo> pedronis: \o/ thanks, looking
[12:47] <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
[13:42] <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>
[14:26] <mup> PR snapcraft#3507 opened: storeapi: add classes for validation sets <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3507>
[14:33] <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>
[15:09] <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:10] <pstolowski> pedronis: thanks, looking
[15:13] <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:15] <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:16] <pedronis> I think
[15:17] <pedronis> the spec has precise details what to do on errors
[15:18] <pstolowski> pedronis: indeed! i missed that. i'd address this in a followup though
[15:18] <pedronis> that's fine
[15:32] <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:34] <pedronis> mborzecki: thanks
[15:35] <cachio__> mborzecki, hey, any idea if it is a problem related to a new version of glibc, https://paste.ubuntu.com/p/mHpktWNVzj/ ?
[15:36] <cachio__> it is happening in the new image of centos8
[15:39] <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:47] <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:54] <ijohnson> hmm though `uptime -s` is returning something sensible, I suppose I can just use that instead
[16:48]  * cachio__ lunch
[17:37] <mup> PR snapcraft#3507 closed: storeapi: add classes for validation sets <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3507>
[19:32] <ijohnson> cachio__: do you know where the source for test-snapd-curl is ?
[19:47] <mup> PR snapcraft#3508 opened: storeapi: add binding for validations-sets <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3508>
=== ogra_ is now known as Guest66031
=== ogra_ is now known as Guest75940