Skip to content

Use --chmod flag instead of separate RUN chmod command in backend Dockerfile#770

Open
mueckinger wants to merge 1 commit intothunderbird:mainfrom
mueckinger:chmod
Open

Use --chmod flag instead of separate RUN chmod command in backend Dockerfile#770
mueckinger wants to merge 1 commit intothunderbird:mainfrom
mueckinger:chmod

Conversation

@mueckinger
Copy link
Copy Markdown

@mueckinger mueckinger commented Apr 25, 2026

It is good practise to do the chmod in the COPY, not in a separate command as this creates an extra layer and is known to not work in every build environment, for example in my setup which is kaniko.


Note

Low Risk
Low risk build change that only affects Docker image layering and file permissions for the entrypoint script. Potential impact is limited to environments/builders that differ in support for COPY --chmod.

Overview
Updates deploy/docker/backend.Dockerfile to apply executable permissions to backend-entrypoint.sh during COPY via --chmod=755, removing the separate RUN chmod +x layer.

This reduces image layers and improves compatibility with builders where post-copy chmod can be unreliable.

Reviewed by Cursor Bugbot for commit 60ab6bf. Bugbot is set up for automated code reviews on this repo. Configure here.

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.

1 participant