| mup | PR snapcraft#2736 opened: cli: prompt for login if required <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2736> | 00:52 |
|---|---|---|
| mup | PR snapcraft#2737 opened: cli: add -s back to clean for legacy <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2737> | 01:34 |
| mborzecki | morning | 05:02 |
| zyga | Good morning | 06:03 |
| zyga | I’ll start late, looking after Lucy while wife is away | 06:04 |
| zyga | Should be back full time by 10 | 06:04 |
| mborzecki | zyga: hey | 06:14 |
| zyga | Hey :-) | 06:15 |
| zyga | Bug triage day for me :-) | 06:15 |
| jamesh | hi zyga | 06:29 |
| 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:41 |
| mborzecki | mvo: morning | 06:42 |
| mvo | mborzecki: good morning | 06:45 |
| === pstolowski|afk is now known as pstolowski | ||
| pstolowski | morning | 07:10 |
| mborzecki | pstolowski: hey | 07:11 |
| 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:14 |
| mborzecki | pstolowski: thanks! | 07:16 |
| 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:33 |
| zyga | re | 07:34 |
| zyga | jamesh: hello :) | 07:34 |
| zyga | sorry for starting late, Lucy was sleeping while hugging me on the floor and I was pinned down until the wife returned :) | 07:35 |
| jamesh | zyga: I'm just back from holiday. I wasn't just ignoring your review comments | 07:37 |
| zyga | jamesh: no worries, I wasn't thinking that at all | 07:38 |
| 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:39 |
| zyga | looking now | 07:40 |
| 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:41 |
| 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:42 |
| zyga | just got an app failure from travis ... | 07:44 |
| zyga | and it turns out they run on heroku! who knew? | 07:44 |
| 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:50 |
| pedronis | mvo: is it related to 7519 ? | 07:53 |
| 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:54 |
| 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:55 |
| 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:56 |
| pedronis | mvo: but the reason was to fix some core18 tests, right? | 07:59 |
| pedronis | we'll need to do something about those | 07:59 |
| 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:01 |
| 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:10 |
| 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:12 |
| zyga | hmmm | 08:13 |
| 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:36 |
| 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:56 |
| zyga | +1 | 08:57 |
| 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:58 |
| mborzecki | pedronis: there's a higher level mocking helper for this already | 08:59 |
| pedronis | ah ok | 08:59 |
| Chipaca | it's impressive how badly gocheck fails when you try to DeepEquals some nil things | 09:21 |
| 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:22 |
| 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:33 |
| mborzecki | zyga: ^^ probably needs a blessing from jdstrand too | 09:34 |
| 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:36 |
| pedronis | mvo: pstolowski: I pushed one more seedwriter PR (the one after that is the actual switch in image) | 09:37 |
| pstolowski | pedronis: thanks, will try to look at one of your remaining PRs today | 09:40 |
| mvo | pedronis: thanks! | 09:48 |
| mvo | pedronis: will look once 2.42 is out | 09:48 |
| 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:56 | |
| mborzecki | Chipaca: in the refresh schedule? | 09:57 |
| Chipaca | mborzecki: yeh | 09:57 |
| mborzecki | Chipaca: something like: mon-fri,22:00-6:00,,sat-sun | 09:58 |
| Chipaca | mborzecki: thanks | 09:58 |
| mup | Bug #1634803 changed: snapcraft register-key doesn't use U1 SSO login credentials <Snapcraft:New> <snapd:Invalid> <https://launchpad.net/bugs/1634803> | 09:59 |
| 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:02 |
| 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:05 |
| 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:07 |
| 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:08 |
| mup | Bug #1637328 changed: Ubuntu Core 16 includes libnss-resolve from universe <Snappy:Fix Released by mvo> <https://launchpad.net/bugs/1637328> | 10:11 |
| 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:14 |
| zyga | I'll ask in lp channels | 10:15 |
| 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:18 |
| 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:20 |
| 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:24 |
| mup | Bug #1646881 changed: Enable pre-caching of snaps in a classic chroot <cpc> <snapd:Triaged> <https://launchpad.net/bugs/1646881> | 10:30 |
| 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:47 |
| 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 | 10:49 |
| 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:10 |
| 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:16 |
| * Chipaca lunches | 11:30 | |
| === ricab is now known as ricab|lunch | ||
| 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:04 |
| zyga | ohhh | 12:12 |
| 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:14 |
| zyga | did something new land there? | 12:17 |
| mborzecki | it's touching a number of paths: https://paste.ubuntu.com/p/C39dpbfyHC/ | 12:17 |
| 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:18 |
| 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:20 |
| mvo | mborzecki: do you get anything useful with FC_DEBUG=5 gnome-logs | 12:22 |
| mvo | mborzecki: or 1 | 12:23 |
| 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:24 |
| mborzecki | hmm Rework Flatpak support interesting, flatpak has extra sugar in there? something we could use too? | 12:25 |
| mvo | mborzecki: a good question | 12:26 |
| zyga | mborzecki: yes, they create fontconfig config | 12:27 |
| zyga | AFAIK | 12:28 |
| mborzecki | hm pango got updated more-less at the same time | 12:29 |
| 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:34 |
| mvo | pedronis: thank you! | 12:35 |
| mvo | pedronis: sounds perfect | 12:35 |
| 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:36 |
| 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> | 12:41 |
| 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:00 |
| zyga | sorry | 13:01 |
| mborzecki | zyga: do the fonts render correctly on TW? | 13:05 |
| 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:06 |
| mborzecki | hmmm | 13:07 |
| mup | Bug #1648712 changed: Complete /etc content from host accessible to the snap <snapd:New> <https://launchpad.net/bugs/1648712> | 13:10 |
| === ricab|lunch is now known as ricab | ||
| mup | Bug #1663103 changed: No snapd API to determine which plugs/slots an app uses <snapd:Confirmed> <https://launchpad.net/bugs/1663103> | 13:16 |
| 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:20 |
| mvo | cachio: this is already in :) | 13:21 |
| 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:22 |
| mvo | doko: in a meeting right now but let me see what this is about | 13:24 |
| mvo | doko: I get to it | 13:24 |
| 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:25 |
| 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:37 |
| 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:38 |
| roadmr | because knuckle fingerprints or nose hair scans are not yet a thing | 13:39 |
| 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:02 |
| mborzecki | mvo: sooo, krita works fine, no font issues | 14:03 |
| mborzecki | mvo: actually looking at gnone-logs snap, it pulls in fonts.conf from the gnome-3-28-1804 connected via content interface | 14:04 |
| mborzecki | anyone know of gtk3 snap that does not use gnome-3-28-1804? | 14:05 |
| 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:07 |
| 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:08 |
| zyga | thank you | 14:10 |
| 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:11 |
| mvo | ijohnson: sounds sensible! | 14:12 |
| mborzecki | mvo: poedit is gtk3, uses gnome-3-26-1604 and works fine, fonts are rendered correctly | 14:15 |
| === pedronis_ is now known as pedronis | ||
| 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> | 14:57 |
| * zyga -> food | 15:08 | |
| 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:14 |
| * cachio lunch | 15:15 | |
| pstolowski | ijohnson: sure, will do, but most likely tomorrow | 15:16 |
| ijohnson | that's fine, thanks! | 15:16 |
| 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:20 |
| 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:49 |
| Chipaca | <3 | 15:50 |
| zyga | nice :) | 15:50 |
| roadmr | ❤ | 15:50 |
| Chipaca | I'm going to soft-eod and try to sneak in a run before the sky falls | 15:56 |
| mup | PR snapd#7542 closed: release: 2.42 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7542> | 16:01 |
| === pstolowski is now known as pstolowski|afk | ||
| mvo | xnox: adding abootimg to initramfs-tools-ubuntu-core in eoan has triggered lp 1846208 | 16:11 |
| 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:26 |
| 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:27 |
| * ogra is surprised abootimg isnt in main yet given it was the most essential bit to actually boot our phones | 16:31 | |
| 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:32 |
| enkayz | hey ladies | 16:49 |
| 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:13 |
| cachio | fail unmounting writable | 17:14 |
| 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:19 |
| 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:22 |
| 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:23 |
| 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:24 |
| 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:25 |
| 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:26 |
| ijohnson | but I could have gotten that wrong | 17:27 |
| 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:29 |
| 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:32 |
| pedronis | ijohnson: because of generic-classic doesn't have a display-model | 17:33 |
| pedronis | ijohnson: in --verobse is strange to have the display-name in model and in display-name | 17:34 |
| 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:35 |
| 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:36 |
| 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:37 |
| 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:38 |
| === ijohnson is now known as ijohnson|lunch | ||
| pedronis | ijohnson|lunch: you can merge with a 2nd review | 17:41 |
| pedronis | ijohnson|lunch: I will look at the final one once the prereqs are in | 17:43 |
| 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 | ? | 17:45 |
| xnox | ogra: phone was built out of universe | 18:02 |
| mvo | cachio: is this new? | 18:28 |
| cachio | mvo, don't know | 18:28 |
| 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:29 |
| 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:30 |
| 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:31 |
| 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:32 |
| 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:33 |
| 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:34 |
| cachio | mvo, the issue related to the memory | 18:35 |
| mvo | cachio: right | 18:36 |
| mvo | cachio: yeah, that is a also a bit worrying | 18:36 |
| 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:37 |
| cachio | mvo, sure, I'll continue reviewing the results and I'll send you a briefa report | 18:38 |
| 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:42 |
| 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> | 18:48 |
| jdstrand | zyga: hi! fyi, I reviewed 7421 with conditional approval | 20:30 |
| === ijohnson|lunch is now known as ijohnson | ||
| 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 :) | 20:54 |
| Greyer | 2mVhw5w841008 | 21:07 |
| jdstrand | zyga: thanks! :) | 21:20 |
| mup | Bug #1669476 changed: content: piece of content interfaces now mandatory in trunk <snapd:Fix Released> <https://launchpad.net/bugs/1669476> | 21:22 |
| 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:25 |
| mup | Bug #1749276 changed: Interface request: dvb <snapd-interface> <snapd:Fix Released> <https://launchpad.net/bugs/1749276> | 22:28 |
| 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:31 |
| 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:34 |
| 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:37 |
| 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:40 |
| 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:43 |
| mup | Bug #1756285 changed: Gamepads only working in devmode <snapd:Confirmed for zyga> <https://launchpad.net/bugs/1756285> | 22:46 |
| mup | Bug #1761363 changed: Add interface to access to Zeitgeist <snapd-interface> <snapd:New> <https://launchpad.net/bugs/1761363> | 22:49 |
| mup | Bug #1833004 changed: Application SNAP is slower than DEB at first launch <snapd:Confirmed> <https://launchpad.net/bugs/1833004> | 22:58 |
| 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:02 |
| mup | Bug #1804281 changed: Cannot refresh snaps if home is in NFS with root_squash <snapd:Incomplete> <https://launchpad.net/bugs/1804281> | 23:04 |
| mup | Bug #1839709 changed: does not work under cgroup v2 <snapd:New> <snapd (Debian):Confirmed> <snapd (Fedora):Unknown> <https://launchpad.net/bugs/1839709> | 23:07 |
| 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:10 |
| 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:12 |
| 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:13 |
| zyga | ok, I'm off :) | 23:14 |
| zyga | night night :) | 23:14 |
| zyga | down to 66 new bugs on snappy | 23:15 |
| mup | Bug #1801567 changed: /home/* access not available <snapd:Triaged> <https://launchpad.net/bugs/1801567> | 23:16 |
| ijohnson | good work! and good night zyga :-) | 23:22 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
