Details
Description
We could add the following command to our s_all script to make sure we keep our evergreen file correct:
diff --git a/dist/s_all b/dist/s_all
|
index 50d1909ee..a4b7ef4c7 100755
|
--- a/dist/s_all
|
+++ b/dist/s_all
|
@@ -81,6 +81,7 @@ run "./s_clang-format"
|
run "python prototypes.py"
|
run "sh ./s_typedef -b"
|
run "python test_tag.py"
|
+run "evergreen validate ../test/evergreen.yml"
|
As of today, it generates the following message:
####################### MESSAGE ############################
|
s_all run of: "evergreen validate ../test/evergreen.yml" resulted in:
|
WARNING: yaml: unmarshal errors:
|
line 2868: mapping key "test_env_vars" already defined at line 2863
|
../test/evergreen.yml is valid with warnings
|
#######################
|
Looking at the corresponding line, test_env_vars is effectively declared twice:
- name: cppsuite-stress-tests
|
display_name: "Cppsuite Stress Tests"
|
batchtime: 480 # 3 times a day
|
run_on:
|
- ubuntu1804-test
|
expansions:
|
test_env_vars: LD_LIBRARY_PATH=$(pwd)/../../.libs
|
make_command: PATH=/opt/mongodbtoolchain/v3/bin:$PATH make
|
posix_configure_flags:
|
--enable-silent-rules --enable-python --enable-zlib --enable-snappy
|
--enable-strict --enable-static
|
test_env_vars:
|
PATH=/opt/mongodbtoolchain/v3/bin:$PATH
|
tasks:
|
- name: compile
|
- name: cppsuite-hs-cleanup-stress
|
- name: cppsuite-base-test-stress
|
I suggest the following scope:
- Add the new evergreen command to s_all or anywhere that would work
- EDIT: Add a check to make sure the command is available
- Fix any warnings raised by the new command when this ticket is addressed