Skip to content

Commit 21bdea6

Browse files
committed
fix intendation
1 parent 4e45c47 commit 21bdea6

File tree

1 file changed

+44
-44
lines changed

1 file changed

+44
-44
lines changed

superset/Dockerfile

Lines changed: 44 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -46,25 +46,25 @@ COPY --from=opa-authorizer-builder /tmp/opa-authorizer/dist/opa_authorizer-0.1.0
4646

4747
RUN microdnf update \
4848
&& microdnf install \
49-
cyrus-sasl-devel \
50-
# Needed by ./configure to work out SQLite compilation flags, see snippet [1] at the end of file
51-
diffutils \
52-
# According to https://stackoverflow.com/q/19530974 normally sqlite3 should be shipped with the Python
53-
# distribution. However, while addig ARM support we noticed that this does not seem to be the case for the
54-
# Python installation shipped in the ARM image variant. So I guess Make is used to find out the sqlite
55-
# compilation flags (and propably to not build sqlite from source(?)), see snippet [1] at the end of file
56-
make \
57-
gcc \
58-
gcc-c++ \
59-
libffi-devel \
60-
openldap-devel \
61-
openssl-devel \
62-
patch \
63-
python${PYTHON} \
64-
python${PYTHON}-devel \
65-
python${PYTHON}-pip \
66-
python${PYTHON}-wheel \
67-
libpq-devel \
49+
cyrus-sasl-devel \
50+
# Needed by ./configure to work out SQLite compilation flags, see snippet [1] at the end of file
51+
diffutils \
52+
# According to https://stackoverflow.com/q/19530974 normally sqlite3 should be shipped with the Python
53+
# distribution. However, while addig ARM support we noticed that this does not seem to be the case for the
54+
# Python installation shipped in the ARM image variant. So I guess Make is used to find out the sqlite
55+
# compilation flags (and propably to not build sqlite from source(?)), see snippet [1] at the end of file
56+
make \
57+
gcc \
58+
gcc-c++ \
59+
libffi-devel \
60+
openldap-devel \
61+
openssl-devel \
62+
patch \
63+
python${PYTHON} \
64+
python${PYTHON}-devel \
65+
python${PYTHON}-pip \
66+
python${PYTHON}-wheel \
67+
libpq-devel \
6868
&& microdnf clean all && \
6969
rm -rf /var/cache/yum
7070

@@ -74,31 +74,31 @@ RUN microdnf update \
7474
RUN python3 -m venv /stackable/app \
7575
&& source /stackable/app/bin/activate \
7676
&& pip install \
77-
--no-cache-dir \
78-
--upgrade \
79-
setuptools==75.2.0 \
80-
pip \
81-
&& pip install \
82-
--no-cache-dir \
83-
--upgrade \
84-
--constraint /tmp/constraints.txt \
85-
apache-superset==${PRODUCT} \
86-
gevent \
87-
psycopg2-binary \
88-
statsd \
89-
pydruid \
90-
python-ldap \
91-
'trino[sqlalchemy]' \
92-
# Add optional dependencies for use in custom Superset configurations.
93-
# Since https://github.com/stackabletech/superset-operator/pull/530
94-
# admins can add custom configuration to superset_conf.py.
95-
Flask_OIDC==2.2.0 \
96-
Flask-OpenID==1.3.1 \
97-
# Redhat has removed `tzdata` from the ubi-minimal images: see https://bugzilla.redhat.com/show_bug.cgi?id=2223028.
98-
# Superset relies on ZoneInfo (https://docs.python.org/3/library/zoneinfo.html#data-sources) to resolve time zones, and this is done
99-
# by searching first under `TZPATH` (which is empty due to the point above) or for the tzdata python package.
100-
# That package is therefore added here (airflow has tzdata in its list of dependencies, but superset does not).
101-
tzdata \
77+
--no-cache-dir \
78+
--upgrade \
79+
setuptools==75.2.0 \
80+
pip \
81+
&& pip install \
82+
--no-cache-dir \
83+
--upgrade \
84+
--constraint /tmp/constraints.txt \
85+
apache-superset==${PRODUCT} \
86+
gevent \
87+
psycopg2-binary \
88+
statsd \
89+
pydruid \
90+
python-ldap \
91+
'trino[sqlalchemy]' \
92+
# Add optional dependencies for use in custom Superset configurations.
93+
# Since https://github.com/stackabletech/superset-operator/pull/530
94+
# admins can add custom configuration to superset_conf.py.
95+
Flask_OIDC==2.2.0 \
96+
Flask-OpenID==1.3.1 \
97+
# Redhat has removed `tzdata` from the ubi-minimal images: see https://bugzilla.redhat.com/show_bug.cgi?id=2223028.
98+
# Superset relies on ZoneInfo (https://docs.python.org/3/library/zoneinfo.html#data-sources) to resolve time zones, and this is done
99+
# by searching first under `TZPATH` (which is empty due to the point above) or for the tzdata python package.
100+
# That package is therefore added here (airflow has tzdata in its list of dependencies, but superset does not).
101+
tzdata \
102102
# We bumped this from 21.2.0 to 22.0.0 to fix CVE-2024-1135
103103
# Superset 4.1.0 will contain at least 22.0.0, the bump was done in https://github.com/apache/superset/commit/4f693c6db0dc5c7286a36b8d23e90541943ff13f
104104
# We only want to bump this for the 4.0.x line, as the others already have updated and we don't want to accidentially downgrade the version

0 commit comments

Comments
 (0)