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

ci: Run GH action CI build+test as non-root #2424

Merged
merged 1 commit into from
Aug 26, 2021

Conversation

cgwalters
Copy link
Member

This is really the standard best practice, matching how
e.g. dpkg/rpm work, as well as most local development
environments (including mine) with e.g. toolbox.

@cgwalters cgwalters force-pushed the test-nonroot-ci branch 2 times, most recently from 294036f to ea2ec16 Compare August 26, 2021 14:08
@cgwalters
Copy link
Member Author

Hmm. Well, that worked.

@lucab
Copy link
Member

lucab commented Aug 26, 2021

LGTM, although I think this skips some other tests which only run as root.

@cgwalters
Copy link
Member Author

Whee right, we have test-basic-root.sh too.

This is really the standard best practice, matching how
e.g. dpkg/rpm work, as well as most local development
environments (including mine) with e.g. `toolbox`.
@cgwalters
Copy link
Member Author

Whee right, we have test-basic-root.sh too.

But it turns out right now the Jenkins flow is doing CI as uid 0, so we should be running this there.

Copy link
Member

@dbnicholson dbnicholson left a comment

Choose a reason for hiding this comment

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

That makes sense. There's definitely value in running tests as root for ostree, but I agree the common case is unprivileged. Good that it's being covered on the jenkins setup.

@cgwalters cgwalters merged commit 77a86cc into ostreedev:main Aug 26, 2021
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.

3 participants