/srv/irclogs.ubuntu.com/2018/10/09/#snappy.txt

=== chihchun_afk is now known as chihchun
mupPR snapcraft#2326 closed: plugins: remove implicit source <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2326>03:37
=== cpaelzer_ is now known as cpaelzer
mborzeckimorning05:09
mupPR core18#74 opened: hooks: use core18.start-snapd.service for ordering <Created by mvo5> <https://github.com/snapcore/core18/pull/74>05:29
mupPR core18#74 closed: hooks: use core18.start-snapd.service for ordering <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/74>05:45
mupPR snapd#5910 closed: snapshotstate: restore to current revision <Snapshots πŸ“Έσ Ÿ> <Created by chipaca> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5910>05:48
mupPR core18#75 opened: static: show snapd firstboot messages on /dev/console  <Created by mvo5> <https://github.com/snapcore/core18/pull/75>05:54
mupPR core18#75 closed: static: show snapd firstboot messages on /dev/console  <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/75>05:56
zygao/06:23
mborzeckizyga: hey06:25
mvohey zyga and mborzecki06:28
mupPR core18#76 opened: static: make run-snapd-from-snap wait for seeding <Created by mvo5> <https://github.com/snapcore/core18/pull/76>06:29
zyga:-)06:29
mupPR core18#76 closed: static: make run-snapd-from-snap wait for seeding <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/76>06:46
mvouh, on the pi I get "backend.go:319: cannot create host snap-confine apparmor configuration: cannot s07:05
mvoynchronize snap-confine apparmor profile: open /var/lib/snapd/apparmor/profiles/07:05
mvosnap-confine.snapd.1122.m8GcXdqQ5cDR~: no such file or directory" quite often07:05
mvoat firstboot seeding07:05
=== pstolowski|afk is now known as pstolowski
* mvo wonders what is going on there07:05
pstolowskimornings07:05
mvohey pstolowski07:05
zygamvo: didn't we fix that?07:15
zygamvo: is the pi using anything modern or is this the release image?07:15
zyga(release as in 16.04)07:15
mvozyga: let me check, it should use master07:16
mvozyga: rev is 2.35.4+git1480.gdb577fc07:17
mvofrom 7h ago07:17
zygahmm, then it looks real07:18
mvoI'm debugging a different issue right now, we look after this but it looks scary as it breaks first boot seeding07:19
zygak07:19
mvoits a race most likely07:20
mvodoes not happen all the time07:20
mvoand I think not at all on quick HW like amd6407:20
mvoplus it is (currently) mask by this other bug that console-conf runs too early07:20
zygamvo: is https://github.com/snapcore/snapd/pull/5917 still something we want (AFAIK you want to branch master so back ports are not needed)07:31
mupPR #5917: cmd/snap: attempt to start the document portal if running with a sess… <Created by zyga> <https://github.com/snapcore/snapd/pull/5917>07:31
mupPR snapd#5927 closed: cmd/snap-update-ns: remove empty placeholders used for mounting <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5927>07:31
mvozyga: I think its something we want07:32
zygak, just land it then07:32
zyganote, the master version landed already07:32
mvozyga: I will branch again once 2.35.4 is in candidate07:32
mvozyga: aha, ok07:32
zygathis is just a back port07:32
mvook07:32
pedronismvo: I still have a small image.go PR to do (now that everything in snapd itself has landed), can be backported after the main branching tough07:35
mvopedronis: ok, thank you07:36
mvozyga: I think I have an idea about the bug, will go full force once I am finished with the current console-conf sync issue07:48
zygalet me know what you find07:48
mupPR snapd#5951 opened: spread-shellcheck: fix interleaved error messages, tweaks <Simple πŸ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5951>07:55
=== chihchun is now known as chihchun_afk
mborzeckianyone wishing to review #5947 and #5948 ? feel free to do one or both08:12
mupPR #5947: many: cleanup remaining parallel installs TODOs <Parallel installs β›“> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5947>08:12
mupPR #5948: asserts, image: ensure kernel, gadget, base and required-snaps use valid snap names <Parallel installs β›“> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5948>08:12
mborzeckipstolowski: any of your hotplug PRs need attention?08:12
mborzeckipstolowski: i have #5860 open08:12
mupPR #5860: interfaces/hotplug: helpers and struct updates <Hotplug πŸ”Œ> <Created by stolowski> <https://github.com/snapcore/snapd/pull/5860>08:12
zygamborzecki: I can do after noon08:15
pstolowskimborzecki: you can review that one, yes; i generally miss Gustavo's reviews on all of them. i'll have new PRs today I think, i'm going to split my code and propose complete feature in 2-3 pieces, so there will be new material to review. thanks for asking!08:15
zygastill fighting that test08:15
=== chihchun_afk is now known as chihchun
mupPR core18#77 opened: hooks: ensure serial-console-conf also runs after core18.start-snapd.service <Created by mvo5> <https://github.com/snapcore/core18/pull/77>08:28
niemeyerpstolowski: Sorry, haven't managed to get to the rest of them yesterday (did get to the key gen one).. will put the others at the top of my queue today08:42
pstolowskiniemeyer: sure, thanks for the device key review, addressed your feedback and trying to land (travis is not cooperating, again)08:43
mupPR core18#77 closed: hooks: ensure serial-console-conf also runs after core18.start-snapd.service <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/77>08:50
mupPR core-build#36 closed: initramfs: fix shellchecks on bionic <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core-build/pull/36>09:07
* zyga -> small break09:21
* zyga is about to throw the towel on the overlayfs test09:25
ogracjwatson, did the builders become stricter with the recent changes ? i just did manual re-builds of some of my snaps that never had probs building and all of a sudden they fail to release ... there are warnings about host libs being pulled in but i think they were there before and did not make the snap get stuck09:29
cjwatsonogra: no09:29
ograweird09:29
cjwatsonogra: "failed to release" is from the store, not from the builders, anyway09:29
ograyeah, but i dont even see the re-built snaps in the store ... like they never arrived09:30
cjwatsonso this is either due to snapcraft or due to the review tools, probably; unlikely to be the builders09:30
cjwatsonogra: it's not fair to ask me to speculate on this without giving me a link09:31
ograhttps://launchpad.net/~build.snapcraft.io/+snap/3fc16b55a813a672666c6ff13690918e-xenial/+build/32386009:32
ograaha !09:32
ograTemporary failure in name resolution09:32
ogralooks liek some internal DC thing09:32
ogra(sorry, i only looked at build.s.io ... could have checked LP directly)09:33
cjwatsonThat's the ENOMEM issue on ackee09:33
cjwatsonIt is SUPER WEIRD09:33
ograare there chances clicking retry will work (else i'll just leave it as is for the moment, there were no important snaps re-built, only because of security notifications)09:34
cjwatsonWe have three workers that run various jobs such as store uploads; as of a few weeks ago, occasionally one of them gets into a state where it starts getting ENOMEM for all attempts to allocate memory, despite there being loads of memory free and all the other usual things you might check in this situation09:35
cjwatsonThis results in a bunch of things failing, including DNS resolution09:35
ograweird indeed ...09:35
ograyeah09:35
cjwatsonHmm, one of the workers is in that state right now09:36
ograit might be a good idea to forward the error message to the build.snapcraft.io log or some such09:36
ograespecially since the only link to look at LP is at the top of the log iself ... not sure how many people will actually use that to find out more09:37
cjwatsonThe BSI log is just the build log09:39
cjwatsonCan't do that09:39
cjwatsonI have a spec for improvements to LP's store upload process09:40
cjwatsonIt's probably not next on my list but maybe the one after that09:40
ograwell, this is mostly about the UI of build.s.io ... not even sure thats in your area of things :)09:40
ograhaving some direct link to the LP build or some more info than a bland "Failed to release" with no further info on the page would help09:42
cjwatsonEvan says we're not allowed :P09:50
ograhaha09:50
cjwatson(to have a direct link to LP)09:50
ograyeah ... i got that09:50
mupPR core18#78 opened: static: show seed progress on console on firstboot <Created by mvo5> <https://github.com/snapcore/core18/pull/78>09:51
mupPR core18#78 closed: static: show seed progress on console on firstboot <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/core18/pull/78>09:55
cjwatsonogra: Bug in the BSI UI, I think - it's meant to show it but there's a confusion about ![] not being true in JS09:56
ograah09:56
mupPR snapd#5952 opened: tests/ifacestate: moved asserts-related mocking into helper <Created by stolowski> <https://github.com/snapcore/snapd/pull/5952>10:05
mborzeckiwhy are timers/sockets enabled in StartServices() rather than AddSnapServices() ?10:07
pstolowskipedronis: can you take a look ^ ?10:07
pedronispstolowski: yes, after lunch10:07
cjwatsonogra: Anyway, you can try using Retry, though due to the various problems outlined in https://docs.google.com/document/d/1vabN2wBNtGdBqShN1xi9a-osfGPtmA8EARdXfs2kfDI it's possible that it will require a full rebuild in order to be uploadable10:09
ogracjwatson, well, i just tried retry and got a permission error10:10
ograseemingly i'm not allowed to retry10:10
cjwatsonA permission error saying what precisely?10:11
cjwatsonOh, from https://launchpad.net/~build.snapcraft.io/+snap/3fc16b55a813a672666c6ff13690918e-xenial/+build/323860 ?  Yeah, LP doesn't know that you own the snap on BSI.10:11
ograright10:11
cjwatsonI've hit Retry but it probably won't work10:11
ograwell, that particular snap is less important anyway ... and i'll trigger rebuilds for the others10:12
cjwatsonYeah, that hit the silly duplicate-snap-content error from the store10:14
cjwatsonWhich is one of the things my spec covers10:14
mupPR core18#79 opened: snap-snapd-from-snap: remove all debug output <Created by mvo5> <https://github.com/snapcore/core18/pull/79>10:37
Chipacaif/when you spot a unit test on travis taking ages talking to the apt mirror, please let me know10:57
ChipacaI want those logs10:58
Chipacae.g. https://travis-ci.org/snapcore/snapd/jobs/43905823210:59
mupPR snapd#5953 opened: apparmor: create SnapAppArmorDir in setupSnapConfineReexec <Created by mvo5> <https://github.com/snapcore/snapd/pull/5953>11:09
ograppisati, hmm, 4.4.0-1099-raspi2 doesnt have the modules fix yet ? (i just had an upgrade on my pi's and the splashes went away)11:11
=== pstolowski is now known as pstolowski|lunch
ogracjwatson, hmm, are we short on arm builders again (all my retried builds now sit in "building soon" state for all amr arches)11:15
ogra*arm11:15
cjwatsonogra: see #is-outage please11:16
cjwatsonwork in progress11:16
ograok11:16
mborzeckihttps://paste.ubuntu.com/p/gTn99BMxhg/ snap services with sockets and timers11:25
niemeyerpstolowski|lunch: Nitpick (and optional, obviously):11:36
niemeyer // attribute name and value are separated by null character11:36
niemeyerThat's exactly what the code below it says11:37
niemeyermore clearly11:37
dot-tobiaszyga: re: https://forum.snapcraft.io/t/snap-layouts/7207/4?u=tobias β†’ Added an answer, but it seems there's something else going on there. I'm running a command in my install hook to copy files over from $SNAP/folder/subfolder/* to $SNAP_DATA/subfolder, could it be that? This happens even when I completely remove the β€œpartly duplicated file name” section from the layout stanza.11:38
niemeyerpstolowski|lunch: Ah, there's something else.. will comment on the PR11:39
zygadot-tobias: I don't think the hook is a factor, it runs after the setup process, in any case, if you can share a snap that I can play with (even a dummy one that has this bug) it would help as I was unable to reproduce this11:40
dot-tobiaszyga: Will cook up something, thanks11:41
cachiopstolowski|lunch, hey, already working in nested suite11:56
mborzeckiChipaca: niemeyer: i see you've discussed including timers in snap services commands, i've posted a note in the topic https://forum.snapcraft.io/t/command-line-interface-to-manipulate-services/262/33 feel free to provide suggestions12:04
mborzeckicachio: hey, https://github.com/snapcore/spread-images/pull/17 had to execute those manually, but afaict the host after those changes appears to work fine12:06
mupPR spread-images#17: tasks/google/update-arch-linux: tweak Arch image update <Created by bboozzoo> <https://github.com/snapcore/spread-images/pull/17>12:06
cachiomborzecki, nice, checking12:09
mupPR snapcraft#2323 closed: go plugin: support for bases <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2323>12:20
niemeyermborzecki: Thanks!12:21
=== pstolowski|lunch is now known as pstolowski
Chipacaoh, what a surprise, opensuse failing agian :-((((((12:23
Chipacacan we add a rule, "to be supported by snapd, your distro's mirror archive must be *this* available"?12:24
mupPR snapcraft#2327 opened: pluginhandler: remove prepare, build and install scriptlets <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2327>12:26
cachiomborzecki, works!!12:29
niemeyermborzecki: Responded12:34
mupPR snapcraft#2328 opened: godeps plugin: support for bases <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2328>12:38
rbasakmvo_: I put a reproducer in bug 179601712:38
mupBug #1796017: git ubuntu build-source fails with missing libreadline.so.6 <usd-importer:In Progress by racb> <https://launchpad.net/bugs/1796017>12:39
rbasakI'm a bit baffled as to why this doesn't work. Is there anything else going on eg. seccomp?12:39
rbasakOr other necessary environment changes that I missed?12:39
cachiomborzecki,12:55
mupPR core18#79 closed: snap-snapd-from-snap: remove all debug output <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/79>12:58
ograargh13:10
ograsergiusens, !!!13:10
ograSnapping 'pi-kiosk' |13:10
ograSnapped pi-kiosk_18-0.1_OrderedDict([('build-on', 'amd64'), ('run-on', 'armhf')]).snap13:10
ograStopping local:snapcraft-sadly-choice-amoeba13:10
ograRetrieved "pi-kiosk_18-0.1_OrderedDict([('build-on', 'amd64'), ('run-on', 'armhf')]).snap"13:10
ograsergiusens, thats a snapcraft cleanbuild of a snap using build-on/run-on ...13:11
ijohnsonogra: stop naming your snap "pi-kiosk_18-0.1_OrderedDict([('build-on', 'amd64'), ('run-on', 'armhf')])"13:11
sergiusensogra: we'll get kyrofa to take a look when he gets in13:11
ograi'd kind of expect the resulting snap file not to contain python code in its name13:11
sergiusensbut, cleanbuild is also deprecated to a baseless world13:12
ograso how do i build my snaps now ?13:12
sergiusensogra: if you set a base, you get a build vm13:15
ograsergiusens, so i need to touich all the snaps i have ?13:15
ogra(for re-builds when getting security notifications ??)13:15
sergiusensogra: no, if you do not change anything, you don't need to touch anything13:15
ogracjwatson, i know you are busy with the outage ... but i see some weird cross build behaviour on the builders that i can not reproduce locally either with a plain build or withz a snapcraft cleanbuild13:17
ograchecking for arm-linux-gcc... /usr/bin/arm-linux-gnueabihf-gcc13:17
ograchecking whether the C compiler works... no13:17
ograconfigure: error: in `/build/pi-kiosk/parts/psplash/build':13:17
ograconfigure: error: C compiler cannot create executables13:17
mupPR snapcraft#2329 opened: pluginhandler: remove legacy plugin loading without project <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2329>13:17
ograthe fun part is that theer is a part before that just used /usr/bin/arm-linux-gnueabihf-gcc to build a binary13:18
cjwatsonogra: I have no idea, sorry.  Try catting config.log after the failure to get more details13:18
ograhmm., k13:19
mborzeckiChipaca: can you take a quick look at https://github.com/snapcore/snapd/pull/5951 ?13:30
mupPR #5951: spread-shellcheck: fix interleaved error messages, tweaks <Simple πŸ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5951>13:30
Chipacamborzecki: what do you think about my 'activator' comment?13:37
mupPR snapcraft#2330 opened: pluginhandler: remove big solidus workaround <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2330>13:38
* Chipaca ~> school run*13:44
Chipaca* wee (maybe)13:44
ackkhi, is there a way to specify the container image/release when running "snapcraft cleanbuild"?13:51
mupPR snapd#5759 closed: ifacestate: implementation of defaultDeviceKey function for hotplug <Hotplug πŸ”Œ> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/5759>13:52
=== chihchun is now known as chihchun_afk
ogracjwatson, here we go https://launchpadlibrarian.net/392453534/buildlog_snap_ubuntu_xenial_amd64_ae16fdc83ee020a5817d7f0243e105b5-xenial_BUILDING.txt.gz seems while my desktop and cleanbuild automatically install binutils-arm-linux-gnueabihf when i add a "build-packages" entry for gcc-arm-linux-gnueabihf, the LP builder doesnt do so14:01
ograat least thats my guess for:14:01
ograconfigure:2973: /usr/bin/arm-linux-gnueabihf-gcc    conftest.c  >&514:01
ogracollect2: error: ld returned 1 exit status14:01
ogragrrr ! IRC14:01
ograconfigure:2973: /usr/bin/arm-linux-gnueabihf-gcc    conftest.c  >&514:02
ogra/usr/lib/gcc-cross/arm-linux-gnueabihf/5/../../../../arm-linux-gnueabihf/bin/ld: cannot find crt1.o: No such file or directory14:02
ogra/usr/lib/gcc-cross/arm-linux-gnueabihf/5/../../../../arm-linux-gnueabihf/bin/ld: cannot find crti.o: No such file or directory14:02
ogracollect2: error: ld returned 1 exit status14:02
ackksergiusens, hi, maybe you have insights on my question above?14:02
ograaha !14:04
ogralibc6-dev-armhf-cross iss a recommends ... ok14:04
ogra*is14:04
ogracjwatson, thanks for pointing in the right direction i think i got a workaround for that one (though it is still weird that cleanbuild doesnt expose that issue)14:05
niemeyerHey folks.. I'm back14:15
ograwhee ! a properly cross built gadget snap on build.snapcraft.io !!!!14:26
ograhttps://launchpad.net/~build.snapcraft.io/+snap/ae16fdc83ee020a5817d7f0243e105b5-xenial/+build/34907714:26
ogravery nice !14:27
sergiusensackk: no, there is not; we are moving to a new model which will be trickled by the use of bases.14:44
mupPR snapd#5954 opened: [wayland] explicitly permit file locking for wayland lock <Created by gerboland> <https://github.com/snapcore/snapd/pull/5954>14:45
ograjdstrand, so i have https://dashboard.snapcraft.io/snaps/dashkiosk-client-image-config/revisions/11/ ... using a self-provuded udisks2 interface, is there any chance you would approve that or should i rip out the code using that interface ?15:00
ograjdstrand, source is at: https://github.com/ogra1/dashkiosk-client-image-config15:02
ackksergiusens, I see, so there's currently no way of cleanbuilding a core18 snap15:03
ondrakyrofa ping15:18
sergiusensackk: no, for that you probably want to be on edge and set SNAPCRAFT_BUILD_ENVIRONMENT=multipass (and set base: core18)15:19
kyrofaondra, hey there15:19
ackksergiusens, ah I see. I do use edge and core18 base15:20
ackksergiusens, so this will only be available on multipass?15:20
sergiusensackk: just missing the env then, but edge should switch to creating the VM by default real soon15:21
ackksergiusens, thanks15:21
sergiusensackk: for the time being yes, it what was agreed to during the roadmap sprint. Next cycle we will bring back lxd15:21
ondrakyrofa hey15:22
ondrakyrofa can you please check https://code.launchpad.net/~ondrak/+snap/toolbox15:22
ondrakyrofa for whatever reason on arm64 it breaks when priming15:22
cjwatsonogra: mm, may be the sort of thing that gets ironed out by moving to a common build container/VM/whatever15:23
ogracjwatson, well, fallout of --no-install-recommends i think15:24
ograapparently the cleanbuild lxd container installs recommends by default15:25
cjwatsonogra: which is in the LP build chroots' apt.conf15:25
cjwatsonogra: hence my comment15:25
ograright15:25
* cachio lunch15:26
kyrofaondra, is peekfd an app?15:29
ondrakyrofa yep15:30
kyrofaondra, looks like you're using usr/bin/peekfd as an app, but for whatever reason on arm64 it doesn't exist (not that the traceback is turned into a nicer error on edge)15:30
ondrakyrofa unless it exist only on armhf and x86 and not on arm6415:30
kyrofas/not/note/15:30
kyrofaYeah not sure why, but that's the issue15:31
ondrakyrofa can one define app per arch in this case?15:32
kyrofaondra, I'm afraid not15:35
ondrakyrofa damn, so need to remove it from all, or bug against package I guess15:35
kyrofaondra, or add a wrapper that can handle it not being there and error out15:41
kyrofaBut yeah, I'd look into why it's not there15:41
ondrakyrofa unrelated question, I have project which has two flavours. Ideally I'd like to build it from same branch, but that means two different snapcraft.yaml files, is there easy way to handle this?15:43
kyrofaondra, snapcraft doesn't have a way to consume two snapcraft.yamls, if that's what you're asking. Launchpad doesn't support it, either15:46
ondrakyrofa yep, something along those lines15:47
roadmrsergiusens: hi! what are the implications of https://bugs.launchpad.net/snapcraft/+bug/1794805 for the snapcraft parts service? https://parts.snapcraft.io/v1/parts.yaml15:55
mupBug #1794805: Remove wiki parts <18.10-build-vm> <Snapcraft:Fix Committed by sergiusens> <https://launchpad.net/bugs/1794805>15:56
jochehe was just about ask my own question :)15:56
jocsergiusens: with the remote parts wiki going, will there be some other method of re-using parts definitions across snaps? and what timescales are we looking at for this being unavailable?15:59
mupPR core18#80 opened: static: show some progress during firstboot <Created by mvo5> <https://github.com/snapcore/core18/pull/80>15:59
sergiusensroadmr: should keep on working, you use the deb, correct?16:01
sergiusensjoc: https://forum.snapcraft.io/t/proposal-templates/601916:01
sergiusensjoc: if you are using bases, they are gone16:01
roadmrsergiusens: I mean for the service itself - will it still be used or do we need to sunset it?16:01
roadmrsergiusens: (but if it's expected to keep working for people using the .deb or something else, not a problem, it can stay up as long as needed)16:02
sergiusensroadmr: when no one else builds with no base defined is the time we can put it to sleep16:02
roadmrsergiusens: gotcha, that's exactly what I wanted to know. Thanks!16:02
roadmr(sounds like that might be a long time in coming, but that's ok - parts service is quite lightweight, it can stay there indefinitely)16:03
jocsergiusens: by "using bases" do you mean having e.g. "base: core18" in my snapcraft.yaml?16:04
sergiusensjoc: yes16:04
jocoof16:04
mvoChipaca: so I would like to have a way to make "progress.Spin()" show stuff even when `snap watch` is not run from a terminal. the easiest way seems to be to add Spin() to quiteProgress but I wonder if that is the right option. maybe an option to waitMixin?16:05
mvoChipaca: wdyt?16:05
mupPR snapcraft#2329 closed: pluginhandler: remove legacy plugin loading without project <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2329>16:05
Chipacamvo: 1 sec16:05
mvoChipaca: no rush :)16:06
Chipacamvo: I like it (in fact I thought it did that already)16:06
Chipacamvo: maybe only show the message if it changes?16:06
mupPR core18#80 closed: static: show some progress during firstboot <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/80>16:06
mvoChipaca: aha, ok - so far it is doing a print on Notify() but not on Spin(msg)16:06
Chipacanot sure how often we're calling Spin with the same message16:06
mvoChipaca: +116:06
Chipacamvo: yep, just saw that16:06
mvoChipaca: awesome, I will make it happen (and +1 on checking if the message changes)16:06
sergiusensjoc: the deb will not break, it is essentially frozen; which means it will get no new features either16:07
Chipacamvo: OTOH, if this is for running it on /dev/console, is that not a tty? (or are we checking stdin instead of stdout?)16:07
mvoChipaca: we check stdin16:07
Chipacamvo: isn't that a bit silly?16:07
mvoChipaca: of course we could simply check stdout16:07
mvoChipaca: I guess :)16:07
Chipacaof course by now there are people depending on us checking stdin16:08
Chipacabut we could check both :)16:08
=== pstolowski is now known as pstolowski|afk
Chipacabah, dunno16:08
Chipacawtyt?16:08
Chipacawe could also switch to checking stdin, and warn people via the forum16:08
mvoChipaca: one slightly complication is that my code does "snap watch --last=seed | tee -a /dev/console"16:08
Chipaca"this was silly, we fixed it, sorry if we broke you but surely you agree it's less silly now"16:08
Chipacamvo: boo :-)16:08
Chipacaer16:08
mvoChipaca: because we still want these messages in journalctl -u16:08
Chipacamvo: so why not 'snap watch --last=seed < /dev/console | tee -a /dev/console?16:09
mvoChipaca: aha, nice16:09
Chipacas/\?/'?/16:09
mvoChipaca: yeah, that should work, let me quickly test16:09
Chipacamvo: also, I'd recommend writing it '--last=seed?'16:09
Chipacaunless you know for sure there is a seed change already16:10
mvoChipaca: yeah, I totally know16:10
Chipacak16:10
mvoChipaca: but good advice, thanks16:10
jdstrandogra: does it honor the slot side of the contract for udisks2? in other words, does it ship udisks2 such that other snaps can connect to it and use udisks2?16:11
* zyga resumes work on user-mounts16:11
zygahey jdstrand, how are you?16:12
ograjdstrand, well, it runs a daemon that spawns udisksd when a USB key is plugged in ...16:12
jdstrandzyga: hey, pretty good. very busy :) how are you?16:12
zygajdstrand: had an interrupt but made a lot of progress for user mounts today16:13
zygajdstrand: expect some patches relatively quickly :)16:13
ograjdstrand, not sure what else the udisks2 "contract" requires ... the daemon-script is rather specifc to import a neplan yaml file if it exists ... and since this is for embedded systems it a) has an install hook check that prevents it from being installed on non-core and b) does not run udisksd permanently16:14
jdstrandogra: if you slots udisks2, then your snap is expected to provide the udisks2 service16:15
jdstrandogra: why are you slotting udisks2?16:16
ograjdstrand, well, how else would i get mounting to work from a snap ?16:16
jdstrandogra: I don't know what your snap does. have it plugs udisks2 and then talk to udisks2 to do that for you?16:17
ograi'd just use a udev monitor script and call mount driectly, but there are no interfaces offering that16:17
mupPR core18#81 opened: static: tweak run-snapd-from-snap to avoid having to change `snap watch` <Created by mvo5> <https://github.com/snapcore/core18/pull/81>16:17
jdstrandogra: this sounds like a conversation for the forum. I suggest stating what it is your snap requires, and how you'd like to get there16:17
ograjdstrand, https://github.com/ogra1/dashkiosk-client-image-config/blob/master/netplan-import ... thats the daemon script ... it watched udev for udb block device plug events ... if one is there, it mounts the usb block device and looks for a netplan config16:18
ogra*watches16:18
ogras/udb/usb/16:18
mupPR core18#81 closed: static: tweak run-snapd-from-snap to avoid having to change `snap watch` <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core18/pull/81>16:19
ograjdstrand, well, i need the snap for a demo at IoT worldcongress next week, i'll just rip out that feature, i doubt i'll have the time to re-implement it differently in time16:19
ograeffectively i just need an interface that allows me to run mount/umount and we dont have that16:20
jdstrandogra: you are calling udisckstl, why not plugs: [ udisks2 ] and have the udisks2 snap installed?16:20
jdstrandthen just talk to that other snap16:21
ogrado we have that for all arches ?16:21
* ogra checks 16:21
ograhmm, k16:21
jdstrandogra: I see an armhf in edge16:21
kyrofaHey ogra, any ideas on this? https://askubuntu.com/questions/1082277/socketcan-device-on-ubuntu-core16:21
jdstrandit's old, but it is there16:21
ograedge ... hmm16:21
zygaroadmr: hey16:21
roadmrhehey zyga ! how goes it!16:22
zygaI need a hand in debugging a store side bug (presumably)16:22
zygaremember that snap?16:22
zygaI tried it again16:22
ograah, also in stable16:22
jdstrandogra: looks like morphis did it. that would be the ce team now I guess16:22
zygafirst time I kept the hack with chmod16:22
zygait passed16:22
zygasecond time it did not16:22
zygaI think something else is wonky16:22
jdstrandyes, and stable16:22
zygaI can show you the error I'm getting, it's very terse though16:22
ograjdstrand, thanks, i'll try to use that one ... effectively simply having snapd allow you to import a netplan config (like we allow system-user assertions) would be the proper solution after all16:23
roadmrzyga: sure, error would be nice16:23
zygaroadmr: let me re-make it, I also had issues with snapcraft in that run16:23
ograjdstrand, trying to create appliance images that do not force you to attach a serial cable to configure wlan ... (and that already have the screen occupied, so you cant use console-conf there)16:24
ograjdstrand, if i reject the held snap, i should be capable to upoad again without them getting stuck in the sotre review ?16:24
ogra*store16:25
ograkyrofa, i think thats actually one for awe, i dont have moch experience with CAN bus stuff (but he has)16:25
jdstrandogra: yes16:26
ograkyrofa, but it looks like we are lacking an interface to even use them on core16:26
ograjdstrand, my prob with adding more snaps to the image is that snapd takes 3-5 min per snap on first boot btw ... i'm already at a 20min first boot with this image16:27
ograbut i guess i'll ave to bit that bullet for now16:28
ogra*bite16:28
zygaroadmr: got them16:28
zygaThe store was unable to accept this snap.16:28
zyga  - __all__: The upload does not appear to be a valid package.16:28
roadmr\o/16:28
zygathat's all I got16:28
roadmryeay16:28
zygathe package can be installed locally obviously16:28
zyganote: I did the upload as root16:29
zygaas a user snapcraft crashes16:29
zygakyrofa: ^ wanna see?16:29
roadmrzyga: hm, can you either run the review tools on the package locally to see what they say, or shoot me a copy of the built snap?16:29
zygaroadmr: could I just send you the snap please?16:30
roadmrzyga: sure thing16:30
kyrofazyga, I'm missing context, what are you doing?16:32
zygakyrofa: I'm using snapcraft push *.snap16:32
zygaand it crashes with ...16:32
zygahttps://www.irccloud.com/pastebin/164aZPmQ/16:33
ograkyrofa, i forwarded the question to tony now ...16:33
ograkyrofa, did you see my issue with the architectures build-on/run-on thing under cleanbuild above ?16:33
ograkyrofa, :16:34
ogra<ogra> Snapping 'pi-kiosk' |                                                                                                                           16:34
ogra<ogra> Snapped pi-kiosk_18-0.1_OrderedDict([('build-on', 'amd64'), ('run-on', 'armhf')]).snap16:34
ogra<ogra> Stopping local:snapcraft-sadly-choice-amoeba16:34
ogra<ogra> Retrieved "pi-kiosk_18-0.1_OrderedDict([('build-on', 'amd64'), ('run-on', 'armhf')]).snap"16:34
ograit somehow includes python code snippets in the final snap name16:34
roadmrzyga: hm we may need jdstrand's help on this16:38
zygaoh/16:38
zygaroadmr: what's the error from the review tools?16:39
roadmrzyga: https://pastebin.canonical.com/p/8Sj2fXx5M4/16:39
zygahmm, wasn't this the same error as before?16:40
roadmrzyga: I think that python traceback from the failure to rm is causing the store to barf (because it expects the tools output to be json - even if I say --json it has that Python thing at the end)16:40
zygathose as not writable by owner16:40
roadmrzyga: yes, I fixed that store-side, but IIRC your earlier snap didn't show this output from the tools16:40
zygaahh16:40
zygaI See16:40
roadmrif the tools can't clean up and barf like this, I can't fix that store-side directly :/16:40
jdstrandzyga: where is this snap?16:41
zygajdstrand: sent via wormhole to roadmr16:41
roadmrjdstrand: the top-level dir ias this mode drwx------16:41
zygait's just a fedora base I was sending before16:41
jdstrandI'd need the snap if I'm going to look at fixing the review tools16:42
zygajdstrand: I can wormhole it to you as well if you don't mind16:42
jdstrandthat's fine16:42
roadmrit's not huge, only 18 MB16:42
zygasent16:43
jdstrandok, I'll take a look at it16:43
roadmrjdstrand: I suspect the fix will be similar to what I did store-side https://bazaar.launchpad.net/~ubuntuone-pqm-team/software-center-agent/trunk/revision/505516:43
jdstrandzyga: while I have you. can I get your opinion on something16:43
jdstrandroadmr: ack, thanks16:43
jdstrandzyga: so, I need to add 'unsafe' to the default template16:44
zygasure, I'm all ears16:44
jdstrandzyga: but it fails spread cause apparently opensuse 42.3 has a too old apparmor_parser16:44
kyrofaogra, yuck, that's interesting16:44
zygaright, I remember that branch16:44
jdstrandzyga: that doesn't know what 'unsafe' is16:44
roadmractually - the review tools' recursive_rm appears to be identical to what we have in the store, so pretty sure my fix will apply :D16:44
jdstrandzyga: so, I think the most correct solution is to use apparmor_parser from the core snap. however, I don't want to do that in this PR because whenever we move to this sort of scenario, it has a long tail of little fixes16:45
jdstrandzyga: we can do that in a future pr16:45
zygawould a parser from core produce binary compatible with the host kernel?16:45
jdstrandzyga: sure, why not?16:46
jdstrandthe parser is fully aware of the kernel16:46
zygathat's good16:46
jdstrandusing the parser in core would mean we could update apparmor in Ubuntu only to fix parser bugs, enhancements, etc, etc16:46
jdstrandkinda like how we pulled back xenial's apparmor to trusty16:47
jdstrandwe could do that 'forever'16:47
zygaok16:47
jdstrandand not worry about other distro's apparmor16:47
jdstrandanyhoo16:47
zygaso that's the preferred solution16:47
jdstrandthat is the ideal16:47
zygaand in the meantime?16:47
jdstrandbut, again, I don't want to do that, so we have some choices16:47
jdstrandI explored the idea of having a parser features list and populating it, just like we do with the features dir16:48
zygaoh, that's interesting16:48
jdstrandit works, but there are a number of test case mockings which got me wondering if there is another way16:48
jdstrand(especially since if we go the apparmor_parser from core route, we just rip out this code)16:49
zygabtw, what is the unsafe vs safe transition about?16:49
jdstrandsecure_exec16:49
jdstrandie, stripping LD_LIBRARY_PATH16:49
jdstrandit is explained in the pr16:49
jdstrandso the other option is, if we are PartialAppArmor already, we know we don't have a new enough parser, so don't use unsafe if PartialAppArmor16:50
zygaah16:50
zygaok16:50
jdstrandthis is cheap, but mildly inaccurate, but since the code is going to be ripped out soon, who cares?16:50
zygathat would partially regress arch and opensuse tumbleweed though16:50
jdstrandso... do you care?16:50
jdstrand:)16:50
zygaha16:50
zygaso that's the question16:50
zygaI think that's fine if you are really going to do the full deal16:51
zygaas in schedule wise16:51
zygaI'm sure you would16:51
jdstrandzyga: that's the thing, arch and tumbleweed should have a new enough apparmor_parser, so I can special case them or do something with downgradeConfinement, etc so they get unsafe16:51
zygain that case I think we're good16:52
jdstrandor not, and just say that they don't get unsafe until we use the parser from core16:52
jdstrandwell, as it happens, I have a new manager now and slowly coming out from the last couple months16:53
jdstrandI can add moving apparmor_parser from core to the roadmap16:53
jdstranderr16:53
jdstrandmoving to apparmor_parser from core to the roadmap16:53
zyga+116:53
zygawe should do the same for snap-seccomp16:53
jdstrandI mean, this has a ton of benefits way beyond this16:53
zygafor statx and the lkike16:53
zyga*like16:53
jdstrandyeah16:54
zygawe should also answer the question of reexec=0 distros16:54
jdstrandsure, but to me, we would always use the apparmor from core, regardless of reexec16:54
jdstrandor do you see a problem with that?16:54
zygatechnically no16:55
zygabut policy wise it could be borderline iffy16:55
jdstrandwho's policy?16:55
jdstrandfedora shouldn't care-- they don't have apparmor16:55
zygabut they have seccomp16:56
zygaI imagine this is the same deal16:56
zygabut anyway, we don't have to discuss this now,  I think technically we are in agreement16:57
zygaand we can keep the code that uses distribution tools16:57
jdstrandzyga: well, if we are planning on keeping the code, probably need parserFeatures16:58
jdstrandwe don't necessarily need to downgrade to PartialConfinement though16:58
jdstrandthat's a 3rd option. I check to see if the parser supports a syntax we know older ones don't have, and then populate something in snapd (other than PartialAppArmor) that we can query when using that one rule16:59
jdstrandsorta sounds like based on this conversation that is possibly the best path forward17:00
zygajdstrand: we can probably use system key for that17:00
zygaor a combination of system key and locally held state in AppArmor backend17:00
jdstrandright, I was initially thinking of that17:00
jdstrandok, I'll just go that route for now17:00
zygaok17:01
jdstrandzyga: thanks17:01
zygamy pleasure :)17:02
ograjdstrand, fyi, works very nicely with the udidks2 snap, thanks for the hint (if only the firstboot wouldnt be 5min longer now)17:08
jdstrandogra: nice! :)17:10
jdstrandogra: I wonder why it takes so long... that seems like a bug?17:11
ograjdstrand, yeah, its a known go bug with checksumming sha256 stuff on arm ... snapd uses that when installing a snap for the first time ... so each snap delays the firstboot17:12
ograjdstrand, ijohnson has sent a fix upstream some months ago but that has still not been included17:13
ogra(go upstream)17:13
ograpicking sha256 for snaps was a bit overzealous ;)17:13
ijohnsonit's actually sha3 hashing17:14
ograah, i thought it was 25617:14
ograsome sha :P17:14
ijohnsonwell it's sha3 with length 25617:14
ograsha-la-la :)17:14
zygaha17:14
zygaI was _just_ typing that17:14
ijohnson(although I thought that snapd used 384, but my fix fixes all sha3)17:14
ogra(sorry, hard day ... tired and silly)17:14
ograzyga, welcome to my level of sillyness then :)17:15
zygaogra: I'm always silly17:15
zygaI feel quite at home17:15
ograheh17:15
ogramvo, i saw yu landed the password expiry stuff ... any chance that goes into some non-edge channel before the weekend ?17:19
ogra(likely not, but i thought i'd ask at least)17:19
mvoogra: there is a good chance actually - to hit beta, once the current beta moves to candidate I will do a new 2.36~pre217:46
mvoogra: and that will move to beta17:46
ograbeta sounds perfect ...17:47
mvosil2100: the various races for the pi firstboot (and dragon) are hopefully all under control now, I guess we just need to dig into the probert issue (and someone needs to test on a dragonboard)17:50
mvosil2100: I will push new test images later tonight or early tomorrow but nothing special about them, just current edge (and soon beta) for core18 and friends17:50
mvosil2100: (mostly just fyi)17:50
zygajdstrand: I subscribed you to https://bugs.launchpad.net/snappy/+bug/179636217:56
zygaplease check my last comment there17:56
mupBug #1796362: snap-confine has elevated permissions and is not confined but should be. Refusing to continue to avoid permission escalation attacks <Snappy:Incomplete> <https://launchpad.net/bugs/1796362>17:56
zygamvo: is dragonboard fixed now? I can test it if you need that17:56
jdstrandroadmr: can you pull r1132. it has the fix for zyga's snap17:57
zygamvo: do you want to do a simple boring review with a function rename17:57
zygajdstrand: oh, that was quick, thank you!17:57
mvozyga: dragonboard - I'm not sure its fixed, let me build a new test image. I fixed some firstboot races18:08
zygak18:08
mupPR snapd#5955 opened:  cmd/snap, tests: snapshots for all <Snapshots πŸ“Έσ Ÿ> <Created by chipaca> <https://github.com/snapcore/snapd/pull/5955>18:21
sil2100mvo: thanks! I'm usually just building images myself to always have the latest core, but I know there are others that were doing some tests on the pre-built ones18:51
sil2100mvo: I saw a lot of good fixes getting merged today, thanks for those!18:51
sil2100mvo: we're still working on some bits and pieces to enable dailies, but now with the official assertions we can push things a bit more forward18:52
mvosil2100: great, thank you. building them yourself is fine of course :)19:21
mvozyga: dragonboard is up at http://people.canonical.com/~mvo/core18/19:21
mvozyga: but no rush19:21
mvozyga: also I'm not sure things will be better than at the sprint, but you never know :)19:22
mvosil2100: fixes> yeah, nothing beats testing on real HW there were some races19:22
mupPR snapcraft#2331 opened: meson plugin: add support for bases <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2331>19:29
jdstrandzyga: hey, can you add this to your list: https://github.com/snapcore/snapd/pull/5739 (waiting for spread tests to finish)19:58
mupPR #5739: interfaces/default: don't scrub with change_profile with classic <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5739>19:58
mupPR snapcraft#2332 opened: waf plugin: support for bases <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2332>20:12
mupPR snapcraft#2328 closed: godeps plugin: support for bases <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2328>20:24
threshpopey, pingie "download button" art20:29
roadmrjdstrand: hi! sure will do20:35
mupPR snapcraft#2330 closed: pluginhandler: remove big solidus workaround <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2330>20:45
popeythresh: i pinged design about that yesterday, and was told it's in the lawyers queue for licensing.21:03
popeythresh: thanks for the ping :)21:03
mupPR snapcraft#2333 opened: catkin, catkin-tools: add support for bases <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2333>22:12
mupPR snapcraft#2248 closed: plugin: update catkin plugin to support melodic <do-not-merge-yet> <Created by NickZ> <Closed by kyrofa> <https://github.com/snapcore/snapcraft/pull/2248>22:15
mupPR snapd#5861 closed: cmd/libsnap: add functions for handling facts <Created by zyga> <Closed by niemeyer> <https://github.com/snapcore/snapd/pull/5861>23:10
mupPR snapd#5859 closed: many: introduce facts, export experimental features as facts <Created by zyga> <Closed by niemeyer> <https://github.com/snapcore/snapd/pull/5859>23:35

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