Unverified Commit 29fd0469 authored by godofredoc's avatar godofredoc Committed by GitHub

Pin dependencies in docker file. (#97466)

* Pin dependencies in docker file.

The docker file had two dependencies that were not locked. An npm
dependency and the debian base image. This PR pins those two
dependencies.

* Use the slim version of debian image.

* Use image with bigger disk to build docker images.

* remove web_engine_tests from cirrus.
parent fd5120bb
...@@ -18,7 +18,7 @@ gcp_credentials: ENCRYPTED[!cc769765170bebc37e0556e2da5915ca64ee37f4ec8c966ec147 ...@@ -18,7 +18,7 @@ gcp_credentials: ENCRYPTED[!cc769765170bebc37e0556e2da5915ca64ee37f4ec8c966ec147
task: task:
gke_container: gke_container:
dockerfile: "dev/ci/docker_linux/Dockerfile" dockerfile: "dev/ci/docker_linux/Dockerfile"
builder_image_name: docker-builder # gce vm image builder_image_name: docker-builder-linux # gce vm image
builder_image_project: flutter-cirrus builder_image_project: flutter-cirrus
cluster_name: test-cluster cluster_name: test-cluster
zone: us-central1-a zone: us-central1-a
...@@ -156,21 +156,3 @@ task: ...@@ -156,21 +156,3 @@ task:
- git fetch origin master:master - git fetch origin master:master
# The actual logic is in a shell script so that it can be shared between CIs. # The actual logic is in a shell script so that it can be shared between CIs.
- (cd dev/customer_testing/; ./ci.sh) - (cd dev/customer_testing/; ./ci.sh)
- name: web_smoke_test
only_if: "changesInclude('.cirrus.yml', 'examples/hello_world/**' ,'dev/**', 'packages/flutter/**', 'packages/flutter_test/**', 'packages/flutter_tools/lib/src/test/**', 'packages/flutter_web_plugins/**', 'bin/**') && $CIRRUS_PR != ''"
environment:
# Tests Hello World with Flutter Web Engine using Flutter Driver. Should not need more resources.
CPU: 2
MEMORY: 2G
CHROME_NO_SANDBOX: true
script:
- flutter config --enable-web
- git clone https://github.com/flutter/web_installers.git
- cd web_installers/packages/web_drivers/
- dart __deprecated_pub get
- dart lib/web_driver_installer.dart chromedriver --install-only
- chromedriver/chromedriver --port=4444 &
- sleep 1
- cd ../../../examples/hello_world/
- flutter drive --target=test_driver/smoke_web_engine.dart -d web-server --profile --browser-name=chrome
...@@ -12,7 +12,7 @@ ...@@ -12,7 +12,7 @@
# Last manual update 2021-09-24 (changing this comment will re-build image) # Last manual update 2021-09-24 (changing this comment will re-build image)
FROM debian:bullseye FROM debian@sha256:4c25ffa6ef572cf0d57da8c634769a08ae94529f7de5be5587ec8ce7b9b50f9c
MAINTAINER Flutter Developers <flutter-dev@googlegroups.com> MAINTAINER Flutter Developers <flutter-dev@googlegroups.com>
RUN apt-get update -y && \ RUN apt-get update -y && \
...@@ -101,7 +101,7 @@ ENV PATH="/usr/bin:${PATH}" ...@@ -101,7 +101,7 @@ ENV PATH="/usr/bin:${PATH}"
RUN dpkg-query -L nodejs RUN dpkg-query -L nodejs
# Install Firebase # Install Firebase
# This is why we need nodejs installed. # This is why we need nodejs installed.
RUN /usr/bin/npm --verbose install -g firebase-tools RUN /usr/bin/npm --verbose install -g firebase-tools@10.1.2
# Install golang more recent than in repo (necessary for dashing to build) # Install golang more recent than in repo (necessary for dashing to build)
RUN curl 'https://dl.google.com/go/go1.13.3.linux-amd64.tar.gz' | tar xvz RUN curl 'https://dl.google.com/go/go1.13.3.linux-amd64.tar.gz' | tar xvz
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment