Skip to content

Commit

Permalink
The entrypoints in Docker Image should be owned by Airflow (#10853)
Browse files Browse the repository at this point in the history
Since we are running the airflow image as airflow user, the
entrypoint and clear-logs scripts should also be set as airflow.

This had no impact if you actually run this as root user or
when your group was root (which was recommended).

(cherry picked from commit d9920fa)
  • Loading branch information
potiuk committed Sep 15, 2020
1 parent 96b58c5 commit 2eafdf5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -350,8 +350,8 @@ RUN mkdir -pv "${AIRFLOW_HOME}"; \

COPY --chown=airflow:root --from=airflow-build-image /root/.local "${AIRFLOW_USER_HOME_DIR}/.local"

COPY scripts/in_container/prod/entrypoint_prod.sh /entrypoint
COPY scripts/in_container/prod/clean-logs.sh /clean-logs
COPY --chown=airflow:root scripts/in_container/prod/entrypoint_prod.sh /entrypoint
COPY --chown=airflow:root scripts/in_container/prod/clean-logs.sh /clean-logs
RUN chmod a+x /entrypoint /clean-logs

# Make /etc/passwd root-group-writeable so that user can be dynamically added by OpenShift
Expand Down

0 comments on commit 2eafdf5

Please sign in to comment.