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 example for non-root ownership for files & parent directories #450

Closed
wants to merge 2 commits into from

Conversation

mohelt
Copy link
Contributor

@mohelt mohelt commented Aug 31, 2022

managing-files: Add example for non root ownership

Add an example for non root ownership for files
and parent directories

fixes: #441

@mohelt mohelt requested a review from travier August 31, 2022 11:52
@mohelt mohelt force-pushed the Ownership branch 2 times, most recently from d082c74 to 9f74c21 Compare August 31, 2022 12:50
Add an example for non root ownership for files
and parent directories
@travier
Copy link
Member

travier commented Aug 31, 2022

Let's re-title this PR to something more descriptive?

@travier
Copy link
Member

travier commented Aug 31, 2022

Can you also update https://docs.fedoraproject.org/en-US/fedora-coreos/tutorial-user-systemd-unit-on-boot/ as mentioned in the issue?

@mohelt mohelt changed the title Ownership Add example for non-root ownership for files & parent directories Aug 31, 2022
Update ownership in example for files and directories.
name: sleeper
group:
name: sleeper
- path: /home/sleeper/
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we need this one as it's created with the user.

name: sleeper
group:
name: sleeper
- path: /home/sleeper/
Copy link
Member

Choose a reason for hiding this comment

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

same here

storage:
files:
# This creates a file, and sets the ownership
- path: /home/directory/you/want.txt
Copy link
Member

Choose a reason for hiding this comment

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

Let's use a path with a name that matches the user and use a real configuration example like https://discussion.fedoraproject.org/t/ignition-when-a-file-is-created-parent-dir-is-still-owned-by-root/41222

@travier
Copy link
Member

travier commented Sep 2, 2022

Also make sure to validate that your example pass Butane config validation. There are errors in CI for this one.

@travier
Copy link
Member

travier commented Sep 12, 2022

Hey @mohelt , do you still want to work on this one or should I take over / give it to someone else? Thanks!

@travier
Copy link
Member

travier commented Sep 19, 2022

Replaced and updated in #459.

@travier travier closed this Sep 19, 2022
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.

Example for non root ownership for files & parent dirs
2 participants