[07:01] <mborzecki> morning
[07:23] <mborzecki> mvo: hey
[07:23] <mborzecki> mvo: how was you weekend?
[07:23] <mvo> good morning mborzecki ! it was very nice, thank you. how was yours?
[07:24] <mborzecki> mvo: quite busy actually, maybe the next one will be smoother
[07:27] <mvo> mborzecki: heh, hope busy in the good way
[07:49] <mborzecki> ah, the new ineffassings are fixed
[07:56] <zyga> good morning
[08:08] <pstolowski> morning
[08:11] <mborzecki> zyga: pstolowski: hey
[08:11] <mborzecki> errand, back in 30 hopefully
[08:11] <zyga> hey guys
[08:12] <mvo> good morning pstolowski
[08:12] <mvo> hey zyga
[09:16] <mup> PR snapd#9968 closed: cmd/snap-bootstrap/triggerwatch: fix returning wrong errors <UC20> <Created by tsunghanliu> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9968>
[10:14] <mvo> pedronis: I updated 9914 (just fyi, not urgent)
[10:14] <pedronis> mvo: should this be assigned to some other project:  https://bugs.launchpad.net/snapd/+bug/1907056 ? it's really about the pi-kernel
[10:14] <mup> Bug #1907056: Please add meta/kernel.yaml <snapd:New> <snapd (Ubuntu):New> <https://launchpad.net/bugs/1907056>
[10:23] <mvo> pedronis: yes! I need to reassign it, I want things to progress a little bit further with the code but then it really needs to move to gadget/kernel
[10:32] <pedronis> mvo: mborzecki: did we get anywhere on this:  https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1915156
[10:32] <mup> Bug #1915156: sudoers file keeps being tracked as part of snapd <snapd (Ubuntu):Confirmed> <https://launchpad.net/bugs/1915156>
[10:37] <pstolowski> pedronis: hey, can you take a look at https://github.com/snapcore/snapd/pull/9964 ? this is needed to address some of the comments you made about account/account-key in assertstate tests in monitor mode PR
[10:37] <mup> PR #9964: asserts: use Fetcher in AddSequenceToUpdate <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9964>
[10:41] <pedronis> pstolowski: is just the last commit that needs reviewing, right?
[10:43] <pstolowski> pedronis: yes
[10:43] <pedronis> ok, thx, will do after lunch
[10:44] <pstolowski> ty
[10:44] <pstolowski> pedronis: btw, pool.CommitTo(db) in resolvePool(..) doesn't return potential error, is this a bug?
[10:45] <pedronis> pstolowski: no, see the doc comment of CommitTo
[10:46] <pstolowski> pedronis: ah, of course, nvm. thank
[10:46] <pstolowski> *thanks
[12:26] <mvo> pedronis: I'm on bug 1915156 thanks for the reminder
[12:26] <mup> Bug #1915156: sudoers file keeps being tracked as part of snapd <snapd (Ubuntu):In Progress by mvo> <https://launchpad.net/bugs/1915156>
[12:43] <pedronis> pstolowski: couple small comments in #9964
[12:43] <mup> PR #9964: asserts: use Fetcher in AddSequenceToUpdate <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9964>
[12:43] <pstolowski> pedronis: thanks, looking
[12:58] <pstolowski> pedronis: updated
[12:59] <pstolowski> is ijohnson off today?
[13:01] <pedronis> pstolowski: re-reviewed, thanks
[13:04] <pstolowski> pedronis: ty!
[13:04] <pstolowski> ijohnson: hey, how is the review of pool going? do you need any clarification?
[13:16] <ijohnson> hey pstolowski yeah it's going okay i will have a few clarification questions in my review, but I am close to finishing it this morning
[13:16] <pstolowski> ijohnson: great, thank you, shout if there is anything
[13:17] <mborzecki> re
[13:17] <mborzecki> heh, that was a long errand, glad i took a day off
[13:18] <mborzecki> anyways, now i need to take care of even more paperwork :/
[13:18] <ijohnson> will do thanks
[13:19] <mborzecki> cachio: fyi, i forked a project that builds appliance images of different opensuse versions, i'll tweak it to generate TW only and use ext4 for rootfs, it's here if you're interested: https://build.opensuse.org/package/show/home:maciek_borzecki:Virtualization:Appliances:Images:openSUSE-Tumbleweed/kiwi-templates-JeOS
[13:45] <ijohnson> mvo: pedronis: I forgot to send a calendar invite after SU on Friday, but just remembered now, so I sent one for later today after SU to discuss time issues in the initrd
[13:46] <pedronis> ijohnson: ok
[13:47] <ijohnson> pedronis: also does #9895 need another +1 from someone, or is it good to land, it has 2 +1s and even ondrak tested it on another qualcomm platform and it didn't fail on sector sizes there
[13:47] <mup> PR #9895: gadget/many: rm, delay sector size + structure size checks to runtime <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9895>
[13:47] <ijohnson> amazingly the pr is green (except hirsute and tumbleweed of course)
[13:48] <pedronis> ijohnson: no, should fine, it has +2
[13:48] <pedronis> *should be
[13:48] <ijohnson> sounds good
[13:48] <ijohnson> oh wait before I merge
[13:49] <ijohnson> should I squash merge it?
[13:49] <ijohnson> it's only 5 commits so maybe it doesn't matter, but also I don't know if we already branched for 2.50 yet
[13:49] <ijohnson> I don't see a release/2.50 branch so I think it's fine
[13:50]  * ijohnson presses the button
[13:52] <mup> PR snapd#9895 closed: gadget/many: rm, delay sector size + structure size checks to runtime <Run nested> <UC20> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9895>
[13:56] <ijohnson> hmm github actions seems stuck on https://github.com/snapcore/snapd/pull/9914
[13:56] <mup> PR #9914: image: write resolved-content from snap prepare-image <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9914>
[14:57] <mup> PR snapd#9970 opened: packaging: drop 99-snapd.conf via dpkg-maintscript-helper <Created by mvo5> <https://github.com/snapcore/snapd/pull/9970>
[15:34] <ijohnson> mvo: pedronis: fwiw `-rtc base=1970-01-01` works with qemu so I'm testing out the systemd timesync file
[15:37] <mvo> ijohnson: \o/
[15:37] <ijohnson> trying to touch that systemd file now
[15:42] <ijohnson> mvo: so it seems touching that /var/lib/systemd/timesync/clock file doesn't just work because that file is only read via systemd.timesyncd.service which is not shipped in the initrd, I'm trying to see if we can work around that manually
[15:42] <ijohnson> otherwise we would need to change how systemd is built into the initrd to add that service
[15:43] <mvo> ijohnson: yeah, I think it's okay if we do things from snap-bootstrap if the amount of work in initrd is big. do we have a file in initrd that gets updated with each initrd build?
[15:43] <mvo> ijohnson: (where we could get the timestamp from?)
[15:43] <ogra> ♪🎵♪ ** ... can't touch this ... ** ♪🎵♪
[15:44] <ijohnson> mvo: hmm not sure, let me have a look
[15:50] <ijohnson> mvo: also not sure if you saw, but there are merge conflicts in #9914 due to my gadget PR which just landed, you'll need to remove references to SectorSize there
[15:50] <mup> PR #9914: image: write resolved-content from snap prepare-image <Created by mvo5> <https://github.com/snapcore/snapd/pull/9914>
[15:55] <mvo> ijohnson: oh, missed that, this needs a second review anyway iirc but I will fix
[15:55] <ijohnson> mvo: ack
[15:59]  * cachio lunch
[16:13] <mup> PR snapd#9728 closed: many: allow disable user creation on system-user auto-import <Needs Samuele review> <Created by slimjim777> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/9728>
[16:13] <mup> PR snapd#9971 opened: daemon,o/c/configcore: introduce users.create.automatic <Created by pedronis> <https://github.com/snapcore/snapd/pull/9971>
[16:13] <pedronis> mvo: ^
[16:15] <mvo> pedronis: nice, thank you
[16:32] <ijohnson> mvo, I looked and all files in the initrd have the same mtime timestamp of 1574166265 (which is Tue Nov 19 2019), not sure if maybe that is by design that all the files in the initrd get timestamped to the same time
[16:36] <mvo> ijohnson: probably for reproducible builds
[16:36] <ijohnson> ah good point I forgot about that
[16:56] <ijohnson> pstolowski: pedronis: I finished reviewing 9930, hopefully my understanding there is correct
[17:01] <pstolowski> ijohnson: thanks! i'll address/clarify your points
[17:06] <ijohnson> yaw
[17:38] <mup> PR snapd#9972 opened: daemon: move the last api_foo_test.go to daemon_test <Created by pedronis> <https://github.com/snapcore/snapd/pull/9972>
[18:30] <mup> Bug #1917348 opened: NFS access not permitted for snap's on LDAP autofs system <Snappy:New> <https://launchpad.net/bugs/1917348>
[18:33] <mup> Bug #1917348 changed: NFS access not permitted for snap's on LDAP autofs system <Snappy:New> <https://launchpad.net/bugs/1917348>
[18:36] <mup> Bug #1917348 opened: NFS access not permitted for snap's on LDAP autofs system <Snappy:New> <https://launchpad.net/bugs/1917348>
[19:00] <mup> Bug #1917348 changed: NFS access not permitted for snap's on LDAP autofs system <Snappy:New> <https://launchpad.net/bugs/1917348>
[19:59] <mup> PR snapcraft#3440 closed: python v2 plugin: install wheel for build environment <Created by cjp256> <Closed by cjp256> <https://github.com/snapcore/snapcraft/pull/3440>
[20:14] <mup> PR snapcraft#3453 opened: python v2 plugin: default python-packages to [pip, setuptools, wheel] <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3453>
[21:14] <mup> PR snapcraft#3454 opened: spread: update electron-builder test <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3454>
[21:59] <mup> PR snapcraft#3453 closed: python v2 plugin: default python-packages to [pip, setuptools, wheel] <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3453>
[22:39] <mup> PR snapd#9973 opened: osutil: add SetTime() w/ 32-bit and 64-bit implementations <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9973>
[22:45] <mup> PR snapcraft#3454 closed: spread: update electron-builder test <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3454>
[22:49] <mup> PR snapd#9974 opened: cmd/snap-bootstrap/initramfs-mounts: move time forward using ref files <Bug> <UC20> <⛔ Blocked> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9974>
[22:50] <mup> PR snapcraft#3437 closed: extensions: check that the platform snap is connected in desktop extensions and bail out if not (LP: #1915712) <Created by oSoMoN> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3437>