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

RFE/Design question: Should the role support setting ownership/permissions of mount point #274

Closed
benblasco opened this issue Jun 5, 2022 · 3 comments

Comments

@benblasco
Copy link

This role creates mount points during execution, but does not offer the option to define the ownership or permissions for those mount points.

Is this lack of functionality a deliberate design decision, a worthwhile RFE, or a bad idea? I have discussed this with a colleague who thought it reasonable.

My example use case is the deployment of OpenShift Local (aka CodeReady Containers) as a regular user. It requires a large volume of data to be available at ~/.crc. It would be reasonable to create this mount point with ownership and appropriate permissions for .

Your feedback would be appreciated, and I may be able to contribute to the codebase if this is deemed worthwhile.

@benblasco benblasco changed the title RFE/Design question: Should the role support setting ownershiip/perms of mount point RFE/Design question: Should the role support setting ownership/permissions of mount point Jun 5, 2022
@benblasco
Copy link
Author

Also referenced here:
https://issues.redhat.com/browse/RHELBU-1739

@Tronde
Copy link

Tronde commented May 4, 2023

Hello @benblasco,
Though I'm not a contributer of this role I like your RFE pretty much.

IMHO it's not uncommon to run applications and services as a non-root user. Still these apps/services need to read/write data. When the role is capable of creating a mount point it seems perfectly reasonable to me that it's capabel of setting ownership and permissions as well.

@briansmith0
Copy link

This should be completed now with #352

@richm or @japokorn can you please close this issue?

@richm richm closed this as completed Jul 7, 2023
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

No branches or pull requests

4 participants