/srv/irclogs.ubuntu.com/2019/12/13/#snappy.txt

mupPR snapd#7900 opened: tests: do reset of tests during restore and add checks to validate the fs <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/7900>03:09
mupPR snapd#7901 opened: run-checks: check multiline string blocks in restore/prepare/execute sections of spread tests <Simple πŸ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7901>06:20
mborzeckimorning06:39
zygamborzecki: hello07:18
mborzeckizyga: hey07:18
mborzeckizyga: #7901 trivial PR07:34
mupPR #7901: run-checks: check multiline string blocks in restore/prepare/execute sections of spread tests <Simple πŸ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7901>07:34
zygamborzecki: thank you and done :-)08:03
mborzeckizyga: yay :) thx08:03
pstolowskigood morning08:08
mborzeckipstolowski: hey08:08
mvohey pstolowski and mborzecki08:14
mborzeckimvo:  hey08:14
mvosdhd-sascha: 7881 looks like it can land. did you sign the CLA already?08:18
sdhd-saschaGood morning08:19
sdhd-saschamvo: i think i have signed it on launchpad and on some homepage. Not sure if it was the same08:19
mvosdhd-sascha: cool, thank you08:21
mupPR snapd#7881 closed: intrefaces: login-session-control - added missing dbus commands <Created by sd-hd> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7881>08:26
mvo7858 looks like an easy win, just needs a second review08:26
mupPR snapd#7901 closed: run-checks: check multiline string blocks in restore/prepare/execute sections of spread tests <Simple πŸ˜ƒ> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7901>08:28
zygahey mvo08:44
mvohey zyga08:45
zygasorry for starting late, I had some things at home that needed my time08:45
mvothanks mborzecki for the 7899 review!08:45
zygaI need to work a little over weekend08:45
zygaand catch up08:45
mborzeckimvo: np, did't want to commit that whitespace change since travis is already running, we can clean it up when doing some later changes08:46
mvomborzecki: yeah, I would prefer to land and fix in one of the followups08:47
mborzeckimvo: maybe you have permissions to push to https://github.com/snapcore/snapd/pull/789608:50
mupPR #7896: interfaces/wayland: Add access to Xwayland's shm files <Created by AlanGriffiths> <https://github.com/snapcore/snapd/pull/7896>08:50
mvomborzecki: I have the same issue, look like only Alan can do this :/08:51
mupPR snapcraft#2797 closed: meson plugin: add support for core20 <Created by sd-hd> <Closed by sd-hd> <https://github.com/snapcore/snapcraft/pull/2797>08:51
mborzeckioh, well, we'll just have to wait then08:52
mupPR snapd#7899 closed: many: pass consistently boot.Device state to boot methods <UC20> <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7899>09:02
Saviqmborzecki: hey, re: the qemu-img AppArmor denial, does this change look sane https://paste.ubuntu.com/p/k8Shd4WyCP/ (where %3…%4 is the full path to qemu-img) ?09:05
Saviqshould we have "m" for all the executables we want to run?09:05
Saviq(and why isn't this required on Ubuntu ΒΏ?)09:05
mborzeckiSaviq: does it work on debian with apparmor?09:22
zygamborzecki: we don't use apparmor on debian09:22
mborzeckiSaviq: or opensuse for that matter09:22
mborzeckizyga: it's multipass, and it's classic, from what i can tell, it'll try to use aapparmor when aa_is_enabled returns it's on09:24
zygaah09:25
zygathat's interesting09:25
zygamore confined than snaps :)09:25
zygaSaviq: on debian all file rules work the same09:25
mborzeckizyga: this is what I got on arch yday: https://github.com/CanonicalLtd/multipass/issues/122309:29
mborzeckizyga: unclear why it's trying to map qemu-img for reading when there's supposed to be ixr permissions09:30
zygalooking09:32
zygaafter some kernel version "m" became required as well09:32
pedronismvo: thanks for landing my PRs, hope they weren't too confusing ...09:34
mupPR snapd#7889 closed: overlord,o/snapstate: make sure we never leave config behind <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7889>09:38
pstolowskipedronis: hi, #7658 is ready for you re-review if you have a moment09:38
mupPR #7658: cmd/snap-preseed: add snap-preseed executable <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7658>09:38
pedronispstolowski: thx, saw that09:40
mvopedronis: no, great stuff, I'm in the weeds of uc20 right now09:40
mborzeckiSaviq: ok, got something that works https://paste.ubuntu.com/p/mV9fR5M3hf/09:41
mborzeckiSaviq: and looks like stderr may not be redirected when you're running qemu-img09:41
mborzeckiSaviq: /snap is a symlink on arch & fedora09:42
mborzeckiSaviq: ok, added a note too09:46
pedronisI created the Done 2.44 lane09:46
Saviqmborzecki: thanks!09:47
mupPR core20#16 opened: bootstrap from locally build snapd snaps too <Created by mvo5> <https://github.com/snapcore/core20/pull/16>09:51
mupPR snapd#7893 closed: daemon,snap: remove screenshot deprecation notice <Created by robert-ancell> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/7893>09:57
sdhd-saschaWhy does the x11 interface has no support to access: @/tmp/.X11-unix/X[0-9]*10:10
sdhd-saschaIn the plug-case10:10
zygapstolowski: https://github.com/snapcore/snapd/pull/7658#pullrequestreview-33176959310:10
zygasdhd-sascha: looking10:10
mupPR #7658: cmd/snap-preseed: add snap-preseed executable <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7658>10:10
sdhd-saschazyga: i see, there is access to Xauthority. But sway's source want to access the socket.10:11
zygasdhd-sascha: did you look at abstractions/X?10:11
pstolowskizyga: thanks!10:11
sdhd-saschazyga: wait10:11
zygasdhd-sascha: there are include rules in that snippet10:11
zygasdhd-sascha: those go to /etc/apparmor.d/abstractions/X10:11
zygaI suspect it is handled there10:11
zygapstolowski: description of https://github.com/snapcore/snapd/pull/7658 seems out of date, it sets SNAPD_PRESEED, not SNAPD_PREBAKE_IMAGE10:12
mupPR #7658: cmd/snap-preseed: add snap-preseed executable <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7658>10:12
pedronispstolowski: I made a countercomment to zyga comment about the help.10:17
zygapedronis: the long description will never capture everything that is done by the implementation, nor should it, it should just be comprehensible10:20
zygathere are better places to capture details, like man pages for example10:21
zygapedronis: but that's a minor point, I didn't block or intend to block over that10:21
pedronisI made a middle ground suggestion10:26
mborzeckipedronis: this could use your input too https://bugs.launchpad.net/snapd/+bug/185606310:27
mupBug #1856063: snap set refresh.hold immediately triggers a refresh when set <snapd:Confirmed> <https://launchpad.net/bugs/1856063>10:27
mborzeckipedronis: perhaps we should also set up automatic 2h refresh.hold for core devices too10:27
pedronismborzecki: no but open to other options10:28
pedronismborzecki: it's by design that core images try to refresh as soon as possible10:28
pedronismborzecki: multipass images are a bit of a odd case, because of how they tend to be used10:30
mborzeckipedronis: maybe something we could do from cloud-init level?10:30
pedronismaybe10:31
pedronismborzecki: anyway I put some guardrails comment in the bug10:34
mborzeckipedronis: cool, thanks!10:34
pedronisme just noticed a different papercut10:43
sdhd-saschazyga: ah, and there it is again.10:54
sdhd-saschaBefore i start to hack 'snapd', a RFC could be nice. I need to split X11 into a X11-provider and a X11-consumer. To make things work.10:54
zygasdhd-sascha: why?10:54
zygasdhd-sascha: provider is the slot, consumer is the plug10:54
zygasdhd-sascha: they have separate properties10:55
sdhd-saschazyga: sway can run under X11, then the abstraction/X is enough10:55
sdhd-saschabut currently it creates a X11 socket...10:55
sdhd-saschaso i need the slot permissions, too10:55
zygasdhd-sascha: if it creates an x11 socket in a common path it needs to have an x11 slot10:55
zygasdhd-sascha: but is it a generic x11 server?10:55
sdhd-saschayes, but both didn't work10:55
sdhd-saschait's Xwayland inside X10:55
zygasdhd-sascha: should it make sense to connect an app to sway instead of to the system x11 slot?10:56
sdhd-saschainside sway10:56
sdhd-saschai run on a system, without any gui10:56
sdhd-saschai have no system x11 slot there10:56
sdhd-saschasway snap is the only desktop10:56
zygasdhd-sascha: I don't know the answers to my questions but I just want to highlight that splitting any interface into a -provide and -consumer is bad idea because it is already part of the existing design10:56
zygasdhd-sascha: then you just should have an x11 slot IMO10:56
ograyou should probably talk to the guys in #mir-server ;)10:57
ogra(the mir-kiosk snap supports XWayland (in fact all apps use i3wm by default for fullscreen)10:57
sdhd-saschaWell, i can look if the slot permission are good enough to work as plug too10:57
ogra)10:57
mupPR snapd#7902 opened: o/hookstate/ctlcmd: fix command name in snapctl -h <Simple πŸ˜ƒ> <Created by pedronis> <https://github.com/snapcore/snapd/pull/7902>11:04
pedronisthat's the papercut fix11:04
sdhd-saschaHey, if the slot permission is not enough. Then i could change to allow same names of slot and plug. E.g. allow X11 as slot and X11 as plug in the same snap. Or ?11:07
sdhd-saschaogra: thanks, i joined to #mir-server11:09
pedronispstolowski: thx for the review, I just noticed this randomly11:09
mborzeckihttps://bugs.launchpad.net/snapd/+bug/1856073 is interesting, systemd records the state of failed service unit and keeps it around even after the unit is gone, imo it's kind of useful, and don't think we should do anything about that11:11
pstolowskiyw11:11
mupBug #1856073: Failed service state recorded by systemd even after snap removal <snapd:Confirmed> <https://launchpad.net/bugs/1856073>11:11
pedronismborzecki: when does it forget though? next boot? never?  or are we missing a daemon-reload?11:15
mborzeckipedronis: when one runs reset-failed or next boot (maybe reexec too)11:16
pedronisok11:16
pedronismborzecki: yea, I agree we shouldn't go to out of our ways to change this, unless it creates problems if you reinstall the snapd or if it was a missing daemon-reload11:17
pedroniss/snapd/snap/11:17
zygamborzecki: it is useful, we can look at why snap install failed because services failed to start :)11:27
zygapedronis: there's a command to explicitly forget11:28
zygapedronis: otherwise it forgets on reboot, it's only in memory11:28
Saviqzyga: is /var/lib/snapd stable across distros?11:43
Saviq(for the /snap symlink)11:44
zygaSaviq: yes11:45
zygaSaviq: (thank god :D11:45
Saviqindeed11:46
zygaSaviq: not sure how it relates to /snap symlink, do you mean the /var/lib/snapd/snap symlink?11:46
zyga(the one that points there)11:46
mborzeckizyga: we haven't checked hannah montana linux though ;)11:46
zygaSaviq: then indeed that is stable :)11:46
Saviqzyga: I mean /snap β†’ /var/lib/snapd/snap, yes11:46
zygaSaviq: that's 100% stable11:46
zygamborzecki: :-)11:47
zygamborzecki: cannot wait for a distro /apps and /system11:47
mborzeckizyga: don't you have one right now? :P11:47
mborzeckinot exacly linux though11:47
zygamborzecki: no, it's /Applications ;-) and it can be localized11:48
zyga(but without changing the filesystem)11:48
zygajdstrand: (just FYI) https://github.com/snapcore/snapd/pull/7875#discussion_r35761166511:54
mupPR #7875: interfaces: refactor path() from raw-volume into utils with comments for old <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7875>11:54
zygadegville: I don't know if you ever worked with i18n in software, if you did it's old news, if you didn't you may find that interesting11:55
degvillezyga: thanks for the link - I did, briefly, when I was writing my own stuff, never from a real docs perspective.11:57
degvilleand that was with Qt, which had a very set way of doing things.11:58
zygabrb, need to get coffee and warm up12:12
zygaI'm so cold this week :/12:12
pokkzyga: it's been around 0 and raining here for a month, not to mention that the sun barely comes up at all :| I hate this time a year12:14
ograzyga, you should move to spain !12:15
zygapokk: indoors? :) ?12:24
zygapokk: my office is not insulated very well so it's ... very fresh in the morning12:24
pokkzyga: nah, we're used to cold weather so indoors it's decently warm. I'd much prefer -20 over this to be honest12:25
zygapokk: where are you based if I may ask?12:25
zygapokk: here it's around 3-7C during daytime and -1-ish at night12:26
pokkzyga: Sweden12:28
zygapokk: ah, I guess you don't build uninsulated anything, unlike us here12:29
mupPR snapd#7903 opened: overlord,boot: follow ups to #7889 and #7899 <Simple πŸ˜ƒ> <Created by pedronis> <https://github.com/snapcore/snapd/pull/7903>12:30
mupPR snapd#7902 closed: o/hookstate/ctlcmd: fix command name in snapctl -h <Simple πŸ˜ƒ> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7902>12:31
pstolowskipedronis: hey, i may need to discuss with you some aspects of services once again, it shouldn't take long. do you have a moment today?12:33
zygapedronis: that follow-up is just extra test, right?12:34
zygapedronis: looks good, just wanted to make sure I understand it12:34
pedroniszyga: yes, just doing the thing people ask to improve the test12:36
zygaunderstood12:36
zygathanks12:36
pedronispstolowski: maybe just after standup12:39
pstolowskipedronis: great12:39
pedronismborzecki: 7903 is probably for you when you have a moment (it's not urgent and it's small)12:44
pokkzyga: no, not really. Especially not the more newly built houses. We can take -30C without much problem here12:48
pedronisdegville: how should I submit comments for the glossary ?12:48
degvillepedronis: would it help if I paste it into a gdoc? or you can make the edits yourself, or leave a comment on the posts. Whichever is easiest for you.12:49
pedronisdegville: I just skimmed for now, I'll know more when I have done a good read through, I'll let you know (probably actually not today)12:50
pedronisdegville: thanks for it btw12:50
degvillepedronis: no problem, thanks!12:51
degvilleI'm still adding things.12:51
pedronisdegville: a quick thing though, I think you go the description of core and core16 reversed, core is the one that includes snapd12:52
pedroniss/you go/you got/12:52
degvillepedronis: oops, thanks!! I'll fix that now.12:52
pedroniscore16 doesn't carry it and is not stable/done yet, that part is correct12:52
zygabrb13:01
om26erHi! Is sergiusens off today (or is it the timezone difference he is not online ?)13:08
om26erI need to talk about https://github.com/snapcore/snapcraft/pull/2843 and know if that change is acceptable in principle13:09
mupPR snapcraft#2843: snap: only ship .pyc files, strip shared objects <Created by om26er> <https://github.com/snapcore/snapcraft/pull/2843>13:09
zygare13:09
zygajdstrand: https://github.com/snapcore/snapd/pull/7875 is close and needs a small amount of iteration from you to land13:10
mupPR #7875: interfaces: refactor path() from raw-volume into utils with comments for old <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7875>13:10
pedronisom26er: he is off until next year, maybe cjp256 can help13:12
cjp256om26er: it's an interesting one! it was in my queue to look at more closely today, but it's something sergiusens will certainly be opinionated about.  :D13:19
mupPR snapd#7896 closed: interfaces/wayland: Add access to Xwayland's shm files <Created by AlanGriffiths> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/7896>13:22
om26ercjp256 cool, we could have something similar for the python plugin as well, that will likely reduce the size of python based snaps13:28
ograom26er, while that is really cool, it should be handled via a snapcraft.yaml option though13:30
om26erogra I was writing a message to you to get your opinion as you had wrote on the forum about a similar topic13:31
om26er;-)13:31
ograi love the approach ... but as i said, it shuld be configurable ...13:32
om26erogra the above proposed change is only for Snapcraft package itself. I will investigate into the python plugin and figure out how to do that for it, so other snaps could use that feature as well13:34
ogra(you might want to build an "arch all" python snap that only ships .py files by default ... i.e. the exact opposite)13:34
ograah,  thought it was for the plugin actually13:34
ogra*I thought13:34
cjp256i agree13:39
cjp256ogra: i would have guessed the pyc was platform independent?  is that not the case?13:39
ogradunno, can you run a pyc file compiled on armhf on an amd64 machine ? i never tried13:40
ograspecially if it comes to HW related stuff like talking to sensors etc i suspect there are many arch specifics in the backend13:41
ogra*especially13:41
om26erpyc are platform independent...13:41
ograk13:42
=== ricab is now known as ricab|lunch
* om26er is working on a story on how he reduced a snap size by more than 60%13:44
om26erstripped shared objects, only shipped pyc files and by using Python 3.6 from core1813:44
cjp256i left my comments in the PR, but generalized stripping should be a flag soon enough (it's on my todo list).  and I think it would be interesting to provide a flag for the *.py code, under circumstances we know it'd be OK13:44
om26erfor the last it was only a matter of adding /snap/core18/current/usr/bin to PATH.13:46
om26erwonder if snaps are mounted at a different mount point on other distros ?13:46
cjp256what was core18 added to path for?13:47
om26erthanks cjp256, I will take a look13:47
om26ercjp256 to reuse python3 from there instead of shipping a copy with my snap (saves 7mbs)13:48
cjp256ah, well that's where I think you might run into problems if shipping only pyc13:48
cjp256if core18 updates with a patched python that's somehow incompatible with the separately shipped pyc?13:49
ograom26er, other distros might use some dir under /var/lib instead of /snap ...13:49
ogra/snap is very ubuntu specific13:50
mupPR snapd#7904 opened: snapcraft.yaml: add type, build-base and adopt-info, rm builddeb plugin <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7904>13:54
om26erwill figure out a solution for that, likely by looking at $SNAP and finding core18 directory relative to that13:55
om26er@cjp256 that's indeed a challenge but I am hoping python in core won't have any API breaking changes, though I need to read a bit more into this to get an idea of how probably a failure is13:56
Saviqmborzecki: would you please try https://github.com/CanonicalLtd/multipass/pull/1228#issuecomment-565418688 out when you have a moment?14:12
mupPR CanonicalLtd/multipass#1228: [utils] resolve symlinks to snap directories (Fixes #1223) <Created by Saviq> <https://github.com/CanonicalLtd/multipass/pull/1228>14:12
Saviqzyga, if you have something like Debian or Fedora on hand, that would be great, too ↑14:12
mborzeckiSaviq: nice, you collect the build artifacts!14:13
Saviqmborzecki: yeah, it was just too tiring to have to rebuild locally all the time :)14:15
Saviqyou're welcome to some code https://github.com/CanonicalLtd/multipass/blob/master/tools/report_build.py :)14:15
Saviqpretty simple to use, too https://github.com/CanonicalLtd/multipass/blob/master/.travis.yml#L13914:16
mupPR snapd#7903 closed: overlord,boot: follow ups to #7889 and #7899 <Simple πŸ˜ƒ> <Created by pedronis> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/7903>14:37
ijohnsonpstolowski: do you want me to just push to your snapcraft.yaml branch then? I think the only thing my branch has that yours doesn't is removal of the x-builddeb plugin14:44
pstolowskiijohnson: nope, it's fine, go ahead with yours (minus type:)14:45
ijohnsonpstolowski: ack thanks, I think I might take a few of your changes to the tests though that I missed when writing it yesterday14:45
ijohnsonthanks!14:45
pstolowskiijohnson: fyi, my branch incorporated some old changes from cmatsuoka, so it's really a team effort ;)14:46
ijohnson:-)14:47
ijohnsonI will credit both of you in my commit message14:47
ijohnsonsil2100: hey I was wondering about the UC images we have on cdimage, I'm wondering specifically what's the difference between http://cdimage.ubuntu.com/ubuntu-core/18/current/ and http://cdimage.ubuntu.com/ubuntu-core/18/stable/current/ ? the former seems regularly updated, but the latter hasn't been updated since august14:49
ijohnson(also let me know if you're the wrong person to ask about this)14:49
* cachio lunch14:51
mborzeckiSaviq: looking good, i was able to launch an instance14:55
Saviqmborzecki: cool, can you try a mount, too, please?14:56
=== ricab|lunch is now known as ricab
mborzeckiSaviq: mount failed: Error enabling mount support in 'lasting-mara' but nothing in dmesg, so probably not AA related14:58
mborzeckiSaviq: any way to get some info why it failed?14:58
Saviqmborzecki: journal for snap.multipass*14:58
SaviqIt try again with -vvv14:58
Saviq*Or14:58
mborzeckiSaviq: https://paste.ubuntu.com/p/5M9QQsCZyQ/14:59
mborzeckioh, and i need sshfs?14:59
sil2100ijohnson: hey, so the ones in the main directory are dailies built against edge, while all the others are built per given channel15:01
Saviqmborzecki: in the instance, that is installed automagically, and also it should have continue regardless of the profile failing to load, since we skip AA on that…15:01
Saviqso fix is incomplete15:01
mborzeckiSaviq: what follows is: gru 13 15:58:45 galeon multipassd[897299]: Failed to install 'sshfs', error message: ''15:01
* Saviq just got a fedora VM so should be able to repro15:01
sil2100ijohnson: so the stable/ ones are built with stable snaps, and we currently only build those before every point-release actually15:02
Saviqmborzecki: ah, so that's the real problem… wonder why, can you try `sudo apt install sshfs` inside the instance?15:02
sil2100ijohnson: we have daily images built against edge (so the ones in /) and against beta (so the ones in /beta)15:02
mborzeckiSaviq: ah, i've launched ubuntu core, let me try the regular distro15:02
Saviqmborzecki: right, that would be it, oof15:03
Saviq(we're working on a static sshfs snap to work on Core, too)15:03
Saviqhttps://snapcraft.io/install/multipass/fedora this is so awesome :D15:04
mborzeckiSaviq: hmm some trouble when stopping the vm https://paste.ubuntu.com/p/ktdPWR7XCX/15:04
Saviqmborzecki: it stopped, didn't it? ;P15:05
* Saviq will try to stop Core15:05
mborzeckiSaviq: you want  /{,var/lib/snapd/}snap/core18/*/{,usr/}lib/@{multiarch}/{,**/}*.so* rm, in those profiles instead of {,/var/lib/snapd/}/snap15:08
mborzeckiSaviq: otherwise the parser complains15:08
Saviqmborzecki: aha, /me fixes15:09
Saviqprobably why it failed to load15:09
Saviqerror: system does not fully support snapd: cannot mount squashfs image using "squashfs": mount:15:12
Saviq       /tmp/sanity-mountpoint-875787176: unknown filesystem type 'squashfs'.15:12
Saviqhrmf15:12
mborzeckiSaviq: multipass mount $PWD/spread-mini saving-chamois:~/foo kind of worked (despite apparmor), but I got an unexpected path in the vm, basically $HOME/~/foo :)15:17
Saviqmborzecki: well, it did what you asked it to do ;P15:19
mborzeckiSaviq: heh, right, presumptions about rsync/ssh like path handling15:20
Saviqmborzecki: please file an issue if you can (I can, too), we should fix15:21
mborzeckiSaviq: there you go: https://github.com/CanonicalLtd/multipass/issues/123015:27
Saviqmborzecki: πŸ‘‹15:27
Saviqnot trivial since we'll need to resolve this inside the instance15:28
mvosil2100: do you think you could look at https://github.com/snapcore/core20/pull/16 ?15:32
mupPR core20#16: bootstrap from locally build snapd snaps too <Created by mvo5> <https://github.com/snapcore/core20/pull/16>15:32
mvosil2100: this would unblock uc20 image creation15:32
mupPR snapd#7905 opened: snap-bootstrap: actually parse snapd_recovery label <Created by xnox> <https://github.com/snapcore/snapd/pull/7905>15:35
mupPR core20#17 opened: Drop encoding digits in units and code paths.  <Created by xnox> <https://github.com/snapcore/core20/pull/17>15:48
ijohnsonsil2100: would it be a lot of work to build the current/stable images everytime there's a stable updated to any of the included snaps for core images? i.e. for UC18, rebuild on core18, snapd, (and pc or pc-kernel on the 18/stable tracks) ?15:53
sil2100mvo: looking!16:09
sil2100mvo: btw. uc20 dailies are now up16:09
sil2100http://cdimage.ubuntu.com/ubuntu-core/20/16:09
mvosil2100: thank you!16:10
mvosil2100: nice16:10
sil2100ijohnson: hmm, well, we don't have such automation right now, we did plan on adding something like that at one point, but nothing available yet16:12
sil2100ijohnson: it's certainly possible16:12
sil2100Just needs cycle for someone to sit down on it16:13
mupPR pc-amd64-gadget#26 opened: Use the correct name for the recovery_system label <Created by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/26>16:17
pstolowskizyga, pedronis updated #765816:23
mupPR #7658: cmd/snap-preseed: add snap-preseed executable <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7658>16:23
pedronispstolowski: thx16:23
cachioijohnson, hey, you already commented on this one https://bugs.launchpad.net/snapd/+bug/185515516:24
mupBug #1855155: snap install hangs if internet is disconnected exactly right after err upload starts <snapd:New> <https://launchpad.net/bugs/1855155>16:24
cachioijohnson, any priority to set?16:24
ijohnsoncachio: sure I'll set the priority, it's low I think16:27
cachionice,16:27
cachioI'll update it in that case16:28
ijohnsonsil2100: it's not critical by any means, just would be nice to have since that's what multipass uses to boot images from and they are always out of date so you boot a new UC VM and it immediately needs to reboot twice for snap updates16:36
mupPR snapcraft#2844 opened: Add punctuation rule for comments <Created by hellsworth> <https://github.com/snapcore/snapcraft/pull/2844>16:40
=== heather is now known as hellsworth
zygapstolowski: ack, looking17:00
zygacachio: hey17:09
zygacachio: wanted to share a sneak peak17:10
zygahttps://www.irccloud.com/pastebin/Km7d1ouy/17:10
cachiozyga, looking17:10
cachiozyga, what is that?17:11
zygahttps://www.irccloud.com/pastebin/KfXWwBsb/17:11
zygacachio: iteration on the leak detector, generalized17:11
cachiozyga, nice17:11
cachiozyga, is there a PR with it?17:12
zygacachio: the tool doesn't know anything about mountinfo, it has pluggable invariant helpers that can check anything17:12
zygacachio: not yet, I'll make one soon but the tool grew a little since I started and I have some more tests to write17:13
zygacachio: I may start a new mini-project for it so that it is easier to iterate separately from snapd17:13
zygacachio: (snapd can have a snapshot that we are happy with)17:13
cachiozyga, nice17:13
zygacachio: it's also nice interactively, show-event can invoke the pager automatically, there's tab completion, it's robust in case of corrupt data, etc17:14
zygacachio: event log is really cool, just give me a sec to collect one from snapd run17:14
cachiozyga, which project is that?17:14
zygacachio: I haven't made one yet17:15
cachioa personal is gonna be?17:15
zygacachio: it's essentially on my laptop for the past two days17:15
cachioahh17:15
zygacachio: I think so, it's easier this way17:15
cachiozyga, when you have something to share just ping me, I'd like to use it17:15
cachioat least to play a bit17:16
zygacachio: I'd love to plug your package leak detector17:16
zygacachio: yeah, I'll make sure to push whatever I have by EOD today17:16
cachiozyga, great, thanks!!17:16
zyga:)17:16
cachiomvo, if you have 5 minutes, could you take a look to this one? 785117:20
cachioso we can merge it within 2.4317:21
zygacachio: event log from a partial run17:30
zygahttps://www.irccloud.com/pastebin/0jxLFhu1/17:30
zygathe -- messages show changing attributes17:31
cachiozyga, nice17:32
cachiothis + -show-output you have the whole picture17:32
zygashow-output?17:34
zygaas in output from the test?17:34
cachiono17:34
cachiothis is something that spread2 provides17:34
zygaaha17:34
zygacool17:34
zygawhat does it do?17:34
cachiowith all the details of hte execution in the remote host17:35
cachiothe output got from ssh17:35
cachiodisplayed on real time17:35
zygaI see, that's cool17:36
cachioI use it a lot for debug17:36
zygahttps://www.irccloud.com/pastebin/YnJQbOAR/18:09
zygacachio: ^18:09
zygaevents from this failed run https://www.irccloud.com/pastebin/TnEbAUhC/18:09
cachiozyga, nice, same outoput than the pr18:10
cachiosome tests failing18:10
cachiozyga, did you test is in a board?18:12
zygacachio: no, not yet18:12
zygacachio: I'm testing it in a side project with spread18:13
zygacachio: and in snapd18:13
zygacachio: though on a board it would be even more cool as the test history accumulates and you can see what past test may have done more clearly18:13
cachiozyga, how do you set the invariants to validate?18:14
zygacachio: there are several ways, the image can ship them in a few locations (/usr, /usr/local, /run)18:14
zygacachio: there's also awareness of $SPREAD_PATH so the project can ship some more18:14
zygacachio: in particular $SPREAD_PATH/tests/lib/testbed-invariants is looked at as well18:15
zygacachio: any executable file there is used18:15
cachiozyga, nice18:15
zygacachio: there's a priority where /run overrides, project which overrides os system18:15
zygacachio: there's a simple protocol where a tool is invoked with an option that points it where to store data18:16
cachiocool18:16
zygacachio: and a few commands to implement (status, set-baseline, has-baseline, compare) and expected behaviuor (either return a status code or print something to stdout)18:16
zygacachio: earlier I wrote a few such tools but all in one monster shell script and I abandoned that18:17
zygacachio: now each can be written in any language18:17
cachiozyga, hehehe18:17
zygacachio: and testbed-tool itself is in python18:17
zygacachio: the one used here, mountinfo, is a small shell wrapper around mountinfo-tool18:18
cachiozyga, would be nice to integrate it on our tests in some way18:18
zygacachio: I integrated it with our spread startup logic in a branch of snapd18:18
zygacachio: it hooks into the right spots18:18
zygacachio: tests are entirely unaffected18:18
zygacachio: tests _can_ hook into this as well, if there's a particular reason18:19
zygacachio: e.g. a test may want to say it will do something "persistent" like re-package the core snap or modify the disk image or whatever18:19
cachiozyga, nice, well it is an easy way to check invariants18:19
zygacachio: you might have noticed that the test log is carried over the re-imaging we do in ubuntu-core-* targets18:20
zygacachio: so you see what happened in ubuntu-classic and what happened after booting into the core image18:20
cachionice18:21
cachiothat's really useful18:21
cachionow that we are going to create uc2018:21
cachiotests18:21
zygayeah18:21
zyga:)18:21
zyganot ready for prime time yet but it's close18:22
cachiozyga, hehehe18:23
zygacachio: the log format is also human readable but supports fancy features like timestamps, custom key=value attributes and more18:23
zygacachio: and in emergency you can even echo >> to it18:23
zygacachio: and the log viewer handles that18:23
cachiowow18:25
cachiothe fact it shows the timestamp is really nice18:26
zygacachio: I only format the hours/minutes but since we have everything we can detect time rollback, huge jumps and adjust the display accordingly18:28
mupPR snapd#7092 closed: packaging: use snapd type and snapcraft 3.x <β›” Blocked> <Created by stolowski> <Closed by anonymouse64> <https://github.com/snapcore/snapd/pull/7092>19:14
mvocachio: re 7851> sorry, didn't managed to review thta yet21:07
cachiomvo, np21:07
cachionext monday21:07
cachioenjoy your weelend21:07
mvocachio: yeah, thank you! you too21:07
mupPR pc-amd64-gadget#26 closed: Use the correct name for the recovery_system label <Created by xnox> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/26>21:24
* zyga EODs22:53

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