Skip to content

Commit cbfb77a

Browse files
authored
fix(ci): fix release_publish(_dry_run) paramters (#1828)
cbaeef2 recently refactored the evergreen configuration file. This subtly introduced some breakage, because the `env` and `silent` options are not command options, but rather need to be nested under the `params` key. I've intentionally left the `silent:` option disabled as this would hide genuinely valuable debug output otherwise. This might be a good point to reconsider fixing the evergreen validation warnings that are displayed in CI; `evergreen validate .evergreen.yml` did catch these issues, but since there are a large number of existing warnings (around dependencies between tasks), these specific warnings were drowned out/not visible in the evergreen UI.
1 parent 4d6df93 commit cbfb77a

File tree

2 files changed

+20
-20
lines changed

2 files changed

+20
-20
lines changed

.evergreen.yml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6561,7 +6561,7 @@ functions:
65616561

65626562
write_preload_script:
65636563
- command: shell.exec
6564-
silent: true
6564+
# silent: true
65656565
type: setup
65666566
params:
65676567
working_dir: src
@@ -6590,7 +6590,7 @@ functions:
65906590
wait: true
65916591
run_pkg_tests_through_ssh:
65926592
- command: shell.exec
6593-
silent: true
6593+
# silent: true
65946594
type: setup
65956595
params:
65966596
working_dir: src
@@ -6844,13 +6844,13 @@ functions:
68446844
file: tmp/expansions.yaml
68456845
redacted: true
68466846
- command: shell.exec
6847-
silent: true
6848-
env:
6849-
devtoolsbot_npm_token: ${devtoolsbot_npm_token}
6850-
node_js_version: ${node_js_version}
6847+
# silent: true
68516848
params:
68526849
working_dir: src
68536850
shell: bash
6851+
env:
6852+
devtoolsbot_npm_token: ${devtoolsbot_npm_token}
6853+
node_js_version: ${node_js_version}
68546854
script: |
68556855
set -e
68566856
.evergreen/run-evergreen-release.sh -- --dry-run
@@ -6862,13 +6862,13 @@ functions:
68626862
file: tmp/expansions.yaml
68636863
redacted: true
68646864
- command: shell.exec
6865-
silent: true
6866-
env:
6867-
devtoolsbot_npm_token: ${devtoolsbot_npm_token}
6868-
node_js_version: ${node_js_version}
6865+
# silent: true
68696866
params:
68706867
working_dir: src
68716868
shell: bash
6869+
env:
6870+
devtoolsbot_npm_token: ${devtoolsbot_npm_token}
6871+
node_js_version: ${node_js_version}
68726872
script: |
68736873
set -e
68746874
.evergreen/run-evergreen-release.sh

.evergreen/evergreen.yml.in

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -513,7 +513,7 @@ functions:
513513

514514
write_preload_script:
515515
- command: shell.exec
516-
silent: true
516+
# silent: true
517517
type: setup
518518
params:
519519
working_dir: src
@@ -542,7 +542,7 @@ functions:
542542
wait: true
543543
run_pkg_tests_through_ssh:
544544
- command: shell.exec
545-
silent: true
545+
# silent: true
546546
type: setup
547547
params:
548548
working_dir: src
@@ -796,13 +796,13 @@ functions:
796796
file: tmp/expansions.yaml
797797
redacted: true
798798
- command: shell.exec
799-
silent: true
800-
env:
801-
devtoolsbot_npm_token: ${devtoolsbot_npm_token}
802-
node_js_version: ${node_js_version}
799+
# silent: true
803800
params:
804801
working_dir: src
805802
shell: bash
803+
env:
804+
devtoolsbot_npm_token: ${devtoolsbot_npm_token}
805+
node_js_version: ${node_js_version}
806806
script: |
807807
set -e
808808
.evergreen/run-evergreen-release.sh -- --dry-run
@@ -814,13 +814,13 @@ functions:
814814
file: tmp/expansions.yaml
815815
redacted: true
816816
- command: shell.exec
817-
silent: true
818-
env:
819-
devtoolsbot_npm_token: ${devtoolsbot_npm_token}
820-
node_js_version: ${node_js_version}
817+
# silent: true
821818
params:
822819
working_dir: src
823820
shell: bash
821+
env:
822+
devtoolsbot_npm_token: ${devtoolsbot_npm_token}
823+
node_js_version: ${node_js_version}
824824
script: |
825825
set -e
826826
.evergreen/run-evergreen-release.sh

0 commit comments

Comments
 (0)