/srv/irclogs.ubuntu.com/2022/04/27/#snappy.txt

mupPR snapcraft#3713 opened: requirements: unpin pyyaml and update dependencies <Created by cmatsuoka> <https://github.com/snapcore/snapcraft/pull/3713>01:33
mupPR snapcraft#3710 closed: tests: use craft-cli lib fixtures <Created by facundobatista> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3710>01:38
mupPR snapd#11729 opened: cmd/snap-update-ns: correctly set sticky bit on created directories where applicable <Needs security review> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/11729>02:54
jameshamurray: if you have a chance, could you have a look at https://github.com/snapcore/snapd/pull/11729? This is to fix the permission problems with the private shared-memory feature.03:08
mupPR #11729: cmd/snap-update-ns: correctly set sticky bit on created directories where applicable <Needs security review> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/11729>03:08
mborzeckimorning05:58
mardyhi mborzecki 07:26
pstolowskimorning07:27
mupPR snapd#11730 opened: tests/lib/tools: add piboot to boot_path() <Simple 😃> <Bug> <Created by Meulengracht> <https://github.com/snapcore/snapd/pull/11730>07:30
mardyhi pstolowski 07:32
mupPR snapd#11721 closed: cmd/snap: handler call verifications for cmd_quota_tests <Simple 😃> <Created by Meulengracht> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/11721>08:45
jameshmvo: hi. For https://github.com/snapcore/snapd/pull/11708, I need a newer libseccomp than snapd is built with. What's the process for getting it updated?08:49
mupPR #11708: interfaces: add a steam-support interface <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/11708>08:49
mvojamesh: ideally we would SRU it into the distro pfsmorigo was working on this last time. we can put it into the image-ppa but we really want a SRU in parallel so that we are in sync with the distro and avoid that the distro gets e.g. a security update and we miss it because our PPA has a newer version08:51
mardyaaand, it's snowing :-)08:52
jameshmvo: I thought that might be the case. Annoyingly, jammy has 2.5.3 when the syscall I want to allow was added in 2.5.408:55
jameshmvo: I also created https://github.com/snapcore/snapd/pull/11729 to address the directory permission bugs related to private shared-memory.09:02
mupPR #11729: cmd/snap-update-ns: correctly set sticky bit on created directories where applicable <Needs security review> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/11729>09:02
jameshI think those problems would likely have affected the x11 interface too in the past, but likely only on some Ubuntu Core systems09:03
mupPR snapd#11731 opened: interfaces/builtin/custom-device: fix unit tests on hosts with different libexecdir <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/11731>09:31
mupPR snapd#11700 closed: sandbox: move profile load/unload to sandbox/apparmor <Simple 😃> <Created by mardy> <Merged by mardy> <https://github.com/snapcore/snapd/pull/11700>10:16
mupPR snapd#11732 opened: image, cmd/snap-preseed: allow passing custom apparmor features path <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/11732>10:51
mupPR snapcraft#3713 closed: requirements: unpin pyyaml and update dependencies <Created by cmatsuoka> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3713>11:04
mupPR snapd#11733 opened: snap/quota: add values for journal quotas (journal quota 2/n) <Created by Meulengracht> <https://github.com/snapcore/snapd/pull/11733>11:16
mupPR snapd#11734 opened: interfaces: network-manager: add AppArmor rule for configuring bridges <Created by IsaacJT> <https://github.com/snapcore/snapd/pull/11734>11:16
jameshmborzecki: thanks for your review on my PR. Do you have any thoughts about https://github.com/snapcore/snapd/pull/11729#discussion_r859652193 ?11:26
mupPR #11729: cmd/snap-update-ns: correctly set sticky bit on created directories where applicable <Needs security review> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/11729>11:26
mupPR snapd#11731 closed: interfaces/builtin/custom-device: fix unit tests on hosts with different libexecdir <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/11731>11:26
mborzeckijamesh: hey, let me reply on github11:31
jameshmborzecki: thanks. This turned out to be even messier than I thought.11:32
mborzeckijamesh: yeah, the downsides of doing syscalls ourselves11:34
mborzeckijamesh: it'd be interesting to see if S_ISVTX is indeed a different bit on other systems, as i suspect that's the primary motivation for use of the helper on go side11:35
jameshmborzecki: it's definitely different to the FileMode constants. 0o777 | fs.ModeSticky == 0o400077711:36
jameshrather than 0o177711:36
jameshmborzecki: the Go FileMode seems to be a union of the file modes of supported platforms (posix, plan9, windows)11:41
jameshnot sure why they decided to keep posix compat for the lower 9 bits but nothing else11:41
mborzeckiheh fun11:41
mborzeckii also found a different value for wasm https://github.com/golang/go/blob/19309779ac5e2f5a2fd3cbb34421dafb2855ac21/src/syscall/syscall_js.go#L18311:42
jameshThat looks like the normal posix bit?11:43
jamesh0o1000 == 0x20011:44
mborzeckiah you're right11:50
jameshjust imagine: if unix had one extra permission on top of read, write, and execute, we'd probably never want to use octal constants.11:56
mupPR snapcraft#3714 opened: spread: enable core22 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3714>11:59
mvojamesh: \o/ for 1172914:41
mupPR snapd#11735 opened: tests/main/user-session-env: for for opensuse <Created by mardy> <https://github.com/snapcore/snapd/pull/11735>14:57
mupPR snapcraft#3715 opened: plugs: install snaps from content plugs <Created by mr-cal> <https://github.com/snapcore/snapcraft/pull/3715>16:39
=== claudio__ is now known as cmatsuoka
mupPR snapcraft#3716 opened: meta: dump unicode yaml data <Created by cmatsuoka> <https://github.com/snapcore/snapcraft/pull/3716>18:09
mupPR snapd#11736 opened: interfaces/builtin: shared-memory drop plugs allow-installation: true <Created by pedronis> <https://github.com/snapcore/snapd/pull/11736>18:28
mupPR snapcraft#3717 opened: commands: release and close <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3717>20:30
mupPR snapcraft#3716 closed: meta: dump unicode yaml data <Created by cmatsuoka> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3716>20:35
mupPR snapcraft#3717 closed: commands: release and close <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3717>22:10
mupPR snapcraft#3715 closed: plugs: install snaps from content plugs <Created by mr-cal> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3715>22:59

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