=== JavaPP is now known as Java-- === Eickmeyer_ is now known as Eickmeyer [06:35] morning [06:39] pstolowski: hi! [07:00] good morning [08:13] morning, i'll be staring around 11 [08:27] good morning mborzecki and zyga [08:27] mborzecki: cool! [08:30] mvo, good morning, how are you doing? [08:30] I hope that this week is productive and smooth [09:11] mvo: can you land https://github.com/snapcore/snapd/pull/10640 ? failed tests are unrelated to the changes in the PR [09:11] PR #10640: cmd, packaging: import BPF headers from kernel, detect whether host headers are usable [09:12] mborzecki: sure [09:12] mborzecki: done [09:12] mvo: thank you [09:13] PR snapd#10640 closed: cmd, packaging: import BPF headers from kernel, detect whether host headers are usable [10:46] Good morning. mvo https://github.com/snapcore/snapd/pull/10617 and https://github.com/snapcore/snapd/pull/10643 have tests failures but they're unrelated to the changes. Can you merge them please? [10:46] PR #10617: cmd/snap: add Size column to refresh --list [10:46] PR #10643: testutil: add DeepUnsortedMatches Checker [10:49] miguelpires: sure, happy to do that [10:50] Thank you [10:58] PR snapd#10643 closed: testutil: add DeepUnsortedMatches Checker [11:03] PR snapd#10658 opened: cmd/libsnap-confine-private: add BPF support helpers [11:06] ehh missed a patch there [11:18] * pstolowski lunch [11:23] PR snapd#10659 opened: store: deal correctly with "assumes" from the store raw yaml [11:23] PR snapd#10660 opened: overlord: add manager test for "assumes" checking [11:32] and the fun part of device cgroup v2 is up now too: https://github.com/snapcore/snapd/pull/10661 [11:32] PR #10661: cmd/libsnap-confine-private: device cgroup v2 support [11:33] PR snapd#10661 opened: cmd/libsnap-confine-private: device cgroup v2 support [12:33] PR snapd#10662 opened: .github/workflows/test.yaml: test github.events key [12:34] alright hopefully we will start getting real coverage for PR's after ^ is merged to master [12:41] trivial PR: https://github.com/snapcore/snapd/pull/10663 [12:41] PR #10663: tests/main/services-install-hook-can-run-svcs: shellcheck issue fix [12:43] PR snapd#10663 opened: tests/main/services-install-hook-can-run-svcs: shellcheck issue fix [13:11] Hi, I cannot start a snap program due to error: open /home/user/.config/ttn-lw-cli/ttn-lw-cli.yml: permission denied ; same for ~/.ttn-lw-cli.yml ; snap connections ttn-lw-stack names home network network-bind [13:11] Anything that can be debugged, configured [13:23] maredebianum: snaps by default cannot access "dot files" which would include the ~/.config/... and the ~/.ttn-lw-cli.yml files you listed [13:24] maredebianum: if you have good reason to think that this particular snap should be able to access those files because i.e. the name of the snap you're using is something like ttn-lw-cli and it "owns" those filepaths, then you could reach out to the snap publisher to see if they would be willing to request a personal-files interfaces access which would allow the snap to access those files [13:24] maredebianum: otherwise you should be able to just move those files into your home directory without the prefix "." and then the snap should work fine with the home interface connected [13:27] PR core#122 closed: hooks: replaces symlinks in /usr/share/doc with the actual files [13:33] PR snapd#10642 closed: interfaces/u2f-devices: add Nitrokey FIDO2 [13:33] PR snapd#10664 opened: tests: set to 10 minutes the kill timeout for tests failing on slow boards === rbasak_ is now known as rbasak [13:48] sorry, chat app crashed. Problem was that .dotfiles are not readable in home from snap. Also, ~/snap/ seems not to be a possible workplace... [13:49] Are there any other funny restrictions to be aware of? [13:50] maredebianum: snaps by default cannot access "dot files" which would include the ~/.config/... and the ~/.ttn-lw-cli.yml files you listed [13:50] maredebianum: if you have good reason to think that this particular snap should be able to access those files because i.e. the name of the snap you're using is something like ttn-lw-cli and it "owns" those filepaths, then you could reach out to the snap publisher to see if they would be willing to request a personal-files interfaces access which would allow the snap to access those files [13:50] maredebianum: otherwise you should be able to just move those files into your home directory without the prefix "." and then the snap should work fine with the home interface connected [13:51] maredebianum: what did you mean by ~/snap/ not being a possible workplace? [13:51] Also restricted/permission denied [13:51] maredebianum: you can use ~/snap//current/ or ~/snap//common/ [13:51] did you try those paths ? [13:52] I do not want to clog up my home dir with non-dot config files. Ah, only this specific pattern is possible? [13:53] maredebianum: yes for any snap named foobar, it can always access any file (including dot files) in ~/snap/foobar/{current,common}/ [13:54] Ah, OK, thanks, that seems to work. [14:09] hmm `Codecov: Failed to properly upload: The process '/home/runner/work/_actions/codecov/codecov-action/v2/dist/codecov' failed with exit code 255` [14:09] hope it's just this one time it failed [14:14] bboozzoo: any more logs from that run ? [14:14] ijohnson: no, just that, i've restarted the run === alan_g_ is now known as alan_g [14:16] mmm that's annoying, since we are supposed to have verbose logs for codecov turned on :-/ [14:39] ijohnson: i've updated https://github.com/snapcore/snapd/pull/10166 let's see if the tests are happy about the change [14:39] PR #10166: systemd: implement restart as a single systemctl restart call <â›” Blocked> [14:40] bboozzoo: nice I'll take a look today at that [14:47] ijohnson: heh https://github.com/snapcore/snapd/runs/3401189645?check_suite_focus=true :) [14:48] hm maybe now it's getting confused as the test results were cached? [14:48] but we don't cache coverage output [14:49] ahhhh yeah I suppose that would make sense [14:49] let me re-try my PR that uses the coverage thing to see if it happens there too [14:51] ijohnson: https://github.com/snapcore/snapd/pull/10665 [14:51] PR #10665: github: do not try to upload coverage when working with cached run [14:52] this should fix it i think [14:53] thanks bboozzoo you beat me to it [14:53] (at least the problem with cached runs) [14:54] PR snapd#10665 opened: github: do not try to upload coverage when working with cached run [15:05] @mvo [15:05] * @mvo: can you land https://github.com/snapcore/snapd/pull/10663? [15:05] PR #10663: tests/main/services-install-hook-can-run-svcs: shellcheck issue fix [15:05] mborzecki: sure [15:08] mvo: thanks! [15:09] PR snapd#10663 closed: tests/main/services-install-hook-can-run-svcs: shellcheck issue fix [15:49] PR snapd#10665 closed: github: do not try to upload coverage when working with cached run [15:49] yay === alan_g_ is now known as alan_g [16:09] PR snapd#10666 opened: snapstate: abort kernel refresh if no gadget update can be found [16:34] PR snapd#10667 opened: cmd/snap-seccomp/syscalls: update syscalls list to libseccomp v2.2.0-428-g5c22d4b <âš  Critical> [16:35] ijohnson: ^^^ trivial update [16:40] ack [17:09] PR snapd#10668 opened: i18n/xgettext-go: preserve already escaped quotes [18:04] PR snapd#10669 opened: o/hookstate/ctlcmd: correct err message if missing root [21:36] * cachio_ afk