From fc2d5994d328f8be3c209ba85e71fb3477be0196 Mon Sep 17 00:00:00 2001 From: Ian Tangney Date: Thu, 27 Mar 2025 18:50:21 +0000 Subject: [PATCH 1/2] KASM-6955 Fix registry login conditional --- ci-scripts/gitlab-ci.template | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ci-scripts/gitlab-ci.template b/ci-scripts/gitlab-ci.template index 23e98d9..927b533 100644 --- a/ci-scripts/gitlab-ci.template +++ b/ci-scripts/gitlab-ci.template @@ -20,8 +20,8 @@ variables: MIRROR_ORG_NAME: "{{ MIRROR_ORG_NAME }}" before_script: - docker login --username $DOCKER_HUB_USERNAME --password $DOCKER_HUB_PASSWORD - - if [ -f "$CI_COMMIT_REF_PROTECTED" == "true" ]; then docker login --username $QUAY_USERNAME --password $QUAY_PASSWORD quay.io; fi - - if [ -f "$CI_COMMIT_REF_PROTECTED" == "true" ]; then docker login --username $GHCR_USERNAME --password $GHCR_PASSWORD ghcr.io; fi + - if [ "$CI_COMMIT_REF_PROTECTED" == "true" ]; then docker login --username $QUAY_USERNAME --password $QUAY_PASSWORD quay.io; fi + - if [ "$CI_COMMIT_REF_PROTECTED" == "true" ]; then docker login --username $GHCR_USERNAME --password $GHCR_PASSWORD ghcr.io; fi - export SANITIZED_BRANCH="$(echo $CI_COMMIT_REF_NAME | sed -r 's#^release/##' | sed 's/\//_/g')" - export BASE_TAG="{{ BASE_TAG }}" From 73e8248f017411e9336cbd62aa467731ad4c69da Mon Sep 17 00:00:00 2001 From: Ian Tangney Date: Fri, 28 Mar 2025 14:43:19 +0000 Subject: [PATCH 2/2] KASM-6955 Fix issue with readme pipeline variables not getting passed into sub-pipeline --- .gitlab-ci.yml | 27 ++++++++++++++++ ci-scripts/gitlab-ci.template | 58 +++++++++++++++-------------------- 2 files changed, 51 insertions(+), 34 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 02cbcd6..6eb793e 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -34,12 +34,39 @@ pipeline: stage: run except: variables: + - $README_USERNAME_RUN + - $README_PASSWORD_RUN + - $QUAY_API_KEY_RUN - $DOCKERHUB_REVERT_RUN - $REVERT_IS_ROLLING_RUN trigger: include: - artifact: gitlab-ci.yml job: template +pipeline_readme: + stage: run + only: + variables: + - $README_USERNAME_RUN + - $README_PASSWORD_RUN + variables: + README_USERNAME: $README_USERNAME_RUN + README_PASSWORD: $README_PASSWORD_RUN + trigger: + include: + - artifact: gitlab-ci.yml + job: template +pipeline_readme_quay: + stage: run + only: + variables: + - $QUAY_API_KEY_RUN + variables: + QUAY_API_KEY: $QUAY_API_KEY_RUN + trigger: + include: + - artifact: gitlab-ci.yml + job: template pipeline_revert: stage: run only: diff --git a/ci-scripts/gitlab-ci.template b/ci-scripts/gitlab-ci.template index 927b533..edb0fdb 100644 --- a/ci-scripts/gitlab-ci.template +++ b/ci-scripts/gitlab-ci.template @@ -41,9 +41,9 @@ build_{{ IMAGE.name }}: {% endfor %}{% endif %} except: variables: - - $README_USERNAME_RUN - - $README_PASSWORD_RUN - - $QUAY_API_KEY_RUN + - $README_USERNAME + - $README_PASSWORD + - $QUAY_API_KEY - $DOCKERHUB_REVERT - $REVERT_IS_ROLLING tags: @@ -67,9 +67,9 @@ build_{{ IMAGE.name }}: {% endfor %}{% endif %} except: variables: - - $README_USERNAME_RUN - - $README_PASSWORD_RUN - - $QUAY_API_KEY_RUN + - $README_USERNAME + - $README_PASSWORD + - $QUAY_API_KEY - $DOCKERHUB_REVERT - $REVERT_IS_ROLLING tags: @@ -94,9 +94,9 @@ test_{{ IMAGE.name }}: {% endfor %}{% endif %} except: variables: - - $README_USERNAME_RUN - - $README_PASSWORD_RUN - - $QUAY_API_KEY_RUN + - $README_USERNAME + - $README_PASSWORD + - $QUAY_API_KEY - $DOCKERHUB_REVERT - $REVERT_IS_ROLLING needs: @@ -124,9 +124,9 @@ test_{{ IMAGE.name }}: {% endfor %}{% endif %} except: variables: - - $README_USERNAME_RUN - - $README_PASSWORD_RUN - - $QUAY_API_KEY_RUN + - $README_USERNAME + - $README_PASSWORD + - $QUAY_API_KEY - $DOCKERHUB_REVERT - $REVERT_IS_ROLLING needs: @@ -158,9 +158,9 @@ manifest_{{ IMAGE.name }}: {% endfor %}{% endif %} except: variables: - - $README_USERNAME_RUN - - $README_PASSWORD_RUN - - $QUAY_API_KEY_RUN + - $README_USERNAME + - $README_PASSWORD + - $QUAY_API_KEY - $DOCKERHUB_REVERT - $REVERT_IS_ROLLING needs: @@ -188,9 +188,9 @@ manifest_{{ IMAGE.name }}: {% endfor %}{% endif %} except: variables: - - $README_USERNAME_RUN - - $README_PASSWORD_RUN - - $QUAY_API_KEY_RUN + - $README_USERNAME + - $README_PASSWORD + - $QUAY_API_KEY - $DOCKERHUB_REVERT - $REVERT_IS_ROLLING needs: @@ -213,11 +213,8 @@ update_readmes_{{ IMAGE.name }}: - bash ci-scripts/readme.sh "{{ IMAGE.name }}" only: variables: - - $README_USERNAME_RUN - - $README_PASSWORD_RUN - variables: - README_USERNAME: $README_USERNAME_RUN - README_PASSWORD: $README_PASSWORD_RUN + - $README_USERNAME + - $README_PASSWORD tags: - oci-fixed-amd {% endfor %} @@ -230,11 +227,8 @@ update_readmes_{{ IMAGE.name }}: - bash ci-scripts/readme.sh "{{ IMAGE.name }}" only: variables: - - $README_USERNAME_RUN - - $README_PASSWORD_RUN - variables: - README_USERNAME: $README_USERNAME_RUN - README_PASSWORD: $README_PASSWORD_RUN + - $README_USERNAME + - $README_PASSWORD tags: - oci-fixed-amd {% endfor %} @@ -248,9 +242,7 @@ update_quay_readmes_{{ IMAGE.name }}: - bash ci-scripts/quay_readme.sh "{{ IMAGE.name }}" only: variables: - - $QUAY_API_KEY_RUN - variables: - QUAY_API_KEY: $QUAY_API_KEY_RUN + - $QUAY_API_KEY tags: - oci-fixed-amd {% endfor %} @@ -263,9 +255,7 @@ update_quay_readmes_{{ IMAGE.name }}: - bash ci-scripts/quay_readme.sh "{{ IMAGE.name }}" only: variables: - - $QUAY_API_KEY_RUN - variables: - QUAY_API_KEY: $QUAY_API_KEY_RUN + - $QUAY_API_KEY tags: - oci-fixed-amd {% endfor %}