Skip to content

Commit

Permalink
Merge #91962
Browse files Browse the repository at this point in the history
91962: ci: fix regression in stress_impl r=srosenberg a=srosenberg

Fixes a regression introduced in [1] which broke nightly Stress tests. bazci is _sensitive_ to the order of command-line arguments. Specifically, --config cannot be in the first position.

[1] 7122bac

Release note: None
Epic: None

Co-authored-by: Stan Rosenberg <[email protected]>
  • Loading branch information
craig[bot] and srosenberg committed Nov 16, 2022
2 parents 563ea39 + df7ca88 commit e3a8850
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion build/teamcity/cockroach/nightlies/stress_impl.sh
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ do
continue
fi
exit_status=0
$BAZEL_BIN/pkg/cmd/bazci/bazci_/bazci -- --config=ci test "$test" \
$BAZEL_BIN/pkg/cmd/bazci/bazci_/bazci -- test --config=ci "$test" \
--test_env=COCKROACH_NIGHTLY_STRESS=true \
--test_timeout="$TESTTIMEOUTSECS" \
--run_under "@com_github_cockroachdb_stress//:stress -bazel -shardable-artifacts 'XML_OUTPUT_FILE=$BAZEL_BIN/pkg/cmd/bazci/bazci_/bazci merge-test-xmls' $STRESSFLAGS" \
Expand Down

0 comments on commit e3a8850

Please sign in to comment.