-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sql: increase tenant testing coverage #141604
sql: increase tenant testing coverage #141604
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the pkg/sql changes lgtm, but some of these tests are owned by the SQL Queries and DR teams, so they should take a look.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 15 of 15 files at r1, 16 of 16 files at r2, all commit messages.
Reviewable status:complete! 1 of 0 LGTMs obtained (waiting on @DarrylWong, @golgeek, @kev-cao, @kyle-a-wong, @mgartner, and @wenyihu6)
(Stacked on cockroachdb#141490.) This PR continues the work from cockroachdb#140447, replacing occurrences of `createTestServerParams` with `createTestServerParamsAllowTenants` to enable tenant testing in these tests. Informs: cockroachdb#140446 Epic: CRDB-38970 Release note: None
bf68b79
to
6f2435c
Compare
⚪ Sysbench [SQL, 3node, oltp_read_write]
Reproducebenchdiff binaries: mkdir -p benchdiff/6f2435c/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/6f2435c712e4ca5c7b72293bfb605a1f70bb998d/bin/pkg_sql_tests benchdiff/6f2435c/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/6f2435c/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
mkdir -p benchdiff/fbcabe0/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/fbcabe029eb2e1701710d08cebbca6d15089f911/bin/pkg_sql_tests benchdiff/fbcabe0/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/fbcabe0/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests benchdiff command: benchdiff --run=^BenchmarkSysbench/SQL/3node/oltp_read_write$ --old=fbcabe0 --new=6f2435c ./pkg/sql/tests ⚪ Sysbench [KV, 1node, local, oltp_read_only]
Reproducebenchdiff binaries: mkdir -p benchdiff/6f2435c/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/6f2435c712e4ca5c7b72293bfb605a1f70bb998d/bin/pkg_sql_tests benchdiff/6f2435c/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/6f2435c/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
mkdir -p benchdiff/fbcabe0/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/fbcabe029eb2e1701710d08cebbca6d15089f911/bin/pkg_sql_tests benchdiff/fbcabe0/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/fbcabe0/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests benchdiff command: benchdiff --run=^BenchmarkSysbench/KV/1node_local/oltp_read_only$ --old=fbcabe0 --new=6f2435c ./pkg/sql/tests ⚪ Sysbench [KV, 1node, local, oltp_write_only]
Reproducebenchdiff binaries: mkdir -p benchdiff/6f2435c/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/6f2435c712e4ca5c7b72293bfb605a1f70bb998d/bin/pkg_sql_tests benchdiff/6f2435c/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/6f2435c/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
mkdir -p benchdiff/fbcabe0/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/fbcabe029eb2e1701710d08cebbca6d15089f911/bin/pkg_sql_tests benchdiff/fbcabe0/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/fbcabe0/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests benchdiff command: benchdiff --run=^BenchmarkSysbench/KV/1node_local/oltp_write_only$ --old=fbcabe0 --new=6f2435c ./pkg/sql/tests Artifactsdownload: mkdir -p new
gcloud storage cp gs://cockroach-microbench-ci/artifacts/6f2435c712e4ca5c7b72293bfb605a1f70bb998d/13456892060-1/\* new/
mkdir -p old
gcloud storage cp gs://cockroach-microbench-ci/artifacts/fbcabe029eb2e1701710d08cebbca6d15089f911/13456892060-1/\* old/ Legend
No regressions detected! built with commit: 6f2435c712e4ca5c7b72293bfb605a1f70bb998d |
After removing the stacked commit, it seems there are no remaining changes that would affect the DR team. So, we should be good to merge. Thanks for the reviews! bors r =rafiss,yuzefovich |
Oops! bors r=rafiss,yuzefovich |
This PR continues the work from #140447, replacing occurrences of
createTestServerParams
withcreateTestServerParamsAllowTenants
to enable tenant testing in these tests.
Informs: #140446
Epic: CRDB-38970
Release note: None