/srv/irclogs.ubuntu.com/2021/03/10/#snappy.txt

mupPR snapd#10022 opened: boot, o/devicestate: split makeBootable20 into two parts  <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10022>02:36
mupPR snapd#10023 opened: interfaces: make cups interface implicit on classic <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/10023>03:36
mborzeckimorning07:01
mborzeckimvo: hey07:34
mvohey mborzecki07:35
mupPR 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
mupPR 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
ograhah, that first one was long overdue ...07:42
mborzeckipstolowski: heya08:05
pstolowskihey!08:05
mvogood morning pstolowski08:10
pstolowskihey mborzecki, mvo !08:10
=== bboozzoo[m] is now known as mborzecki
mupBug #1918408 opened: Allow the installation of unasserted kernel snaps <Snappy:New> <https://launchpad.net/bugs/1918408>09:38
mupPR snapcraft#3470 opened: ci: run spread store tests when secret is available <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3470>10:26
mupPR 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>10:37
mborzeckisomewhat nice progress on recovery systems, the seed gets created now, but grub does not boot, as i forgot about grubenv for the recovery system11:47
mupPR snapd#10026 opened: tests: use retry tool instead a loops <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10026>11:48
pedronismborzecki: hi, could you review  https://github.com/snapcore/snapd/pull/10018/files it's a revert of a PR you reviewed12:47
mupPR #10018: boot: revert #10009 <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10018>12:47
mborzeckipedronis: sure, i actually wanted to take a look at https://github.com/snapcore/snapd/pull/10022 first to understand some background12:48
mupPR #10022: boot, o/devicestate: split makeBootable20 into two parts  <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10022>12:48
zygatest13:18
mborzeckizyga: testing you back13:27
zygacool, just checking if this works :)13:34
mupPR snapd#10018 closed: boot: revert #10009 <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10018>13:43
mborzeckipedronis: wdyt about https://github.com/snapcore/snapd/pull/10016#discussion_r591508255 ?13:45
mupPR #10016:  boot: helper for inspecting the outcome of a recovery system try  <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10016>13:45
mupPR snapd#10027 opened: snap-seccomp: add new `close_range` syscall <Created by mvo5> <https://github.com/snapcore/snapd/pull/10027>13:53
* diddledan posts it in here, too (was in #snapcraft) https://gitlab.com/diddledan/snapcraft-multiarch-gitlab-ci14:39
mupPR snapcraft#3471 opened: Add flutter-stable and -beta extension variants <Created by MarcusTomlinson> <https://github.com/snapcore/snapcraft/pull/3471>14:46
pedronismborzecki: I answered now14:54
mborzeckipedronis: thanks!14:57
* cachio lunch15:09
mupPR snapcraft#3465 closed: [backport] store: improve platform detection (#2931) <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3465>15:26
mupPR snapd#10027 closed: snap-seccomp: add new `close_range` syscall <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10027>15:38
dokomvo: please drop the dh-systemd b-d for snapd in hirsute, it's gone15:40
mvodoko: in a meeting right now but I put it on my todo15:41
mvopstolowski: put one quick question into the 10007 - maybe I'm confused, if so sorry16:03
pstolowskimvo: you're right, the test was missing a check, thanks!16:21
mvopstolowski: the spread test also seems to not panic16:22
pstolowskimvo: this check is an extra measure16:22
pstolowskimvo: the problem is prevented at higher levels. so if you remove just this, the spread test would still pass16:23
pstolowskimvo: see marshalSnapConfig16:24
mvopstolowski: thanks!16:25
pstolowskimvo: SetSnapConfig has this extra measure in case anyone ever calls SetSnapConfig from other places16:25
* mvo nods16:29
=== Savicq is now known as Saviq
mupPR 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>17:59
mvothanks for this pedronis18:03
=== 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
mupPR 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>20:37
mupPR snapcraft#3466 closed: store: do not unnecessarily catch/rewrite exceptions <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3466>21:47
mupPR snapcraft#3472 opened: storeapi: move http client and auth to http_clients package <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3472>21:52
=== popey3 is now known as popey
mupPR 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:20
ijohnsonNice thanks jdstrand_ !23:21
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 me23:23
=== jdstrand_ is now known as jdstrand
jdstrandijohnson: 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:25
ijohnsonjdstrand_: 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 me23:30
jdstrandheh :)23:31
jdstrandtechalchemy: fyi, I think you'll be interested in ^23:31
mwhudsoni'd like to move the go snap to core20, but it has a custom plugin23:53
mwhudsonare there docs on how to make a local v2 plugin?23:53
ijohnsonmwhudson: 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 things23:55
ijohnsonso you at least wouldn't be alone23:55
mwhudsonijohnson: got an example ?23:55
mwhudsoni am currently grepping around snapcraft source, this isn't very efficient :)23:56
ijohnsonmwhudson the cert team for one has a checkbox snap, you could try reaching out to them, also ondra has one too23:56
ijohnsonBoth of those folks ran into the problem if I'm remembering correctly23:57
mwhudson        # v2 requires plugin implementation to be named "PluginImpl".23:57
mwhudsonah ha23:57

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!