kgabryje commented on code in PR #30000:
URL: https://github.com/apache/superset/pull/30000#discussion_r1778155441
##########
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 think cypress installs its own chromium binary. A proper way to test it
would be to run a report
--
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]
