[01:30] <mup> PR snapd#7916 closed: interfaces/browser-support: add more product/vendor paths <Created by Erick555> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/7916>
[04:10] <ctOS> Hi. I may be misunderstanding something. Why can the Firefox snap see contents of file:///home/me/Documents/ ? The :home connection is not connected to a slot. Shouldn’t that  stop access?
[04:48] <ctOS> Au, auto-connections and stuff. Okay, got it.
[05:37] <mup> PR snapd#7941 closed: snap-bootstrap: read only stdout when parsing the sfdisk json <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7941>
=== Girtablulu_ is now known as Girtablulu
[06:39] <mup> PR pc-amd64-gadget#32 opened: gadget.yaml: increase default size of ubuntu-data to 3G <Created by mvo5> <https://github.com/snapcore/pc-amd64-gadget/pull/32>
[06:40] <mborzecki> morning
[06:45] <mvo> hey mborzecki
[06:46] <mborzecki> mvo: hey
[06:46] <mborzecki> mvo: any PRs you'd like me to look at?
[06:49] <mvo> mborzecki: thank you, all the criticial stuff has landed
[06:49] <mborzecki> yay :)
[06:50] <mvo> mborzecki: exactly
[06:50] <mvo> mborzecki: and I have a (hacked) spread setup that runs hello world in uc20
[06:52] <mborzecki> mvo: oh cool, as in boots into uc20 and runs a test?
[06:54] <mvo> mborzecki: correct
[06:54] <mvo> mborzecki: it's very raw right now (and needs a hacked spread for uefi support with qemu). but it's getting there :)
[06:55] <mvo> mborzecki: I hope to proposed a slightly cleaned up version today
[06:56] <mborzecki> mvo: wondering whether gcp supports booting with uefi
[06:58] <mborzecki> mvo: btw. somebody is trying out gadget updates https://forum.snapcraft.io/t/gadget-schema-for-ubuntu-image-parser-might-be-broken/14723
[07:00] <mvo> mborzecki: yeah, cachio explored this last night
[07:00] <mvo> mborzecki: we have a ubuntu-1804-64-uefi-enabled image for this now
[07:00] <mvo> mborzecki: oh, interessting!
[07:01] <mvo> mborzecki: looks like we really need to take over ubuntu-image ;)
[07:01] <mvo> mborzecki: or rather make it use our gadget parser
[07:01] <mvo> mborzecki: but that's for later
[07:01] <mborzecki> mvo: yeah, maybe, i can look into adding that to the schema
[07:02] <mvo> mborzecki: if it's not too much distraction that would be nice
[07:02] <mvo> mborzecki: I should be able to look at reviews today again, anything you would like me to prioritize?
[07:02] <mborzecki> mvo: hopefully sil2100 is still around today ;)
[07:02] <mborzecki> mvo: this one would unblock rest of snapd on core: https://github.com/snapcore/snapd/pull/7772
[07:03] <mup> PR #7772: wrappers: write and undo snapd services on core <Remodel 🚋> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7772>
[07:03] <mvo> mborzecki: cool, I have a look, this is really exctiin g for me
[07:48] <mborzecki> quick errand, back in 30
[08:01] <ctOS> mborzecki: (quick follow-up from yday): the beta channel resolved the font issue, but not the tiny mouse cursor.
[08:03] <mvo> ctOS: nice, thanks for the update. I think for the cursor me need someone from the desktop team to help, maybe kenvandine when he is around (in the US timezone so not up yet)
[08:05] <mup> PR snapd#7943 opened: tests: add core20 tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/7943>
[08:14] <mvo> mborzecki: when you are back - where is OFMF.fd on fedora/arch?
[08:14] <mborzecki> re
[08:14] <mvo> mborzecki: on ubuntu it's /usr/share/OVMF/OVFM_CODE.ms.fd
[08:14] <mvo> mborzecki: but I wonder if I can build something for spread that is portable
[08:15] <mborzecki> mvo: let me see
[08:16] <mborzecki> mvo: ovmf /usr/share/ovmf/x64/OVMF_CODE.fd
[08:16] <mborzecki> ctOS: if it's not too much hassle, can you check that the cursor fix that was linked yday is include in the firefox snap from beta channel?
[08:17] <mvo> mborzecki: ta
[08:19] <ackk> hi, is there any peculiar difference between the environment a snap runs in on ubuntu vs ubuntu core? I have a snap which run sshd and I can ssh in if I install it on my PC. but running it on a rpi with ubuntu core I get disconnected right away (with no error)
[08:21] <ctOS> mborzecki: the patch is included in the beta (287). I’ve also double-checked that it is indeed not fixed in this version.
[08:23] <mborzecki> ctOS: thank you!
[08:23] <mborzecki> mvo: on fedora i have /usr/share/edk2/ovmf/OVMF_CODE.fd and there's also  /usr/share/edk2/ovmf/OVMF_CODE.secboot.fd
[08:25] <mborzecki> mvo: on arch the package is called 'ovmf', on fedora it's 'edk2-ovmf'
[08:28] <mvo> mborzecki: thanks
[08:28] <mvo> mborzecki: still scratching my head about how to do this in a portable way, maybe an environment is the simplest and just documenting it
[08:29] <mup> PR snapd#7944 opened: test: extract code that modifies "writable" for test prep <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7944>
[08:29] <pstolowski> morning
[08:29] <mborzecki> pstolowski: hey
[08:32] <mup> PR snapd#7686 closed: systemd: handle preseed mode <Preseeding 🍞> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7686>
[08:39] <mvo> hey pstolowski
[08:39] <mvo> pstolowski: nice to see this merged!
[08:41] <pstolowski> yeah!
[08:55] <mup> PR snapd#7945 opened: tests: unify/rename services-related spread tests to start with services- prefix <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7945>
[08:55] <pstolowski> mvo, mborzecki ^ trivial and hopefully uncontroversial
[08:55] <mvo> pstolowski: nice
[09:02] <mborzecki> pstolowski: +1
[09:02] <pstolowski> thx
[09:54] <mup> PR pc-amd64-gadget#32 closed: gadget.yaml: increase default size of ubuntu-data to 3G <Created by mvo5> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/32>
[09:59] <mvo> niemeyer: when you have some spare cycles a review/feedback on https://github.com/snapcore/spread/pull/95 and https://github.com/snapcore/spread/pull/96 would be great. happy to adjust as needed. we need uefi/virtio for spread testing uc20
[09:59] <mup> PR spread#95: spread: add support to define a custom bios with the qemu backend <Created by mvo5> <https://github.com/snapcore/spread/pull/95>
[09:59] <mup> PR spread#96: spread: add support for system specific "flags" and use in qemu <Created by mvo5> <https://github.com/snapcore/spread/pull/96>
[10:05] <pstolowski> travis is super slow.. or is it my PR?
[10:05] <mup> PR snapd#7944 closed: test: extract code that modifies "writable" for test prep <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7944>
[10:22] <niemeyer> mvo: Of course
[10:24] <niemeyer> mvo: Is that the same thing zyga mentioned last week? I was waiting for the ping on Friday
[10:25] <mvo> niemeyer: it's slightly different
[10:25] <mvo> niemeyer: I think zyga wants to stop using "kvm" to launch qemu because it's ubuntu/debian specific. my bit are new features for the qemu backend, i.e. I need the ability to enable uefi/virtio for uc20 testing
[10:26] <niemeyer> mvo: Ack
[10:27] <mvo> niemeyer: it's not super critical, I can use my local spread for testing for now but eventually it would be nice to be able to have it as part of the default
[10:27] <mvo> niemeyer: also does not affect GCE testing so no need to do a new release and all that :)
[10:27] <niemeyer> Cool, thanks
[10:28] <mvo> thank you!
[10:44] <pstolowski> mvo, mborzecki any particular PRs you would like to land today & want reviewed?
[10:45] <mborzecki> pstolowski: https://github.com/snapcore/snapd/pull/7772 if you would
[10:45] <mup> PR #7772: wrappers: write and undo snapd services on core <Remodel 🚋> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7772>
[10:45] <pstolowski> sure
[11:14] <mup> PR snapd#7945 closed: tests: unify/rename services-related spread tests to start with services- prefix <Simple 😃> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7945>
[11:36] <mborzecki> sil2100: hi, any chance you can take a look at this PR before the break? https://github.com/CanonicalLtd/ubuntu-image/pull/180
[11:36] <mup> PR CanonicalLtd/ubuntu-image#180: ubuntu_image: update schema validator to allow gadget update specific keys & little cleanups <Created by bboozzoo> <https://github.com/CanonicalLtd/ubuntu-image/pull/180>
[11:37] <sil2100> mborzecki: o/
[11:38] <mborzecki> sil2100: got one more patch with nicer error messages for all validation, but no clue whether older releases of voluptous raise useful exceptions
[11:39] <mborzecki> mvo: ^^
[11:42] <mvo> mborzecki: nice one!
[11:45] <cmatsuoka> good morning. hopefully my internet will be fast and stable today
[11:46] <pstolowski> hi cmatsuoka !
[11:46] <cmatsuoka> because yesterday it was just weird
[11:52] <cmatsuoka> a friend of mine is moving to Canada and is selling a NUC, I'm checking if it has TPM
[12:58] <sil2100> mborzecki: do you have an LP bug for the PR? Could you fill in a bug for https://github.com/CanonicalLtd/ubuntu-image/pull/180/ and add a changelog entry to it with the bug linked?
[12:58] <mup> PR CanonicalLtd/ubuntu-image#180: ubuntu_image: update schema validator to allow gadget update specific keys & little cleanups <Created by bboozzoo> <https://github.com/CanonicalLtd/ubuntu-image/pull/180>
[13:00] <mborzecki> sil2100: ha, there's one https://bugs.launchpad.net/ubuntu-image/+bug/1856903
[13:00] <mup> Bug #1856903: ubuntu_image/parser.py Gadget YAML Definition needs updating <gadget> <Ubuntu Image:New> <https://launchpad.net/bugs/1856903>
[13:00] <sil2100> mborzecki: hah, excellent
[13:01] <sil2100> mborzecki: could you add a changelog entry for the change with the LP: # added?
[13:01] <sil2100> mborzecki: or actually, I guess I'll just do that instead, so nevermind o/
[13:02] <mborzecki> sil2100: cool, thanks!
[13:03] <sil2100> mborzecki: oh, though some code-style stuff needs to be fixed first
[13:03] <sil2100> mborzecki: did you run `tox` before submitting the changes? Since the qa test-suite seems to fail due to pep8 errors
[13:05] <sil2100> mborzecki: example log: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-bionic-sil2100-ubuntu-image-ci-deps/bionic/amd64/u/ubuntu-image/20191219_124506_43ed7@/log.gz
[13:06] <mborzecki> sil2100: i run specific tests only, the whole suite never completed on my system
[13:16] <sil2100> mborzecki: could you address those pep8 errors and re-push?
[13:16] <mborzecki> sil2100: sure, will do
[13:18] <sil2100> Thanks!
=== ricab is now known as ricab|lunch
[13:29] <mborzecki> sil2100: pushed
[13:34] <mborzecki> mvo: have you seen something like this: https://paste.ubuntu.com/p/YcrWJxsK5d/
[13:35] <mborzecki> mvo: that's core18 snapd failover, maybe systemd doesn't like that much when a service changes it's own unit file
[13:57] <mvo> mborzecki: uh, haven't seen this one
[13:58] <mborzecki> mvo: i can reproduce it locally
[13:58] <mborzecki> mvo: https://paste.ubuntu.com/p/y2SWwD38rF/
[13:59] <mvo> mborzecki: uh, so we need a new strategy for this it seems
[13:59] <mvo> mborzecki: that's unfortunate
[14:39] <jdstrand> mborzecki, ijohnson: hey, note, I'm just passing through in the forum (I'm on holiday). fyi since I responded to a couple of topics you responded to
[14:40] <mborzecki> jdstrand: thanks!
[14:40] <mborzecki> jdstrand: fyi, that library has setuid set
[14:40] <jdstrand> mborzecki: yeah, that sounds a bit scary. wonder if it was copy and waste somewhere...
[14:41] <jdstrand> mborzecki: it could also be an overly ambitious postinst script. grepping /var/lib/dpkg/info/* for the lib might provide a clue
[14:44] <mborzecki> jdstrand: no happy postinst script, but found this in lintian/overrides: https://paste.ubuntu.com/p/k5Zprvx92V/
[14:45] <mborzecki> jdstrand: and there are hooks that setup LD_PRELOAD in Xsession.d :/
[14:45] <jdstrand> mborzecki: that sounds terrible
[14:46] <jdstrand> mborzecki: what is shipping that?
[14:47] <mborzecki> jdstrand: https://launchpad.net/ubuntu/+source/gtk3-nocsd
[14:47] <jdstrand> mborzecki: look at all the lintian overrides. that is a terrible hack
=== ricab|lunch is now known as ricab
[14:48] <jdstrand> mborzecki: I would never want arbitrary library for my setuid binaries. ping, snap-confine, nothing
[14:50] <jdstrand> mborzecki: that has to be against policy. if it isn't, it should be. at best, the packaging should put something in place that allows the user to configure setting the setuid bit, with appropriate warnings
[14:51] <jdstrand> mborzecki: but not by default
[14:51] <mborzecki> jdstrand: i can file a bug about it, or would you prefer to do it and include some security perspective?
[14:52] <jdstrand> mborzecki: perhaps you can file a public security bug that just describes the problem wrt snap-confine, along with the lintian bits. then a member of our team will look at it
[14:52] <mborzecki> jdstrand: ok
[14:53] <jdstrand> it's a little more than I want to chase down while on holiday :) either a member of the team will act on it or it will be in my inbox nagging me to do something about it :)
[15:06] <mborzecki> jdstrand: filed: https://bugs.launchpad.net/ubuntu/+source/gtk3-nocsd/+bug/1857022
[15:06] <mup> Bug #1857022: gtk3-nocsd preloads a setuid library <gtk3-nocsd (Ubuntu):New> <https://launchpad.net/bugs/1857022>
[15:08] <jdstrand> mborzecki: thanks!
[15:08] <mborzecki> jdstrand: np
[15:26] <kenvandine> ctOS: wayland or X?
[15:45] <cachio> mvo, https://paste.ubuntu.com/p/h8t4vMYfWv/
[15:47] <cachio> mvo, dd: writing '/dev/sda': No space left on device
[15:48] <cachio> this is failing
[15:59] <cachio> mvo, trying with a metter instance now
[16:02] <mup> PR snapd#7946 opened: tests: fix partition creation test <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/7946>
[16:05] <cmatsuoka> cachio: it was not the problem I thought it was, it was something much simpler
[16:05] <cachio> cmatsuoka, yes I saw that
[16:05] <cachio> I gave +1 already
[16:05] <cmatsuoka> thanks!
[16:22] <cachio> mvo, well, with more disk I got this error https://paste.ubuntu.com/p/XYxQpCNKK4/
[16:22] <cachio> mvo, it is needed a change in the spread.yaml
[16:22] <cachio> https://paste.ubuntu.com/p/P6tbMRpxMs/
[16:23] <cachio> it is needed more storage
[16:25] <cachio> mvo, just changing the storage it seems to work until console conf
[16:26] <cachio> mvo, this is the full log: https://paste.ubuntu.com/p/3bCZx6dDrw/
[16:48] <mvo> cachio: in a meeting, will get back to you
[16:55] <cmatsuoka> cachio: I wasn't completely wrong, after fixing that trivial problem the other one appears, so I'm fixing the other as well
[16:55] <cmatsuoka> s/fixing/adding a workaround for/
[17:01] <mvo> cachio: the full log looks kind of ok, I mean, it looks like it going into run mode eventually,
[17:01] <mvo> cachio: like e.g. [   32.825784] audit: type=1400 audit(1576772640.792:12): apparmor="STATUS" operation="profile_load" profile="unconfined" name="snap.pc.hook.configure" pid=1585 comm="apparmor_parser" in line 4107
[17:01] <mvo> cachio: but I guess you don't get a connection to this?
[17:03] <cachio> mvo, hey
[17:03] <cachio> I can access to the console
[17:03] <cachio> mvo, I just triggered a new run
[17:06] <mvo> cachio: ok, if you have access to a debug console (you can e.g. add "systemd.debug-shell=1" in the grub commandline. then it would be nice to see the output of "snap changes" or journalctl -u snapd-seeding or snapd
[17:09] <cachio> mvo, sure
[17:22] <cachio> mvo, is it any way to scape from console-conf?
[17:22] <cachio> escape
[17:23] <mvo> cachio: I think not right now - but you should be able ssh into the instance
[17:26] <cachio> mvo, it does not allow me to ssh
[17:27] <cachio> mvo, which user/pass should I use?
[17:27] <cachio> I tried with root and didn't work
[17:30] <cachio> test user nither
[17:30] <cachio> neither
[17:41] <mvo> cachio: meh, ok
[17:41] <cachio> trying again
[17:49] <ctOS> kenvandine: x
[18:17] <mup> PR snapd#7946 closed: tests: fix partition creation test <Simple 😃> <UC20> <Created by cmatsuoka> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/7946>
[18:43] <cmatsuoka> bbin30min
=== ijohnson is now known as ijohnson|lunch
[19:38]  * cachio afk
[20:43]  * cachio afk
=== heather is now known as hellsworth
=== ijohnson|lunch is now known as ijohnson
[23:12] <mup> PR snapd#7947 opened: boot/many: support new UC20 style kernel extraction <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7947>
[23:16] <ijohnson> have a nice holiday break everyone!
=== ijohnson is now known as ijohnson|EOY