[00:52] <mup> PR snapcraft#2736 opened: cli: prompt for login if required <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2736>
[01:34] <mup> PR snapcraft#2737 opened: cli: add -s back to clean for legacy <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2737>
[05:02] <mborzecki> morning
[06:03] <zyga> Good morning
[06:04] <zyga> I’ll start late, looking after Lucy while wife is away
[06:04] <zyga> Should be back full time by 10
[06:14] <mborzecki> zyga:  hey
[06:15] <zyga> Hey :-)
[06:15] <zyga> Bug triage day for me :-)
[06:29] <jamesh> hi zyga
[06:41] <mup> PR snapd#7537 closed: tests: fix ubuntu-core-device-reg test for arm devices on core18 <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7537>
[06:42] <mborzecki> mvo: morning
[06:45] <mvo> mborzecki: good morning
[07:10] <pstolowski> morning
[07:11] <mborzecki> pstolowski: hey
[07:14] <mborzecki> pstolowski: since you're already reviewed it, can you take another look at https://github.com/snapcore/snapd/pull/7443 ?
[07:14] <mup> PR #7443: timeutil: fix schedules with ambiguous nth weekday spans <Bug> <Needs Samuele review> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7443>
[07:14] <pstolowski> mborzecki: will do today
[07:16] <mborzecki> pstolowski: thanks!
[07:33] <mup> PR snapd#7538 opened: tests: use `snap model` instead of `snap known model` in tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/7538>
[07:34] <zyga> re
[07:34] <zyga> jamesh: hello :)
[07:35] <zyga> sorry for starting late, Lucy was sleeping while hugging me on the floor and I was pinned down until the wife returned :)
[07:37] <jamesh> zyga: I'm just back from holiday.  I wasn't just ignoring your review comments
[07:38] <zyga> jamesh: no worries, I wasn't thinking that at all
[07:39] <jamesh> zyga: I think https://github.com/snapcore/snapd/pull/7197 has all the review ticks it needs.  Would you mind landing it for me when you have time?
[07:39] <mup> PR #7197: usersession: track connections to session agent for exit on idle and peer credential checks <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/7197>
[07:40] <zyga> looking now
[07:41] <zyga> done :)
[07:41] <zyga> I'm very happy about what's coming to snapd recently
[07:41] <zyga> exciting new features are brewing
[07:41] <mup> PR snapd#7197 closed: usersession: track connections to session agent for exit on idle and peer credential checks <Created by jhenstridge> <Merged by zyga> <https://github.com/snapcore/snapd/pull/7197>
[07:41] <zyga> mvo: could you please double check https://github.com/snapcore/snapd/pull/7533 is safe for landing
[07:42] <mup> PR #7533: client: add support to use the new "download" API <Created by mvo5> <https://github.com/snapcore/snapd/pull/7533>
[07:42] <zyga> it's got +2 and green but there's a comment there that may need attention
[07:44] <zyga> just got an app failure from travis ...
[07:44] <zyga> and it turns out they run on heroku! who knew?
[07:50] <mvo> pedronis: hey, I need a quick advice - in "snap debug boot-vars" - if a boot vars is not set, should we print "snap_bootvar=" or simply omit printing anything?
[07:53] <pedronis> mvo: is it related to 7519 ?
[07:54] <mvo> pedronis: yeah
[07:54] <pedronis> mvo: I'm not quite sure 7519 makes completely sense
[07:54] <mvo> pedronis: fair enough, I can kill it
[07:55] <pedronis> mvo: I think snap debug boot-vars though should probably return something specific on classic
[07:55] <pedronis> not print the env var as empty
[07:55] <pedronis> boot-vars atm don't make sense on classic
[07:55] <pedronis> mvo: to be clear my comment about 7519 it's that it moves from code not trusting snapd to code that does
[07:56] <pedronis> not sure it's a good idea for tests in general
[07:56] <pedronis> or maybe it needs to be changed test by test
[07:59] <pedronis> mvo: but the reason was to fix some core18 tests, right?
[07:59] <pedronis> we'll need to do something about those
[08:01] <mup> PR snapd#7519 closed: tests: use `snap debug boot-vars` in "ubuntu-core-upgrade" test <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/7519>
[08:10] <mup> PR snapd#7277 closed: overlord/snapstate: fix undo on firstboot seeding <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7277>
[08:12] <Chipaca> pedronis: mvo: imo 'snap debug boot-vars' could error with 'wtf, u classic'
[08:12] <Chipaca> on classic i mean
[08:12] <Chipaca> no need to print anything there
[08:13] <zyga> hmmm
[08:36] <mborzecki> pedronis: #7468 is ready to land, there's a typo in the comments, but i'm not sure it's worth another spread run
[08:36] <mup> PR #7468: seed/seedwriter,snap: support local snaps <Created by pedronis> <https://github.com/snapcore/snapd/pull/7468>
[08:56] <mup> PR snapd#7539 opened: sandbox/cgroup: avoid dependency on dirs <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7539>
[08:56] <mborzecki> zyga: super simple fix for dependency cycle when importing sandbox/cgroup in release ^^
[08:57] <zyga> +1
[08:58] <mborzecki> thx
[08:58] <pedronis> mborzecki: is there no other place using this that will need to mock root?
[08:58] <pedronis> for it
[08:59] <mborzecki> pedronis: there's a higher level mocking helper for this already
[08:59] <pedronis> ah ok
[09:21] <Chipaca> it's impressive how badly gocheck fails when you try to DeepEquals some nil things
[09:22] <mup> PR snapd#7468 closed: seed/seedwriter,snap: support local snaps <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7468>
[09:33] <mup> PR snapd#7540 opened: interfaces/seccomp: query apparmor sandbox helper rather than aggregate info <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7540>
[09:33] <mup> PR snapd#7541 opened: seed/seedwriter: support for extra snaps <Created by pedronis> <https://github.com/snapcore/snapd/pull/7541>
[09:34] <mborzecki> zyga: ^^ probably needs a blessing from jdstrand too
[09:36] <pedronis> mborzecki: yes
[09:36] <zyga> mborzecki: done
[09:36] <pedronis> mborzecki: I applied that typo fix in the follow up, also applied some of pstolowski comments in it and previous
[09:37] <pedronis> mvo: pstolowski: I pushed one more seedwriter PR  (the one after that is the actual switch in image)
[09:40] <pstolowski> pedronis: thanks, will try to look at one of your remaining PRs today
[09:48] <mvo> pedronis: thanks!
[09:48] <mvo> pedronis: will look once 2.42 is out
[09:56] <Chipaca> mborzecki: how would you say "update daily from 10pm to 6am, or any time on the weekends"?
[09:56] <mup> Bug #1635258 changed: ubuntu-image should reserve 1-5% for root  when creating filesystems <snapd:Triaged> <Ubuntu Image:New> <https://launchpad.net/bugs/1635258>
[09:56]  * Chipaca is bad at writing these things because coming up with it from examples is hard
[09:57] <mborzecki> Chipaca: in the refresh schedule?
[09:57] <Chipaca> mborzecki: yeh
[09:58] <mborzecki> Chipaca: something like: mon-fri,22:00-6:00,,sat-sun
[09:58] <Chipaca> mborzecki: thanks
[09:59] <mup> Bug #1634803 changed: snapcraft register-key doesn't use U1 SSO login credentials <Snapcraft:New> <snapd:Invalid> <https://launchpad.net/bugs/1634803>
[10:02] <mup> Bug #1637324 changed: snapd does not provide a REST API to log out <snapd:Fix Released> <snapd (Ubuntu):Fix Released> <https://launchpad.net/bugs/1637324>
[10:05] <mup> Bug #1637046 changed: requesting "quiet" mode <canonical-is> <papercut> <snapd:Confirmed> <https://launchpad.net/bugs/1637046>
[10:05] <mup> Bug #1642669 changed: PolicyKit doesn't work inside snaps, preventing snap installation in unity8 <personal> <Canonical System Image:Confirmed for thomas-voss> <snapd-glib:Confirmed> <Snappy:Won't Fix> <Unity8 Session Snap:Confirmed> <https://launchpad.net/bugs/1642669>
[10:07] <mup> Issue pc-amd64-gadget#20 opened:  GRUB shows "Trying to terminate EFI services again" and blocks boot for a while <Created by zyga> <https://github.com/snapcore/pc-amd64-gadget/issue/20>
[10:08] <mup> Bug #1642090 changed: crontab like snaps or interfaces <snapd:New> <https://launchpad.net/bugs/1642090>
[10:08] <mup> Bug #1642183 changed: GRUB shows "Trying to terminate EFI services again" and blocks boot for a while <snapd:Triaged> <https://launchpad.net/bugs/1642183>
[10:11] <mup> Bug #1637328 changed: Ubuntu Core 16 includes libnss-resolve from universe <Snappy:Fix Released by mvo> <https://launchpad.net/bugs/1637328>
[10:14] <zyga> hmmm
[10:14] <zyga> I think launchpad ate my bug
[10:14] <zyga> mvo: can you see bug https://bugs.launchpad.net/snapd/+bug/1643893 ?
[10:14] <mvo> zyga: in a meeting, but no
[10:14] <zyga> thanks
[10:15] <zyga> I'll ask in lp channels
[10:18] <zyga> ijohnson: please look at this bug for your work concerning services https://bugs.launchpad.net/snappy/+bug/1645271
[10:18] <mup> Bug #1645271: User unable to disable service <snapd:Confirmed> <https://launchpad.net/bugs/1645271>
[10:20] <mup> Bug #1644074 changed: auto connect none auto-connect interfaces of snaps in gadget snap <snapd:Fix Released> <https://launchpad.net/bugs/1644074>
[10:20] <mup> Bug #1645271 changed: User unable to disable service <snapd:Confirmed> <https://launchpad.net/bugs/1645271>
[10:24] <mup> Bug #1646968 changed: There's something weird about /tmp <snapd:Won't Fix> <Ubuntu Image:Fix Released by barry> <https://launchpad.net/bugs/1646968>
[10:30] <mup> Bug #1646881 changed: Enable pre-caching of snaps in a classic chroot <cpc> <snapd:Triaged> <https://launchpad.net/bugs/1646881>
[10:47] <mborzecki> mvo: i've moved the f31 cgroupv2 minimal work to done, we don't seem to have a an umbrella card for cgroupv2 work
[10:49] <mvo> mborzecki: feel free to create one
[10:49] <mborzecki> mvo: added one for hierarchy support and one for device filtering (cc zyga)
[10:49] <mvo> mborzecki: and *thank you*
[10:49] <mup> PR snapd#7542 opened: release: 2.42 <Created by mvo5> <https://github.com/snapcore/snapd/pull/7542>
[10:49] <zyga> ack
[11:10] <ogra> zyga, while going over all these 3 year old bugs you should probably first ask if it can still be repoduced at all :P
[11:10] <zyga> yeah, I'm doing that :)
[11:16] <mup> PR snapd#7539 closed: sandbox/cgroup: avoid dependency on dirs <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7539>
[11:30]  * Chipaca lunches
[12:04] <mborzecki> somebody just added a comment in the AUR page of snapd that fonts are rendered as boxes
[12:04] <mborzecki> well, they are https://i.imgur.com/5sLAJTs.png
[12:12] <zyga> ohhh
[12:14] <mborzecki> added a note under: https://forum.snapcraft.io/t/snapped-app-not-loading-fonts-on-fedora-and-arch/12484 still, no clue how to debug fontconfig rendering little boxes :/
[12:17] <zyga> did something new land there?
[12:17] <mborzecki> it's touching a number of paths: https://paste.ubuntu.com/p/C39dpbfyHC/
[12:18] <zyga> perhaps we can get our desktop teams to help us understand the stack more?
[12:18] <mborzecki> [2019-09-24 07:09] [ALPM] upgraded fontconfig (2:2.13.1+12+g5f5ec56-1 -> 2:2.13.91+23+g65087ac-1)
[12:18] <mborzecki> so the package got updated just recently
[12:20] <mborzecki> hm maybe i can downgrade on my laptop
[12:20] <mup> PR snapd#7543 opened: release: make forced dev mode look at cgroupv2 support <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7543>
[12:20] <mborzecki> zyga: ^^
[12:22] <mvo> mborzecki: do you get anything useful with FC_DEBUG=5 gnome-logs
[12:23] <mvo> mborzecki: or 1
[12:24] <mborzecki> mvo:  https://paste.ubuntu.com/p/83CNv4NncM/
[12:24] <mvo> mborzecki: https://www.freedesktop.org/wiki/Software/fontconfig/Devel/ fwiw has the high level changes
[12:25] <mborzecki> hmm Rework Flatpak support interesting, flatpak has extra sugar in there? something we could use too?
[12:26] <mvo> mborzecki: a good question
[12:27] <zyga> mborzecki: yes, they create fontconfig config
[12:28] <zyga> AFAIK
[12:29] <mborzecki> hm pango got updated more-less at the same time
[12:34] <pedronis> mvo: I commented in #7536
[12:34] <mup> PR #7536: gadget: accept system-seed role and ubuntu-data label <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/7536>
[12:35] <mvo> pedronis: thank you!
[12:35] <mvo> pedronis: sounds perfect
[12:36] <pedronis> mvo: probably best first to introduce ModelConstraints with just Classic
[12:36] <pedronis> and then do the UC20 change on top
[12:36] <mvo> pedronis: makes sense (cc cmatsuoka )
[12:41] <pedronis> mborzecki: ^ that probaby is interesting for #7509
[12:41] <mup> PR #7509: gadget, snap/pack: perform extended validation of gadget metadata and contents <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7509>
[13:00] <zyga> joining
[13:00] <mup> Bug #1645606 changed: Default ubuntu core Pi2 configuration doesn't allow to use fullscreen browser <snapd:New> <https://launchpad.net/bugs/1645606>
[13:01] <zyga> sorry
[13:05] <mborzecki> zyga: do the fonts render correctly on TW?
[13:06] <zyga> mborzecki: yes
[13:06] <zyga> I checked yesterday
[13:06] <zyga> odd?
[13:06] <zyga> mborzecki: I can recheck today
[13:06] <mborzecki> zyga: zypper dup? :)
[13:06] <zyga> yeah, daily
[13:07] <mborzecki> hmmm
[13:10] <mup> Bug #1648712 changed: Complete /etc content from host accessible to the snap <snapd:New> <https://launchpad.net/bugs/1648712>
[13:16] <mup> Bug #1663103 changed: No snapd API to determine which plugs/slots an app uses <snapd:Confirmed> <https://launchpad.net/bugs/1663103>
[13:20] <cachio> mvo, #7537 for 2.42 please
[13:20] <mup> PR #7537: tests: fix ubuntu-core-device-reg test for arm devices on core18 <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7537>
[13:20] <mvo> cachio: thanks, will cherry-pick it
[13:21] <mvo> cachio: this is already in :)
[13:22] <mup> Bug #1663787 changed: Libreoffice 5.3 instalado via snapd. NO accede a root ni a la partición con ntfs <Snappy:Invalid> <https://launchpad.net/bugs/1663787>
[13:22] <doko> mvo, whoever: https://bugs.launchpad.net/ubuntu/+source/abootimg/+bug/1846208
[13:22] <mup> Bug #1846208: [MIR] abootimg (dependency of initramfs-tools-ubuntu-core) <eoan> <rls-ee-incoming> <abootimg (Ubuntu):Incomplete by snappy-dev> <https://launchpad.net/bugs/1846208>
[13:24] <mvo> doko: in a meeting right now but let me see what this is about
[13:24] <mvo> doko: I get to it
[13:25] <mup> Bug #1669000 changed: classic snap can't use confinement override <amd64> <apport-bug> <xenial> <snapd:New> <snapd (Ubuntu):Confirmed> <https://launchpad.net/bugs/1669000>
[13:37] <mup> Bug #1669151 changed: No way to discover one's own appID <personal> <Canonical System Image:New> <snapd:Incomplete> <https://launchpad.net/bugs/1669151>
[13:37] <zyga> camera angle on the thinpad is really poor
[13:37] <roadmr> zyga: hopefully better than the XPS 13 nostril-cam though :)
[13:37] <zyga> hahaha
[13:37] <zyga> yeah, that's true
[13:37] <zyga> it showed my forehead and ceiling
[13:38] <roadmr> I prefer foreheads over knuckles blocking the person's face haha
[13:38] <roadmr> I mean, it was so bad that Dell actually had to move it to the top bezel at the expense of Windows Hello support
[13:39] <roadmr> because knuckle fingerprints or nose hair scans are not yet a thing
[14:02] <mup> PR snapcraft#2735 closed: extensions: add gsettings plug to gnome-3-28 extension <Created by galgalesh> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2735>
[14:03] <mborzecki> mvo: sooo, krita works fine, no font issues
[14:04] <mborzecki> mvo: actually looking at gnone-logs snap, it pulls in fonts.conf from the gnome-3-28-1804 connected via content interface
[14:05] <mborzecki> anyone know of gtk3 snap that does not use gnome-3-28-1804?
[14:07] <ijohnson> zyga: thanks for the ping, I use irccloud now so I don't need to be logged in to get pings :-)
[14:07] <zyga> :)
[14:08] <zyga> cool
[14:08] <ijohnson> and I closed that bug as FixReleased, the command exists, it's just called `snap stop --disable`
[14:08] <ijohnson> unclear if that command existed when the bug was filed but it exists now
[14:08] <mup> PR snapcraft#2737 closed: cli: add -s back to clean for legacy <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2737>
[14:10] <zyga> thank you
[14:11] <ijohnson> mvo: you were right it was a bunch of binaries I had in my git tree which made it run out of space, should we perhaps include `git clean -xfd` in the prepare section for spread?
[14:12] <mvo> ijohnson: sounds sensible!
[14:15] <mborzecki> mvo: poedit is gtk3, uses gnome-3-26-1604 and works fine, fonts are rendered correctly
[14:57] <mup> PR snapd#7544 opened: tests: fix snapd-failover test for core18 tests on boards <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/7544>
[15:08]  * zyga -> food
[15:14] <ijohnson> pstolowski: mind giving a review to #7437 when you get a chance?
[15:14] <mup> PR #7437:  wrappers/services.go: add disabled svc list arg to AddSnapServices <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7437>
[15:15]  * cachio lunch
[15:16] <pstolowski> ijohnson: sure, will do, but most likely tomorrow
[15:16] <ijohnson> that's fine, thanks!
[15:20] <pedronis> Chipaca: I did a pass on #7445, looking good but tests need some addition/tweaks
[15:20] <mup> PR #7445: overlord/snapstate/policy, etc: introduce policy, move canRemove to it <Created by chipaca> <https://github.com/snapcore/snapd/pull/7445>
[15:20] <Chipaca> pedronis: thanks
[15:49] <Chipaca> “Thanks all, love the work you guys are doing. It’s helped us reach users and saves us a lot of time and energy”
[15:49] <Chipaca> https://forum.snapcraft.io/t/autoconnection-of-camera-for-wickrpro/13413/7?u=chipaca
[15:50] <Chipaca> <3
[15:50] <zyga> nice :)
[15:50] <roadmr> ❤
[15:56] <Chipaca> I'm going to soft-eod and try to sneak in a run before the sky falls
[16:01] <mup> PR snapd#7542 closed: release: 2.42 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7542>
[16:11] <mvo> xnox: adding abootimg to initramfs-tools-ubuntu-core in eoan has triggered lp 1846208
[16:26] <ijohnson> pedronis: looking at mvo's usage of `snap model` in #7538 shows me that there are a number of cases where we can't use `--verbose` and have to use `--assertion` even in the scripted case like we have in the tests, is it okay if I ping you to look at that PR (can be tomorrow) just to think about the current design for `snap model` ?
[16:26] <mup> PR #7538: tests: use `snap model` instead of `snap known model` in tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/7538>
[16:27] <xnox> mvo:  yes. it's naughty to use things in the image-build PPA that are from universe, as there is no security support for them.
[16:31]  * ogra is surprised abootimg isnt in main yet given it was the most essential bit to actually boot our phones
[16:32] <ogra> we have used and supported it for at least 5 years before
[16:32] <ogra> seems it simply never moved to the right archive component
[16:49] <enkayz> hey ladies
[17:13] <cachio> mvo, hey, I see this error when I reboot core 18 https://paste.ubuntu.com/p/WpNftfG5G2/
[17:13] <cachio> on my pi3
[17:14] <cachio> fail unmounting writable
[17:19] <pedronis> ijohnson: mmh
[17:19] <pedronis> ijohnson: I'm probably missing something
[17:19] <ijohnson> let me submit my PR review if you want to look at it right now
[17:22] <ijohnson> pedronis: look at https://github.com/snapcore/snapd/pull/7538#pullrequestreview-295727879
[17:22] <mup> PR #7538: tests: use `snap model` instead of `snap known model` in tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/7538>
[17:22] <pedronis> yes?
[17:23] <ijohnson> so there are instances where we can't use `snap model --verbose` to look at authority-id for example
[17:23] <ijohnson> so firstly I was wondering if we should include `authority-id` in `--verbose`
[17:24] <ijohnson> secondly, we can't use `snap model --verbose` to easily look at the `model`, since we have the display-name before the actualy model value
[17:25] <pedronis> ijohnson: mmh, I probably misunderstood something
[17:25] <pedronis> when we finalized the other PR
[17:25] <ijohnson> so since this is the first time I've seen someone other than myself try to use snap model in a programmatic way, I'm wondering if we should revisit the discussion around the display-name for the model and whether we should use `authority-id` because here's a use case where we couldn't do that
[17:26] <ijohnson> s/should use/should show/
[17:26] <pedronis> ijohnson: I don't think display-name work as intented
[17:26] <pedronis> for --verbose
[17:26] <ijohnson> oh that's fair I seem to recall you wanted `model : display-name (value)` for --verbose just like without --verbose
[17:27] <ijohnson> but I could have gotten that wrong
[17:29] <ijohnson> pedronis: I need to break for lunch right now, but if you want to just comment on the PR what you think in your morning we can discuss tomorrow if needed
[17:32] <pedronis> ijohnson: the minsunderstanding was here: https://github.com/snapcore/snapd/pull/7411#discussion_r323704447
[17:32] <mup> PR #7411: cmd/model: output tweaks, add'l tests <Created by anonymouse64> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/7411>
[17:33] <pedronis> ijohnson: because of generic-classic doesn't have a display-model
[17:34] <pedronis> ijohnson: in --verobse is strange to have the display-name in model and in display-name
[17:35] <ijohnson> ok so it does sound like a bug with `snap model --verbose`, I should have done the similar thing with model/display-name that we did with brand/brand-id
[17:35] <ijohnson> so then, `snap model` -> shows `display-name (value)`
[17:35] <ijohnson> `snap model --verbose` -> show `model: value` and also `display-name: display-name-thing`
[17:35] <ijohnson> is that correct?
[17:36] <pedronis> yes
[17:36] <pedronis> I thought it was already implemented like that
[17:36] <pedronis> because of my comment
[17:36] <ijohnson> no I don't think it is implemented like that I just tried in a UC vm with snapd edge and it looks like this:
[17:36] <pedronis> but the discussion uses an example without a display-name
[17:36] <pedronis> it's not
[17:36] <pedronis> I looked at the code
[17:37] <pedronis> the x.Serial case was split intoo
[17:37] <ijohnson> https://pastebin.ubuntu.com/p/6cwvVwf28k/
[17:37] <pedronis> in two
[17:37] <ijohnson> alright I will submit a fix to snap model for that then
[17:38] <ijohnson> also quick, while you're here, do you need to review #7437 or can I merge when I get a 2nd review?
[17:38] <ijohnson> pedronis: ^
[17:38] <mup> PR #7437:  wrappers/services.go: add disabled svc list arg to AddSnapServices <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7437>
[17:41] <pedronis> ijohnson|lunch: you can merge with a 2nd review
[17:43] <pedronis> ijohnson|lunch: I will look at the final one once the prereqs are in
[17:45] <pedronis> ijohnson|lunch: what's the use case for authority-id in --verbose apart tests, at least for model is always the same as brand-id
[17:45] <pedronis> ?
[18:02] <xnox> ogra:  phone was built out of universe
[18:28] <mvo> cachio: is this new?
[18:28] <cachio> mvo, don't know
[18:29] <cachio> because I saw this on my device
[18:29] <cachio> and bata and edge validation are done on the lab devices
[18:29] <mvo> ijohnson|lunch: is snap model in 2.42?
[18:30] <mvo> cachio: ok, would be great to check if its a regression compared to 2.31
[18:30] <mvo> cachio: eh 2.41
[18:30] <mvo> cachio: and if so I think we need to see if its reproducible and if so we probably need a 2.42.1 :/
[18:31] <cachio> mvo, I'll check now against stable
[18:31] <cachio> it is 2.41
[18:31] <mvo> cachio: oh? so this happend on 2.41?
[18:32] <cachio> no
[18:32] <cachio> sorry
[18:32] <cachio> is it 2.42
[18:32] <cachio> my fault ;)
[18:32] <mvo> cachio: ok, thanks. let double check if it happens all the time and also if it happen with 2.41
[18:33] <cachio> mvo, sure, thanks
[18:33] <mvo> cachio: thanks, I will soon eod, but if you could mail me a brief summary of your findings that would be great
[18:34] <cachio> mvo, sure, so far we are ok but I had to re-execute some tests because of the issue that we saw running 2.42~pre
[18:35] <cachio> mvo, the issue related to the memory
[18:36] <mvo> cachio: right
[18:36] <mvo> cachio: yeah, that is a also a bit worrying
[18:37] <mvo> cachio: the umount issue is also something we need to double check
[18:37] <mvo> cachio: but not rush, I won't get a chance to look at any of this today anyway :)
[18:37] <mvo> (my today)
[18:38] <cachio> mvo, sure, I'll continue reviewing the results and I'll send you a briefa report
[18:42] <mup> Bug #1672740 changed: Netplan replug function is incompatible with ath9k_htc module <verification-done-xenial> <verification-done-yakkety> <verification-done-zesty> <netplan:Fix Released by cyphermox> <Snappy:Fix Released> <nplan (Ubuntu):Fix Released> <nplan (Ubuntu Xenial):Fix Released> <nplan
[18:42] <mup> (Ubuntu Yakkety):Fix Released> <nplan (Ubuntu Zesty):Fix Released> <https://launchpad.net/bugs/1672740>
[18:48] <mup> Bug #1672803 changed: Console-conf restarts on db when applying wrong wlan SSID or password <Snappy:Fix Released> <subiquity (Ubuntu):Fix Released> <https://launchpad.net/bugs/1672803>
[20:30] <jdstrand> zyga: hi! fyi, I reviewed 7421 with conditional approval
[20:54] <zyga> jdstrand: oooh, thank you!
[20:54] <zyga> jdstrand: I got the note that you are off for the week
[20:54] <zyga> have a fantastic anniversary :)
[21:07] <Greyer> 2mVhw5w841008
[21:20] <jdstrand> zyga: thanks! :)
[21:22] <mup> Bug #1669476 changed: content: piece of content interfaces now mandatory in trunk <snapd:Fix Released> <https://launchpad.net/bugs/1669476>
[22:25] <mup> Bug #1674468 changed: More than 36% overhead running tests for dbus interface in snap with devmode confinement <snapd:Triaged> <https://launchpad.net/bugs/1674468>
[22:25] <mup> Bug #1674634 changed: After initial 'refresh', snapd doesn't work <snapd:Incomplete> <https://launchpad.net/bugs/1674634>
[22:28] <mup> Bug #1749276 changed: Interface request: dvb <snapd-interface> <snapd:Fix Released> <https://launchpad.net/bugs/1749276>
[22:31] <mup> Bug #1676614 changed: snap install canonical-livepatch fails on system with nvidia driver <snapd:Fix Released> <https://launchpad.net/bugs/1676614>
[22:31] <mup> Bug #1749028 changed: snapd and disks <snapd:Fix Released> <https://launchpad.net/bugs/1749028>
[22:34] <mup> Bug #1680011 changed: snapd support for xdg-desktop-portal <cross-distro> <snapd:Fix Released> <https://launchpad.net/bugs/1680011>
[22:34] <mup> Bug #1683061 changed: Support wayland <Snappy:Won't Fix> <https://launchpad.net/bugs/1683061>
[22:37] <mup> Bug #1651090 changed: snap names starting with capital letters aren't supported <capitalized-app-name> <Snapcraft:Opinion> <Snappy:Won't Fix> <https://launchpad.net/bugs/1651090>
[22:40] <mup> Bug #1648478 changed: 'snap run --shell snap.hello-world.env' tries to read /snap/snap/current <snapd:Confirmed> <https://launchpad.net/bugs/1648478>
[22:43] <mup> Bug #1651936 changed: autopkgtests fail on ppc64el <snapd:Confirmed> <https://launchpad.net/bugs/1651936>
[22:43] <mup> Bug #1655594 changed:  expect based tests fails on ppc64el <snapd:Confirmed> <https://launchpad.net/bugs/1655594>
[22:43] <mup> Bug #1749374 changed: The /bin/sync command is not allowed by default <snapd:New> <https://launchpad.net/bugs/1749374>
[22:46] <mup> Bug #1756285 changed: Gamepads only working in devmode <snapd:Confirmed for zyga> <https://launchpad.net/bugs/1756285>
[22:49] <mup> Bug #1761363 changed: Add interface to access to Zeitgeist <snapd-interface> <snapd:New> <https://launchpad.net/bugs/1761363>
[22:58] <mup> Bug #1833004 changed: Application SNAP is slower than DEB at first launch <snapd:Confirmed> <https://launchpad.net/bugs/1833004>
[23:02] <zyga> kenvandine: can you please enqueue a sanity check of https://bugs.launchpad.net/snappy/+bug/1834061
[23:02] <mup> Bug #1834061: qt apps in hidpi looks tiny <Snappy:Triaged> <https://launchpad.net/bugs/1834061>
[23:04] <mup> Bug #1804281 changed: Cannot refresh snaps if home is in NFS with root_squash <snapd:Incomplete> <https://launchpad.net/bugs/1804281>
[23:07] <mup> Bug #1839709 changed: does not work under cgroup v2 <snapd:New> <snapd (Debian):Confirmed> <snapd (Fedora):Unknown> <https://launchpad.net/bugs/1839709>
[23:10] <mup> Bug #1828175 changed: Lack of proxy support for snap prepare-image <snapd:Triaged> <https://launchpad.net/bugs/1828175>
[23:10] <mup> Bug #1834723 changed: snap auto-import causes part-probe to hang while running in a VMware VM <snapd:Triaged> <https://launchpad.net/bugs/1834723>
[23:10] <mup> Bug #1841327 changed: Install snaps in Dockerfile <wishlist> <Snappy:Won't Fix> <https://launchpad.net/bugs/1841327>
[23:12] <zyga> ijohnson: can you please enqueue a quick pass over https://bugs.launchpad.net/snapd/+bug/1795947
[23:12] <mup> Bug #1795947: `snap services` should indicate when services are a oneshot <snapd:Triaged> <https://launchpad.net/bugs/1795947>
[23:12] <ijohnson> zyga: sure as long as you can enqueue some sleep :-)
[23:13] <zyga> ijohnson: yeah, I should
[23:13] <zyga> one last and I'm off
[23:13] <ijohnson> famous last words!
[23:13] <mup> Bug #1795947 changed: `snap services` should indicate when services are a oneshot <snapd:Triaged by anonymouse67> <https://launchpad.net/bugs/1795947>
[23:14] <zyga> ok, I'm off :)
[23:14] <zyga> night night :)
[23:15] <zyga> down to 66 new bugs on snappy
[23:16] <mup> Bug #1801567 changed: /home/* access not available <snapd:Triaged> <https://launchpad.net/bugs/1801567>
[23:22] <ijohnson> good work! and good night zyga :-)