Skip to content

Comments

extra-files: add udev rule for snap auto-import#132

Open
mardy wants to merge 1 commit intocanonical:masterfrom
mardy:udev-auto-import
Open

extra-files: add udev rule for snap auto-import#132
mardy wants to merge 1 commit intocanonical:masterfrom
mardy:udev-auto-import

Conversation

@mardy
Copy link

@mardy mardy commented May 24, 2022

This file has always been shipped by snapd, but we are about to remove
it from there because it causes issues on recent classic systems.

See also: https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1966203

This file has always been shipped by snapd, but we are about to remove
it from there because it causes issues on recent classic systems.

See also: https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1966203
Copy link
Member

@alfonsosanchezbeato alfonsosanchezbeato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! Which is the PR removing from snapd?

@bboozzoo
Copy link

Core22 has this: canonical/core-base#35 should we add something similar here too? Does udev/systemd from 16.04 even has the necessary support?

@mardy
Copy link
Author

mardy commented May 27, 2022

LGTM, thanks! Which is the PR removing from snapd?

It will come later, after the core snap has been released. By the way, how often does that happen?

@mardy
Copy link
Author

mardy commented May 27, 2022

Core22 has this: snapcore/core-base#35 should we add something similar here too? Does udev/systemd from 16.04 even has the necessary support?

No, it doesn't. In core22 that change was needed to workaround the security restrictions recently added to the udev unit file; but in previous Ubuntu distributions these restrictions are not there, so we can safely use unshare and call mount(). :-)

@valentindavid
Copy link

Has this been opened also in snapcore/core20 and snapcore/core18? I think those will need the fix too.

@mardy
Copy link
Author

mardy commented May 31, 2022

Has this been opened also in snapcore/core20 and snapcore/core18? I think those will need the fix too.

No, they already have their udev rules. Links in https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1966203/comments/13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants