fix: dry run on grind (#12088)

Prevents the docker push from failing in grind jobs where DOCKERHUB_PASSWORD isnt set
AztecProtocol · Feb 18, 2025 · 3b8f876 · 3b8f876 · AztecBot · Feb 18, 2025
1 parent 8471b54
commit 3b8f876
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/.github/workflows/ci3.yml b/.github/workflows/ci3.yml
@@ -185,6 +185,7 @@ jobs:
           ARCH: amd64
           LOG_ID: ${{ github.run_id }}
           INSTANCE_POSTFIX: ${{ matrix.number }}
+          DRY_RUN: 1
           NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
         run: |
           ./ci.sh ec2-test

diff --git a/ci3/bootstrap_ec2 b/ci3/bootstrap_ec2
@@ -99,12 +99,13 @@ container_script=$(cat <<EOF
   git checkout FETCH_HEAD
   git checkout -b $REF_NAME
   export CI_FULL=$CI_FULL
+  export DRY_RUN=${DRY_RUN:-0}
   source ci3/source
 
   if [ -n "${DOCKERHUB_PASSWORD:-}" ] && ([ "$REF_NAME" == "master" ] || semver check "$REF_NAME"); then
     echo ${DOCKERHUB_PASSWORD:-} | docker login -u aztecprotocolci --password-stdin
   fi
-  echo "env: REF_NAME=$REF_NAME COMMIT_HASH=$COMMIT_HASH CURRENT_VERSION=$CURRENT_VERSION CI_FULL=$CI_FULL"
+  echo "env: REF_NAME=$REF_NAME COMMIT_HASH=$COMMIT_HASH CURRENT_VERSION=$CURRENT_VERSION CI_FULL=$CI_FULL DRY_RUN=${DRY_RUN:-0}"
   if semver check "$REF_NAME"; then
     echo "Performing a release because $REF_NAME is a semver."
   fi