Skip to content

Commit

Permalink
The entrypoints in Docker Image should be owned by Airflow (apache#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).
  • Loading branch information
@potiuk
potiuk authored Sep 12, 2020
1 parent 35840ff commit d9920fa
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Dockerfile
View file
Original file line number Diff line number Diff line change
Expand Up @@ -349,8 +349,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 d9920fa

Please sign in to comment.