Skip to content

fix(dogfood/Dockerfile): change ownership of /etc/sudoers.d to root #13793

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

Merged
merged 2 commits into from
Jul 5, 2024

Conversation

johnstcn
Copy link
Member

@johnstcn johnstcn commented Jul 5, 2024

to work around coder/envbuilder#70

This is normally a no-op when building on Docker.

  • Verified by building locally and executing sudo su
  • Verified with coder-envbuilder template in combination with local modifications
  • Verified by pulling the built image from depot.dev and running sudo su

@johnstcn johnstcn self-assigned this Jul 5, 2024
@johnstcn johnstcn changed the title fix(dogfood/Dockerfile): manually chown /etc/sudoers.d fix(dogfood/Dockerfile): change ownership of /etc/sudoers.d to root Jul 5, 2024
@johnstcn johnstcn merged commit da89114 into main Jul 5, 2024
40 checks passed
@johnstcn johnstcn deleted the cj/dogfood-chown-sudoers branch July 5, 2024 11:42
@github-actions github-actions bot locked and limited conversation to collaborators Jul 5, 2024
@johnstcn
Copy link
Member Author

johnstcn commented Jul 5, 2024

Verified also by running

depot pull b1zph7tvtf

and checking sudo su inside container.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants