villebro commented on code in PR #30000:
URL: https://github.com/apache/superset/pull/30000#discussion_r1729490034
##########
Dockerfile:
##########
@@ -140,25 +140,15 @@ RUN apt-get update -qq \
libxtst6 \
git \
pkg-config \
+ wget \
+ bzip2 \
&& rm -rf /var/lib/apt/lists/*
+# Instlalling headless browsers
RUN --mount=type=cache,target=/root/.cache/pip \
pip install playwright
-RUN playwright install-deps
-RUN playwright install chromium
-
-# Install GeckoDriver WebDriver
-ARG GECKODRIVER_VERSION=v0.34.0 \
- FIREFOX_VERSION=125.0.3
-
-RUN apt-get update -qq \
- && apt-get install -yqq --no-install-recommends wget bzip2 \
- && wget -q
https://github.com/mozilla/geckodriver/releases/download/${GECKODRIVER_VERSION}/geckodriver-${GECKODRIVER_VERSION}-linux64.tar.gz
-O - | tar xfz - -C /usr/local/bin \
- # Install Firefox
- && wget -q
https://download-installer.cdn.mozilla.net/pub/firefox/releases/${FIREFOX_VERSION}/linux-x86_64/en-US/firefox-${FIREFOX_VERSION}.tar.bz2
-O - | tar xfj - -C /opt \
- && ln -s /opt/firefox/firefox /usr/local/bin/firefox \
- && apt-get autoremove -yqq --purge wget bzip2 && rm -rf /var/[log,tmp]/*
/tmp/* /var/lib/apt/lists/*
-# Cache everything for dev purposes...
+RUN playwright install chromium --with-deps
+RUN playwright install firefox --with-deps
Review Comment:
I'm curious, does with work with multiarch builds? I'm assuming the chromium
and firefox binaries are different on `x86_64` and `arm64`. It seems previously
we were only installing the `x86_64` versions.
##########
Dockerfile:
##########
@@ -140,25 +140,15 @@ RUN apt-get update -qq \
libxtst6 \
git \
pkg-config \
+ wget \
+ bzip2 \
&& rm -rf /var/lib/apt/lists/*
+# Instlalling headless browsers
Review Comment:
```suggestion
# Installing headless browsers
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
