mupPR snapd#7916 closed: interfaces/browser-support: add more product/vendor paths <Created by Erick555> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/7916>01:30
ctOSHi. 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:10
ctOSAu, auto-connections and stuff. Okay, got it.04:48
mupPR 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>05:37
=== Girtablulu_ is now known as Girtablulu
mupPR 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:39
mvohey mborzecki06:45
mborzeckimvo: hey06:46
mborzeckimvo: any PRs you'd like me to look at?06:46
mvomborzecki: thank you, all the criticial stuff has landed06:49
mborzeckiyay :)06:49
mvomborzecki: exactly06:50
mvomborzecki: and I have a (hacked) spread setup that runs hello world in uc2006:50
mborzeckimvo: oh cool, as in boots into uc20 and runs a test?06:52
mvomborzecki: correct06:54
mvomborzecki: it's very raw right now (and needs a hacked spread for uefi support with qemu). but it's getting there :)06:54
mvomborzecki: I hope to proposed a slightly cleaned up version today06:55
mborzeckimvo: wondering whether gcp supports booting with uefi06:56
mborzeckimvo: btw. somebody is trying out gadget updates https://forum.snapcraft.io/t/gadget-schema-for-ubuntu-image-parser-might-be-broken/1472306:58
mvomborzecki: yeah, cachio explored this last night07:00
mvomborzecki: we have a ubuntu-1804-64-uefi-enabled image for this now07:00
mvomborzecki: oh, interessting!07:00
mvomborzecki: looks like we really need to take over ubuntu-image ;)07:01
mvomborzecki: or rather make it use our gadget parser07:01
mvomborzecki: but that's for later07:01
mborzeckimvo: yeah, maybe, i can look into adding that to the schema07:01
mvomborzecki: if it's not too much distraction that would be nice07:02
mvomborzecki: I should be able to look at reviews today again, anything you would like me to prioritize?07:02
mborzeckimvo: hopefully sil2100 is still around today ;)07:02
mborzeckimvo: this one would unblock rest of snapd on core: https://github.com/snapcore/snapd/pull/777207:02
mupPR #7772: wrappers: write and undo snapd services on core <Remodel 🚋> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7772>07:03
mvomborzecki: cool, I have a look, this is really exctiin g for me07:03
mborzeckiquick errand, back in 3007:48
ctOSmborzecki: (quick follow-up from yday): the beta channel resolved the font issue, but not the tiny mouse cursor.08:01
mvoctOS: 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:03
mupPR snapd#7943 opened: tests: add core20 tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/7943>08:05
mvomborzecki: when you are back - where is OFMF.fd on fedora/arch?08:14
mvomborzecki: on ubuntu it's /usr/share/OVMF/OVFM_CODE.ms.fd08:14
mvomborzecki: but I wonder if I can build something for spread that is portable08:14
mborzeckimvo: let me see08:15
mborzeckimvo: ovmf /usr/share/ovmf/x64/OVMF_CODE.fd08:16
mborzeckictOS: 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:16
mvomborzecki: ta08:17
ackkhi, 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:19
ctOSmborzecki: the patch is included in the beta (287). I’ve also double-checked that it is indeed not fixed in this version.08:21
mborzeckictOS: thank you!08:23
mborzeckimvo: on fedora i have /usr/share/edk2/ovmf/OVMF_CODE.fd and there's also  /usr/share/edk2/ovmf/OVMF_CODE.secboot.fd08:23
mborzeckimvo: on arch the package is called 'ovmf', on fedora it's 'edk2-ovmf'08:25
mvomborzecki: thanks08:28
mvomborzecki: still scratching my head about how to do this in a portable way, maybe an environment is the simplest and just documenting it08:28
mupPR 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
mborzeckipstolowski: hey08:29
mupPR snapd#7686 closed: systemd: handle preseed mode <Preseeding 🍞> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7686>08:32
mvohey pstolowski08:39
mvopstolowski: nice to see this merged!08:39
mupPR 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
pstolowskimvo, mborzecki ^ trivial and hopefully uncontroversial08:55
mvopstolowski: nice08:55
mborzeckipstolowski: +109:02
mupPR 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:54
mvoniemeyer: 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 uc2009:59
mupPR 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
mupPR spread#96: spread: add support for system specific "flags" and use in qemu <Created by mvo5> <https://github.com/snapcore/spread/pull/96>09:59
pstolowskitravis is super slow.. or is it my PR?10:05
mupPR 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:05
niemeyermvo: Of course10:22
niemeyermvo: Is that the same thing zyga mentioned last week? I was waiting for the ping on Friday10:24
mvoniemeyer: it's slightly different10:25
mvoniemeyer: 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 testing10:25
niemeyermvo: Ack10:26
mvoniemeyer: 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 default10:27
mvoniemeyer: also does not affect GCE testing so no need to do a new release and all that :)10:27
niemeyerCool, thanks10:27
mvothank you!10:28
pstolowskimvo, mborzecki any particular PRs you would like to land today & want reviewed?10:44
mborzeckipstolowski: https://github.com/snapcore/snapd/pull/7772 if you would10:45
mupPR #7772: wrappers: write and undo snapd services on core <Remodel 🚋> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7772>10:45
mupPR 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:14
mborzeckisil2100: hi, any chance you can take a look at this PR before the break? https://github.com/CanonicalLtd/ubuntu-image/pull/18011:36
mupPR 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:36
sil2100mborzecki: o/11:37
mborzeckisil2100: got one more patch with nicer error messages for all validation, but no clue whether older releases of voluptous raise useful exceptions11:38
mborzeckimvo: ^^11:39
mvomborzecki: nice one!11:42
cmatsuokagood morning. hopefully my internet will be fast and stable today11:45
pstolowskihi cmatsuoka !11:46
cmatsuokabecause yesterday it was just weird11:46
cmatsuokaa friend of mine is moving to Canada and is selling a NUC, I'm checking if it has TPM11:52
sil2100mborzecki: 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
mupPR 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>12:58
mborzeckisil2100: ha, there's one https://bugs.launchpad.net/ubuntu-image/+bug/185690313:00
mupBug #1856903: ubuntu_image/parser.py Gadget YAML Definition needs updating <gadget> <Ubuntu Image:New> <https://launchpad.net/bugs/1856903>13:00
sil2100mborzecki: hah, excellent13:00
sil2100mborzecki: could you add a changelog entry for the change with the LP: # added?13:01
sil2100mborzecki: or actually, I guess I'll just do that instead, so nevermind o/13:01
mborzeckisil2100: cool, thanks!13:02
sil2100mborzecki: oh, though some code-style stuff needs to be fixed first13:03
sil2100mborzecki: did you run `tox` before submitting the changes? Since the qa test-suite seems to fail due to pep8 errors13:03
sil2100mborzecki: 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.gz13:05
mborzeckisil2100: i run specific tests only, the whole suite never completed on my system13:06
sil2100mborzecki: could you address those pep8 errors and re-push?13:16
mborzeckisil2100: sure, will do13:16
=== ricab is now known as ricab|lunch
mborzeckisil2100: pushed13:29
mborzeckimvo: have you seen something like this: https://paste.ubuntu.com/p/YcrWJxsK5d/13:34
mborzeckimvo: that's core18 snapd failover, maybe systemd doesn't like that much when a service changes it's own unit file13:35
mvomborzecki: uh, haven't seen this one13:57
mborzeckimvo: i can reproduce it locally13:58
mborzeckimvo: https://paste.ubuntu.com/p/y2SWwD38rF/13:58
mvomborzecki: uh, so we need a new strategy for this it seems13:59
mvomborzecki: that's unfortunate13:59
jdstrandmborzecki, 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 to14:39
mborzeckijdstrand: thanks!14:40
mborzeckijdstrand: fyi, that library has setuid set14:40
jdstrandmborzecki: yeah, that sounds a bit scary. wonder if it was copy and waste somewhere...14:40
jdstrandmborzecki: it could also be an overly ambitious postinst script. grepping /var/lib/dpkg/info/* for the lib might provide a clue14:41
mborzeckijdstrand: no happy postinst script, but found this in lintian/overrides: https://paste.ubuntu.com/p/k5Zprvx92V/14:44
mborzeckijdstrand: and there are hooks that setup LD_PRELOAD in Xsession.d :/14:45
jdstrandmborzecki: that sounds terrible14:45
jdstrandmborzecki: what is shipping that?14:46
mborzeckijdstrand: https://launchpad.net/ubuntu/+source/gtk3-nocsd14:47
jdstrandmborzecki: look at all the lintian overrides. that is a terrible hack14:47
=== ricab|lunch is now known as ricab
jdstrandmborzecki: I would never want arbitrary library for my setuid binaries. ping, snap-confine, nothing14:48
jdstrandmborzecki: 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 warnings14:50
jdstrandmborzecki: but not by default14:51
mborzeckijdstrand: i can file a bug about it, or would you prefer to do it and include some security perspective?14:51
jdstrandmborzecki: 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 it14:52
mborzeckijdstrand: ok14:52
jdstrandit'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 :)14:53
mborzeckijdstrand: filed: https://bugs.launchpad.net/ubuntu/+source/gtk3-nocsd/+bug/185702215:06
mupBug #1857022: gtk3-nocsd preloads a setuid library <gtk3-nocsd (Ubuntu):New> <https://launchpad.net/bugs/1857022>15:06
jdstrandmborzecki: thanks!15:08
mborzeckijdstrand: np15:08
kenvandinectOS: wayland or X?15:26
cachiomvo, https://paste.ubuntu.com/p/h8t4vMYfWv/15:45
cachiomvo, dd: writing '/dev/sda': No space left on device15:47
cachiothis is failing15:48
cachiomvo, trying with a metter instance now15:59
mupPR snapd#7946 opened: tests: fix partition creation test <Simple 😃> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/7946>16:02
cmatsuokacachio: it was not the problem I thought it was, it was something much simpler16:05
cachiocmatsuoka, yes I saw that16:05
cachioI gave +1 already16:05
cachiomvo, well, with more disk I got this error https://paste.ubuntu.com/p/XYxQpCNKK4/16:22
cachiomvo, it is needed a change in the spread.yaml16:22
cachioit is needed more storage16:23
cachiomvo, just changing the storage it seems to work until console conf16:25
cachiomvo, this is the full log: https://paste.ubuntu.com/p/3bCZx6dDrw/16:26
mvocachio: in a meeting, will get back to you16:48
cmatsuokacachio: I wasn't completely wrong, after fixing that trivial problem the other one appears, so I'm fixing the other as well16:55
cmatsuokas/fixing/adding a workaround for/16:55
mvocachio: the full log looks kind of ok, I mean, it looks like it going into run mode eventually,17:01
mvocachio: 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 410717:01
mvocachio: but I guess you don't get a connection to this?17:01
cachiomvo, hey17:03
cachioI can access to the console17:03
cachiomvo, I just triggered a new run17:03
mvocachio: 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 snapd17:06
cachiomvo, sure17:09
cachiomvo, is it any way to scape from console-conf?17:22
mvocachio: I think not right now - but you should be able ssh into the instance17:23
cachiomvo, it does not allow me to ssh17:26
cachiomvo, which user/pass should I use?17:27
cachioI tried with root and didn't work17:27
cachiotest user nither17:30
mvocachio: meh, ok17:41
cachiotrying again17:41
ctOSkenvandine: x17:49
mupPR snapd#7946 closed: tests: fix partition creation test <Simple 😃> <UC20> <Created by cmatsuoka> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/7946>18:17
=== ijohnson is now known as ijohnson|lunch
* cachio afk19:38
* cachio afk20:43
=== heather is now known as hellsworth
=== ijohnson|lunch is now known as ijohnson
mupPR snapd#7947 opened: boot/many: support new UC20 style kernel extraction <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7947>23:12
ijohnsonhave a nice holiday break everyone!23:16
=== ijohnson is now known as ijohnson|EOY

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!