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

feat: add chown (linux only) #695

Merged
merged 2 commits into from
Aug 6, 2024
Merged

Conversation

cdrage
Copy link
Contributor

@cdrage cdrage commented Jul 31, 2024

feat: add chown (linux only)

What does this PR do?

Screenshot / video of UI

Screenshot 2024-07-31 at 2 42 38 PM

What issues does this PR fix or reference?

Closes #472

How to test this PR?

  1. Be on Linux
  2. Build an image
  3. See that it is chown (ex. ls -l /outputdir)

Signed-off-by: Charlie Drage [email protected]

@cdrage cdrage requested a review from a team as a code owner July 31, 2024 18:48
@cdrage
Copy link
Contributor Author

cdrage commented Jul 31, 2024

FYI based on #684 since it requires certain Linux features to be in for this to work.

@cdrage
Copy link
Contributor Author

cdrage commented Jul 31, 2024

Update: Fixing a test, but otherwise ready for review 👍 Fixed!

@cdrage
Copy link
Contributor Author

cdrage commented Aug 1, 2024

@deboer-tim added chown detection / fill in with a new commit

cdrage added 2 commits August 6, 2024 13:20
### What does this PR do?

* Adds chown to the advanced settings page for build
* Only applicable to Linux (see:
  osbuild/bootc-image-builder#576)

### Screenshot / video of UI

<!-- If this PR is changing UI, please include
screenshots or screencasts showing the difference -->

### What issues does this PR fix or reference?

<!-- Include any related issues from Podman Desktop
repository (or from another issue tracker). -->

Closes podman-desktop#472

### How to test this PR?

<!-- Please explain steps to reproduce -->

1. Be on Linux
2. Build an image
3. See that it is chown (ex. ls -l /outputdir)

Signed-off-by: Charlie Drage <[email protected]>
Signed-off-by: Charlie Drage <[email protected]>
@cdrage cdrage merged commit 357038a into podman-desktop:main Aug 6, 2024
4 of 5 checks passed
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.

Support chown from bootc-image-builder
2 participants