remove dry_run env flag (#14992)

(cherry picked from commit 46364e9)
aptos-labs · Oct 28, 2024 · 9c922eb · 9c922eb
1 parent 9a44d73
commit 9c922eb
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/.github/workflows/copy-images-to-dockerhub.yaml b/.github/workflows/copy-images-to-dockerhub.yaml
@@ -71,5 +71,4 @@ jobs:
           AWS_ACCOUNT_ID: ${{ secrets.AWS_ECR_ACCOUNT_NUM }}
           GCP_DOCKER_ARTIFACT_REPO: ${{ vars.GCP_DOCKER_ARTIFACT_REPO }}
           IMAGE_TAG_PREFIX: ${{ inputs.image_tag_prefix }}
-          DRY_RUN: ${{ inputs.dry_run }}
         run: ./docker/release-images.mjs --wait-for-image-seconds=3600 ${{ inputs.dry_run && '--dry-run' || '' }}
diff --git a/docker/release-images.mjs b/docker/release-images.mjs
@@ -249,6 +249,7 @@ async function main() {
           const imageTarget = `${targetRegistry}/${image}:${joinTagSegments(parsedArgs.IMAGE_TAG_PREFIX, profilePrefix, featureSuffix)}`;
           console.info(chalk.green(`INFO: copying ${imageSource} to ${imageTarget}`));
           if (parsedArgs.DRY_RUN) {
+            console.info(chalk.yellow(`INFO: skipping copy of ${imageSource} to ${imageTarget} due to dry run`));
             continue;
           }
           await waitForImageToBecomeAvailable(imageSource, parsedArgs.WAIT_FOR_IMAGE_SECONDS);