[00:17] <dariball> can someone tell me where I can find the sources for the "pi-kernel" snap? core18 and pi-gadget are on github, but cannot find pi-kernel there
[00:22] <cjwatson> dariball: I'm not an expert here but https://code.launchpad.net/~canonical-kernel-snaps/+git/kernel-snaps-uc20/+ref/pi seems to be set up to build that snap
[00:23] <cjwatson> And I guess the "master" branch there is the actual kernel code that gets built; the "pi" branch seems to just be the snapcraft.yaml
[00:24] <cjwatson> oh, or maybe just some Makefiles ... I'm out of my depth at this point
[00:27] <tianon> I have no idea how releases for the "legacy" branch of snapcraft are managed (appears to be manually backported to xenial and bionic uploads?) but it'd probably be good to get https://github.com/snapcore/snapcraft/pull/3429 into a release soonish ("plugin: python" is unusable on "base: core" without it /o\)
[00:27] <mup> PR snapcraft#3429: plugins: Pin pip to supported versions <Created by philroche> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3429>
[00:29] <tianon> (I manually applied that patch locally to fix my own dev environment, but I imagine my GitHub Actions are probably gonna fail too)
[00:32] <dariball> cjwatson: mmmh more than I found, but can't really find the kernel sources there, just makefile using apt & snapcraft.yamls ... but at least a point to start, thanks
[02:20] <sbeattie> cjwatson, dariball: I'm no expert in the kernel snaps, but my understanding is that is repackaging the linux-raspi (https://launchpad.net/ubuntu/+source/linux-raspi2) archive package; the source tree would be https://git.launchpad.net/~ubuntu-kernel/ubuntu/+source/linux-raspi/+git/focal?h=master though it seems master is behind what's in the archive (though the appropriate tag is on master-next)
[06:50] <mborzecki> morning
[07:24] <mborzecki> mvo: hey
[07:35] <mvo> good morning mborzecki
[07:59] <mborzecki> mvo: snapd apaprently ftbfs on rawhide, looking into it
[08:00] <mborzecki> pstolowski: hey
[08:00] <pstolowski> morning
[08:31] <mborzecki> hm so go modules are for real this time when 1.16 gets released
[08:32] <mborzecki> rawhide already has 1.16beta what broke building snapd
[08:43] <pedronis> mborzecki: let's chat in the standup I suppose
[08:43] <mborzecki> pedronis: for now i'm using GO111MODULE=off which allows the build to complete successfuly, i'm not aware of plans to remove the switch (yet)
[08:48] <mup> PR snapd#9682 closed: data: fix case when $PATH has spaces in snapd.sh.in <Created by ExE-Boss> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/9682>
[08:50] <pstolowski> pedronis: hi, i've rebased #9853
[08:50] <mup> PR #9853: api: validate snaps against validation set assert from the store <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9853>
[08:51] <pedronis> pstolowski: thx, I saw that, I put it in my queue
[08:51] <pstolowski> ty
[08:58] <mup> PR snapd#9888 opened: data/env/snapd: use quoting in case PATH contains spaces <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9888>
[09:07] <mborzecki> any clue who creates /run/netns? is it lxd?
[09:08] <mup> PR snapd#9830 closed: Clarify error message <⛔ Blocked> <Created by SeanColombo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/9830>
[09:15] <mborzecki> heh: [43451.411268] audit: type=1400 audit(1612257161.339:593): apparmor="DENIED" operation="open" profile="/usr/lib/snapd/snap-confine" name="/proc/457397/attr/apparmor/current" pid=457397 comm="snap-confine" requested_mask="r" denied_mask="r" fsuid=0 ouid=0
[09:16] <mborzecki> but the profile has: `@{PROC}/[0-9]*/attr/{,apparmor/}current r,`
[10:03] <mup> PR snapd#9766 closed: interfaces/fwupd: allow any distros to access fw files via fwupd <Created by woodrow-shen> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9766>
[10:03] <pstolowski> pedronis: does revision 0 in asserts have any meaning anywhere? i'm wondering wrt RevisionNotKnown = -1, and if for sequences we should have an equivalent SequenceNotKnown = -1, or simply > 0 and anything <=0 is unset
[10:09] <pedronis> pstolowski: revision 0 is a valid revision for assertions, it's the default value
[10:10] <pedronis> pstolowski: sequences start at 1 though
[10:10] <pstolowski> pedronis: i see, great, thanks
[10:10] <mborzecki> mvo: can you take a look at https://github.com/snapcore/snapd/pull/9878 ? should be quite trivial
[10:10] <mup> PR #9878: tests/core/uc20-recovery: move recover mode helpers to generic testslib script <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9878>
[10:11] <pedronis> pstolowski: it's a bit confusing because snap revision start at 1 as well, otoh the reason for this default is that some assertions should never get a 2nd revision, so having revision: 1 in them would be a bit misleading
[10:12] <pstolowski> got it
=== benfrancis0 is now known as benfrancis
[10:36] <pedronis> mvo: I reviewed the dtbs PRs
[10:41] <mvo> pedronis: \o/
[10:42] <mvo> mborzecki: sure, looking (was in meetings, sorry)
[10:45] <mborzecki> mvo: also, i think we can land https://github.com/snapcore/snapd/pull/9883 the failures are unrelated
[10:45] <mup> PR #9883: cmd/snap-preseed: initialize snap.SanitizePlugsSlots for gadget in seeds <Bug> <Preseeding 🍞> <Squash-merge> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9883>
[10:45] <mborzecki> (needs your superpowers tho)
[10:46] <mvo> mborzecki: failures are unrelated? or shall I check?
[10:46] <mborzecki> mvo: unrelated, tests/unit/go failed on gpg, snap-mgmt on fedora failed in restore :/
[10:47] <mborzecki> same for sid
[10:48] <mup> PR snapd#9878 closed: tests/core/uc20-recovery: move recover mode helpers to generic testslib script <Simple 😃> <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9878>
[10:51] <mvo> mborzecki: thanks, merged and cherry-picked
[10:52] <mborzecki> and our snap-mgmt --purge is spotty, not sure how to fix that nicely, if user services are running, we'll miss them when cleaning up
[10:52] <mborzecki> (that's what happened in the failed snap-mgmt spread test)
[10:53] <ogra> hmm, wasnt UC20 supposed to hold back the starting of snaps until console-conf is done ?
[10:53] <mup> PR snapd#9883 closed: cmd/snap-preseed: initialize snap.SanitizePlugsSlots for gadget in seeds <Bug> <Preseeding 🍞> <Squash-merge> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9883>
[10:54] <ogra> trying to build an RPi kodi appliance image i always end up with the kodi UI without a chance to set up network and user
[10:55] <ogra> (note that kodi runs natively on the GPU, no Mir or anything)
[11:44] <pedronis> ogra: we don't have any feature that blocks services based on console-conf. we do have a feature nowadays to have services that arer not started/enabled by snapd by default, in which case you can use a different service and snapctl to control when they start
[11:46] <ogra> pedronis, i just found that my snap can see /var/lib/snapd/hostfs/var/lib/console-conf/complete without creating denials, so i'll simply check if i'm on core and let the service sleep until the file appears from a wrapper
[11:47] <ogra> but i thougth holding back snap startup after console-conf was theer in UC18 ... that's why i was surprised to have it just start
[11:48] <ogra> (though i admittedly only played with Mir in UC1😎
[11:48] <ogra> tsk
[11:48] <pedronis> I'm not aware of such a feature
[11:48] <ogra> *UC18
[11:48] <ogra> k, then perhaps something was slow back in my testing 🙂
[11:49] <ogra> (and i was just lucky or so)
[11:49] <pedronis> I'm surprised that you don't get a denial for that file though
[11:50] <ogra>  if [ -e "/var/lib/snapd/hostfs/var/lib/console-conf/completes" ]; then echo "console-conf done !"; fi
[11:50] <ogra> from within "snap run --shell ..." ... journal stays quiet
[11:51] <ogra> i can *not* ls the dir without denial ... seems only thet specific filenale is allowed
[11:51] <ogra> *that
[11:51] <ogra> *filemane
[11:51] <ogra> *GOD !
[11:52]  * ogra stamps foot about his typing
[12:05] <mup> PR snapcraft#3430 closed: More improvements for using python3.8 from within a snap <Created by kenvandine> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3430>
[12:39] <ogra> bah .. except that on my appliance image console-confi doesnt create the "complete" file anymore !
[13:06] <pedronis> pstolowski: I did a pass on #9853
[13:06] <mup> PR #9853: api: validate snaps against validation set assert from the store <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9853>
[13:06] <pstolowski> pedronis: ty
[13:09] <mup> PR snapd#9887 closed: seed: add LoadEssentialMeta to seed16 and allow all of its implementations to be called multiple times <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9887>
[13:09] <pedronis> pstolowski: mborzecki: thansk for the reviews
[13:09] <mborzecki> yw
[13:56] <jdstrand> mborzecki: that might suggest the profile is using an old cache file or wasn't applied (eg, another one was being used) at the time of access
[13:58] <mborzecki> jdstrand: hm maybe it's something about cache, the problem went away after i restarted the apparmor.service
[13:59] <mup> PR snapd#9545 closed: devicestate: keep log from install-mode on installed system <Run nested> <UC20> <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9545>
[14:00] <mborzecki> jdstrand: anyways, i'll try find out under what conditions this happens, also maybe it's something about the setup on my dev box, i haven't seen reports from users with a similar problem
[14:52] <jdstrand> mborzecki: re users> that's good to hear
[14:54] <ijohnson> mvo: I landed #9885 but forgot to squash-merge sorry /o\ do you want me to open a backport pr?
[14:54] <mup> PR #9885: snap/info.go: ignore unknown daemons in SortSnapServices <Bug> <Squash-merge> <⚠ Critical> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9885>
[14:54] <mup> PR snapd#9885 closed: snap/info.go: ignore unknown daemons in SortSnapServices <Bug> <Squash-merge> <⚠ Critical> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9885>
[14:55] <ijohnson> mvo: for what it's worth, I just tested cherry-picking the 3 commits from that pr to release/2.49 and they all cleanly applied there
[14:57] <mvo> ijohnson: no problem
[14:57] <ogra> my first UC20 appliance attempt ... : https://people.canonical.com/~ogra/UC20/pi-mediacenter-appliance/
[14:57] <mvo> ijohnson: feel free to cherry-pick/push yourself to the branch but I can do it as well
[14:58] <ogra> (still need an interface that allows /dev/input without Mir/X11 so i can switch kodi to strict mode)
[14:59] <ijohnson> mvo: I don't have permissions to push to the release branches, so maybe you could cherry-pick for me
[15:00] <mvo> ijohnson: sure, will do
[15:05] <mvo> ijohnson: all four commits from https://github.com/snapcore/snapd/pull/9885/commits - right?
[15:05] <mup> PR #9885: snap/info.go: ignore unknown daemons in SortSnapServices <Bug> <Squash-merge> <⚠ Critical> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/9885>
[15:06] <ijohnson> mvo: only 3 of them are necessary, e563dd413875a556a9eff6e0276f6ab57a672c16, 88d1e8aa5a00e7071bfc0449ab40b799c0d07da6 and f41ed8094010e74c691f4368aec7d547d286fbff, but the last one is just a typo fix
[15:06] <ijohnson> so up to you if you wanna cherry-pick the typo fix on 2.49
[15:06] <ijohnson> I'd say yes simply because it reduces likelihood of conflicts
[15:06] <mvo> +2
[15:06] <mvo> or +4 even ;)
[15:06] <mvo> anyway, done
[15:08] <zyga> mvo did I hear that core 20 is out?
[15:08] <mvo> zyga: I did!
[15:09] <zyga> hey guys, I worked from the kitchen all day, so I was not on IRC
[15:09] <zyga> (worked from windows laptop)
[15:09] <zyga> congratulations!
[15:09] <zyga> is there any publicity going on as well?
[15:11] <pedronis> zyga: yes, https://ubuntu.com/blog/ubuntu-core-20-secures-linux-for-iot
[15:14] <zyga> really nice work!
[15:18] <mvo> zyga: thank you! but congrats to you as well, you helped make this happen :)
[15:18] <zyga> I'm sorry I didn't endure to stay till the end
[15:19] <zyga> I'm looking into some zmk patches for bitbake compatibility, too bad Debian freeze is in effect, I cannot even upload things nwo
[15:19] <zyga> *now
=== seb128_ is now known as seb128
[16:51] <pedronis> actually there were tests for what I mentioned in the SU but the suite was never hooked up :/
[17:18] <ogra> mvo, is there a bug open about the fact that pi-config options do not seem to work at all anymore in UC20 (pre-seeding disable-overscan sets the config but does not change config.txt)
[17:44] <pedronis> ogra: there is a bug,  ijohnson has been looking into it
[17:45] <ogra> perfect, then i dont need to file one 😉
[17:50] <sbeattie> dariball: [I think you dropped when I wrote this yesterday] I'm no expert in the kernel snaps, but my understanding is that is repackaging the linux-raspi (https://launchpad.net/ubuntu/+source/linux-raspi2) archive package; the source tree would be https://git.launchpad.net/~ubuntu-kernel/ubuntu/+source/linux-raspi/+git/focal?h=master though it seems master is behind what's in the archive (though the appropriate tag is on master-next)
[17:50] <sbeattie> bah, actually https://launchpad.net/ubuntu/+source/linux-raspi (not raspi2)
[18:02] <ijohnson> ogra: pedronis the bug is https://bugs.launchpad.net/snapd/+bug/1909353
[18:02] <mup> Bug #1909353: pi-config not working in UC 20 <snapd:Triaged by pedronis> <https://launchpad.net/bugs/1909353>
[18:02] <ijohnson> I was hoping to get a little bit more free time and propose what I think is a more useful fix to get the pi gadget fixed instead of hacking in snapd, but alas I have not gotten my fix to work on 32-bit armhf yet, mainly due to no time to test it out yet
=== ijohnson is now known as ijohnson|lunch
[18:04] <ogra> thx !
[18:05]  * ogra hits "me too"
[21:46] <mup> PR snapd#9889 opened: cmd/snap-bootstrap/initramfs-mounts: write realistic modeenv for recover+install <Needs Samuele review> <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9889>
=== ijohnson|lunch is now known as ijohnson
[21:46] <ijohnson> whoops forget to leave lunch mode :-)
[22:36] <mup> PR snapd#9890 opened: misc: little tweaks <Simple 😃> <Skip spread> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9890>