[04:27] <mup> Bug #1868620 changed: Snaps based on Wine fail with nvidia driver 440  <Snappy:Expired> <https://launchpad.net/bugs/1868620>
[04:33] <mup> Bug #1868620 opened: Snaps based on Wine fail with nvidia driver 440  <Snappy:Expired> <https://launchpad.net/bugs/1868620>
[04:39] <mup> Bug #1868620 changed: Snaps based on Wine fail with nvidia driver 440  <Snappy:Expired> <https://launchpad.net/bugs/1868620>
[05:28] <mborzecki> morning
[07:02] <mup> PR snapd#8993 closed: bootloader, boot: compose kernel command line passed by grub bootloader <UC20> <⛔ Blocked> <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/8993>
[07:07] <pstolowski> morning
[07:10] <mborzecki> pstolowski: hey
[07:43] <mborzecki> phew, rebased and restiched all the patches
[07:52] <mup> PR snapd#9005 opened: boot: support setting extra command line argument, bootloader interface tweaks <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9005>
[08:00] <mborzecki> pstolowski: need to take a short break from coding, do you need any reviews?
[08:00] <pstolowski> mborzecki: sure, ty! #9001 ?
[08:00] <mup> PR #9001: o/snapshotstate: helpers for calculating disk space needed for an automatic snapshot (2/N) <Disk space awareness> <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9001>
[08:00] <mborzecki> pstolowski: ok, will do
[08:00] <pstolowski> thanks!
[08:01] <pstolowski> i'll do the same; anything in particular you want reviewed?
[08:01] <pstolowski> mborzecki: ^
[08:02] <mborzecki> pstolowski: got some uc20 PRs if you're interested
[08:03] <mup> PR snapd#9006 opened: bootloader: compose command line with mode and extra arguments <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9006>
[08:03] <mborzecki> pstolowski: but nothing in particular, perhaps there's some PRs from claudio
[08:03] <pstolowski> ack
[08:03] <pstolowski> zyga is off today?
[08:06] <mborzecki> pstolowski: iirc he said he has a doctor appointment?
[08:06] <mborzecki> pstolowski: mvo too
[08:07] <pstolowski> same doctor? ;)
[08:07] <pstolowski> ok, got it
[08:07] <mborzecki> pretty sure a different one ;P
[08:20] <zyga-mbp> good morning
[08:21] <zyga-mbp> sorry for starting late, the night was pretty rough
[08:21] <pstolowski> zyga-mbp: hi! sorry to hear that
[08:22] <zyga-mbp> I got some sleep after 6AM and woke up around 10 again
[08:22] <zyga-mbp> I'll be back shortly, just making some tea and coffee for the day
[08:23] <pstolowski> zyga: will have some questions for you re some bugs
[08:23] <zyga-mbp> sure
[08:23] <pstolowski> zyga: is this fixed? https://bugs.launchpad.net/snapd/+bug/1867193
[08:23] <mup> Bug #1867193: failure refreshing snap with content interface <snapd:In Progress by zyga> <https://launchpad.net/bugs/1867193>
[08:25] <pstolowski> zyga-mbp: and this looks like something for you? https://bugs.launchpad.net/snapd/+bug/1887255
[08:25] <mup> Bug #1887255: all snaps won't start suddenly with permission denied errors <Snapcraft:Invalid> <snapd:Triaged> <https://launchpad.net/bugs/1887255>
[08:29] <zyga-mbp> hmmm
[08:29] <zyga-mbp> weird stuff, maybe mint broke snapd somehow?
[08:29] <mborzecki> zyga-mbp: heya
[08:30] <zyga-mbp> o/
[08:31] <zyga-mbp> replied there
[08:31] <zyga-mbp> though launchpad is still spinning
[08:31] <zyga-mbp> well, let me make that coffee now
[08:32] <pstolowski> zyga-mbp: didn't we similar case between ubuntu upgrades?
[08:32] <pstolowski> thanks
[08:34] <zyga-mbp> pstolowski I don't recall so, we had a different issue that resulted in the .real suffix
[08:34] <zyga-mbp> but since then it was all quiet
[08:41] <jamesh> could it have got that far if the setuid bit had been removed from snap-confine?
[08:42] <zyga-mbp> jamesh oh, interesting
[08:42] <zyga-mbp> hey :)
[08:42] <zyga-mbp> perhaps
[08:42] <zyga-mbp> but I don't know how that would happen, apart from mint package alterations
[08:42] <jamesh> it's weird that it isn't the same error from each snap
[08:42] <murthy> gwenview snap can't display webp images
[08:43] <murthy> the normally packaged one on ubuntu uses the package qt5-image-formats-plugins package to get the webp support
[08:44] <murthy> show can i install that specific plugin/package or how to get gwenview snap app to display webp images
[08:44] <jamesh> murthy: it might need to be fixed in the snap itself.  The contact info for that snap directs you to https://bugs.kde.org/enter_bug.cgi?product=neon&component=Snaps
[08:45] <murthy> jamesh: ok will try to file a bug report
[08:45] <murthy> jamesh: ty
[08:52] <pstolowski> zyga-mbp: should https://bugs.launchpad.net/snapd/+bug/1866855 be set back to confirmed (not in progress)?
[08:52] <mup> Bug #1866855: nvidia driver integration is incompatible with Debian <snapd:In Progress by zyga> <https://launchpad.net/bugs/1866855>
[08:52] <zyga-mbp> yes please, I cannot do anything about that now
[09:01] <mborzecki> pstolowski: would simplifying a bit make sense there? https://github.com/snapcore/snapd/pull/9001#pullrequestreview-447919000
[09:01] <mup> PR #9001: o/snapshotstate: helpers for calculating disk space needed for an automatic snapshot (2/N) <Disk space awareness> <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9001>
[09:03] <murthy> bug report filed for gwenview's missing webp capability https://bugs.kde.org/show_bug.cgi?id=424185
[09:13] <pstolowski> mborzecki: thanks, i need to think. i was following the logic of snapshot save method, which was a little bit convoluted ;)
[09:17] <mborzecki> pstolowski: added some comments https://github.com/snapcore/snapd/pull/9002#pullrequestreview-447923236
[09:17] <mup> PR #9002: o/snapstate: integrate free space checks with install/refresh and remove (3/N) <Disk space awareness> <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9002>
[09:18] <mborzecki> pstolowski: there's snapshot.automatic.retention=no which one can set to disable snapshots, imo we should not fail when there's not enough space for snapshots in such case
[09:51] <pstolowski> mborzecki: yep, +1, thanks
[10:02] <mborzecki> omg, arch switched the default compression alg for built pacakges
[10:02] <mborzecki> so now all spread jobs are failing
[10:05] <mborzecki> heh and the change did not hit my mirror yet, so I had not noticed it earlier
=== ricab is now known as ricab_
[10:25] <mborzecki> zyga-mbp: pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/9007 ?
[10:25] <mup> PR #9007: tests/lib: account for changes in arch package file name extention <Simple 😃> <Test Robustness> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9007>
[10:25] <zyga-mbp> sure
[10:25] <mborzecki> mvo: we'll need to cherry-pick that to 2.45 ^^
[10:25] <mborzecki> thanks!
[10:25] <mvo> mborzecki: sure, will do
[10:28] <mup> PR snapd#9007 opened: tests/lib: account for changes in arch package file name extention <Simple 😃> <Test Robustness> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9007>
[10:28] <mborzecki> huh, static checks failed? wth?
[10:30] <mvo> mborzecki: probably an apt error, I had that yesterday too
[10:31] <mborzecki> mvo: nah, a silly typo i did :/
[11:09] <zyga-mbp> brb
[11:47] <mborzecki> mvo: can you cherry pick https://github.com/snapcore/snapd/pull/9007 to 2.45?
[11:47] <mup> PR #9007: tests/lib: account for changes in arch package file name extension <Simple 😃> <Test Robustness> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9007>
[11:48] <mup> PR snapd#9007 closed: tests/lib: account for changes in arch package file name extension <Simple 😃> <Test Robustness> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9007>
[12:04] <mvo> mborzecki: sure, done
[12:04] <mborzecki> mvo: thanks
[12:06] <mvo> thank *you*
=== ijohnson|EOD is now known as ijohnson
=== JanC_ is now known as JanC
[14:19]  * zyga-mbp lunch break
[15:02] <zyga-mbp> uh
[15:02] <zyga-mbp> darn pain
[15:02] <zyga-mbp> one hour for next round
[15:27] <mup> PR snapcraft#3210 closed: docker: install snapd dependency <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3210>
[15:32]  * cachio lunch
[15:36] <ijohnson> jamesh: just an FYI since I saw your desktop team update about having difficulty building core20 or snaps that use device nodes with lxd, you can use the same trick from my PR here to get across that: https://github.com/snapcore/core20/pull/74
[15:36] <mup> PR core20#74: .travis.yml: use snapcraft w/ lxd to build the snap <Created by anonymouse64> <https://github.com/snapcore/core20/pull/74>
[15:37] <ijohnson> jamesh: it's not great, but it works well enough in CI, so I figured I would share
[15:39] <ijohnson> jamesh: also re: checking for an interface connected from inside snapctl code for the theme work, I implemented something like that for a PR that wasn't merged if you are curious about an example of how to do that (but perhaps Samuele has already explained that aspect to you)
[15:42] <mup> PR snapcraft#3212 closed: tests: expand spread coverage for stage-snaps <Created by cjp256> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/3212>
[16:01] <zyga-mbp> mvo I wrote some more code for cerberus and I found a bug that made asynchronous notifications fail, they work correctly now!
[16:01] <mvo> zyga-mbp: aha, nice
[16:01] <mvo> zyga-mbp: that is good to know
[16:02] <zyga-mbp> I pushed all the fixes now
[16:05]  * zyga-mbp -> meds & break
[16:30] <mup> PR snapd#9008 opened: boot/initramfs_test.go: add Commentf to more Assert()'s <Simple 😃> <Test Robustness> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9008>
=== msalvatore_ is now known as msalvatore
[17:02] <mup> PR snapcraft#3211 closed: pluginhandler: fix stage-snaps for v2 plugins <bug> <Created by jhenstridge> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3211>
[17:07] <mup> PR snapcraft#3209 closed: extensions/desktop/common: add snapd gl vdpau dir to LD_LIBRARY_PATH <maintenance> <Created by anonymouse64> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3209>
[17:11] <kyrofa> Hey ogra_, is there a way to take an existing ubuntu core image and make SSO NOT required? Somehow seed it with an account after I write it to disk, maybe?
[17:12] <ijohnson> kyrofa: the only way to do that would be with cloud-init, you cannot create a normal snap user the way you would with console-conf except by using console-conf or a snap that uses snapd-control
[17:13] <ijohnson> kyrofa: look at ubuntu-image --help there is a cloud-init option IIRC
[17:14] <kyrofa> ijohnson, yeah that involves actually creating a new image, was wondering if there was a way to do that on an existing image
[17:14] <ijohnson> kyrofa: what ubuntu-image does can be done on an existing image
[17:14] <kyrofa> Oh? I didn't realize ubuntu-image could operate on an image
[17:14] <ijohnson> kyrofa: it involves writing the cloud-init stuff you would pass to ubuntu-image somewhere in like /var/lib/cloud/seed/nocloud-net I think
[17:14] <ijohnson> kyrofa: no sorry I mispoke
[17:15] <kyrofa> Oh oh, I got it now
[17:15] <ijohnson> kyrofa: you can do that yourself by mounting the image and copying files, ubuntu-image will not do that for you
[17:15] <kyrofa> I can do what it does on my OWN
[17:15] <ijohnson> yes precisely
[17:15] <ijohnson> kyrofa: out of curiosity what's the use case here?
[17:15] <kyrofa> Okay so all I need to do is whatever cloud-init needs me to do to make it run
[17:15] <kyrofa> Someone asked me how they can use an ubuntu appliance without an SSO account. I wanted to be able to give them an answer
[17:16] <kyrofa> An answer that wasn't "build your own image"
[17:16] <ijohnson> kyrofa: yes setup some kind of cloud-init configuration that creates the user you want, but note that I would strongly suggest that this is erm a bit hackish since they are modifying the image
[17:16] <ijohnson> just a small disclaimer
[17:16] <kyrofa> Of course, I agree
[17:17] <ijohnson> cool
[17:17] <ijohnson> kyrofa: is this on a forum or something or just PM?
[17:17] <kyrofa> But hacking up an existing image might be a bit easier than figuring out what's in there and how to use ubuntu-image to duplicate it
[17:17] <kyrofa> Twitter, the messaging platform of professionals, of course
[17:17] <ijohnson> haha fair enough
[17:31] <ogra_> kyrofa, you can surely put a system-user assertion onto a USB key to crate the user
[17:33] <ogra_> but beyond that i dont know how you would do anything like that with an *existing* image ... for core16 and 18 images you can use the --disable-consoleconf option to ubuntu-image ... but that indeed means you need to roll your own
[17:35] <ogra_> kyrofa, oh, wait ... you *can* create a user by simply using the account control interface too ... take a look at the connect-plug-account-control hook in this snap https://github.com/ogra1/config-snap
[17:36] <ogra_> but that also means you somehow need to seed that snap into an image ... (and need to do the interface connection)
[17:36] <ogra_> (beyond the fact that you hardcode username and password in the snap indeed)
[17:39] <zyga> I've update cerberus one more time and will EOD now
[18:22]  * zyga EODs for real now
[18:22] <zyga> bye everyone!
[18:52] <mup> PR snapcraft#3213 opened: cli: use prefix in store name registration hint <bug> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3213>
[18:57] <mup> PR snapcraft#3214 opened: snap: specify 'libxslt1-dev' than virtual 'libxslt-dev' build-package <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3214>
[19:07] <mup> PR snapcraft#3215 opened: build providers: use PEP-440 compliant version comparison operator <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3215>
[19:52] <mup> PR snapcraft#3216 opened: build providers: tweak environment clean detection and logging <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3216>
[20:36] <mup> PR snapd#9009 opened: tests/cmd/snap-bootstrap/initramfs-mounts: rm duplicated env ref kernel tests <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9009>
[20:52] <mup> PR snapcraft#3213 closed: cli: use prefix in store name registration hint <bug> <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3213>
[20:56] <mup> PR snapd#9010 opened: cmd/snap-bootstrap/initramfs-mounts: call systemd-mount instead of the-tool <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9010>
[21:12] <mup> PR snapcraft#3214 closed: snap: specify 'libxslt1-dev' than virtual 'libxslt-dev' build-package <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3214>
[22:28] <mup> PR snapcraft#3215 closed: build providers: use PEP-440 compliant version comparison operator <bug> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3215>
[22:28] <mup> PR snapcraft#3217 opened: cmake v2 plugin: add stage to CMAKE_PREFIX_PATH <bug> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3217>