NotificationsYou must be signed in to change notification settings
Fork928
Star10.1k

fix(dogfood/Dockerfile): add explicit --chown to COPY directive#13569

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

johnstcn merged 2 commits intomainfromcj/dogfood-dockerfile-copy-chown

Jun 13, 2024

Merged

fix(dogfood/Dockerfile): add explicit --chown to COPY directive#13569

johnstcn merged 2 commits intomainfromcj/dogfood-dockerfile-copy-chown

Jun 13, 2024

Conversation

Copy link

Member

johnstcn commentedJun 13, 2024

Problem: when buildingdogfood/Dockerfile with envbuilder,/etc/sudoers.d/nopasswd was owned bycoder:coder until I did this.

This may be a bug in kaniko, but I haven't been able to figure out an explicit test to reproduce it. For now I'm just adding an explicit--chown to the COPY directive.

fix(dogfood): Dockerfile: add explicit --chown to COPY directive

87f6f2d

johnstcn requested a review fromdeansheather

June 13, 2024 13:39

johnstcn self-assigned this

Jun 13, 2024

johnstcn changed the title~~fix(dogfood): Dockerfile: add explicit --chown to COPY directive~~fix(dogfood/Dockerfile): add explicit --chown to COPY directive

Jun 13, 2024

fixup! fix(dogfood): Dockerfile: add explicit --chown to COPY directive

79b1ca8