[00:25] <mup> PR snapd#9673 closed: bootloader/many: rm ConfigFile, add Present for indicating presence of bloader <UC20> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9673>
[06:12] <mup> PR snapd#9685 closed: tests: run tests against 2.48 for the SRU <⛔ Blocked> <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/9685>
[06:37] <mup> PR snapd#9686 opened: tests: remove workarounds that add "ubuntu-save" if missing <Run nested> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9686>
[07:01] <mborzecki> morning
[07:02] <zyga> good morning
[07:24] <mvo> good morning zyga and mborzecki
[07:25] <zyga> hey :)
[07:48] <mborzecki> mvo: hey
[07:49] <mborzecki> 18.04 on my in-laws computer went crazy and was showing year 2037, ofc all https traffic was dead, systemd-timedated was not able to set the date from ntp either
[07:50] <mborzecki> probably the cmos battery is dying slowly
[07:56] <mvo> mborzecki: woah
[08:01] <pstolowski> morning
[08:06] <mborzecki> pstolowski: hey hey
[08:10] <mvo> pstolowski: good morning
[08:17] <zyga-mbp> hey mvo, I will need sponsorship in debian
[08:17] <zyga-mbp> mvo I'd like to maintain spread there
[08:18] <mvo> zyga-mbp: sure, happy to do that
[08:18] <zyga-mbp> I'll make a few more changes so that there's a sid-happy and focal happy version
[08:18] <zyga-mbp> and will ping you
[08:28] <mborzecki> mvo: i've added blocked label to #9686, it appears the gadget in 20/edge does not have ubuntu-save yet
[08:28] <mup> PR #9686: tests: remove workarounds that add "ubuntu-save" if missing <Run nested> <⛔ Blocked> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9686>
[08:38] <pstolowski> woot, we don't run nested/manual on 20.10
[09:08] <mup> PR snapd#8943 closed: wrappers: generate D-Bus service activation files <Created by jhenstridge> <Merged by jhenstridge> <https://github.com/snapcore/snapd/pull/8943>
[09:08] <mup> PR snapd#9687 opened: osutil: add helper for getting the kernel command line <Simple 😃> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9687>
[09:13] <mup> PR snapd#9688 opened: boot: add kernel command lines to the modeenv file <Run nested> <Simple 😃> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9688>
[09:28] <mup> PR snapd#9689 opened: boot: add helper for generating candidate kernel lines for recovery system <Simple 😃> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9689>
[09:48] <mup> PR snapd#9690 opened: tests: enable nested on 20.10 <Run nested> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9690>
[09:49] <zyga> mvo there will be a pair of new deps
[10:14]  * pstolowski pstolowski not feeling well (stomach bug), calling it a sick day o/
[10:45] <mvo> zyga: a pair of new deps for what exactly (sorry for the slow reply, was in a meeting)
[11:21] <zyga> mvo: spread package is in https://launchpad.net/~zyga/+archive/ubuntu/oh-tools/+packages
[11:22] <zyga> mvo: when you have time, I'd love some early review
[11:22] <zyga> mvo: it depends on  golang-github-niemeyer-pretty and golang-gopkg-niemeyer-ynext.v3 - also present in the same archive
[11:22] <zyga> I tried to follow the right policy but it's aimed at focal for now
[11:28] <zyga> eh, this constant irc disconnect
[11:30] <niemeyer> For the record, ynext is actually gopkg.in/yaml.v3 by now
[11:31] <zyga> niemeyer oh, that's cool, we should upgrade spread then
[11:31] <niemeyer> Yeah
[11:31] <zyga> niemeyer hi :)
[11:31] <zyga> niemeyer I can send a PR if you have time to land it
[11:32] <niemeyer> It's okay, I'll just pack it with whatever next update takes place.. I need to check if the API changed since then as well
[11:33] <zyga> niemeyer do you plan to release a v3.0.0 as well?
[11:33] <zyga> https://github.com/go-yaml/yaml/releases
[11:33] <zyga> would be good for go modules and versions
[11:34] <niemeyer> Yeah,  I've left it settle for a while so the issues would show up before tagging
[11:34] <niemeyer> Now there are a few (simple, luckily) open issues that I need to look into before tagging
[11:41] <zyga> niemeyer if you ping me when you release the new yaml and the new spread I will make adjustments
[11:43] <zyga> I've packaged the snapshot of yaml.v3 and should be ready to update both spread and yaml.v3 when the time comes
[11:52] <zyga> cachio hey
[11:52] <cachio> zyga, hey
[11:53] <cachio> zyga, how are you doing?
[11:53] <mup> PR snapd#9691 opened: spread: bump delta ref, tweak repacking to make smaller delta archives <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9691>
[11:54] <zyga> cachio: good, making progress :)
[11:54] <zyga> cachio I've packaged spread as a debian package
[11:54] <zyga> cachio if you need it it's at https://launchpad.net/~zyga/+archive/ubuntu/oh-tools/+packages
[11:54] <zyga> I'll work with mvo to push it to debian
[11:54] <cachio> zyga, nice, this is really good
[11:55] <cachio> zyga, too much time downloading that binary
[11:58] <mup> PR snapd#6258 closed: interfaces: support D-Bus activation <:birthday:> <Needs security review> <Created by jhenstridge> <Closed by jhenstridge> <https://github.com/snapcore/snapd/pull/6258>
[12:03] <mup> PR snapd#9692 opened: client,snapctl: add primite support for "stdin" <Squash-merge> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9692>
[12:11] <zyga> cachio the package has built now, it should be published in a few minutes
[12:14] <zyga> and it's there now
[12:18] <zyga> cachio I can install this build on the spread runner box
[12:19] <zyga> will make updates easier
[12:20] <zyga> cc mvo ^
[12:20] <zyga> feel free to nack, I will do that this evening otherwise
[12:22] <cachio> zyga, it is ok for me
[12:22] <cachio> I have more runners using the binary in the local environment
[12:23] <cachio> which I need to update in that case
[12:23] <zyga> cachio do you use focal or a mixture of focal/bionic?
[12:24] <mup> PR snapd#9684 closed: devicestate: support "storage-safety" defaults during install (2.48) <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9684>
[12:24] <cachio> zyga, bionic
[12:24] <zyga> ok
[12:24] <cachio> zyga, got some issues using focal with lxc
[12:24] <cachio> lxd
[12:24] <zyga> oh?
[12:24] <cachio> I am going to migreate that to a new environment soon
[12:24] <zyga> my workers run on focal IIRC
[12:25] <cachio> zyga, in the new environment I'll try using focal
[12:26] <cachio> still trying to fix some networking permissions there
[12:26] <zyga> I've copied spread to bionic as well
[12:26] <zyga> but it's untested
[12:26] <cachio> I can test it today
[12:28] <zyga> cachio I'll import the sources to salsa later so that we can file bugs like normal people
[12:28] <zyga> salsa.debian.org
[12:30] <cachio> zyga, nice, thanks
[13:24] <mup> PR snapd#9693 opened: data/selinux: update the policy to allow operations on non-tmpfs /tmp <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9693>
[15:11] <pedronis> mborzecki: I reviewed #9688
[15:11] <mup> PR #9688: boot: add kernel command lines to the modeenv file <Run nested> <Simple 😃> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9688>
[15:11] <mborzecki> pedronis: thanks
[15:13] <mborzecki> pedronis: i take that the field should also be named CurrentKernelComamndLines then?
[15:13] <pedronis> yes
[15:14] <mborzecki> ack
[15:14]  * cachio lunch
[15:18] <ijohnson> mborzecki: I reviewed 9688 too
[15:45] <mvo> pedronis: I updated 9692, hope I got all the things you suggested
[15:46] <mborzecki> ijohnson: thanks!
[15:52] <mvo> 9670 needs a second review please :)
[15:57] <zyga> mvo, hey
[15:57] <zyga> mvo, do you have a moment to talk about spread
[15:58] <zyga> I wanted to discuss ITP and related paperwork
[15:58] <mvo> zyga: yes
[15:59] <zyga> mvo, so I see that we have two options
[15:59] <zyga> mvo, do ITP dance, rebuild the packages I prepared for sid and have you sponsor
[16:00] <zyga> mvo, or wait for the yaml.v3 changes and release and use that instead of ynext
[16:00] <mvo> zyga: what is needed for yaml.v3?
[16:00] <zyga> mvo, the former can start now, the latter is unclear
[16:00] <mvo> zyga: I slightly prefer (2) because without we will need to do a package for 3next?
[16:00] <zyga> mvo, I think gustavo needs to review some patches for yaml, release it, then spread needs to be updated to use it
[16:00] <mvo> zyga: is there a yaml.v3 pacakge yet?
[16:01] <zyga> I made one today
[16:01] <zyga> all the packages are read and need review and sponsorship
[16:01] <mvo> zyga: with the in-progress v3? that should be fine
[16:01] <zyga> I made both
[16:01] <zyga> ynext and yaml.v3
[16:01] <mvo> zyga: hm, what is the relation of ynext to yaml.v3? I mean, ideally we would not dput ynext because it's extra burden for us and ftp-master
[16:02] <zyga> ynext is precursor to yaml.v3
[16:02] <zyga> I agree
[16:02] <mvo> zyga: could we make spread work with the current yaml.v3 ? is that easy?
[16:02] <zyga> yaml.v3 is not released yet officially from what I understand
[16:02] <zyga> it should be a drop in replacement though
[16:03] <mvo> zyga: then we could push a yaml.v3 pacakge and spread build against this. and once v3 is fully released we push the two packages again but that does not require any work from ftp-master
[16:03] <zyga> there's a similar question for kr-pretty
[16:03] <zyga> there's a fork in gustavo's github that spread is using
[16:03] <zyga> I also packaged that
[16:03] <mvo> zyga: what the difference of that vs gustavos version?
[16:03] <zyga> mvo, in that case, could you review yaml.v3 that I uploaded there
[16:03] <zyga> mvo, some more patches
[16:03] <zyga> it's go, a bit of a wild west perhaps
[16:04] <mvo> zyga: do we need those? mostly asking because any new binary/source pacakge requires that a human with limited time/patience (ftp-master) looks at it. so the least amount of extra work we create the faster it will go :)
[16:04] <pedronis> mvo: re-reviewed
[16:04] <mvo> pedronis: \o/
[16:04] <ijohnson> pedronis: hey I wrote a quick doc for the recovery systems rest api, could you take a look?
[16:04] <ijohnson> I just shared it with you and graham
[16:05] <zyga> mvo, I understand that but I wanted to be true to the source
[16:05] <zyga> I suspect gustavo can answer that in detail
[16:05] <ijohnson> it ocurred to me that actually it's rather complicated to explain what the difference between "do" and "reboot" action are
[16:05] <ijohnson> I wanted to make sure I got it right
[16:05] <ijohnson> hey zyga
[16:05] <zyga> hey ijohnson :)
[16:05] <ijohnson> having fun with spread I see :-)
[16:06] <zyga> wanted to help a little :)
[16:06] <mvo> zyga: sure, sure, jsut asking if for the debian version we could use spread with upstream -pretty and not with the fork
[16:06] <ijohnson> always appreciated :-)
[16:07] <zyga> mvo, I don't know, the answer is "maybe"
[16:07] <zyga> it probably only matters in -vv mode
[16:07] <zyga> packaging is rather simple so if you could do a pass and let me know we can iterate
[16:08] <zyga> mvo, I'd be comfortable with non-forked versions iff those land in spread upstream
[16:08] <mup> PR snapcraft#3386 opened: meta: don't overwrite preconfigured hooks with stubs <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3386>
[16:11] <zyga> mvo, let me know please
[16:11] <zyga> I'll run away now
[16:11] <mvo> zyga: we can start with the yaml.v3 I guess, that should be pretty simple
[16:11] <zyga> mvo, yeah
[16:11] <mvo> zyga: haha, run!
[16:11] <zyga> we can also rev it
[16:11] <mvo> zyga: let's talk when you are back
[16:11] <zyga> when more patches land upstream I'm happy to release it
[16:11] <zyga> ideally if gustavo tags v3.x.y
[16:11] <mvo> zyga: more patches for what?
[16:11] <zyga> mvo, for yaml.v3, I understand there are PRs for him to review
[16:12] <mvo> zyga: it's totally fine to have yaml.v3 3.0~pre1.gitXXYYYZZZ
[16:12] <zyga> but unless those add deps it's super easy to update
[16:12] <zyga> yeah, this is what I packaged now
[16:12] <mvo> zyga: exactly
[16:12] <mvo> zyga: yeah, that should be fine
[16:12] <zyga> v3-0~20221124...
[16:12] <mvo> zyga: +1
[16:12] <zyga> so it's good to go IMO
[16:13] <zyga> mvo, I packaged both spread and humbox in the other package
[16:13] <zyga> go packaging is fairly painless now
[16:14] <zyga> so if you have a moment to look and provide feedback, that would rock
[16:14] <pedronis> ijohnson: I got the share, I'll look at it tomorrow in my morning if that's ok?
[16:14] <zyga> I did the painful copyright file in all cases
[16:14] <ijohnson> pedronis: yes that's ok
[16:14] <ijohnson> pedronis: also I'm working on adding support for i.e. `snap reboot --recover` without the label specified
[16:14] <mvo> zyga: what's the url again? sorry, misplaced
[16:14] <pedronis> ijohnson: I thought we supported some of that already
[16:14] <ijohnson> because that doesn't work, I was just going to make a POST to `/v2/systems` DTRT and figure out the right system to act on
[16:15] <ijohnson> pedronis: no, you have to specify the system
[16:15] <pedronis> mmh
[16:15] <ijohnson> anonymouse67@ubuntu:~$ snap reboot --recover
[16:15] <ijohnson> error: cannot request system reboot: method "POST" not allowed
[16:15] <pedronis> I'm probably misremembering something
[16:15] <ijohnson> I think it was just missed in code review
[16:15] <zyga> mvo, https://launchpad.net/~zyga/+archive/ubuntu/oh-tools/+packages
[16:15] <ijohnson> the client just uses "" as the recovery system when performing the POST
[16:15] <zyga> sorry for not using a more neutral location but it's just practical for me
[16:16] <ijohnson> pedronis: which I think is totally fair to provide "" as the recovery system in the request and let the daemon figure out the right current system
[16:16] <pedronis> ijohnson: sounds we miss a spread tests ?
[16:16] <ijohnson> well we found it in spread tests when trying to use `snap reboot --recover` and I think there's even a TODO about it not working and need to grep `snap recovery`
[16:17] <ijohnson> but yes we should add such a spread test
[16:17] <ijohnson> and/or adapt the current ones to not find the label using `snap recovery`
[16:22] <pedronis> ijohnson: to be clear the intention was to make this work, seems we didn't dot the i, I'm bit surprised/worried that we are missing bits also in devicestate
[16:22] <ijohnson> pedronis: no I don't think we are missing the bits in devicestate
[16:23] <ijohnson> I just made it work by adding the path for POST to /v2/systems to api_systems.go
[16:23] <mvo> zyga: packaging looks fine, homepage is not quite right, see https://paste.ubuntu.com/p/3TQtMMvRRw/ (ignore the changelog diff, just added that for debdiff benefits)
[16:23] <ijohnson> I think it's just that little bit of daemon that 's missing
[16:23] <pedronis> ah, then I misunderstood your comments
[16:23] <ijohnson> sorry yes we are _just_ missing the daemon bit
[16:23] <ijohnson> I will have a PR shortly that uses this after I finish some tests
[16:24] <ijohnson> would be "nice" to have for release since it's a much more clear CLI user story to just do `snap reboot --recover` and have to find the label from `snap recovery` first, etc.
[16:24] <ijohnson> but I don't think it's a blocker
[16:34] <mvo> pedronis: thanks! I updated 9692 to use embedding now
[16:36] <mvo> also 9692 needs a second review now :)
[16:38] <ijohnson> oh hmmm actually maybe devicemgr is doing the wrong thing :-(
[16:59] <ijohnson> ok I think I have it sorted finally, we are missing bits in both devicemgr and in daemon
[17:02] <zyga> mvo, ah, good catch
[17:02] <zyga> mvo, thanks, I'll open an ITP and complete the package
[17:09] <zyga-mbp> mvo let me weak packaging now
[17:10] <diddledan> why are the gnome platform snaps missing from `snap find`?
[17:10] <diddledan> https://www.irccloud.com/pastebin/YN3ycIMs/
[17:11] <ogra> i think thats on purpose
[17:11] <ogra> (forgot the reason though ... but theer probably is one even 🙂 )
[17:13] <diddledan> oic. that means I need to guess which ones are available? ;-p
[17:15] <ogra> diddledan, snapcraft list-extensions
[17:16] <diddledan> that only works for extensions that have stabilised :-)
[17:16] <diddledan> and there's no guarantee that an extension is named exactly the same as their platform snap
[17:17] <diddledan> also. kenvandine pretty please get a desktop extension for core20 out the door 🙏🏻
[17:18] <kenvandine> diddledan: indeed, it's in the works but waiting on some changes in snapcraft from sergio
[17:18] <diddledan> bah :-p
[17:19] <kenvandine> i have a WIP branch for gnome-3-38-2004{,-sdk} but tons of build failures for things using autotools
[17:19] <diddledan> dang
[17:19] <kenvandine> i was working around them but sergio said he would restore some of the stuff that was dropped from the autotools plugin
[17:20] <diddledan> aah. yeah the new v2 plugins are much simpler than their v1 equivalents IIRC
[17:20] <diddledan> that probably means they're doing less fixup
[17:21]  * diddledan makes a note to bother segio ;-p
[17:21] <diddledan> sergio*
[17:24] <kenvandine> yup
[17:24] <diddledan> I'm working on improving snapcraft.io/fakecam by adding hardware acceleration, but cuda is complaining about GCC being too new - I'm hoping when I can make the jump to core20 the newer cuda libraries will be better behaved
[17:35] <mup> PR snapd#9694 opened: o/devicestate,daemon: fix reboot system action to not require a system label <Bug> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9694>
[17:36] <ijohnson> pedronis: ^ is the pr, you were right there was a small bit in devicestate that needed to be fixed in addition to the daemon bits being weird
[17:38] <mup> PR snapcraft#3372 closed: tests: make each static test available as a make target <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3372>
[17:38] <mup> PR snapcraft#3376 closed: tests: add missing mock for rust unit tests <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3376>
[19:55]  * cachio afk
[20:38] <mup> PR snapcraft#3373 closed: ci: migrate static & unit tests from travis to github actions <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3373>
[20:53] <mup> PR snapcraft#3387 opened: ci: migrate snap publishing to github actions <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3387>
[21:01] <mup> PR snapd#9689 closed: boot: add helper for generating candidate kernel lines for recovery system <Simple 😃> <UC20> <Created by bboozzoo> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9689>
[21:13] <mup> PR snapcraft#3388 opened: ci: migrate snap publishing to github actions <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3388>
[21:44] <mup> PR snapcraft#3368 closed: ci: use github actions for cla check <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3368>
[23:26] <mup> PR snapd#9695 opened: bootloader/lk: add support for UC20 lk bootloader with V2 lkenv structs <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9695>
[23:52] <mup> PR snapd#9696 opened: osutil/disks: allow building on mac os <Simple 😃> <Skip spread> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9696>