CI: speed up the docker_check stage
authorBenjamin Tissoires <benjamin.tissoires@gmail.com>
Mon, 18 Jun 2018 07:34:10 +0000 (09:34 +0200)
committerBenjamin Tissoires <benjamin.tissoires@gmail.com>
Tue, 19 Jun 2018 08:41:43 +0000 (10:41 +0200)
There is no point in login in to the registry if there is no need to
create a new docker image.

Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
.gitlab-ci.yml

index 189461d..9860ad6 100644 (file)
@@ -191,12 +191,12 @@ ubuntu:18.04@docker-check:
   stage: docker_prep
   services:
     - docker:dind
-  before_script:
-    - docker login -u $CI_REGISTRY_USER -p $CI_REGISTRY_PASSWORD $CI_REGISTRY
   script:
     # if the check was successful, we just skip recreating the docker image
     - test -e .img_ready && exit 0
 
+    - docker login -u $CI_REGISTRY_USER -p $CI_REGISTRY_PASSWORD $CI_REGISTRY
+
     # create a Dockerfile with our dependencies
     - echo "FROM fedora:$FEDORA_VERSION" > Dockerfile
     - echo "WORKDIR /app" >> Dockerfile
@@ -233,12 +233,12 @@ fedora:27@docker-prep:
   stage: docker_prep
   services:
     - docker:dind
-  before_script:
-    - docker login -u $CI_REGISTRY_USER -p $CI_REGISTRY_PASSWORD $CI_REGISTRY
   script:
     # if the check was successful, we just skip recreating the docker image
     - test -e .img_ready && exit 0
 
+    - docker login -u $CI_REGISTRY_USER -p $CI_REGISTRY_PASSWORD $CI_REGISTRY
+
     # create a Dockerfile with our dependencies
     - echo "FROM ubuntu:$UBUNTU_VERSION" > Dockerfile
     - echo "WORKDIR /app" >> Dockerfile