Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add initContainer to delete automount udev rule file #305

Merged
merged 1 commit into from
Apr 28, 2020

Conversation

timoreimann
Copy link
Contributor

@timoreimann timoreimann commented Apr 27, 2020

The automount udev rule shipping on all DigitalOcean droplets can conflict with CSI-managed volumes, leading to "resource busy" errors. As the file won't be needed, we can safely delete it when the Node service starts up.

Fixes #297

@timoreimann timoreimann requested a review from adamwg April 27, 2020 20:21
Copy link
Contributor

@adamwg adamwg left a comment

Choose a reason for hiding this comment

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

This is a pretty tidy solution to the problem 👍

The automount udev rule shipping on all DigitalOcean droplets can
conflict with CSI-managed volumes, leading to "resource busy" errors. As
the file won't be needed, we can safely delete it when the Node service
starts up.
@timoreimann timoreimann force-pushed the add-initcontainer-to-delete-automount-udev-rule branch from c3e6974 to 3b4e455 Compare April 28, 2020 07:16
@timoreimann timoreimann merged commit bcbd4aa into master Apr 28, 2020
@timoreimann timoreimann deleted the add-initcontainer-to-delete-automount-udev-rule branch April 28, 2020 10:15
@jhoelzel
Copy link

jhoelzel commented Aug 21, 2023

Just a headsup, this is still relevant today, and hardened clusters will struggle with the the file.
Under current security suggestions, using hostpath is out of the picture in general and the init container does not really justify an exception :/

log

rm: can't stat '/etc/udev/rules.d/99-digitalocean-automount.rules': Permission denied

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.

MountVolume.MountDevice failed for volume
3 participants