-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Use chown(8) only once #1263
Use chown(8) only once #1263
Conversation
It's possible to pass multiple files to chown utility, so run it only once to make changes.
Deploy request for nginx-kubernetes-ingress rejected. Rejected with commit 03e3e27 https://docs.netlify.com/configure-builds/environment-variables/#sensitive-variable-policy |
&& chown -R nginx:0 /var/log/app_protect/ \ | ||
&& chown -R nginx:0 /opt/app_protect/ \ | ||
&& chown -R nginx:0 /var/log/nginx/ \ | ||
/etc/nginx \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the order of folders here is a bit different from the file build/appprotect/DockerfileWithAppProtectForPlus
.
Could we use the same order for consistency?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pleshakov I've sorted that out cause in that order it has better readility, however I can revert that one.
Requsted by: pleshakov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick, but could you repace spaces with tabs so it's consistent with the rest of the file?
Requested by: lucacome
It's possible to pass multiple files to chown utility, so run it
only once to make changes.
Proposed changes
Describe the use case and detail of the change. If this PR addresses an issue on GitHub, make sure to include a link to that issue here in this description (not in the title of the PR).
Checklist
Before creating a PR, run through this checklist and mark each as complete.