[07:01] morning [07:23] mvo: hey [07:23] mvo: how was you weekend? [07:23] good morning mborzecki ! it was very nice, thank you. how was yours? [07:24] mvo: quite busy actually, maybe the next one will be smoother [07:27] mborzecki: heh, hope busy in the good way [07:49] ah, the new ineffassings are fixed [07:56] good morning [08:08] morning [08:11] zyga: pstolowski: hey [08:11] errand, back in 30 hopefully [08:11] hey guys [08:12] good morning pstolowski [08:12] hey zyga [09:16] PR snapd#9968 closed: cmd/snap-bootstrap/triggerwatch: fix returning wrong errors [10:14] pedronis: I updated 9914 (just fyi, not urgent) [10:14] 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] Bug #1907056: Please add meta/kernel.yaml [10:23] 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] mvo: mborzecki: did we get anywhere on this: https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1915156 [10:32] Bug #1915156: sudoers file keeps being tracked as part of snapd [10:37] 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] PR #9964: asserts: use Fetcher in AddSequenceToUpdate

[10:41] pstolowski: is just the last commit that needs reviewing, right? [10:43] pedronis: yes [10:43] ok, thx, will do after lunch [10:44] ty [10:44] pedronis: btw, pool.CommitTo(db) in resolvePool(..) doesn't return potential error, is this a bug? [10:45] pstolowski: no, see the doc comment of CommitTo [10:46] pedronis: ah, of course, nvm. thank [10:46] *thanks [12:26] pedronis: I'm on bug 1915156 thanks for the reminder [12:26] Bug #1915156: sudoers file keeps being tracked as part of snapd [12:43] pstolowski: couple small comments in #9964 [12:43] PR #9964: asserts: use Fetcher in AddSequenceToUpdate

[12:43] pedronis: thanks, looking [12:58] pedronis: updated [12:59] is ijohnson off today? [13:01] pstolowski: re-reviewed, thanks [13:04] pedronis: ty! [13:04] ijohnson: hey, how is the review of pool going? do you need any clarification? [13:16] 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] ijohnson: great, thank you, shout if there is anything [13:17] re [13:17] heh, that was a long errand, glad i took a day off [13:18] anyways, now i need to take care of even more paperwork :/ [13:18] will do thanks [13:19] 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] 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] ijohnson: ok [13:47] 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] PR #9895: gadget/many: rm, delay sector size + structure size checks to runtime [13:47] amazingly the pr is green (except hirsute and tumbleweed of course) [13:48] ijohnson: no, should fine, it has +2 [13:48] *should be [13:48] sounds good [13:48] oh wait before I merge [13:49] should I squash merge it? [13:49] 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] I don't see a release/2.50 branch so I think it's fine [13:50] * ijohnson presses the button [13:52] PR snapd#9895 closed: gadget/many: rm, delay sector size + structure size checks to runtime [13:56] hmm github actions seems stuck on https://github.com/snapcore/snapd/pull/9914 [13:56] PR #9914: image: write resolved-content from snap prepare-image [14:57] PR snapd#9970 opened: packaging: drop 99-snapd.conf via dpkg-maintscript-helper [15:34] mvo: pedronis: fwiw `-rtc base=1970-01-01` works with qemu so I'm testing out the systemd timesync file [15:37] ijohnson: \o/ [15:37] trying to touch that systemd file now [15:42] 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] otherwise we would need to change how systemd is built into the initrd to add that service [15:43] 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] ijohnson: (where we could get the timestamp from?) [15:43] ♪🎵♪ ** ... can't touch this ... ** ♪🎵♪ [15:44] mvo: hmm not sure, let me have a look [15:50] 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] PR #9914: image: write resolved-content from snap prepare-image [15:55] ijohnson: oh, missed that, this needs a second review anyway iirc but I will fix [15:55] mvo: ack [15:59] * cachio lunch === pedronis_ is now known as pedronis [16:13] PR snapd#9728 closed: many: allow disable user creation on system-user auto-import [16:13] PR snapd#9971 opened: daemon,o/c/configcore: introduce users.create.automatic [16:13] mvo: ^ [16:15] pedronis: nice, thank you [16:32] 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] ijohnson: probably for reproducible builds [16:36] ah good point I forgot about that [16:56] pstolowski: pedronis: I finished reviewing 9930, hopefully my understanding there is correct [17:01] ijohnson: thanks! i'll address/clarify your points [17:06] yaw [17:38] PR snapd#9972 opened: daemon: move the last api_foo_test.go to daemon_test [18:30] Bug #1917348 opened: NFS access not permitted for snap's on LDAP autofs system [18:33] Bug #1917348 changed: NFS access not permitted for snap's on LDAP autofs system [18:36] Bug #1917348 opened: NFS access not permitted for snap's on LDAP autofs system === ijohnson is now known as ijohnson|lunch [19:00] Bug #1917348 changed: NFS access not permitted for snap's on LDAP autofs system [19:59] PR snapcraft#3440 closed: python v2 plugin: install wheel for build environment [20:14] PR snapcraft#3453 opened: python v2 plugin: default python-packages to [pip, setuptools, wheel] === ijohnson|lunch is now known as ijohnson [21:14] PR snapcraft#3454 opened: spread: update electron-builder test [21:59] PR snapcraft#3453 closed: python v2 plugin: default python-packages to [pip, setuptools, wheel] [22:39] PR snapd#9973 opened: osutil: add SetTime() w/ 32-bit and 64-bit implementations [22:45] PR snapcraft#3454 closed: spread: update electron-builder test [22:49] PR snapd#9974 opened: cmd/snap-bootstrap/initramfs-mounts: move time forward using ref files <⛔ Blocked> [22:50] PR snapcraft#3437 closed: extensions: check that the platform snap is connected in desktop extensions and bail out if not (LP: #1915712)