Skip to content

Commit ac6efd8

Browse files
committed
fix(NODE-7512): set CLIENT_ENCRYPTION expansion for Windows node-latest variant
The node-latest smoke test build variant was missing CLIENT_ENCRYPTION from its expansions when clientEncryption is false (as it is on Windows per NODE-3401). run-tests.sh requires the var to be exactly "true" or "false" and exits 1 otherwise, causing all Windows node-latest tasks to fail before any tests run. Align with the regular variant loop by always emitting String(!!clientEncryption) instead of conditionally setting it only when truthy.
1 parent bbf75ca commit ac6efd8

2 files changed

Lines changed: 9 additions & 5 deletions

File tree

.evergreen/config.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3687,7 +3687,8 @@ buildvariants:
36873687
run_on: rhel80-large
36883688
expansions:
36893689
NODE_LTS_VERSION: latest
3690-
CLIENT_ENCRYPTION: true
3690+
CLIENT_ENCRYPTION: 'true'
3691+
TEST_CSFLE: 'true'
36913692
tasks:
36923693
- test-latest-server
36933694
- test-latest-replica_set
@@ -3873,6 +3874,8 @@ buildvariants:
38733874
run_on: windows-2022-latest-large
38743875
expansions:
38753876
NODE_LTS_VERSION: latest
3877+
CLIENT_ENCRYPTION: 'false'
3878+
TEST_CSFLE: 'false'
38763879
tasks:
38773880
- test-latest-server
38783881
- test-latest-replica_set

.evergreen/generate_evergreen_tasks.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -424,12 +424,13 @@ for (const {
424424
name: `${osName}-node-latest`,
425425
display_name: `${osDisplayName} Node Latest`,
426426
run_on,
427-
expansions: { NODE_LTS_VERSION: 'latest' },
427+
expansions: {
428+
NODE_LTS_VERSION: 'latest',
429+
CLIENT_ENCRYPTION: String(!!clientEncryption),
430+
TEST_CSFLE: String(!!clientEncryption)
431+
},
428432
tasks: tasks.map(({ name }) => name)
429433
};
430-
if (clientEncryption) {
431-
buildVariantData.expansions.CLIENT_ENCRYPTION = true;
432-
}
433434
BUILD_VARIANTS.push(buildVariantData);
434435
}
435436
}

0 commit comments

Comments
 (0)