=== popey8 is now known as popey
[06:06] <mborzecki> morning
[07:14] <pstolowski> morning
[07:16] <mup> PR snapd#10094 opened: overlord/snapstate: skip catalog refresh when snappy testing is enabled <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10094>
[07:16] <mborzecki> pstolowski: hey
[07:43] <zyga> good morning
=== popey8 is now known as popey
[07:55] <mborzecki> zyga: hey
[07:56] <zyga> mborzecki there was some spam last evening
[07:56] <zyga> but I think it got sorted out
[07:56] <zyga> on one of the high-profile bugs
[08:00] <mborzecki> zyga: did you manage to get that landscape thing sorted out?
[08:04] <zyga> mborzecki: yes I did, I have several machines now
[08:04] <zyga> mborzecki: some small issue remains but we're trying to sort that one as well
[08:04] <mborzecki> zyga: just a couple? not a 100s like before? :)
[08:05] <zyga> mborzecki: I seem to have access to employee subscriptions
[08:05] <zyga> mborzecki: and can see hardware stats as well
[08:06] <mborzecki> yay
[08:59] <mborzecki> hmm refreshed the github review page and all fonts are bitmap, looks like they request helvetica now?
[09:19] <mborzecki> and a false alarm, fontconfig update moved the config files to /usr/share/fontconfig/conf.avail but not all symlinks in /etc/fonts/conf.d were updated
[09:45] <pstolowski> hmm is cla-check broken everywhere or is it just me and my email address?
[09:47] <pstolowski> hmm must be me, dang
[09:59] <pstolowski> so, apparently it is https://bugs.launchpad.net/launchpad/+bug/1921727
[09:59] <mup> Bug #1921727: Error 503 when trying to iterate over lp.people[team].participants <Launchpad itself:New> <https://launchpad.net/bugs/1921727>
[10:00] <zyga> oh
[10:00] <zyga> looks like a recent regression
[10:01] <cjwatson> Yeah, except that nothing relevant changed :(
[10:03] <zyga> is there a server-side oops with a backtrace?
[10:05] <pstolowski> mhm
[10:05] <cjwatson> zyga: yes
[10:07] <cjwatson> it's a slow query that really shouldn't be particularly slow and that performs reasonably on staging, so needs a fair bit of digging
[10:41] <cjwatson> Looks like a query tipped over the edge into using a completely inappropriate plan, perhaps because the team is large
[10:41] <cjwatson> I don't think it was a change in Launchpad, but maybe just adding more people to the team
[10:44] <cjwatson> Staging's PostgreSQL does things in a reasonable order; production's PostgreSQL (when querying for that particular team) takes the index of all user accounts sorted by names and goes through that looking for people who are in the relevant team, which is, uh, not as fast
[10:45] <pstolowski> yeah sounds very ineffective
[10:58] <zyga> cjwatson nice analysis!
[11:12] <mup> PR snapd#10095 opened: tests: add simple sanity check for systemctl show --property=UnitFileState for unknown service <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10095>
=== alan_g_ is now known as alan_g
[11:43] <mborzecki> hm refresh endpoint is rate limited too? https://paste.ubuntu.com/p/9WVn6cjD36/
[11:57] <mup> PR snapd#10091 closed: overlord/snapstate, overlord/ifacestate: move late security profile removal to ifacestate <Created by bboozzoo> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/10091>
[11:57] <mborzecki> pedronis: thanks!
[12:07] <pstolowski> pedronis: what's the purpose of refreshCandidates() inside `func (r *refreshHints) refresh()` ? we are not modifying any local state are we?
[12:27] <pstolowski> cjwatson: thanks for updating the bug report and looking into it
[12:32] <mup> PR snapd#10096 opened: gadget: add support for kernel command line provided by the gadget <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10096>
[13:37]  * pstolowski lunch
[13:42] <mup> PR snapd#9940 closed:  boot: cmd/snap-bootstrap: handle a candidate recovery system v2 <Run nested> <UC20> <Created by bboozzoo> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9940>
[13:42] <mup> PR snapd#10087 closed: release: 2.49.2 <Simple 😃> <Skip spread> <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/10087>
[13:42] <mup> PR snapd#10094 closed: overlord/snapstate: skip catalog refresh when snappy testing is enabled <Simple 😃> <Created by bboozzoo> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/10094>
[13:42] <pedronis> mborzecki: ^
[13:47] <ijohnson> pedronis: can you land #10089 and #10073 ? the failures there are all unrelated
[13:47] <mup> Bug #10089: please don't depend on libstdc++5-dev <xfree86 (Ubuntu):Fix Released by daniels> <https://launchpad.net/bugs/10089>
[13:47] <mup> PR #10089: o/configstate/configcore: simple refactors in preparation for new function <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10089>
[13:47] <mup> PR #10073: o/devicestate/devicemgr: register install-device hook, run if present in install <Needs Samuele review> <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10073>
[13:47] <mup> Bug #10073: kernel-package: new changes from Debian require merging <kernel-package (Ubuntu):Fix Released by mdz> <https://launchpad.net/bugs/10073>
[13:51] <ijohnson> hey cachio if you have time, I've seen ubuntu-20.04-64:tests/main/interfaces-desktop-document-portal fail because of a leftover dbus session daemon, can you try porting that test to use tests.session for systems that support using the user session (`tests.session has-session-systemd-and-dbus`)? I think that would make the leftover dbus session daemon go away on i.e. ubuntu-20.04
[13:51] <cachio> ijohnson, hey
[13:51] <cachio> sure
[13:51] <ijohnson> on systems which don't support session systemd probably it's fine to just keep testing as-is with `su -l -c ...` and accept that maybe sometimes the test there will fail
[13:52] <ijohnson> over time those systems will fall out of support
[13:52] <ijohnson> thanks cachio
[14:26] <mborzecki> pedronis: thanks!
[14:27] <mup> PR snapd#10090 closed: interfaces/apparmor: followup cleanups, comments and tweaks <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10090>
=== jdstrand_ is now known as jdstrand
[17:03] <mup> PR snapd#10097 opened: tests: use tests.session tool on interfaces-desktop-document-portal test <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10097>
=== ijohnson is now known as ijohnson|lunch
=== ijohnson|lunch is now known as ijohnson
[17:29] <ijohnson> cachio: a comment on that pr
[17:29] <cachio> ijohnson, nice, thanks for reviewing
[17:44] <mup> PR snapcraft#3479 closed: Use has-signed-canonical-cla GitHub Action <Created by MarcusTomlinson> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3479>
[17:44] <mup> PR snapcraft#3481 closed: tests: stub job to get autokpgtest for edge <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/3481>
[18:04] <mup> PR snapcraft#3446 closed: extensions: suppress realpath noise on headless systems <Created by MrCarroll> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3446>
[19:06] <cachio> ijohnson, pr updated
[19:13] <ijohnson> cachio thanks I'll take a look in a bit
[19:13] <cachio> tx
[19:43] <mup> PR snapd#10098 opened: tests: updated restore phase of spread tests - part 1 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10098>
[23:19] <mup> PR snapcraft#3489 closed: errors: introduce details_from_called_process_error() helper <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3489>