[02:36] <mup> PR snapd#10022 opened: boot, o/devicestate: split makeBootable20 into two parts  <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10022>
[03:36] <mup> PR snapd#10023 opened: interfaces: make cups interface implicit on classic <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/10023>
[07:01] <mborzecki> morning
[07:34] <mborzecki> mvo: hey
[07:35] <mvo> hey mborzecki
[07:42] <mup> PR snapd#10017 closed: gadget: remove `device-tree{,-origin}` from gadget tests <Simple 😃> <Skip spread> <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10017>
[07:42] <mup> PR snapd#10024 opened: o/hookstate/ctlcmd: add a new exit code for is-connected for cases when the peer is from the same snap <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/10024>
[07:42] <ogra> hah, that first one was long overdue ...
[08:05] <mborzecki> pstolowski: heya
[08:05] <pstolowski> hey!
[08:10] <mvo> good morning pstolowski
[08:10] <pstolowski> hey mborzecki, mvo !
=== bboozzoo[m] is now known as mborzecki
[09:38] <mup> Bug #1918408 opened: Allow the installation of unasserted kernel snaps <Snappy:New> <https://launchpad.net/bugs/1918408>
[10:26] <mup> PR snapcraft#3470 opened: ci: run spread store tests when secret is available <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3470>
[10:37] <mup> PR snapd#10025 opened: api: local assertion fallback when it's not in the store <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10025>
[11:47] <mborzecki> somewhat nice progress on recovery systems, the seed gets created now, but grub does not boot, as i forgot about grubenv for the recovery system
[11:48] <mup> PR snapd#10026 opened: tests: use retry tool instead a loops <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10026>
[12:47] <pedronis> mborzecki: hi, could you review  https://github.com/snapcore/snapd/pull/10018/files it's a revert of a PR you reviewed
[12:47] <mup> PR #10018: boot: revert #10009 <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10018>
[12:48] <mborzecki> pedronis: sure, i actually wanted to take a look at https://github.com/snapcore/snapd/pull/10022 first to understand some background
[12:48] <mup> PR #10022: boot, o/devicestate: split makeBootable20 into two parts  <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10022>
[13:18] <zyga> test
[13:27] <mborzecki> zyga: testing you back
[13:34] <zyga> cool, just checking if this works :)
[13:43] <mup> PR snapd#10018 closed: boot: revert #10009 <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10018>
[13:45] <mborzecki> pedronis: wdyt about https://github.com/snapcore/snapd/pull/10016#discussion_r591508255 ?
[13:45] <mup> PR #10016:  boot: helper for inspecting the outcome of a recovery system try  <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10016>
[13:53] <mup> PR snapd#10027 opened: snap-seccomp: add new `close_range` syscall <Created by mvo5> <https://github.com/snapcore/snapd/pull/10027>
[14:39]  * diddledan posts it in here, too (was in #snapcraft) https://gitlab.com/diddledan/snapcraft-multiarch-gitlab-ci
[14:46] <mup> PR snapcraft#3471 opened: Add flutter-stable and -beta extension variants <Created by MarcusTomlinson> <https://github.com/snapcore/snapcraft/pull/3471>
[14:54] <pedronis> mborzecki: I answered now
[14:57] <mborzecki> pedronis: thanks!
[15:09]  * cachio lunch
[15:26] <mup> PR snapcraft#3465 closed: [backport] store: improve platform detection (#2931) <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3465>
[15:38] <mup> PR snapd#10027 closed: snap-seccomp: add new `close_range` syscall <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10027>
[15:40] <doko> mvo: please drop the dh-systemd b-d for snapd in hirsute, it's gone
[15:41] <mvo> doko: in a meeting right now but I put it on my todo
[16:03] <mvo> pstolowski: put one quick question into the 10007 - maybe I'm confused, if so sorry
[16:21] <pstolowski> mvo: you're right, the test was missing a check, thanks!
[16:22] <mvo> pstolowski: the spread test also seems to not panic
[16:22] <pstolowski> mvo: this check is an extra measure
[16:23] <pstolowski> mvo: the problem is prevented at higher levels. so if you remove just this, the spread test would still pass
[16:24] <pstolowski> mvo: see marshalSnapConfig
[16:25] <mvo> pstolowski: thanks!
[16:25] <pstolowski> mvo: SetSnapConfig has this extra measure in case anyone ever calls SetSnapConfig from other places
[16:29]  * mvo nods
=== Savicq is now known as Saviq
[17:59] <mup> PR snapd#10028 opened: daemon:  move out /v2/login|logout and errToResponse tests from api_test.go <Created by pedronis> <https://github.com/snapcore/snapd/pull/10028>
[18:03] <mvo> thanks for this pedronis
=== ijohnson is now known as ijohnson|lunch
=== ijohnson|lunch is now known as ijohnson
=== JmaJeremy0 is now known as JmaJeremy
=== not_phunyguy is now known as phunyguy
[20:37] <mup> PR snapcraft#3470 closed: ci: run spread store tests when secret is available <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3470>
[21:47] <mup> PR snapcraft#3466 closed: store: do not unnecessarily catch/rewrite exceptions <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3466>
[21:52] <mup> PR snapcraft#3472 opened: storeapi: move http client and auth to http_clients package <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3472>
=== popey3 is now known as popey
[23:20] <mup> PR snapd#10029 opened: RFC: introduce alternate snap-confine for classic exec transitions (LP: #1849753) <Created by jdstrand> <https://github.com/snapcore/snapd/pull/10029>
[23:21] <ijohnson> Nice thanks jdstrand_ !
[23:23] <jdstrand_> ijohnson: I didn't realize quite how painful that was until I started using the go snap in earnest with vim-go. It's been slowly killing me
=== jdstrand_ is now known as jdstrand
[23:25] <jdstrand> ijohnson: it's too bad I was never able to find the time before, but at least now there is code to demonstrate the approach I detailed in the bug :)
[23:30] <ijohnson> jdstrand_: Yah, I just recently started using vscode as a snap, and was running into the same thing and ended up writing a super hacky wrapper script around Go which closes and redirects all the file descriptors which works around the problem for me
[23:31] <jdstrand> heh :)
[23:31] <jdstrand> techalchemy: fyi, I think you'll be interested in ^
[23:53] <mwhudson> i'd like to move the go snap to core20, but it has a custom plugin
[23:53] <mwhudson> are there docs on how to make a local v2 plugin?
[23:55] <ijohnson> mwhudson: there are not docs, but there's a good number of folks who have tried to do the same thing of moving their custom snapcraft plugin to core20 and had to adjust a good amount of things
[23:55] <ijohnson> so you at least wouldn't be alone
[23:55] <mwhudson> ijohnson: got an example ?
[23:56] <mwhudson> i am currently grepping around snapcraft source, this isn't very efficient :)
[23:56] <ijohnson> mwhudson the cert team for one has a checkbox snap, you could try reaching out to them, also ondra has one too
[23:57] <ijohnson> Both of those folks ran into the problem if I'm remembering correctly
[23:57] <mwhudson>         # v2 requires plugin implementation to be named "PluginImpl".
[23:57] <mwhudson> ah ha