| mborzecki | morning | 06:10 |
|---|---|---|
| mborzecki | back from school run | 06:58 |
| mborzecki | mvo: morning! | 06:58 |
| 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 | 06:59 |
| 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:00 |
| zyga | (I bet my wife would do too) | 07:01 |
| mborzecki | yeah, need to make use of those swap and carry over days | 07:01 |
| mvo | zyga, mborzecki ++ ! | 07:02 |
| 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:03 |
| 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:07 |
| 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:08 |
| mvo | mborzecki: I should have capacity | 07:10 |
| mborzecki | mvo: it's super simple ;) | 07:10 |
| mvo | mborzecki: \o/ | 07:10 |
| 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:11 |
| mvo | mborzecki: but this will not fix the fc issue on fedora, right? | 07:12 |
| mvo | mborzecki: it's just a drive-by? | 07:12 |
| mborzecki | mvo: unfortunately, just a drive by | 07:13 |
| mup | PR snapd#8229 opened: overlord: disable Test..AbortShortlyAfterStartOfOperation for 2.44 <Created by mvo5> <https://github.com/snapcore/snapd/pull/8229> | 07:15 |
| mvo | hrm, session-tool terminates on centos-8-64 in master | 07:18 |
| 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:19 |
| mvo | a review for 8229 would be nice, this is strictly 8229 | 07:22 |
| mborzecki | mvo: proposed a little change in 8229 | 07:51 |
| 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 | 07:52 |
| 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:12 |
| 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:13 |
| 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:15 |
| 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:17 |
| 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:45 |
| 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:46 |
| 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:47 |
| mvo | mwhudson: a picture with a mobile phone is probably also fine :) | 08:48 |
| 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:49 |
| 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:50 |
| zyga | we should go back to autoexec.bat | 08:51 |
| zyga | at least it was deterministic | 08:51 |
| mwhudson | er does the service file get deleted again if installing the snap fails? | 08:55 |
| zyga | mborzecki: it should | 08:56 |
| zyga | er | 08:56 |
| zyga | mwhudson: ^ | 08:56 |
| mwhudson | zyga: this is not maximally helpful :) | 08:56 |
| zyga | mwhudson: yeah, I can understand that, perhaps you can prolong the process with a long sleep (assuming you can tty in) | 08:57 |
| mwhudson | zyga: in this case what's going on is fairy clear | 09:09 |
| zyga | mwhudson: how would it help you if the service file sticked around? | 09:23 |
| zyga | mwhudson: perhaps I misunderstand but perhaps a general "--install-even-if-services-fail" mode would be useful for development | 09:24 |
| 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:07 |
| zyga | mborzecki: yes | 10:09 |
| zyga | mborzecki: the fix is not that hard, just thinking if something else needs to be updated | 10:12 |
| zyga | brb | 10:23 |
| 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:32 |
| mborzecki | mvo: i can post a patch if you like | 10:33 |
| mborzecki | zyga: do you have /etc/pulse/client.conf.d/<files-inside> in focal? | 10:35 |
| 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:36 |
| mborzecki | zyga: trivial PR ^^ | 10:39 |
| zyga | mborzecki: checking | 10:43 |
| zyga | yes, I have two files there | 10:44 |
| zyga | 00-disable-autospawn.conf | 10:44 |
| zyga | and 01-enable-autospawn.conf | 10:44 |
| 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:45 |
| 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 | 10:48 |
| mborzecki | ogra_: hey, does the zoom-client app start at all? just tried it and it coredumps when starting | 11:23 |
| mvo | mborzecki: sorry for the delay, had a call | 11:30 |
| mvo | mborzecki: looking at backlog now | 11:30 |
| 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:31 |
| 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:35 |
| mborzecki | mvo: yeah, he's away today, but i'll ping him tomorrow | 11:36 |
| ogra | mborzecki, it definitely does for me ... | 11:44 |
| ogra | let me purge and re-install to be sure | 11:45 |
| mvo | mborzecki: sounds great, thank you | 11:47 |
| 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:48 |
| 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:49 |
| ogra | but for snap run --shell to look at pacmd you luckily dont need to run itself | 11:50 |
| 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) | 11:54 |
| 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:52 |
| 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:53 |
| 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:54 |
| 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 | 12:55 |
| 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:07 |
| 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:08 |
| 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:13 |
| 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:14 |
| 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:15 |
| ondra | sergiusens ah, good to know I think one customer is using it as well | 13:19 |
| zyga | re | 13:25 |
| 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:26 |
| 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:27 |
| 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:28 |
| 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:29 |
| 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:30 |
| jdstrand | mvo: I'll do that today (maybe split into multiple PRs) | 13:31 |
| 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:32 |
| jdstrand | mvo: this will be in a separate PR as soon as I figure it out | 13:33 |
| joedborg | Morning jdstrand, we’re mostly all on swap. What’s the issue? | 13:35 |
| 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:40 |
| mvo | jdstrand: thanks for this update | 13:59 |
| 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:02 |
| 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:03 |
| mvo | roadmr: is the above question from clmsy something you could help with? | 14:25 |
| 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:37 |
| mup | PR snapcraft#2963 opened: project: add fallbacks for os.sched_getaffinity <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2963> | 14:44 |
| * zyga -> quick coffee | 15:38 | |
| === genii_ is now known as genii | ||
| * zyga EODs the online part, will focus on coding for some more time offline | 17:22 | |
| mup | PR snapd#8231 opened: interfaces/{docker,kubernetes}-support: updates for lastest k8s <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8231> | 20:15 |
| mup | PR snapcraft#2964 opened: build providers: remove LXD specific env setup <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2964> | 21:00 |
| mup | PR snapd#8232 opened: interfaces: miscellaneous policy updates <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8232> | 21:49 |
| 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:13 |
| 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:15 |
| mup | PR snapcraft#2965 opened: rosdep: include EOL ROS distros <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2965> | 22:31 |
| mup | PR snapcraft#2966 opened: build providers: move to buildd images <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2966> | 22:34 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!