[01:33] PR snapcraft#2551 closed: project: read local plugins from build-aux [03:43] PR snapcraft#2555 closed: extensions: block direct use of private extensions [05:20] morning [05:29] kernel update, brb [06:03] Good morning [06:03] zyga: hey [06:03] zyga: so when do we get to work on snaps for windows? [06:04] Exactly [06:04] I wonder if they will just work [06:08] Quick walk and I will be in the office [06:55] good morning [06:58] Good morning pedronis, mvo === pstolowski|afk is now known as pstolowski [07:07] morning [07:08] pstolowski: mvo: pedronis: hey [07:08] pstolowski: any branches for review? [07:09] hey pawel [07:10] mborzecki: hey, #6793 needs 2nd review, ty! [07:10] PR #6793: cmd/snap: support for --ensure argument for snap debug timings [07:11] pstolowski: k, looking [07:17] 6819 needs a review and is quite simple [07:19] unless you guys disagree I would like to delete some labels, the list gets a bit unwieldy. I would like to kill: "reviewed", "decaying", "parallel installs", "selinux", "complex", "core18" - wdyt? [07:21] PR snapd#6813 closed: data: update XDG_DATA_DIRS via the systemd environment.d mechanism too [07:23] PR snapd#6832 opened: gadget: more validation checks for legacy MBR structure type & role [07:23] another super simple PR ^^ [07:29] mborzecki: any concerns if I delete the "selinux" label? [07:29] mvo: no, go ahead, we can always add it back if there's more selinux work [07:30] mborzecki: same for parallel installs? [07:30] GH projects work nice for concentrators [07:30] labels feel more like "bug" "feature" "simple" "critical" [07:30] zyga: aha, I need to look at those haven't done so yet :/ [07:30] zyga: yeah, this is why I wanted to thin them a bit [07:31] mvo: I use them (better to ask for forgiveness as they say :) [07:31] mvo: we'll have some parallel installs work, but it's ok to drop this label too [07:31] PR snapd#6833 opened: many: intrroduce assertstest.SigningAccounts and AddMany test helpers [07:32] PR snapd#6819 closed: osutil: fix TestReadBuildGo test in sbuild [07:33] mvo: #6833 is the refactoring I started on Friday about signing models, fun ride [07:34] PR #6833: many: introduce assertstest.SigningAccounts and AddMany test helpers [07:34] pedronis: cool [07:34] maybe :) [07:35] heh :) [07:35] pedronis: heh [07:35] pedronis: 6817 is the one that needs the next review, right? [07:36] mvo: yes, next in chain (then the chain becomes a tree a couple of times) [07:37] https://github.com/snapcore/snapd/pull/6796 is an easy one [07:37] PR #6796: cmd/snap-update-ns: add no-op load/save current user profile logic [07:48] pstolowski: can you ask for snap debug timings --ensure=foo ? [07:49] mborzecki: atm no, i dropped it in favor of getting the latest ensure or all of them. i can add it back though [07:51] pstolowski: ok, but if you do snap debug timings --ensure=foo you can still get timing data on related changes right? [07:54] yes [08:39] PR snapd#6834 opened: daemon: pass the model to the create known user helpers (instead of full Overlord) [08:41] pstolowski: mborzecki: I did the thing we discussed yesterday in the last commit here ^ [08:41] ack [08:44] pedronis: ty [08:49] mvo: I'm preparing proper demo snaps, I have a CUDA snap that works in both legacy (current nvidia sharing) and modern (content based sharing) mode now [08:49] mvo: I need to take a break for an hour because my father in-law just sold the desk I'm using for my testbench [08:49] so... small inconvenience there [08:51] zyga: nice [08:51] zyga: well, that your desk is sold is not so nice [08:51] mvo: we wanted to sell it for several weeks now [08:51] but now his work colleague just decided he could use it :) [08:51] it's an older desk and I wanted to get rid of it because it doesn't fit the rest of office furniture and is too small for my needs [08:52] I just used it to place the testbench on it :) [08:52] my main desk is safe :D [08:52] mvo: on the up side I will get a standing desk [08:52] just ordered one from ikea :) [09:02] zyga: hahah afer all :P [09:02] zyga: bekant? [09:02] mborzecki: exactly [09:15] PR snapd#6832 closed: gadget: more validation checks for legacy MBR structure type & role [09:19] mborzecki: actually, I just cancelled and got IDASEN [09:20] zyga: ha, they have those again? [09:20] apparently, there are three in the ikea closest to us [09:20] the legs look more sturdy [09:20] zyga: last i checked (last staurday), they were unavailable online [09:20] and the price is the same [09:21] zyga: mhm, nice, it should be more sturdy than bekant [09:22] mborzecki: I also got the TROLLBERGET stool because it will look nice together :) [09:25] zyga: troll's hill? :) [09:25] haha, does that mean troll's hill? [09:25] am I the troll now :D [09:26] hill/mountain iirc [09:35] mborzecki: pushed changes to 6793 [09:36] PR core#38 closed: Add another pi-config option [09:36] PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build [09:36] PR core#104 closed: snapcraft.yaml: use remote fc-cache-builder [09:37] PR core#38 opened: Add another pi-config option [09:37] PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build [09:37] PR core#104 opened: snapcraft.yaml: use remote fc-cache-builder [09:38] * mvo thinks mup is a bit confused [10:24] fatal: unable to access 'https://github.com/snapcore/snapd.git/': The requested URL returned error: 503 [10:24] hm so a day off then? [10:31] hmm yeah i see a pony [10:31] and a unicorn: https://i.imgur.com/ikJTURq.png [10:33] right, s/pony/unicorn/ :) [10:33] https://www.githubstatus.com all green [10:36] ok, not anymore [10:36] pnycorn [10:39] re [10:39] back to work :) [10:39] well, I mean, I stopped doing the office changes and desk departure preparations [10:40] github is still down [10:52] hi, I have a weird multipass issue, is discussing here fine or is there a better channel? [10:53] actually, it seems related to snapd [10:54] it seems snapd is running 2 versions of the multipass snap in parallel [10:54] how do I remove a snap revision that's unknown to snapd? [10:59] ah, I had to kill all processes and squashfuse to make it go away [11:01] heh git fetch still broken [11:09] is there any chance of snapcraft growing the ability to verify that a source tarball is signed by a particular key? [11:09] i'm sure i found a bug for this at some point but can't find it now [11:10] ah https://bugs.launchpad.net/snapcraft/+bug/1626632 [11:10] Bug #1626632: snapcraft part sources are not verified for authenticity [11:10] not quite a year since i commented on it... [11:11] mwhudson: I think your best chance is to catch sergio or claudio early in your day [11:12] PR snapd#6835 opened: snapstate: allow removal of non-model kernels [11:16] mvo: ^ reviewed [11:17] zyga: thank you! [11:19] pedronis: #6817 can land [11:20] PR #6817: overlord,overlord/devicestate: do without GadgetInfo/KernelInfo in devicestate [11:20] thx [11:20] hmmm [11:20] LD_LIBRARY_PATH=/var/lib/snapd/lib/gl:/var/lib/snapd/lib/gl32:/var/lib/snapd/void:/snap/snapd-cuda-sample/x6/usr/lib/cuda-runtime::/snap/snapd-cuda-sample/x6/usr/lib [11:20] how did we add /var/lib/snapd/void to LD_LIBRARY_PATH? [11:22] aha https://github.com/snapcore/snapd/pull/2732#pullrequestreview-18827193 [11:22] PR #2732: snapenv: do not append ":" to the SNAP_LIBRARY_PATH [11:49] PR snapd#6817 closed: overlord,overlord/devicestate: do without GadgetInfo/KernelInfo in devicestate [11:54] PR core#38 closed: Add another pi-config option [11:54] PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build [11:54] PR core#104 closed: snapcraft.yaml: use remote fc-cache-builder [11:55] PR core#38 opened: Add another pi-config option [11:55] PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build [11:55] PR core#104 opened: snapcraft.yaml: use remote fc-cache-builder [12:09] * cachio afk [12:09] afk for lunch and walk [12:14] PR snapd#6836 opened: overlord/snapstate: add update-gadget task when needed, block other changes [12:15] mvo: pedronis: ^^ conflicts check & snapstate changes [12:16] mvo: where? [12:16] sory [12:16] mborzecki: where, in your PR ? === ricab is now known as ricab|lunch [12:17] pedronis: yeah, the PR adding the task handler is actually separate, but that shouldn't be a problem [12:22] zyga: any chance of snapd working with wsl2? not many kernel configuration details at this point, but do you see any big show stopper? [12:28] cmatsuoka: probably hard to tell now knowing their kernel config, but they claim to be able to run docker and fuse [12:30] pity they don't share the details about how it's all wired with windows kernel, wheether that's some elaborate hyper-v setup or something else [12:31] yep, I see the possibility there, but I think there are many things that could go wrong [12:33] mborzecki: whoops, posted a comment to the PR and forgot to +1 [12:34] ok [12:39] niemeyer: diddledan addressed most the issues you raised for the gnome extension PR (only the error message wording open for discussion) [12:40] * diddledan lurks like a lurky thing lurking [12:41] zyga ping [13:00] standup [13:00] ondra: what's up? :) [13:00] zyga hey, wanted to ask about spi device node name check [13:01] zyga you added there /dev/spidev[0-9].[0-9]+$ [13:01] zyga I have device with /dev/spidev32766.0 [13:02] zyga I wonder it this is usual or we are too conservative about name sanity check [13:02] ondra: is that number stable? where does it come from? [13:02] ondra: it looks ugly TBH but then again, it's hardware, if there's a good reason for it perhaps that's just the case [13:03] zyga I can't comment how stable is that number, or do you mean over reboots? [13:03] yes, ideally we'd know exactly where it comes from [13:03] is it hardcoded in the driver? [13:03] is it coming from the device tree? [13:03] etc [13:03] zyga let me ask dev working with that device if number is persistent over reboots [13:03] this will help us weigh on the decision [13:15] zyga I asked, but I think it's persistent, given it's 0x7FFE, so not exactly random [13:15] 0xCAFEBABE [13:15] ;-) [13:15] ondra: thanks, please let me know once you know more [13:15] ondra: a forum topic is a nice way to record the conversation for proposal to pedronis later on [13:16] zyga sure :) [13:16] thank you! [13:16] I'm sorry for being slow on some older bugs you reported, I really want to get to them but for now sprint preparation takes priority [13:21] PR # closed: snapd#5644, snapd#5822, snapd#5915, snapd#6108, snapd#6258, snapd#6325, snapd#6327, snapd#6341, snapd#6347, snapd#6360, snapd#6367, snapd#6404, snapd#6436, snapd#6541, snapd#6564, snapd#6588, snapd#6618, snapd#6648, snapd#6653, snapd#6666, snapd#6679, snapd#6680, snapd#6681, [13:21] snapd#6691, snapd#6695, snapd#6697, snapd#6703, snapd#6705, snapd#6708, snapd#6714, snapd#6721, snapd#6734, snapd#6750, snapd#6751, snapd#6759, snapd#6760, snapd#6767, snapd#6775, snapd#6776, snapd#6790, snapd#6793, snapd#6795, snapd#6796, snapd#6798, snapd#6803, snapd#6804, snapd#6805, snapd#6811, [13:21] snapd#6816, snapd#6820, snapd#6821, snapd#6822, snapd#6823, snapd#6825, snapd#6826, snapd#6828, snapd#6829, snapd#6831, snapd#6833, snapd#6834, snapd#6835, snapd#6836 [13:21] someone just had a productive afternoon :-p [13:22] PR # opened: snapd#5644, snapd#5822, snapd#5915, snapd#6108, snapd#6258, snapd#6325, snapd#6327, snapd#6341, snapd#6347, snapd#6360, snapd#6367, snapd#6404, snapd#6436, snapd#6541, snapd#6564, snapd#6588, snapd#6618, snapd#6648, snapd#6653, snapd#6666, snapd#6679, snapd#6680, snapd#6681, [13:22] snapd#6691, snapd#6695, snapd#6697, snapd#6703, snapd#6705, snapd#6708, snapd#6714, snapd#6721, snapd#6734, snapd#6750, snapd#6751, snapd#6759, snapd#6760, snapd#6767, snapd#6775, snapd#6776, snapd#6790, snapd#6793, snapd#6795, snapd#6796, snapd#6798, snapd#6803, snapd#6804, snapd#6805, snapd#6811, [13:22] snapd#6816, snapd#6820, snapd#6821, snapd#6822, snapd#6823, snapd#6825, snapd#6826, snapd#6828, snapd#6829, snapd#6831, snapd#6833, snapd#6834, snapd#6835, snapd#6836 [13:22] diddledan: mup had a long nap ;) [13:22] bah. and now a less productive one :-p [13:22] haha [13:23] holy crap :) [13:29] cmatsuoka, diddledan: The response about the adapter seems like a blocker to me [13:31] cmatsuoka, diddledan: The question was asked precisely because the logic is obsolete and needs to go away.. Sergio seems to confirm that.. we should not release a brand new extension that cooks into it behavior that is obsolete and needs to be removed. Extensions need to respect their interface forever, as long as the base is the same. === ricab|lunch is now known as ricab [13:49] cmatsuoka: re wsl2, I strongly believe it will work, the question remains on kernel configuration cooked by microsoft; I suspect someone at canonical might know but that's not me [13:54] PR core#38 closed: Add another pi-config option [13:54] PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build [13:54] PR core#104 closed: snapcraft.yaml: use remote fc-cache-builder [13:55] PR core#38 opened: Add another pi-config option [13:55] PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build [13:55] PR core#104 opened: snapcraft.yaml: use remote fc-cache-builder [13:57] PR # closed: snapcraft#2176, snapcraft#2229, snapcraft#2239, snapcraft#2398, snapcraft#2413, snapcraft#2433, snapcraft#2470, snapcraft#2495, snapcraft#2500, snapcraft#2514, snapcraft#2518, snapcraft#2527, snapcraft#2537, snapcraft#2539, snapcraft#2541, snapcraft#2544, snapcraft#2556, snapcraft#2557 [14:00] PR # opened: snapcraft#2176, snapcraft#2229, snapcraft#2239, snapcraft#2398, snapcraft#2413, snapcraft#2433, snapcraft#2470, snapcraft#2495, snapcraft#2500, snapcraft#2514, snapcraft#2518, snapcraft#2527, snapcraft#2537, snapcraft#2539, snapcraft#2541, snapcraft#2544, snapcraft#2556, snapcraft#2557 [14:07] PR # closed: snapd#5644, snapd#5822, snapd#5915, snapd#6108, snapd#6258, snapd#6325, snapd#6327, snapd#6341, snapd#6347, snapd#6360, snapd#6367, snapd#6404, snapd#6436, snapd#6541, snapd#6564, snapd#6588, snapd#6618, snapd#6648, snapd#6653, snapd#6666, snapd#6679, snapd#6680, snapd#6681, [14:07] snapd#6691, snapd#6695, snapd#6697, snapd#6703, snapd#6705, snapd#6708, snapd#6714, snapd#6721, snapd#6734, snapd#6750, snapd#6751, snapd#6759, snapd#6760, snapd#6767, snapd#6775, snapd#6776, snapd#6790, snapd#6793, snapd#6795, snapd#6796, snapd#6798, snapd#6803, snapd#6804, snapd#6805, snapd#6811, [14:07] snapd#6816, snapd#6820, snapd#6821, snapd#6822, snapd#6823, snapd#6825, snapd#6826, snapd#6828, snapd#6829, snapd#6831, snapd#6833, snapd#6834, snapd#6835, snapd#6836 [14:07] re [14:08] niemeyer: thanks! I'll coordinate with sergiusens and diddledan to address the adapter issue [14:08] PR # opened: snapd#5644, snapd#5822, snapd#5915, snapd#6108, snapd#6258, snapd#6325, snapd#6327, snapd#6341, snapd#6347, snapd#6360, snapd#6367, snapd#6404, snapd#6436, snapd#6541, snapd#6564, snapd#6588, snapd#6618, snapd#6648, snapd#6653, snapd#6666, snapd#6679, snapd#6680, snapd#6681, [14:08] snapd#6691, snapd#6695, snapd#6697, snapd#6703, snapd#6705, snapd#6708, snapd#6714, snapd#6721, snapd#6734, snapd#6750, snapd#6751, snapd#6759, snapd#6760, snapd#6767, snapd#6775, snapd#6776, snapd#6790, snapd#6793, snapd#6795, snapd#6796, snapd#6798, snapd#6803, snapd#6804, snapd#6805, snapd#6811, [14:08] snapd#6816, snapd#6820, snapd#6821, snapd#6822, snapd#6823, snapd#6825, snapd#6826, snapd#6828, snapd#6829, snapd#6831, snapd#6833, snapd#6834, snapd#6835, snapd#6836 [14:10] Looks like GitHub is unhappy lately [14:11] yep, many unicorns today [14:25] PR # closed: core-build#11, core-build#22, core-build#26, core-build#37, core-build#38 [14:27] PR # opened: core-build#11, core-build#22, core-build#26, core-build#37, core-build#38 [14:39] PR # closed: snapcraft#2176, snapcraft#2229, snapcraft#2239, snapcraft#2398, snapcraft#2413, snapcraft#2433, snapcraft#2470, snapcraft#2495, snapcraft#2500, snapcraft#2514, snapcraft#2518, snapcraft#2527, snapcraft#2537, snapcraft#2539, snapcraft#2541, snapcraft#2544, snapcraft#2556, snapcraft#2557 [14:42] PR # opened: snapcraft#2176, snapcraft#2229, snapcraft#2239, snapcraft#2398, snapcraft#2413, snapcraft#2433, snapcraft#2470, snapcraft#2495, snapcraft#2500, snapcraft#2514, snapcraft#2518, snapcraft#2527, snapcraft#2537, snapcraft#2539, snapcraft#2541, snapcraft#2544, snapcraft#2556, snapcraft#2557 [14:49] PR # closed: core-build#11, core-build#22, core-build#26, core-build#37, core-build#38 [14:50] PR # opened: core-build#11, core-build#22, core-build#26, core-build#37, core-build#38 [14:53] pedronis: could you merge master into #6821? [14:53] PR #6821: many: make which store to use contextual [14:53] pstolowski: yes, in a little bit (in a meeting) [14:53] k [15:00] pstolowski: done [15:00] ty [15:00] PR # closed: snapd#5644, snapd#5822, snapd#5915, snapd#6108, snapd#6258, snapd#6325, snapd#6327, snapd#6341, snapd#6347, snapd#6360, snapd#6367, snapd#6404, snapd#6436, snapd#6541, snapd#6564, snapd#6588, snapd#6618, snapd#6648, snapd#6653, snapd#6666, snapd#6679, snapd#6680, snapd#6681, [15:00] snapd#6691, snapd#6695, snapd#6697, snapd#6703, snapd#6705, snapd#6708, snapd#6714, snapd#6721, snapd#6734, snapd#6750, snapd#6751, snapd#6759, snapd#6760, snapd#6767, snapd#6775, snapd#6776, snapd#6790, snapd#6793, snapd#6795, snapd#6796, snapd#6798, snapd#6803, snapd#6804, snapd#6805, snapd#6811, [15:00] snapd#6816, snapd#6820, snapd#6821, snapd#6822, snapd#6823, snapd#6825, snapd#6826, snapd#6828, snapd#6829, snapd#6831, snapd#6833, snapd#6834, snapd#6835, snapd#6836 [15:02] PR # opened: snapd#5644, snapd#5822, snapd#5915, snapd#6108, snapd#6258, snapd#6325, snapd#6327, snapd#6341, snapd#6347, snapd#6360, snapd#6367, snapd#6404, snapd#6436, snapd#6541, snapd#6564, snapd#6588, snapd#6618, snapd#6648, snapd#6653, snapd#6666, snapd#6679, snapd#6680, snapd#6681, [15:02] snapd#6691, snapd#6695, snapd#6697, snapd#6703, snapd#6705, snapd#6708, snapd#6714, snapd#6721, snapd#6734, snapd#6750, snapd#6751, snapd#6759, snapd#6760, snapd#6767, snapd#6775, snapd#6776, snapd#6790, snapd#6793, snapd#6795, snapd#6796, snapd#6798, snapd#6803, snapd#6804, snapd#6805, snapd#6811, [15:02] snapd#6816, snapd#6820, snapd#6821, snapd#6822, snapd#6823, snapd#6825, snapd#6826, snapd#6828, snapd#6829, snapd#6831, snapd#6833, snapd#6834, snapd#6835, snapd#6836 [15:04] PR core#38 closed: Add another pi-config option [15:04] PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build [15:04] PR core#104 closed: snapcraft.yaml: use remote fc-cache-builder [15:05] PR core#38 opened: Add another pi-config option [15:05] PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build [15:05] PR core#104 opened: snapcraft.yaml: use remote fc-cache-builder [15:12] Issue # closed: core18#56, core18#86, core18#89, core18#117 [15:12] PR # closed: core18#43, core18#72, core18#90, core18#98, core18#122, core18#126, core18#127 [15:13] Issue # opened: core18#56, core18#86, core18#89, core18#117 [15:13] PR # opened: core18#43, core18#72, core18#90, core18#98, core18#122, core18#126, core18#127 [15:14] mvo: btw, I solved that issue I mentioned during the standup :) [15:17] mvo: and in an elegant way as well, I'd love to show you in a moment [15:18] zyga: in a meeting - but sounds amazing [15:18] pstolowski, hey [15:18] mvo: sure, nothing urgent :) [15:18] pstolowski, a test failed in a board https://paste.ubuntu.com/p/s9xrbG6mgc/ [15:18] cachio: hey, what's up? [15:18] for 2.39 [15:18] cachio: oh, interesting. looking [15:18] pstolowski, perhaps you have an idea [15:19] it happened in a caracalla [15:19] it is amd64 [15:19] cachio: i implemented the option, not sure what happened. need to check [15:20] pstolowski, sure, thanks [15:34] PR pc-amd64-gadget#10 closed: Add mmx64.efi (MokManager) to support mokutil [15:34] PR pc-amd64-gadget#11 closed: Add mmx64.efi (MokManager) to support mokutil [15:35] PR pc-amd64-gadget#10 opened: Add mmx64.efi (MokManager) to support mokutil [15:35] PR pc-amd64-gadget#11 opened: Add mmx64.efi (MokManager) to support mokutil [15:40] mvo, hey, could you please include this #6753 on 2.39? [15:40] PR #6753: tests: fix spaces issue in the base snaps names to remove during reset phase [15:40] mvo, for next cycle? [15:52] cachio: https://github.com/snapcore/snapd/pull/6837 [15:52] PR #6837: tests: extra debug for snapshot-basic test [15:52] PR snapd#6837 opened: tests: extra debug for snapshot-basic test [15:58] cachio: sure, thank you [15:59] mvo: do you have moment for this trivial https://github.com/snapcore/snapd/pull/6837 ? i'll then prep 2.39 PR [15:59] PR #6837: tests: extra debug for snapshot-basic test [15:59] cachio: merging [15:59] pstolowski: in a meeting [16:16] sergiusens: is there a way to tell snapcraft to use my apt proxy? [16:20] PR core#38 closed: Add another pi-config option [16:20] PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build [16:20] PR core#104 closed: snapcraft.yaml: use remote fc-cache-builder [16:21] PR core#38 opened: Add another pi-config option [16:21] PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build [16:21] PR core#104 opened: snapcraft.yaml: use remote fc-cache-builder [16:27] PR snapd#6838 opened: overlord/devicestate: introduce remodel kinds and contexts [16:31] mvo: in #6838 interesting things start to happen [16:31] PR #6838: overlord/devicestate: introduce remodel kinds and contexts