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