| mup | Bug #1906327 opened: aws-iot-greengrass snap *was temporarily gone <Snappy:New> <https://launchpad.net/bugs/1906327> | 01:03 |
|---|---|---|
| mup | Bug #1906327 changed: aws-iot-greengrass snap *was temporarily gone <Snappy:New> <https://launchpad.net/bugs/1906327> | 01:12 |
| mup | Bug #1906327 opened: aws-iot-greengrass snap *was temporarily gone <Snappy:New> <https://launchpad.net/bugs/1906327> | 01:15 |
| mborzecki | morning | 07:05 |
| mborzecki | brb, reboot for a new kernel | 07:15 |
| mborzecki | re | 07:17 |
| mup | PR 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 |
| mborzecki | mvo: morning | 07:30 |
| mvo | good morning mborzecki | 07:30 |
| pstolowski | mornings | 07:57 |
| mvo | good morning pstolowski | 08:01 |
| mborzecki | pstolowski: hey | 08:18 |
| mup | PR 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 |
| pedronis | mvo: I commented on #9718 | 08:49 |
| mup | PR #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 |
| pedronis | mvo: mborzecki: hi, #9713 needs 2nd reviews/re-reviews, my fix seems to have worked | 08:50 |
| mup | PR #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 |
| pedronis | mvo: mborzecki: thanks for reviewing my daemon cleanups | 08:51 |
| mborzecki | pedronis: 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 store | 08:52 |
| mup | PR #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 |
| mborzecki | pedronis: and https://github.com/snapcore/snapd/pull/9719 is trivial and could use your review | 08:53 |
| mup | PR #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 |
| pedronis | mborzecki: it's in my queue | 09:03 |
| mborzecki | pedronis: thanks | 09:10 |
| pedronis | mvo: I re-reviewed #9715 , I made a non-blocking suggestion but maybe is too complicated for now | 09:27 |
| mup | PR #9715: hookstate: add new HookManager.EphemeralRunHook() <Needs Samuele review> <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9715> | 09:27 |
| pedronis | mborzecki: I asked a question in #9719 | 09:36 |
| mup | PR #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 |
| zyga | good morning guys | 09:59 |
| zyga | got out of most of the planning mess and I will be back to normal on-irc working on foss-y bits tomorrow | 10:00 |
| zyga | I will resume snapd patches as well | 10:00 |
| zyga | the lat few days were sort of an internal planning sprint where I'm just always in meetings and planning | 10:01 |
| zyga | mvo spread has no missing deps in Debian anymore | 10:04 |
| zyga | coffee done and back to planning :) | 10:05 |
| mvo | zyga: woah! did you update it to use only pkgs that are already packaged? | 10:05 |
| zyga | mvo yeah, I just need to give you the package | 10:05 |
| mvo | zyga: let me know when you need sponsoring | 10:05 |
| zyga | given that I run part of my own planning now | 10:05 |
| zyga | I added a task group for handling debian packaging | 10:05 |
| zyga | so I will definitely have time for shared work | 10:05 |
| zyga | it's just not today, time to write down all decisions and fill in various comms systems for others to know | 10:06 |
| zyga | good luck and see you soon :) | 10:06 |
| mborzecki | pedronis: thanks, let me take a look | 10:15 |
| mvo | zyga: sure, no rush! see you | 10:15 |
| pedronis | mborzecki: what go version we have on on amazon linux? | 10:22 |
| mborzecki | pedronis: let me double check but iirc it was 1.11+ | 10:23 |
| pedronis | 1.15 | 10:23 |
| pedronis | sorry, we print nowadays | 10:23 |
| mborzecki | pedronis: yup that's quite likely | 10:23 |
| mborzecki | pedronis: centos 7 should have 1.15 too | 10:27 |
| mup | PR 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 |
| mborzecki | pedronis: i've updated https://github.com/snapcore/snapd/pull/9719 | 10:43 |
| mup | PR #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 |
| pedronis | mborzecki: +1 | 11:45 |
| mvo | thanks pedronis ! | 11:49 |
| mvo | 9715 needs a second review, maybe pstolowski can have a look, hopefully not too long/difficult | 11:49 |
| pstolowski | mvo: sure, will do | 11:50 |
| mvo | thanks a lot! | 11:50 |
| mborzecki | pedronis: thanks! | 11:55 |
| pedronis | mvo: I commented on the cleanup: https://github.com/snapcore/snapd/pull/9715#discussion_r533353541 | 11:57 |
| mup | PR #9715: hookstate: add new HookManager.EphemeralRunHook() <Needs Samuele review> <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9715> | 11:57 |
| mborzecki | looks like apt hooks and snap advise is failing | 11:57 |
| mborzecki | (the tests i mean) | 11:58 |
| mvo | pedronis: excellent, I poke on this next | 11:58 |
| mborzecki | heh HTTP/1.0 429 Too Many Requests | 11:58 |
| mborzecki | when updating snap catalog | 11:59 |
| pstolowski | mvo: 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 |
| pedronis | pstolowski: we want to run a hook from snapd without a change or task | 12:18 |
| pedronis | synchronously | 12:18 |
| pstolowski | pedronis: ah, i see, makes sense, thanks | 12:19 |
| mup | PR 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 |
| mup | PR 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 |
| mup | Bug #1906327 changed: aws-iot-greengrass snap *was temporarily gone <Snappy:Invalid> <Snap Store Server:Incomplete by nataliabidart> <https://launchpad.net/bugs/1906327> | 15:13 |
| ijohnson | mborzecki: do you know if we have unit tests for snap-confine? if so where are they? | 15:20 |
| mborzecki | ijohnson: we have some, they're in cmd/snap-confine/*-test.c files | 15:21 |
| pedronis | mborzecki: I commented on #9724, some questions there | 15:21 |
| mup | PR #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 |
| ijohnson | mborzecki: ah thanks I think snap-confine-invocation-test.c is what I was looking for | 15:21 |
| mborzecki | ijohnson: though it's not per se the whole of snap-confine functionality covered by the test, rather some smaller individual bits | 15:21 |
| mborzecki | pedronis: thanks | 15:21 |
| pedronis | ijohnson: you asked about unit tests, there's also spread tests for it in case, under spread-tests | 15:22 |
| ijohnson | yeah I saw the spread tests, just wondering if we also had unit test coverage for snap-confine | 15:23 |
| mborzecki | w8, do we run those tests? | 15:23 |
| ijohnson | haaaaa | 15:23 |
| ijohnson | 🤦 | 15:24 |
| ijohnson | mborzecki: uhhhh no we don't seem to run them in github actions that is | 15:24 |
| ijohnson | perhaps 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 |
| pedronis | mmh, I don't see anything mentioning them | 15:30 |
| mborzecki | Before snap-confine was merged with snapd it used to have its own spread | 15:39 |
| mborzecki | tests | 15:39 |
| mborzecki | that's from commit e3d6f6aae6bb48e585ff3de1a25277b680e736b4 | 15:39 |
| mborzecki | https://github.com/bboozzoo/snapd/commit/e3d6f6aae6bb48e585ff3de1a25277b680e736b4 | 15:39 |
| * cachio lunch | 15:41 | |
| ijohnson | mborzecki: I updated #9720, could you take a look? thanks | 15:42 |
| mup | PR #9720: many: rename disks.FindMatching... to FindMatching...WithFsLabel and err type <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9720> | 15:42 |
| mborzecki | ijohnson: +1, little copy paste typo :P | 15:47 |
| ijohnson | thanks mborzecki ! | 15:49 |
| mup | PR 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 |
| mup | PR 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 |
| pedronis | vidal72[m]: not atm, it would need some discussion because it kind of sidesteps/overlap with the configuration support for snaps | 16:51 |
| mup | PR snapd#9728 opened: Disable user creation on system-user auto-import <Created by slimjim777> <https://github.com/snapcore/snapd/pull/9728> | 17:10 |
| om26er | Hey 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 |
| ijohnson | om26er: /var/lib/snapd/hostfs will be empty if you are running outside of snap confinement | 17:19 |
| om26er | While 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 |
| ijohnson | om26er: that directory has "/" bind mounted to it when snaps run | 17:20 |
| ijohnson | om26er: so whatever is in "/" is what will show up in /var/lib/snapd/hostfs | 17:20 |
| om26er | hm, interesting. So that means the opengl interface might need some update(s) to support Yocto based environments | 17:21 |
| om26er | ? | 17:21 |
| ijohnson | well you mentioned /var/lib/snapd/lib/gl and /var/lib/snapd/hostfs | 17:22 |
| ijohnson | /var/lib/snapd/hostfs should not need any changes for yocto since that's just a simple bind mount | 17: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, yes | 17:23 |
| om26er | is it possible that my snapd is not able to see /var/lib/snapd/hostfs, due to a bug or something ? | 17:23 |
| ijohnson | what do you mean that your snapd is not able to see /var/lib/snapd/hostfs ? | 17:23 |
| om26er | I 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 |
| ijohnson | om26er: 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 |
| om26er | root@jetson-xavier-nx-devkit:~# snap run --shell kivy-sample -c "ls -lah /var/lib/snapd/hostfs" | 17:25 |
| om26er | ls: cannot open directory '/var/lib/snapd/hostfs': Permission denied | 17:25 |
| om26er | also: on that system libGLESv2.so.2 is under /usr/lib/libGLESv2.so.2 which isn't the case on Ubuntu at least. | 17:28 |
| om26er | also 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 host | 17:29 |
| om26er | s/directory/file | 17:29 |
| om26er | ijohnson btw, that "permission denied" also happens on my desktop, so looks like its "expected" ? | 17:31 |
| ijohnson | om26er: oh yeah you will need to run a devmode snap, can you reinstall the snap in devmode | 17:32 |
| ijohnson | and then try that same command again | 17:32 |
| om26er | yeah, now it prints the directory structure | 17:33 |
| om26er | well in devmode, my snap starts anyways, I have been working on enable confinement though | 17:34 |
| mup | PR 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 |
| om26er | actually 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 |
| om26er | btw, 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/b4aaf190f5ce3629067bd18b5ff502f2 | 17:41 |
| mup | PR 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 |
| thresh | hello. 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 |
| thresh | https://code.videolan.org/-/snippets/1301/raw is the log output | 18:11 |
| ijohnson | thresh: 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/docs | 18:31 |
| === ijohnson is now known as ijohnson|lunch | ||
| thresh | thanks ijohnson|lunch | 18:46 |
| thresh | why 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 |
| thresh | X-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 core18 | 18:54 |
| cjwatson | (yes, this is a bit confusing. It's more of an API version nowadays) | 18:54 |
| mup | PR snapd#9730 opened: hookstate: refactor around EphemeralRunHook <Squash-merge> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9730> | 19:11 |
| mup | PR 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 |
| mup | PR 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 |
| thresh | cjwatson, 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 out | 19:51 |
| thresh | thanks a lot and have a good day :) | 19:58 |
| * cachio afk | 21:30 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!