mupBug #1906327 opened: aws-iot-greengrass snap *was temporarily gone <Snappy:New> <https://launchpad.net/bugs/1906327>01:03
mupBug #1906327 changed: aws-iot-greengrass snap *was temporarily gone <Snappy:New> <https://launchpad.net/bugs/1906327>01:12
mupBug #1906327 opened: aws-iot-greengrass snap *was temporarily gone <Snappy:New> <https://launchpad.net/bugs/1906327>01:15
mborzeckibrb, reboot for a  new kernel07:15
mupPR snapd#9421 closed: interfaces/utf: Add Ledger to U2F devices <Needs security review> <Created by TamtamHero> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9421>07:18
mborzeckimvo: morning07:30
mvogood morning mborzecki07:30
mvogood morning pstolowski08:01
mborzeckipstolowski: hey08:18
mupPR snapd#9722 closed: daemon: split apps and logs endpoints to api_apps.go and tests <Cleanup :broom:> <Created by pedronis> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/9722>08:23
pedronismvo: I commented on #971808:49
mupPR #9718: secboot,devicestate: add scaffoling for "fde-reveal-key" support <Needs Samuele review> <Squash-merge> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9718>08:49
pedronismvo: mborzecki: hi,  #9713 needs 2nd reviews/re-reviews, my fix seems to  have worked08:50
mupPR #9713: tests: sign new nested-18|20* models to allow for generic serials <Run nested> <UC20> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9713>08:50
pedronismvo: mborzecki: thanks for reviewing my daemon cleanups08:51
mborzeckipedronis: hi, i think you can force merge https://github.com/snapcore/snapd/pull/9723 when the spread jobs finish, i've restarted them but one of the tests still failed on amazon due to 408 from the store08:52
mupPR #9723: daemon: move some more api tests from daemon to daemon_test <Cleanup :broom:> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9723>08:52
mborzeckipedronis: and https://github.com/snapcore/snapd/pull/9719 is trivial and could use your review08:53
mupPR #9719: boot: set kernel command line in modeenv during install  <Run nested> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9719>08:53
pedronismborzecki: it's in my queue09:03
mborzeckipedronis: thanks09:10
pedronismvo: I re-reviewed #9715 , I made a non-blocking suggestion but maybe is too complicated for now09:27
mupPR #9715: hookstate: add new HookManager.EphemeralRunHook() <Needs Samuele review> <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9715>09:27
pedronismborzecki: I asked a question in #971909:36
mupPR #9719: boot: set kernel command line in modeenv during install  <Run nested> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9719>09:36
zygagood morning guys09:59
zygagot out of most of the planning mess and I will be back to normal on-irc working on foss-y bits tomorrow10:00
zygaI will resume snapd patches as well10:00
zygathe lat few days were sort of an internal planning sprint where I'm just always in meetings and planning10:01
zygamvo spread has no missing deps in Debian anymore10:04
zygacoffee done and back to planning :)10:05
mvozyga: woah! did you update it to use only pkgs that are already packaged?10:05
zygamvo yeah, I just need to give you the package10:05
mvozyga: let me know when you need sponsoring10:05
zygagiven that I run part of my own planning now10:05
zygaI added a task group for handling debian packaging10:05
zygaso I will definitely have time for shared work10:05
zygait's just not today, time to write down all decisions and fill in various comms systems for others to know10:06
zygagood luck and see you soon :)10:06
mborzeckipedronis: thanks, let me take a look10:15
mvozyga: sure, no rush! see you10:15
pedronismborzecki: what go version we have on on amazon linux?10:22
mborzeckipedronis: let me double check but iirc it was 1.11+10:23
pedronissorry, we print nowadays10:23
mborzeckipedronis: yup that's quite likely10:23
mborzeckipedronis: centos 7 should have 1.15 too10:27
mupPR snapd#9723 closed: daemon: move some more api tests from daemon to daemon_test <Cleanup :broom:> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9723>10:28
mborzeckipedronis: i've updated https://github.com/snapcore/snapd/pull/971910:43
mupPR #9719: boot: set kernel command line in modeenv during install  <Run nested> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9719>10:43
pedronismborzecki: +111:45
mvothanks pedronis !11:49
mvo9715 needs a second review, maybe pstolowski can have a look, hopefully not too long/difficult11:49
pstolowskimvo: sure, will do11:50
mvothanks a lot!11:50
mborzeckipedronis: thanks!11:55
pedronismvo: I commented on the cleanup: https://github.com/snapcore/snapd/pull/9715#discussion_r53335354111:57
mupPR #9715: hookstate: add new HookManager.EphemeralRunHook() <Needs Samuele review> <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9715>11:57
mborzeckilooks like apt hooks and snap advise is failing11:57
mborzecki(the tests i mean)11:58
mvopedronis: excellent, I poke on this next11:58
mborzeckiheh HTTP/1.0 429 Too Many Requests11:58
mborzeckiwhen updating snap catalog11:59
pstolowskimvo: can you explain what does it mean to run hooks in ephemeral mode? maybe i'm confused by what epehemeral meant for apps & snapctl. what is the problem this is addressing?12:16
pedronispstolowski: we want to run a hook from snapd without a change or task12:18
pstolowskipedronis: ah, i see, makes sense, thanks12:19
mupPR snapd#9724 opened: boot: observe successful command line update, provide a default  <Run nested> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9724>12:39
mupPR snapd#9725 opened: bootloader: remove installableBootloader interface and methods <Cleanup :broom:> <Simple 😃> <Skip spread> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9725>14:50
mupBug #1906327 changed: aws-iot-greengrass snap *was temporarily gone <Snappy:Invalid> <Snap Store Server:Incomplete by nataliabidart> <https://launchpad.net/bugs/1906327>15:13
ijohnsonmborzecki: do you know if we have unit tests for snap-confine? if so where are they?15:20
mborzeckiijohnson: we have some, they're in cmd/snap-confine/*-test.c files15:21
pedronismborzecki: I commented on #9724, some questions there15:21
mupPR #9724: boot: observe successful command line update, provide a default  <Run nested> <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9724>15:21
ijohnsonmborzecki: ah thanks I think snap-confine-invocation-test.c is what I was looking for15:21
mborzeckiijohnson: though it's not per se the whole of snap-confine functionality covered by the test, rather some smaller individual bits15:21
mborzeckipedronis: thanks15:21
pedronisijohnson: you asked about unit tests, there's also spread tests for it in case, under spread-tests15:22
ijohnsonyeah I saw the spread tests, just wondering if we also had unit test coverage for snap-confine15:23
mborzeckiw8, do we run those tests?15:23
ijohnsonmborzecki: uhhhh no we don't seem to run them in github actions that is15:24
ijohnsonperhaps we still run them in nightly travis runs? cachio do you know if we run the spread tests in ./cmd/snap-confine/spread-tests anywhere?15:25
pedronismmh, I don't see anything mentioning them15:30
mborzeckiBefore snap-confine was merged with snapd it used to have its own spread15:39
mborzecki    tests15:39
mborzeckithat's from commit e3d6f6aae6bb48e585ff3de1a25277b680e736b415:39
* cachio lunch15:41
ijohnsonmborzecki: I updated #9720, could you take a look? thanks15:42
mupPR #9720: many: rename disks.FindMatching... to FindMatching...WithFsLabel and err type <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9720>15:42
mborzeckiijohnson: +1, little copy paste typo :P15:47
ijohnsonthanks mborzecki !15:49
mupPR snapd#9726 opened: cmd/snap: suppress a case of spurious stdout logging from tests <Simple 😃> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9726>16:25
vidal72[m]is it possible to persistently set some env variable for specific snap by user other than editing desktop file?16:47
mupPR snapd#9727 opened: seed: cleanup/drop some no longer valid TODOS, clarify some other points <Run nested> <UC20> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9727>16:50
pedronisvidal72[m]: not atm, it would need some discussion because it kind of sidesteps/overlap with the configuration support for snaps16:51
mupPR snapd#9728 opened: Disable user creation on system-user auto-import <Created by slimjim777> <https://github.com/snapcore/snapd/pull/9728>17:10
om26erHey my /var/lib/snapd/hostfs/ is empty, is that expected ? Also how can I check what my snap could "see" in that directory ?17:16
ijohnsonom26er: /var/lib/snapd/hostfs will be empty if you are running outside of snap confinement17:19
om26erWhile trying to run a strict snap on Nvidia Xavier NX, my snap won't start, but if I copy my libGL* files to /var/lib/snapd/lib/gl then it starts fine.17:20
ijohnsonom26er: that directory has "/" bind mounted to it when snaps run17:20
ijohnsonom26er: so whatever is in "/" is what will show up in /var/lib/snapd/hostfs17:20
om26erhm, interesting. So that means the opengl interface might need some update(s) to support Yocto based environments17:21
ijohnsonwell you mentioned /var/lib/snapd/lib/gl and /var/lib/snapd/hostfs17:22
ijohnson /var/lib/snapd/hostfs should not need any changes for yocto since that's just a simple bind mount17:22
ijohnson /var/lib/snapd/lib/gl will most likely require changes to snap-confine to do extra mounts depending on where the gl files are on the host on yocto, yes17:23
om26eris it possible that my snapd is not able to see /var/lib/snapd/hostfs, due to a bug or something ?17:23
ijohnsonwhat do you mean that your snapd is not able to see /var/lib/snapd/hostfs ?17:23
om26erI mean't is it possible that due to a bug in snapd/confine, my snaps are not able to "access" stuff on the rootfs ?17:24
ijohnsonom26er: what do you see when you run `snap run --shell <your-snap>.<an-app-in-your-snap> -c "ls -lah /var/lib/snapd/hostfs"` ?17:25
om26erroot@jetson-xavier-nx-devkit:~# snap run --shell kivy-sample -c "ls -lah /var/lib/snapd/hostfs"17:25
om26erls: cannot open directory '/var/lib/snapd/hostfs': Permission denied17:25
om26eralso: on that system libGLESv2.so.2 is under /usr/lib/libGLESv2.so.2 which isn't the case on Ubuntu at least.17:28
om26eralso when I run `strace`, it's not able to find this directory as well https://github.com/snapcore/snapd/blob/master/interfaces/builtin/opengl.go#L56 -- it clearly exists on the host17:29
om26erijohnson btw, that "permission denied" also happens on my desktop, so looks like its "expected" ?17:31
ijohnsonom26er: oh yeah you will need to run a devmode snap, can you reinstall the snap in devmode17:32
ijohnsonand then try that same command again17:32
om26eryeah, now it prints the directory structure17:33
om26erwell in devmode, my snap starts anyways, I have been working on enable confinement though17:34
mupPR snapd#9729 opened: bootloader/lkenv: specify backup file as arg to NewEnv(), use "" as path+"bak" <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9729>17:35
om26eractually I take that back, it does't work in devmode as well unless I put the libGL* files under `/var/lib/snapd/lib/gl/`17:35
om26erbtw, these are the minimal set of .so files that I copied from /usr/lib to /var/lib/snapd/lib/gl/ for the Kivy to work from a snap environment https://gist.github.com/om26er/b4aaf190f5ce3629067bd18b5ff502f217:41
mupPR snapd#9715 closed: hookstate: add new HookManager.EphemeralRunHook() <Needs Samuele review> <Squash-merge> <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9715>17:45
threshhello.  i'm installing snapcraft in a ubuntu 20.04 docker container (RUN apt install snapcraft...), and it seems it has issues connecting to snap store while running a debian preinstall script.  It tries asking me on a TTY what to do, but there is no terminal.  How can install it without user interaction?18:11
threshhttps://code.videolan.org/-/snippets/1301/raw is the log output18:11
ijohnsonthresh: you shouldn't be using snapcraft as a deb package, you should be using snapcraft as a snap, there's examples of how to do this from inside a docker container on snapcraft.io/docs18:31
=== ijohnson is now known as ijohnson|lunch
threshthanks ijohnson|lunch18:46
threshwhy does https://snapcraft.io/docs/build-on-docker mention FROM ubuntu:bionic as builder, but later on the Dockerfile it's using FROM ubuntu:xenial ?18:46
threshX-Ubuntu-Series: 16 is also suspicious wrt core18...18:47
=== pedronis__ is now known as pedronis
cjwatson"X-Ubuntu-Series: 16" is normal even for core1818:54
cjwatson(yes, this is a bit confusing.  It's more of an API version nowadays)18:54
mupPR snapd#9730 opened: hookstate: refactor around EphemeralRunHook <Squash-merge> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9730>19:11
mupPR snapd#9731 opened: daemon: split out snapctl support and snap configuration support to their own api_*.go files <Cleanup :broom:> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9731>19:26
mupPR snapd#9726 closed: cmd/snap: suppress a case of spurious stdout logging from tests <Simple 😃> <Skip spread> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9726>19:46
threshcjwatson, ah yeah, it's still a bit weird that it doesnt fetch core18, while says so... I've found https://github.com/snapcore/snapcraft/blob/master/docker/stable.Dockerfile which helped be to figure it out19:51
threshthanks a lot and have a good day :)19:58
* cachio afk21:30

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