/srv/irclogs.ubuntu.com/2020/03/30/#snappy.txt

mupPR snapd#8374 opened: .github: register a problem matcher to detect spread failures <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8374>04:40
mupPR snapd#8375 opened: github: always run the "Discard spread workers" step, even if the job fails <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8375>05:39
mborzeckimorning05:43
mborzeckihmm core20 hangs in spread tests again?06:15
mborzeckilooks like it's hitting job kill-timeout when rebooting during prepare06:16
mborzeckihmm perhaps https://github.com/snapcore/snapd/pull/8373 fixes that06:21
mupPR #8373: tests/lib/prepare.sh: use only initrd from the kernel snap <UC20> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8373>06:21
zygao/06:26
jameshmorning zyga06:26
zygahey james06:26
jameshzyga: here's a simple PR for the github actions CI: https://github.com/snapcore/snapd/pull/837506:27
mupPR #8375: github: always run the "Discard spread workers" step, even if the job fails <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8375>06:27
jameshmake the cleanup run even if the tests fail06:27
zygasuper nice06:27
zygathank you James06:27
zygaI didn't know about the implicit if06:27
jameshI also had a go at adding a problem matcher: it adds the error to the annotation list, but doesn't jump to the log line as I thought it did: https://github.com/snapcore/snapd/pull/837406:28
jamesh(maybe I was misremembering how that worked)06:28
mupPR #8374: .github: register a problem matcher to detect spread failures <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/8374>06:28
zygaI love this :)06:28
zygalittle by little but even this is so much better than what we had before06:29
zygathat's fantastic06:29
zygaI think we could move more stuff over this week06:29
zygalet's see what we can get06:29
mborzeckizyga: jamesh: hey06:30
jameshhi mborzecki06:30
mborzeckiwish the uc20 spread job actually worked06:32
mborzeckimvo: hey06:32
zygahey mvo06:32
* zyga is pre breakfast still so will go away for a while 06:33
jameshzyga: Longer term, it might make sense to have Spread output error annotations when it thinks it is running under the Github runner06:33
jameshit could provide extra info like the task file name06:33
mborzeckijamesh: hopefully the runner makes it easy to detect such scenario06:34
zygajamesh: yeah, I think that would be great, spread has specific travis features, it could have github features06:34
zygaI think the goal should be to phase out travis06:34
zygaso that nobody is stuck waiting06:34
zygathen optimize06:34
mborzeckiotherwise it's ugnly job level  env vars again or somesuch :/06:34
jameshmborzecki: there are environment variables it is documented to set, similar to Travis06:34
jameshhttps://help.github.com/en/actions/configuring-and-managing-workflows/using-environment-variables says it will always set GITHUB_ACTIONS=true06:36
mborzeckiha nice06:38
mvohey mborzecki and zyga !06:38
zygahow are you doing? it's quite cold today06:39
zygayesterday was both +18 and -2 at night (and snow)06:39
mborzeckizyga: it was ~13C before noon, then 2 with a snow/rain mix in the afternoon here06:40
mborzeckifelt like the afternoon was a completely different day06:40
zygabreakfast time06:51
mupPR snapd#8375 closed: github: always run the "Discard spread workers" step, even if the job fails <Created by jhenstridge> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8375>06:53
mupPR snapd#8365 closed: seed: add Info() method for seed.Snap <Simple 😃> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8365>06:58
pstolowskimorning07:03
mvohey pstolowski - good morning07:04
mvofwiw lxd tests are broken right now07:04
mvonot sure why yet, but it seems pretty consistent07:04
mupPR snapd#8374 closed: .github: register a problem matcher to detect spread failures <Created by jhenstridge> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8374>07:12
jameshmvo: thanks for doing the merges.  I think the problem matcher is about as good as it can get: further improvements would probably need to happen in Spread itself07:18
mvojamesh: yeah, I was thinking that we might need to modify spread07:19
mvojamesh: that's fine, thanks again for this, it's so much better than what we had before07:19
jameshmvo: e.g. it would be nice to tie the error to the task.yaml file, but the problem matcher can only use information present in the error message07:19
mborzeckipower outage :/ moved to my parent's07:24
mborzeckitravis job failed again in #837307:31
mupPR #8373: tests/lib/prepare.sh: use only initrd from the kernel snap <UC20> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8373>07:31
mvomborzecki: probably the lxd issue07:31
mvomborzecki: also, does it have two reviews?07:31
mborzeckikill-timeout reached after mar300623-305765 (google:ubuntu-core-20-64:tests/regression/) reboot request07:31
mborzeckimvo: all test suites failed on uc2007:32
mborzecki(on prepare)07:32
mvomborzecki: uh, no that then :)07:32
mvomborzecki: right, because of the kernel/initramfs issue?07:32
mborzeckiwish the console was easily accessible07:32
mborzeckimvo: yeah07:32
mvomborzecki: does it only fail in gce or also with qemu?07:32
mborzeckimvo: haven't tried qemu yet, i'll try to obtain the image from gce07:33
zygare07:33
mvomborzecki: yeah, qemu should give a lot more visiblity into this, so we still don't know what is going on :( ?07:33
zygasorry, it's hard to organize kids still07:33
pedronismvo: hi, #8325 has a conflict now also as we know needs a bit more work07:58
mupPR #8325: snap-bootstrap: copy auth data from real ubuntu-data in recovery mode <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8325>07:58
mvopedronis: thanks, look today (many meetings this morning unfortunately)07:59
mborzeckiany clue why it's no longer possible to connect via ssh to a spread node?08:29
mborzeckispread can do it, but once i got the debug shell, i cannot establish any new ssh connections08:30
pedronismborzecki: hi, I made a comment in the stand up notes about a thing that landed Friday, also did review some of your open PRs08:39
mborzeckipedronis: thanks!08:39
zygare08:40
mborzeckipedronis: also wondering whether we should support GET on /v2/systems/{label}, but there does not seem to be any potential use cases atm08:42
pedronismborzecki: we can add it later, but yes would leave it alone for now08:42
mborzeckiack08:42
mupPR snapd#8376 opened: daemon: make POST /v2/systems/<label> root only <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8376>08:48
pedronisdegville: hi, could you look at this when you have a moment: https://github.com/snapcore/snapd/pull/8372#discussion_r40002649808:52
mupPR #8372: [RFC] devicestate: generate warning if seeding fails <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8372>08:52
degvillepedronis: morning - looking now!08:54
mborzeckisooo when i apt install openssh-server to pull in the update of sshd, i can log in over ssh again09:08
zygaBrb09:28
mborzeckii pulled the uc20 image built from master branch and it seems to work09:35
zygamborzecki: cgroup code I mentined yesterday is in 837709:58
mborzeckizyga: ack09:58
mupPR snapd#8377 opened: sandbox/cgroup: add ProcessPathInTrackingCgroup <Created by zyga> <https://github.com/snapcore/snapd/pull/8377>09:58
zygaI'll get some mint10:47
zygabrb10:47
mborzeckimoving back home10:52
zygashit, I forgot about the mint/tea10:53
zygabrb10:53
zygareally10:53
zygaI'll push a fix for session-tool, sorry for annoying failures everyone10:53
mupPR snapd#8378 opened: o/configcore: introduce core config handlers (3/N) <Created by stolowski> <https://github.com/snapcore/snapd/pull/8378>11:06
mborzeckiand back11:12
* zyga stresses session-tool to see if the issue is gone11:21
pedronismvo: commented on #8325 also there is stil a conflict11:43
mupPR #8325: snap-bootstrap: copy auth data from real ubuntu-data in recovery mode <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8325>11:43
mupPR snapcraft#2997 opened: specifications: progressive delivery <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2997>11:55
* zyga is surpsised by the surge of snow outdoors!!!11:56
threshsince https://snapcraft.io/docs/personal-files-interface says it requires snapd 2.37+, how can I check for snapd version in the wrapper script that launches my app?12:00
zygathresh: don't use assumes: [snapd2.37]12:00
zygahttps://snapcraft.io/docs/snapcraft-yaml-reference and search for "assumes"12:01
threshzyga, will that force user to upgrade to the needed version if they somehow fetch my snap on an older machine?12:01
zygathresh: it will prevent installation on an older snapd12:01
threshthat is not something I would want, then12:01
zygathresh: but we're not seeing old snapd's in the wild12:01
zygathresh: why?12:01
zygathresh: snapd has a self-update mechanism12:02
threshOK12:02
zygathresh: virtually everyone runs the current stable version12:02
threshI mean, I assumed not everyone updates12:02
threshThanks12:02
zygagood luck :)12:02
* zyga cannot reproduce session-tool failure on ubuntu, I'll try some other distro and send a PR next12:02
mborzeckiwell, there are those that use snapd from distro repositories12:10
mborzeckibut most major distros should be covered and have something quite recent available12:10
pedronismvo: this is fixed, right? https://bugs.launchpad.net/snapd/+bug/186775512:12
mupBug #1867755: snapd fails to build in focal, unit test clientSuite.TestClientFindFromPathErrIsWrapped fails <snapd:New> <https://launchpad.net/bugs/1867755>12:13
mvopedronis: yes, let me close it12:42
ijohnsonmorning folks12:42
* ijohnson is super tired today for some reason12:42
cwayneijohnson: same here, but i think just cause it's rainy and gloomy and monday12:53
ijohnsoncwayne: since this is happening to both of us I think it's totally fair to blame the CDG airport12:53
cwayneway ahead of you12:53
ijohnson(in addition to the mondayness of today)12:53
cwayneit is pretty peak monday12:54
ijohnsonhow's your treadmill working out12:54
cwaynenot too badly, averaging about ~4mi per day so far on it i think12:55
ijohnsonnice12:55
cwayneeveryone laughed at me for panic buying a treadmill, then they closed the gyms the next day and now i'm supposed to stay home so HA12:55
* ijohnson has gotten really bad at not exercising more with the lockdown and all12:55
cwaynei've been exercising more since hte lockdown, but ive also eaten everything on planet earth.12:56
cwaynelike my wife keeps stockpiling food12:56
cwayneand them im like OH SHIT FOOD12:56
cwayneand then its gone.12:56
ijohnsonit's a good thing you're not in MN otherwise we both would have eaten all the food here, it'll probably take us a while to eat our way through the eastern US and new england before there's none left12:57
cwaynechristina goes "that should last us a week" and then im just like "lol sure watch this"12:59
* ijohnson imagines cwayne going "hold my treadmill"12:59
cwaynetotal re-emergence of 2012 vintage fat cwayne13:00
cwaynefun times13:00
mborzeckicmatsuoka: cachio: standup is now ;) tz was switched over the weekend13:02
cachioah13:04
cachiook13:04
cmatsuokaoh ouch13:05
=== tai271828__ is now known as tai271828_
mupPR core20#22 closed: run-snapd-from-snap: don't try to load a snapd snap from the seed anymore <Created by anonymouse64> <Merged by xnox> <https://github.com/snapcore/core20/pull/22>13:33
zygabrb, coffee13:42
zygaamazing sun + snow experience toda13:42
ijohnsonmvo did you see that folks responded on the lxd bug ?13:43
ijohnsonzyga: ackk what's the special mount ns experimental setting I need to set to let the maas snap refresh ?13:56
ijohnson`snap set system experimental.<something> true` ?13:56
ijohnsonnvm I see that it's `robust-mount-namespace-updates`13:57
zyga ijohnson snap set core experimental.robust-mount-namespace-updates=true13:57
ijohnsonyep that's it13:57
mvoijohnson: I did not, let me check13:58
zygawall of snow outdoors!14:03
roadmryay! I want snow, all we have had is 2 crappy days of rain (even indoors, stupid roof leak)14:07
cachiomvo, I tested the lxd test installing lxd from edge and hte error is the same14:24
cachiomvo, I still see + lxd.lxc exec my-nesting-ubuntu -- lxd.lxc exec my-inner-ubuntu -- echo from-the-INSIDE-inside14:25
cachioError: EOF14:25
mvocachio: nice, let's add this to the LP bugreport14:37
cachioI already did ti14:39
cachioit14:39
cachiomvo, I forgot to ask about 2.44.1 to stable today14:42
cachioit is ok to strart the pregressive release today?14:42
mvocachio: either today or tomorrow, if all looks good14:42
mvocachio: fine to start phasing today14:43
cachiomvo, nice, thanks14:43
mborzeckipedronis: we want /var/lib/snapd/seed RO, but /run/mnt/ubuntu-seed is TBD15:00
pedronisyes15:01
mborzeckiok, cool15:01
mupPR pc-amd64-gadget#39 opened: Drop no longer used snap_kernel fallback <Created by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/39>15:06
mupPR snapd#8379 opened: many: comment or avoid cryptic snap-ids in tests <Created by pedronis> <https://github.com/snapcore/snapd/pull/8379>15:22
alan_gcachio, looks like the rawhide image needs refreshing: https://api.travis-ci.com/v3/job/308689304/log.txt15:23
cachioalan_g, hui, fedora refresh is scheduled for today, it is gonna happen in few hours15:25
cachioalan_g, I'll let you know when it is ready15:26
alan_g:)15:26
mupIssue core20#29 opened: core20 ubuntu-seed mount point should be read only <Created by xnox> <https://github.com/snapcore/core20/issue/29>15:32
mupPR core20#30 opened: Make /var/lib/snapd/seed bind-mount ro <Created by xnox> <https://github.com/snapcore/core20/pull/30>15:34
zygabrbb15:35
mupIssue core20#29 closed: core20 ubuntu-seed mount point should be read only <Created by xnox> <Closed by xnox> <https://github.com/snapcore/core20/issue/29>15:50
mupPR core20#30 closed: Make /var/lib/snapd/seed bind-mount ro <Created by xnox> <Merged by xnox> <https://github.com/snapcore/core20/pull/30>15:50
mupPR pc-amd64-gadget#39 closed: Drop no longer used snap_kernel fallback <Created by xnox> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/39>15:51
threshso I've been rereading the snapcraft.yaml syntax docs, and found out I didnt set license: and title:.  I did, but now I get "Issues while validating snapcraft.yaml: Additional properties are not allowed ('title', 'license' were unexpected)"15:52
threshyml is at https://code.videolan.org/thresh/vlc/-/blob/snap-fixes/extras/package/snap/snapcraft.yaml#L615:53
threshit's on snapcraft, version 2.43.1+18.415:56
threshand an unrelated question, do I need an avahi-observe interface if I already have network?15:59
zygare16:02
zygamborzecki: I've reduced the failing case and added logging to the tool itself16:02
zygamborzecki: now it doesn't fail, I wonder if that's because of (perhaps) synchronization due to logging and set -x16:03
zygathresh: I think license has a long-standing issue, as for title, I think it's gone (but I could be wrong)16:03
* zyga goes away to eat dinner16:03
zygare16:20
zygaINFURIATING it keeps working16:20
zygamvo: as for debian snapd package, you didn't commit the change to vcs16:24
zygamvo: the salsa git repo seems out of date (2.37 vs 2.44.1)16:24
=== pedronis_ is now known as pedronis
threshhow do I set the license that is shown in snap info vlc then?16:27
threshnow it's "unset"16:27
zygathresh: you can set it but I don't know if this will fly through the store,16:27
zygaCC degville ^ do you know if licenses work?16:28
pedronisit's meant to be set through the store atm16:28
threshIt's set in the store: License GPL-2.0+16:28
threshas from https://snapcraft.io/vlc16:28
degvillezyga / thresh: yeah, it's in the store page for a snap (https://snapcraft.io/docs/using-the-snap-store)16:30
threshdegville, thanks, I see it's set in the store: https://i.imgur.com/WKFIOLz.png, but I don't see it in "snap info vlc": https://i.imgur.com/uwIGcd8.png16:32
mupPR snapd#8380 opened: tests: add LXD_CHANNEL environment <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8380>16:33
pedronisthresh: snapd takes it from the snap16:33
pedronisnot all the dots are connected here16:34
threshthat sounds like a mess :-)16:34
threshwhere do I file a bug to get it fixed?16:34
pedronisthresh: well, snapcraft should let you set a license16:35
pedronisthe snapd side is this: https://bugs.launchpad.net/snappy/+bug/185367016:35
mupBug #1853670: snap always shows license as 'unset' after install <Snappy:Triaged by pedronis> <https://launchpad.net/bugs/1853670>16:35
pedronisthere's a long forum thread as well16:35
pedronispointed by that bug16:35
threshsnapcraft doesnt let me set one, I get "Additional properties are not allowed" when providing it16:36
threshso I guess the consensus moved to setting it on the store only16:36
mvozyga: I did commit to my repo I think16:36
mvozyga: and iirc there is a MP for my repo to the shared one16:37
zygaah, I see16:37
zygaI guess that's why16:37
pedronisthresh: there is no final consesus yet16:37
zygathresh: so I just checked16:37
zygathresh: I made a snap last weekend16:37
zygaand I did set a license16:37
zygawhen I don't have it16:37
zygasnap info ...16:37
zygadoesn't show the license (it says unknown)16:38
zygawhen I install it16:38
zygaI *do* see the license16:38
zygathresh: I set the top-level "license" field to a valid SPDX expression16:39
mupPR snapd#8381 opened: tests: skip lxd test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8381>16:39
threshhmm16:39
threshthanks for your help16:40
ijohnsonthresh: also note that you may have an older version of snapcraft which doesn't understand new keys, are you using snapcraft from the deb or from the snap ?16:49
zygaok, 20K iterations of the tool, I suspect there's something I did that fixed this bug16:49
zyganow let me think what I did16:49
* zyga checks diff16:49
threshijohnson, I'm using the snapcraft version from Ubuntu bionic, 2.43.1+18.0416:50
threshfrom a .deb16:50
zygathresh: snap install snapcraft --classic16:50
mupPR snapd#8382 opened: packaging: detect broken seed during focal and disable it <Created by mvo5> <https://github.com/snapcore/snapd/pull/8382>16:51
ijohnsonthresh: then your snapcraft is certainly out of date16:51
ijohnsonsee zyga's comment16:51
threshnot sure that's possible since my build environment is in a docker container16:51
threshbut we'll see16:52
zygaah16:52
zygathen probably not16:52
zygacan you not use docker?16:52
threshnot really, VLC CI uses it extensively16:52
zygahum16:52
zygathresh: please ask in #snapcraft16:52
zygaI don't really know16:52
ijohnsonthresh: there are ways to use modern snapcraft inside a docker container16:53
zygaI haven't tried docker in years16:54
zygaand not following the set of current trends16:54
zygaas to which fork to use16:54
zygathresh: note that there _is_ one thing that may help you16:55
zygathresh: snapcraft has a mode where it "destroys" the environment by installing deps directly onto the system16:55
threshI assume those ways are: setting up systemd inside, and dropping all security boundaries docker (might) provide16:55
zygathresh: but please ask in #snapcraft for details16:56
threshthank you zyga!16:56
* zyga reproduced the error and has more logs17:00

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