/srv/irclogs.ubuntu.com/2021/08/23/#snappy.txt

=== JavaPP is now known as Java--
=== Eickmeyer_ is now known as Eickmeyer
pstolowskimorning06:35
mardypstolowski: hi!06:39
zygagood morning07:00
mborzeckimorning, i'll be staring around 1108:13
mvogood morning mborzecki and zyga 08:27
mvomborzecki: cool!08:27
zygamvo, good morning, how are you doing?08:30
zygaI hope that this week is productive and smooth 08:30
mborzeckimvo: can you land https://github.com/snapcore/snapd/pull/10640 ? failed tests are unrelated to the changes in the PR09:11
mupPR #10640: cmd, packaging: import BPF headers from kernel, detect whether host headers are usable <Simple 😃> <cgroupv2> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10640>09:11
mvomborzecki: sure09:12
mvomborzecki: done09:12
mborzeckimvo: thank you09:12
mupPR snapd#10640 closed: cmd, packaging: import BPF headers from kernel, detect whether host headers are usable <Simple 😃> <cgroupv2> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10640>09:13
miguelpiresGood 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
mupPR #10617: cmd/snap: add Size column to refresh --list <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10617>10:46
mupPR #10643: testutil: add DeepUnsortedMatches Checker <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10643>10:46
mvomiguelpires: sure, happy to do that10:49
miguelpiresThank you10:50
mupPR snapd#10643 closed: testutil: add DeepUnsortedMatches Checker <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10643>10:58
mupPR snapd#10658 opened: cmd/libsnap-confine-private: add BPF support helpers <Needs security review> <cgroupv2> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10658>11:03
mborzeckiehh missed a patch there11:06
* pstolowski lunch11:18
mupPR snapd#10659 opened: store: deal correctly with "assumes" from the store raw yaml <Created by mvo5> <https://github.com/snapcore/snapd/pull/10659>11:23
mupPR snapd#10660 opened: overlord: add manager test for "assumes" checking <Created by mvo5> <https://github.com/snapcore/snapd/pull/10660>11:23
mborzeckiand the fun part of device cgroup v2 is up now too: https://github.com/snapcore/snapd/pull/1066111:32
mupPR #10661: cmd/libsnap-confine-private: device cgroup v2 support <Complex> <Needs security review> <cgroupv2> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10661>11:32
mupPR snapd#10661 opened: cmd/libsnap-confine-private: device cgroup v2 support <Complex> <Needs security review> <cgroupv2> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10661>11:33
mupPR snapd#10662 opened: .github/workflows/test.yaml: test github.events key <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10662>12:33
ijohnson[m]alright hopefully we will start getting real coverage for PR's after ^ is merged to master12:34
mborzeckitrivial PR: https://github.com/snapcore/snapd/pull/1066312:41
mupPR #10663: tests/main/services-install-hook-can-run-svcs: shellcheck issue fix <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10663>12:41
mupPR snapd#10663 opened: tests/main/services-install-hook-can-run-svcs: shellcheck issue fix <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10663>12:43
maredebianumHi, 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-bind13:11
maredebianumAnything that can be debugged, configured13:11
ijohnson[m]maredebianum: snaps by default cannot access "dot files" which would include the ~/.config/... and the ~/.ttn-lw-cli.yml files you listed13:23
ijohnson[m]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 files13:24
ijohnson[m]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 connected13:24
mupPR core#122 closed: hooks: replaces symlinks in /usr/share/doc with the actual files <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/core/pull/122>13:27
mupPR snapd#10642 closed: interfaces/u2f-devices: add Nitrokey FIDO2 <Simple 😃> <Created by kkeijzer> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10642>13:33
mupPR snapd#10664 opened: tests: set to 10 minutes the kill timeout for tests failing on slow boards <Simple 😃> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10664>13:33
=== rbasak_ is now known as rbasak
maredebianumsorry, chat app crashed. Problem was that .dotfiles are not readable in home from snap. Also, ~/snap/ seems not to be a possible workplace...13:48
maredebianumAre there any other funny restrictions to be aware of?13:49
ijohnson[m]maredebianum: snaps by default cannot access "dot files" which would include the ~/.config/... and the ~/.ttn-lw-cli.yml files you listed13:50
ijohnson[m]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 files13:50
ijohnson[m]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 connected13:50
ijohnson[m]maredebianum: what did you mean by ~/snap/ not being a possible workplace?13:51
maredebianumAlso restricted/permission denied13:51
ijohnson[m]maredebianum: you can use ~/snap/<name-of-snap>/current/ or ~/snap/<name-of-snap>/common/13:51
ijohnson[m]did you try those paths ?13:51
maredebianumI do not want to clog up my home dir with non-dot config files. Ah, only this specific pattern is possible?13:52
ijohnson[m]maredebianum: yes for any snap named foobar, it can always access any file (including dot files) in ~/snap/foobar/{current,common}/13:53
maredebianumAh, OK, thanks, that seems to work.13:54
mborzeckihmm `Codecov: Failed to properly upload: The process '/home/runner/work/_actions/codecov/codecov-action/v2/dist/codecov' failed with exit code 255`14:09
mborzeckihope it's just this one time it failed14:09
ijohnson[m]bboozzoo: any more logs from that run ?14:14
mborzeckiijohnson: no, just that, i've restarted the run14:14
=== alan_g_ is now known as alan_g
ijohnson[m]mmm that's annoying, since we are supposed to have verbose logs for codecov turned on :-/14:16
mborzeckiijohnson: i've updated https://github.com/snapcore/snapd/pull/10166 let's see if the tests are happy about the change14:39
mupPR #10166: systemd: implement restart as a single systemctl restart call <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10166>14:39
ijohnson[m]bboozzoo: nice I'll take a look today at that 14:40
mborzeckiijohnson: heh https://github.com/snapcore/snapd/runs/3401189645?check_suite_focus=true :)14:47
mborzeckihm maybe now it's getting confused as the test results were cached?14:48
mborzeckibut we don't cache coverage output14:48
ijohnson[m]ahhhh yeah I suppose that would make sense14:49
ijohnson[m]let me re-try my PR that uses the coverage thing to see if it happens there too14:49
mborzeckiijohnson: https://github.com/snapcore/snapd/pull/10665 14:51
mupPR #10665: github: do not try to upload coverage when working with cached run <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10665>14:51
mborzeckithis should fix it i think14:52
ijohnson[m]thanks bboozzoo you beat me to it14:53
mborzecki(at least the problem with cached runs)14:53
mupPR snapd#10665 opened: github: do not try to upload coverage when working with cached run <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10665>14:54
mborzecki@mvo15:05
mborzecki * @mvo: can you land https://github.com/snapcore/snapd/pull/10663?15:05
mupPR #10663: tests/main/services-install-hook-can-run-svcs: shellcheck issue fix <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10663>15:05
mvomborzecki: sure15:05
mborzeckimvo: thanks!15:08
mupPR snapd#10663 closed: tests/main/services-install-hook-can-run-svcs: shellcheck issue fix <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10663>15:09
mupPR snapd#10665 closed: github: do not try to upload coverage when working with cached run <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/10665>15:49
mborzeckiyay15:49
=== alan_g_ is now known as alan_g
mupPR snapd#10666 opened: snapstate: abort kernel refresh if no gadget update can be found <Created by mvo5> <https://github.com/snapcore/snapd/pull/10666>16:09
mupPR snapd#10667 opened: cmd/snap-seccomp/syscalls: update syscalls list to libseccomp v2.2.0-428-g5c22d4b <⚠ Critical> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10667>16:34
mborzeckiijohnson: ^^^ trivial update16:35
ijohnson[m]ack16:40
mupPR snapd#10668 opened: i18n/xgettext-go: preserve already escaped quotes <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10668>17:09
mupPR snapd#10669 opened: o/hookstate/ctlcmd: correct err message if missing root <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10669>18:04
* cachio_ afk21:36

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