[06:10] <mborzecki> morning
[06:58] <mborzecki> back from school run
[06:58] <mborzecki> mvo: morning!
[06:59] <zyga> Hey
[06:59] <zyga> I will start around 9
[06:59] <zyga> Looking after Lucy for now
[06:59] <mborzecki> zyga: hey!
[06:59] <zyga> How are you all? :-)
[06:59] <mborzecki> zyga: still a bit tired
[07:00] <mvo> mborzecki: good morning
[07:00] <mvo> zyga: hey!
[07:00] <mborzecki> zyga: though happy i'm no longer on hotel food ;)
[07:00] <zyga> Or hotel coffee
[07:00] <zyga> Though I could use a few days of no-work sleep and rest away from family life too :P
[07:01] <zyga> (I bet my wife would do too)
[07:01] <mborzecki> yeah, need to make use of those swap and carry over days
[07:02] <mvo> zyga, mborzecki ++ !
[07:03] <mup> PR snapd#8223 closed: interfaces/u2f: Add Titan USB-C key <Created by stgraber> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8223>
[07:07] <mup> PR snapd#8222 closed: wrappers: import /etc/environment in all services <Bug> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8222>
[07:08] <mborzecki> https://github.com/snapcore/snapd/pull/8221 needs a 2nd review
[07:08] <mup> PR #8221: ovelord/snapstate: update only system wide fonts cache <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8221>
[07:10] <mvo> mborzecki: I should have capacity
[07:10] <mborzecki> mvo:  it's super simple ;)
[07:10] <mvo> mborzecki: \o/
[07:11] <mup> PR snapd#8221 closed: ovelord/snapstate: update only system wide fonts cache <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8221>
[07:11] <mborzecki> yay
[07:12] <mvo> mborzecki: but this will not fix the fc issue on fedora, right?
[07:12] <mvo> mborzecki: it's just a drive-by?
[07:13] <mborzecki> mvo: unfortunately, just a drive by
[07:15] <mup> PR snapd#8229 opened: overlord: disable Test..AbortShortlyAfterStartOfOperation for 2.44 <Created by mvo5> <https://github.com/snapcore/snapd/pull/8229>
[07:18] <mvo> hrm, session-tool terminates on centos-8-64 in master
[07:19] <mvo> c.f. https://api.travis-ci.org/v3/job/658196422/log.txt
[07:19] <mborzecki> zyga: ^^
[07:19] <mborzecki> zyga: isn't that the thing you investigated on friday?
[07:22] <mvo> a review for 8229 would be nice, this is strictly 8229
[07:51] <mborzecki> mvo: proposed a little change in 8229
[07:52] <mvo> mborzecki: oh, nice idea!
[07:52] <mvo> mborzecki: it's not quite true, it will be fixed in master :) but yeah, I think this is much nicer
[08:12] <zyga> re
[08:12] <zyga> mvo: I'll check
[08:12] <zyga> mborzecki: yes but I could not reproduce it after -repeat 100 :/
[08:12] <mborzecki> duh
[08:13] <zyga> but yeah
[08:13] <zyga> that's weird
[08:13] <zyga> it implies PAM module that sets XDG_RUNTIME_DIR just didn't do it
[08:13] <zyga> but why?
[08:15] <mborzecki> hmm all the relevant bits should be fairly recent
[08:15] <mborzecki> maybe we're missing some confi piece that's enabled for desktops?
[08:17] <zyga> mborzecki: are you talking about pam?
[08:17] <zyga> mborzecki: but it passes like 99,9% the tiem
[08:17] <zyga> it's probably a race or a bug we're hitting (or a mistake in my assumptions)
[08:45] <mwhudson> hm i have seeding problems in today's live server daily
[08:45] <mvo> mwhudson: any logs?
[08:45] <mwhudson> the journal refers to OOPSID
[08:45] <mwhudson> where are they?
[08:46] <mwhudson> mvo: "reported install problem for "core" as ..."
[08:46] <mvo> mwhudson: does "journal -u snapd" report anything? also "snap changes"
[08:46] <mwhudson> mvo: the journal refers to these OOPSes
[08:46] <mwhudson> snap changes has "initialize system state" in Error
[08:46] <mvo> mwhudson: do you have an oops-id for me? I can look at it
[08:47] <mwhudson> ah i think this may be my fault
[08:47] <mvo> mwhudson: also "snap change 1" (the error change is probably id 1, right?). a pastebin of this would be great
[08:47] <mwhudson> mvo: all slightly tedious in the live session :)
[08:47] <mvo> mwhudson: indeed, sorry for that
[08:48] <mvo> mwhudson: a picture with a mobile phone is probably also fine :)
[08:49] <zyga> I picked up the thing I was looking at on Friday
[08:49] <mwhudson> the task in error in is "failed to start snap.subiquity.subquity-service.service"
[08:49] <zyga> https://github.com/snapcore/snapd/pull/8230 is just a test case, I'll check how we handle environment across the stack and try to fix it as well
[08:49] <mup> PR #8230: cmd/snap-exec: add test case for LP:#1860369 <Created by zyga> <https://github.com/snapcore/snapd/pull/8230>
[08:49] <mwhudson> and "transaction order is cyclic"
[08:49] <mup> PR snapd#8230 opened: cmd/snap-exec: add test case for LP:#1860369 <Created by zyga> <https://github.com/snapcore/snapd/pull/8230>
[08:50] <zyga> mborzecki: ^
[08:50] <mwhudson> and i was messing with after/before nonsense on friday
[08:50] <mvo> mwhudson: aha, that makes sense
[08:50] <mwhudson> so yeah, i think this is my fault :)
[08:50] <mwhudson> sigh, why is boot so hard
[08:51] <zyga> we should go back to autoexec.bat
[08:51] <zyga> at least it was deterministic
[08:55] <mwhudson> er does the service file get deleted again if installing the snap fails?
[08:56] <zyga> mborzecki: it should
[08:56] <zyga> er
[08:56] <zyga> mwhudson: ^
[08:56] <mwhudson> zyga: this is not maximally helpful :)
[08:57] <zyga> mwhudson: yeah, I can understand that, perhaps you can prolong the process with a long sleep (assuming you can tty in)
[09:09] <mwhudson> zyga: in this case what's going on is fairy clear
[09:23] <zyga> mwhudson: how would it help you if the service file sticked around?
[09:24] <zyga> mwhudson: perhaps I misunderstand but perhaps a general "--install-even-if-services-fail" mode would be useful for development
[10:07] <mborzecki> zyga: will you be pushing a fix to #8230 as well?
[10:07] <mup> PR #8230: cmd/snap-exec: add test case for LP:#1860369 <Created by zyga> <https://github.com/snapcore/snapd/pull/8230>
[10:09] <zyga> mborzecki: yes
[10:12] <zyga> mborzecki: the fix is not that hard, just thinking if something else needs to be updated
[10:23] <zyga> brb
[10:32] <mborzecki> mvo: added some comments in #8100
[10:32] <mup> PR #8100: httputil: add support for extra snapd certs <Created by mvo5> <https://github.com/snapcore/snapd/pull/8100>
[10:33] <mborzecki> mvo: i can post a patch if you like
[10:35] <mborzecki> zyga: do you have /etc/pulse/client.conf.d/<files-inside> in focal?
[10:36] <mborzecki> mvo: #8227 for 2.44?
[10:36] <mup> PR #8227: interfaces/audio_playback: Fix pulseaudio config access <Simple 😃> <Created by Erick555> <https://github.com/snapcore/snapd/pull/8227>
[10:39] <mborzecki> zyga: trivial PR ^^
[10:43] <zyga> mborzecki: checking
[10:44] <zyga> yes, I have two files there
[10:44] <zyga> 00-disable-autospawn.conf
[10:44] <zyga> and 01-enable-autospawn.conf
[10:45] <zyga> mborzecki: the latter is a symlink (dangling) to /run/pulseaudio-enable-autospawn
[10:45] <zyga> weird
[10:45] <zyga> perhaps we need the /run rule as well?
[10:48] <mborzecki> zyga: idk, the fix in 8227 seems to fix the problem for the author, so i'd start with that (and it's alrady what the `pulseaudio` iface does)
[10:48] <zyga> mborzecki: reviewed
[11:23] <mborzecki> ogra_: hey, does the zoom-client app start at all? just tried it and it coredumps when starting
[11:30] <mvo> mborzecki: sorry for the delay, had a call
[11:30] <mvo> mborzecki: looking at backlog now
[11:31] <mvo> mborzecki: re 8227> yes, let's double check with jamie though
[11:31] <mvo> mborzecki: just to be sure it's not something deliberate
[11:35] <mvo> mborzecki: I like the LoadInfo() idea - I think it makes sense. feel free to push a patch. I modelled the interface after samueles suggestion so might be worth checking with him first before spending too much time on this though)
[11:36] <mborzecki> mvo: yeah, he's away today, but i'll ping him tomorrow
[11:44] <ogra> mborzecki, it definitely does for me ...
[11:45] <ogra> let me purge and re-install to be sure
[11:47] <mvo> mborzecki: sounds great, thank you
[11:48] <ogra> mborzecki, starts fine here https://paste.ubuntu.com/p/vBnx3CpZBM/ ... (at that point i have a UI on screen (even though nothing would work without connecting the interfaces)
[11:49] <mborzecki> ogra: i'm trying this on arch, maybe it's a similar case to spotify from snap that segfaults
[11:49] <ogra> hmm, yeah
[11:49] <ogra> sadly zoom is 100% proporietary so i cant change any of its code
[11:50] <ogra> but for snap run --shell to look at pacmd you luckily dont need to run itself
[11:54] <ogra> mborzecki, also note that i can touch /var/run/1000/pulse/native (the socket) just fine without error
[11:54] <ogra> (forgot to add that to the forum)
[12:52] <zyga> mborzecki: so I did some digging
[12:52] <zyga> mborzecki: and it's not great
[12:52] <mborzecki> zyga: hm?
[12:52] <zyga> mborzecki: when environment contains duplicate variables
[12:52] <zyga> mborzecki: programs disagree as to what is set
[12:52] <mborzecki> hahah
[12:53] <zyga> mborzecki: for instance, python prefers the first value
[12:53] <zyga> mborzecki: but shell prefers the last value
[12:53] <zyga> mborzecki: C is more like python
[12:53] <zyga> mborzecki: I think fixing this may "bite"
[12:54] <zyga> mborzecki: I'll go ahead and see if fixing it breaks anything visibly
[12:54] <mborzecki> hm wnder if we should maybe do it in 2.45?
[12:55] <zyga> mborzecki: IIRC it breaks some stuff for snapcraft now
[12:55] <mborzecki> or 44 and then patch release if things are turn for the worse?
[12:55] <zyga> mborzecki: let's decide once we know some more
[13:07] <zyga> mborzecki: https://bugs.launchpad.net/snapd/+bug/1860369/comments/13
[13:07] <mup> Bug #1860369: Python apps broken in MicroStack (a classic snap) after Jan 14 snapcraft update <core20> <snapd:Triaged by anonymouse67> <https://launchpad.net/bugs/1860369>
[13:07] <zyga> I'll grab lunch and fix it after
[13:08] <zyga> sergiusens: https://bugs.launchpad.net/snapd/+bug/1860369/comments/13 can you comment on this and advise us if this will break snapcraft-built snaps?
[13:08] <mup> Bug #1860369: Python apps broken in MicroStack (a classic snap) after Jan 14 snapcraft update <core20> <snapd:In Progress by zyga> <https://launchpad.net/bugs/1860369>
[13:13] <sergiusens> zyga: I suspect that it would be mostly ondra's apps as they use adapter: none... mostly everything else has a shell in the middle either through command-chain or a wrapper
[13:14] <ondra> sergiusens yes, I use heavily adapter: none
[13:14] <sergiusens> zyga: but yes, python prefers "old", shell prefers "new"...which is the origin for the bug report
[13:14] <ondra> sergiusens are we removing support of it?
[13:14] <sergiusens> ondra: adapter: none and "environment: " is buggy in snapd
[13:14] <mup> PR snapd#8227 closed: interfaces/audio_playback: Fix pulseaudio config access <Security-High> <Simple 😃> <Created by Erick555> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8227>
[13:15] <mborzecki> mvo: can you cherry-pick https://github.com/snapcore/snapd/pull/8227 to 2.44?
[13:15] <mup> PR #8227: interfaces/audio_playback: Fix pulseaudio config access <Security-High> <Simple 😃> <Created by Erick555> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8227>
[13:15] <sergiusens> ondra: just read a few lines above
[13:19] <ondra> sergiusens ah, good to know I think one customer is using it as well
[13:25] <zyga> re
[13:26] <zyga> ondra: are you okay with us fixing the bug without any extra special compatibility features?
[13:26] <zyga> ondra: we will correctly override environment along the whole chain
[13:27] <ondra> zyga sure, I do not think we have hit it so far, and if we have defined env twice, it's also error on our side :)
[13:27] <zyga> ondra: I think it's not an error in general
[13:27] <zyga> ondra: let's take PATH
[13:27] <ondra> zyga true
[13:27] <zyga> ondra: you have one to begin with, from either system or snap-confine
[13:28] <zyga> ondra: then you can change it (e.g. extend or override) in snap global environment
[13:28] <zyga> ondra: then you can do the same again, at the level of a specific application
[13:28] <zyga> ondra: we're just inconsistent for now
[13:28] <ondra> zyga ah, that makes sense
[13:29] <ondra> zyga so I think from I'm aware, we usually use one or another, do not combine both
[13:29] <zyga> ondra: sure, my point is that combining is fine IMO
[13:29] <ondra> zyga this was case in one snap I was doing about year ago and that one is dead for now :)
[13:29] <ondra> zyga agree with you, you should be able to combine
[13:29] <ondra> it makes sense
[13:30] <jdstrand> mvo: hey, did you see https://github.com/snapcore/snapd/pull/8100#discussion_r389109976 ? I feel like I am missing something?
[13:30] <ondra> zyga but it should be predictable :)
[13:30] <mup> PR #8100: httputil: add support for extra snapd certs <Created by mvo5> <https://github.com/snapcore/snapd/pull/8100>
[13:30] <jdstrand> mvo: also, saw the timeline for 2.44. I need to prepare a PR for policy updates (wanted to earlier, but alas, focused on PR review backlog)
[13:31] <jdstrand> mvo: I'll do that today (maybe split into multiple PRs)
[13:32] <jdstrand> mvo: there is a timing problem though. there is a weird rule I'm trying to chase down for joedborg and the k8s team and until I understand it, I don't know where I want to put it
[13:33] <jdstrand> mvo: this will be in a separate PR as soon as I figure it out
[13:35] <joedborg> Morning jdstrand, we’re mostly all on swap.  What’s the issue?
[13:40] <jdstrand> joedborg: there is no new issue. just the timing of when I saw mvo say that he wanted to cut a final 2.44
[13:59] <mvo> jdstrand: thanks for this update
[14:02] <clmsy> hi everyone
[14:02] <clmsy> having a bit of a trouble at the moment
[14:02] <clmsy> i accidentaly uploaded a package to global store
[14:02] <clmsy> than on the details page i just want to remove and get rid of it
[14:03] <clmsy> details page being dashboard.snapcraft.io/snaps/(nameofthesnap)
[14:03] <clmsy> if i try edit name or any edit in any part i get a 404 back
[14:25] <mvo> roadmr: is the above question from clmsy something you could help with?
[14:37] <mup> PR snapd#8229 closed: overlord: disable Test..AbortShortlyAfterStartOfOperation for 2.44 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8229>
[14:44] <mup> PR snapcraft#2963 opened: project: add fallbacks for os.sched_getaffinity <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2963>
[15:38]  * zyga -> quick coffee 
[17:22]  * zyga EODs the online part, will focus on coding for some more time offline
[20:15] <mup> PR snapd#8231 opened: interfaces/{docker,kubernetes}-support: updates for lastest k8s <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8231>
[21:00] <mup> PR snapcraft#2964 opened: build providers: remove LXD specific env setup <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2964>
[21:49] <mup> PR snapd#8232 opened: interfaces: miscellaneous policy updates <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8232>
[22:13] <mup> PR snapd#8233 opened: snap-confine: unconditionally add /dev/net/tun to the device cgroup <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8233>
[22:15] <mup> Bug #1859084 changed: tun module is not auto-loaded  on UC18/pi3 despite network-control interface being connected <snapd:In Progress by jdstrand> <https://launchpad.net/bugs/1859084>
[22:31] <mup> PR snapcraft#2965 opened: rosdep: include EOL ROS distros <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2965>
[22:34] <mup> PR snapcraft#2966 opened: build providers: move to buildd images <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2966>