| zyga-mbp | goood morning :) | 05:06 |
|---|---|---|
| * zyga-mbp spent all of yesterday deep inside the cloud but is back now | 05:07 | |
| mardy | zyga-mbp: 'morning! | 05:47 |
| zyga-mbp | o/ :) | 05:47 |
| mborzecki | morning | 05:48 |
| mborzecki | mardy: zyga-mbp: hey | 05:48 |
| mborzecki | hm opensuse.org is down | 05:59 |
| zyga-mbp | yeah, it looks down from here as well | 06:03 |
| mup | PR snapd#10837 opened: cmd/libsnap-confine-private: fix set but unsed variable in the unit tests <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10837> | 06:14 |
| mborzecki | trivial PR ^^^ | 06:14 |
| mborzecki | and should make clang 13 happy, apparently it's more picky now | 06:15 |
| zyga-mbp | mborzecki +1 | 06:18 |
| mborzecki | ehh so https://forum.snapcraft.io/t/systemd-services-running-snap-apps-fail-to-associate-journal-logs/26714 someone tries to use a snap application as a service (defined separately, outside of snapd's knowledge), then tracking gets in the way and moves the app (since it's an app) to a separate sope | 06:35 |
| mborzecki | woah, opensuse landed a rebuild with new glibc too | 06:55 |
| mardy | mborzecki: hi! | 07:01 |
| pstolowski | morning | 07:01 |
| mvo | good morning pstolowski and mardy | 07:03 |
| mborzecki | pstolowski: mvo: hey | 07:03 |
| mvo | hey mborzecki | 07:04 |
| zyga-mbp | mborzecki ouch, interesting | 07:08 |
| zyga-mbp | mborzecki there's a gnome protocol for app tracking, also based on scopes | 07:08 |
| zyga-mbp | I guess it just doesn't matter there because it's a part of a desktop app | 07:08 |
| zyga-mbp | perhaps app tracking should bail-out if invoked in a systemd service context? | 07:09 |
| mborzecki | that would break the user, perhaps we shoudl just show a warning and continue | 07:10 |
| mardy | mvo: hi! | 07:21 |
| mardy | I need to leave for about 1 hour | 07:21 |
| mup | PR snapd#10838 opened: Component tests <Created by mardy> <https://github.com/snapcore/snapd/pull/10838> | 07:24 |
| mup | PR snapd#10831 closed: osutil/disks: add new `CreateLinearMapperDevice` helper <Skip spread> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10831> | 07:29 |
| mup | PR snapd#10834 closed: tests: fix netplan test on i386 architecture <Simple 😃> <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10834> | 07:34 |
| mup | PR snapd#10835 closed: tests: fix lxd-mount-units test which is based on core20 in ubuntu focal system <Simple 😃> <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10835> | 07:34 |
| miguelpires | mvo: hi, can you merge this https://github.com/snapcore/snapd/pull/10812 please? | 07:51 |
| mup | PR #10812: o/ifacestate: don't fail remove if disconnect hook fails <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10812> | 07:51 |
| pstolowski | mardy: hey, can you take a look at https://github.com/snapcore/snapd/pull/10737 again? | 07:56 |
| mup | PR #10737: o/snapstate: validation sets enforcing on update <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10737> | 07:56 |
| mborzecki | quick errand, back in 30 | 08:03 |
| mardy | pstolowski: approved -- I also have a question, but it's just to make sure the branches are intentional | 08:35 |
| mborzecki | mardy: can you take another look at https://github.com/snapcore/snapd/pull/10826 ? | 08:40 |
| mup | PR #10826: cmd/libsnap-confine-private: workaround BPF memory accounting, update apparmor profile <Security-High> <Needs security review> <cgroupv2-impish> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10826> | 08:40 |
| mborzecki | heh native mattermost app is kinda broken, whenever i click on a link it tries to open it in firefox, but in such a way that firefox shows a dialog that profile is in use | 08:42 |
| mardy | mborzecki: approved | 08:46 |
| mborzecki | had to restart the CI job in #10837, seems like there were network problems | 08:50 |
| mup | Bug #10837: Segfault when try to print using cups. <samba (Ubuntu):Fix Released> <samba (Debian):Fix Released> <https://launchpad.net/bugs/10837> | 08:50 |
| mup | PR #10837: cmd/libsnap-confine-private: fix set but unsed variable in the unit tests <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10837> | 08:50 |
| mborzecki | mardy: thanks! | 08:50 |
| pstolowski | damn, this new github ui is confusing indeed | 08:58 |
| mborzecki | hahah | 08:59 |
| mup | PR snapcraft#3584 opened: extension/desktop: Prevent keeping updating when there is no user use… <Created by didrocks> <https://github.com/snapcore/snapcraft/pull/3584> | 09:09 |
| zyga-mbp | mborzecki nice work on that .1 | 10:28 |
| zyga-mbp | I'll happily review, I've been following that thread | 10:29 |
| mborzecki | hm? | 10:30 |
| mup | PR snapd#10839 opened: cmd/snap-confine: update s-c apparmor profile to allow versioned ld.so <Security-High> <Needs security review> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10839> | 10:40 |
| zyga-mbp | yes, this :) | 10:41 |
| zyga-mbp | done | 10:43 |
| mborzecki | ah, haha | 10:50 |
| mup | PR snapd#10837 closed: cmd/libsnap-confine-private: fix set but unused variable in the unit tests <Simple 😃> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10837> | 11:10 |
| jamesh | mvo: would you be able to merge https://github.com/snapcore/snapd/pull/10571? It has received 2+ reviews, but I'm getting some unrelated CI failures that prevent me from landing it myself. | 11:16 |
| mup | PR #10571: daemon: implement access checkers for themes API <Needs Samuele review> <Needs security review> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/10571> | 11:16 |
| mvo | jamesh: sure | 11:16 |
| jamesh | mvo: most recent CI run is from today after merging master | 11:17 |
| jd | hello, is it possible to disable ssl_verify for a `snapcraftctl pull` in a snapcraft.yaml ? | 11:58 |
| JonathanEsk-Ridd | ijohnson: thanks for looking at kdenlive for me. looking in the journald output I see the supporting binary kdenlive_renderer is crashing after it fails to load /snap/core20/.../libacl etc and I've no idea why it would be loading it from there instead of /usr/lib/libacl etc there's various scripts in the command chain that alter lots of environment variables so meh maybe I'll have to play with lots of those and see what happens | 11:59 |
| JonathanEsk-Ridd | https://invent.kde.org/packaging/snapcraft-kde-applications/-/blob/Neon/release/kdenlive/kdenlive-launch/kdenlive-launch is the only one specific to kdenlive but I don't think setting those would cause the issue | 12:01 |
| mup | PR snapd#10788 closed: many: rename systemd.Kind to Backend for a bit more clarity <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/10788> | 12:10 |
| mborzecki | hm did we land a change in spread to run with a single worker? | 12:12 |
| pstolowski | does anyone remember the trick for time.Time mocking in the tests to make DeepEquals happy about time fields and not choke about its internal bits that differ? | 12:19 |
| zyga | pstolowski, I only did this once but IIRC you can round the nanonseconds/internal noise | 12:21 |
| zyga | and then perform the comparison | 12:21 |
| pstolowski | zyga: will try, thanks... i've a vague memory there was something else but completely forgot | 12:24 |
| mup | PR snapd#10840 opened: tests/main/security-device-cgroups: fix when both variants run on the same host <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10840> | 12:50 |
| zyga | hmm | 13:13 |
| zyga | mborzecki, not signed into github and away from token but https://github.com/snapcore/snapd/blob/f33c0179a4bef62c224160d77c996e5f9307d6ca/tests/main/cgroup-tracking/task.yaml#L77 | 13:14 |
| zyga | mborzecki, nitpicky a bit but we compare systemd version in one more place | 13:14 |
| zyga | and I wanted to make sure you were at least aware of the way | 13:15 |
| mborzecki | mardy: let's sync monday morning about the app tracking stuff | 13:41 |
| mborzecki | maybe it's actually one problem | 13:41 |
| mardy | mborzecki: OK! | 13:54 |
| mborzecki | i suspect it's ok for normal user, but maybe not so ok for root | 13:55 |
| mup | PR snapd#10841 opened: interfaces/dsp: add a usb rule to the ambarella flavor <⚠Critical> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10841> | 14:06 |
| zyga | mborzecki root doesn't have app tracking, we equate root to services, no? | 14:09 |
| mborzecki | zyga: not really iirc | 14:10 |
| zyga | hmm, perhaps I remember wrong then | 14:10 |
| mup | PR snapd#10842 opened: interfaces/apparmor/template.go: allow inspection of dbus mediation level <Simple 😃> <Needs security review> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10842> | 14:11 |
| zyga | ah right https://github.com/snapcore/snapd/blob/master/cmd/snap/cmd_run.go#L1158 | 14:11 |
| mup | PR snapd#10839 closed: cmd/snap-confine: update s-c apparmor profile to allow versioned ld.so <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10839> | 14:16 |
| mborzecki | yay | 14:16 |
| mborzecki | hm we just continue if tracking could not be established, which probably is ok | 14:18 |
| mborzecki | i should make thhe devie cgroup code verify that we're in a scope/service of a snap, i.e. there's a `snap.` prefix and `.scope` || `.service` suffix | 14:18 |
| ijohnson[m] | Jonathan Esk-Riddell: maybe something to discuss in the snapcraft clinic today | 14:18 |
| zyga | mborzecki, mmm | 14:19 |
| zyga | yeah | 14:19 |
| zyga | that is a good idea | 14:19 |
| zyga | those are not synchronized today | 14:19 |
| mup | PR snapd#10840 closed: tests/main/security-device-cgroups: fix when both variants run on the same host <Simple 😃> <Created by bboozzoo> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10840> | 20:32 |
| sarnold | diddledani: hello :) is there a way to go from https://snapstats.org/snaps/multipass to a list of releases for this snap? | 20:54 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!