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

drop permissions of container to match host system #8007

Merged
merged 1 commit into from
Mar 10, 2023

Conversation

gdams
Copy link
Contributor

@gdams gdams commented Mar 9, 2023

When you run Techempower as a non-elevated user, it will create report files that that the user cannot clean up.

This PR ensures that the Group/User IDs match the host systems.

@gdams
Copy link
Contributor Author

gdams commented Mar 9, 2023

Hi @nbrady-techempower - could you please review/merge (or tag an appropriate reviewer)? Thanks so much!

@NateBrady23 NateBrady23 merged commit 1cf4924 into TechEmpower:master Mar 10, 2023
@gdams gdams deleted the docker_permission branch March 11, 2023 10:12
joanhey added a commit to joanhey/FrameworkBenchmarks that referenced this pull request Mar 11, 2023
NateBrady23 pushed a commit that referenced this pull request Mar 13, 2023
#8007)" (#8013)

* Revert "drop permissions of container to match host system (#8007)"

This reverts commit 1cf4924.

* [ci lang-only php] To test CI

* [ci run-all] To test CI
franz1981 pushed a commit to franz1981/FrameworkBenchmarks that referenced this pull request Jun 23, 2023
franz1981 pushed a commit to franz1981/FrameworkBenchmarks that referenced this pull request Jun 23, 2023
TechEmpower#8007)" (TechEmpower#8013)

* Revert "drop permissions of container to match host system (TechEmpower#8007)"

This reverts commit 1cf4924.

* [ci lang-only php] To test CI

* [ci run-all] To test CI
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.

2 participants