From abf056df709c1b63ee93dfc61e0afb86627b0e87 Mon Sep 17 00:00:00 2001 From: DeeDeeG Date: Sat, 5 Sep 2020 15:02:57 -0400 Subject: [PATCH] CI: Patch to really skip uploading artifacts to S3 On the "release-branch-build.yml" pipeline, in the "IsSignedZipBranch" condition, we were not passing the re-mapped env variables. This patch fixes the `env:` block to send the re-mapped env vars, as was intended. This fixes an oversight in the previous PR. --- script/vsts/release-branch-build.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/script/vsts/release-branch-build.yml b/script/vsts/release-branch-build.yml index e9dbeac3992..b2f700e31c8 100644 --- a/script/vsts/release-branch-build.yml +++ b/script/vsts/release-branch-build.yml @@ -65,8 +65,8 @@ jobs: node $(Build.SourcesDirectory)/script/vsts/upload-artifacts.js --assets-path "$(System.ArtifactsDirectory)" --s3-path "vsts-artifacts/$(Build.BuildId)/" env: ATOM_RELEASE_VERSION: $(ReleaseVersion) - ATOM_RELEASES_S3_KEY: $(ATOM_RELEASES_S3_KEY) - ATOM_RELEASES_S3_SECRET: $(ATOM_RELEASES_S3_SECRET) - ATOM_RELEASES_S3_BUCKET: $(ATOM_RELEASES_S3_BUCKET) + ATOM_RELEASES_S3_KEY: $(_ATOM_RELEASES_S3_KEY) + ATOM_RELEASES_S3_SECRET: $(_ATOM_RELEASES_S3_SECRET) + ATOM_RELEASES_S3_BUCKET: $(_ATOM_RELEASES_S3_BUCKET) displayName: Upload CI Artifacts to S3 condition: and(succeeded(), eq(variables['IsSignedZipBranch'], 'true'))