From 90d3711cbe4d9f05e7484b46d374264e3f27619a Mon Sep 17 00:00:00 2001 From: Stan Rosenberg Date: Mon, 18 Apr 2022 03:03:21 +0000 Subject: [PATCH] ci: fix optimizer_tests_impl.sh Confusingly bazci uses `--artifacts_dir` whereas roachtest uses `--artifacts`. The recent refactoring of optimizer_tests_impl.sh erroneously used the latter instead of the former. Consequently, (sql) Optimizer Tests stopped working in CI. Fixes: https://github.com/cockroachdb/cockroach/pull/77688 Release justification: ensure CI can run nightly (sql) Optimizer Tests Release note: None --- build/teamcity/cockroach/nightlies/optimizer_tests_impl.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build/teamcity/cockroach/nightlies/optimizer_tests_impl.sh b/build/teamcity/cockroach/nightlies/optimizer_tests_impl.sh index de20575bcf75..ac0ccebff6d3 100755 --- a/build/teamcity/cockroach/nightlies/optimizer_tests_impl.sh +++ b/build/teamcity/cockroach/nightlies/optimizer_tests_impl.sh @@ -14,7 +14,7 @@ ARTIFACTS_DIR=/artifacts/fast_int_set_large mkdir $ARTIFACTS_DIR GO_TEST_JSON_OUTPUT_FILE=$ARTIFACTS_DIR/test.json.txt exit_status_large=0 -$BAZEL_BIN/pkg/cmd/bazci/bazci_/bazci --config=ci --artifacts $ARTIFACTS_DIR \ +$BAZEL_BIN/pkg/cmd/bazci/bazci_/bazci --config=ci --artifacts_dir $ARTIFACTS_DIR \ test //pkg/sql/opt:opt_test -- \ --define gotags=bazel,crdb_test,fast_int_set_large \ --test_env=GO_TEST_JSON_OUTPUT_FILE=$GO_TEST_JSON_OUTPUT_FILE || exit_status_large=$?