Skip to content

Commit 4841427

Browse files
authored
Change style of Airflow to be consistent with other Dockerfiles (#960)
Change style to be consistent with other Dockerfiles
1 parent 7e34d80 commit 4841427

File tree

1 file changed

+23
-21
lines changed

1 file changed

+23
-21
lines changed

airflow/Dockerfile

Lines changed: 23 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -100,27 +100,29 @@ ENV PATH=$PATH:/bin:$HOME/app/bin
100100
ENV AIRFLOW_HOME=$HOME/airflow
101101

102102
# Update image and install needed packages
103-
RUN microdnf update && \
104-
microdnf install \
105-
ca-certificates \
106-
cyrus-sasl \
107-
# Needed for the gitsync functionality
108-
git \
109-
libpq \
110-
# Needed for authentication of clients against LDAP servers
111-
openldap \
112-
openldap-clients \
113-
# We need the openssh libs for the gitsync functionality (the clone target could be e.g. git@github.com:org/repo.git)
114-
openssh-clients \
115-
openssl-libs \
116-
openssl-pkcs11 \
117-
# Airflow needs Python
118-
python${PYTHON} \
119-
socat \
120-
unixODBC && \
121-
microdnf clean all && \
122-
rm -rf /var/cache/yum && \
123-
bash <<EOF
103+
RUN <<EOF
104+
microdnf update
105+
106+
# git: Needed for the gitsync functionality
107+
# openldap: Needed for authentication of clients against LDAP servers
108+
# openssh-clients: We need the openssh libs for the gitsync functionality (the clone target could be e.g. git@github.com:org/repo.git)
109+
# python: Airflow needs Python
110+
microdnf install \
111+
ca-certificates \
112+
cyrus-sasl \
113+
git \
114+
libpq \
115+
openldap \
116+
openldap-clients \
117+
openssh-clients \
118+
openssl-libs \
119+
openssl-pkcs11 \
120+
python${PYTHON} \
121+
socat \
122+
unixODBC
123+
microdnf clean all
124+
rm -rf /var/cache/yum
125+
124126
# Get the correct `tini` binary for our architecture.
125127
# It is used as an init alternative in the entrypoint
126128
curl -o /usr/bin/tini "https://repo.stackable.tech/repository/packages/tini/tini-${TINI}-${TARGETARCH}"

0 commit comments

Comments
 (0)