Skip to content

The last release of the EL* build docker containers should not have set USER #824

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.

Sign up for GitHub

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

Jump to bottom
Closed
trevor-vaughan opened this issue May 27, 2022 · 1 comment · Fixed by #825
Closed

The last release of the EL* build docker containers should not have set USER #824

trevor-vaughan opened this issue May 27, 2022 · 1 comment · Fixed by #825
Assignees
@trevor-vaughan
Labels
bug

Comments

@trevor-vaughan
Copy link
Member

Most of the automated tasks that use the docker images expect to inject commands as root.

Forcing the user to build_user caused these processes to break.
@trevor-vaughan trevor-vaughan self-assigned this May 27, 2022
@trevor-vaughan trevor-vaughan moved this to In Progress in SIMP 6.6.0 Release May 27, 2022
trevor-vaughan added a commit to trevor-vaughan/simp-core that referenced this issue May 27, 2022
@trevor-vaughan
Revert addition of build container USER
e8b42b9 
* No longer set the default user in the build containers to `build_user`
  since this caused dependent processes to break.

Closes simp#824
@trevor-vaughan trevor-vaughan mentioned this issue May 27, 2022
Merged
@trevor-vaughan
Copy link
Member Author

Corrected images have been pushed to DockerHub

@trevor-vaughan trevor-vaughan moved this from In Progress to Ready for Review in SIMP 6.6.0 Release May 27, 2022
op-ct pushed a commit that referenced this issue May 27, 2022
@trevor-vaughan @op-ct
Revert addition of build container USER
7498c44 
* No longer set the default user in the build containers to `build_user`
  since this caused dependent processes to break.

Closes #824
Repository owner moved this from Ready for Review to Done in SIMP 6.6.0 Release May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trevor-vaughan trevor-vaughan

Labels
bug
Projects
SIMP 6.6.0 Release
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Revert addition of build container USER trevor-vaughan/simp-core
1 participant
@trevor-vaughan