Skip to content

Commit 96b269c

Browse files
fix: Default CI_USE_BUILD_INSTANCE_KEY to 1 in bootstrap_ec2
Per Adam's feedback, default the env var at the top of bootstrap_ec2 rather than in ci3.yml. This is consistent with how CI=1 is set there. External CI (ci3-external.yml) explicitly sets it to "0" to disable build instance key features for external contributors.
1 parent 248ebc2 commit 96b269c

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.github/workflows/ci3.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,6 @@ jobs:
8585
PR_COMMITS: ${{ github.event.pull_request.commits }}
8686
PR_NUMBER: ${{ github.event.pull_request.number }}
8787
GITHUB_REF_NAME: ${{ github.ref_name }}
88-
# Enable build instance key features (disk logging, benchmark uploads, etc.)
89-
CI_USE_BUILD_INSTANCE_KEY: "1"
9088
# NOTE: $CI_MODE is set in the Determine CI Mode step.
9189
run: ./.github/ci3.sh $CI_MODE
9290

ci3/bootstrap_ec2

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ NO_TERMINATE=${NO_TERMINATE:-0}
88

99
# We're always "in CI" if we're running on a remote instance.
1010
export CI=1
11+
# Enable build instance key features (disk logging, benchmark uploads) by default.
12+
# Set to 0 in ci3-external.yml for external contributors who don't have SSH access.
13+
export CI_USE_BUILD_INSTANCE_KEY=${CI_USE_BUILD_INSTANCE_KEY:-1}
1114

1215
if [ "$arch" == "arm64" ]; then
1316
cores=64

0 commit comments

Comments
 (0)