cockroachdb / cockroach

CockroachDB — the cloud native, distributed SQL database designed for high availability, effortless scale, and control over data placement.
https://www.cockroachlabs.com
Other
30.15k stars 3.81k forks source link

kv: reconsider performance asserting roachtests #106344

Open kvoli opened 1 year ago

kvoli commented 1 year ago

Is your feature request related to a problem? Please describe. There are roachtests which assert that performance is no less than X, where X is usually a performance number the test gathers before changing the cluster (killing a node, restarting etc).

These tests are sensitive to flakes unrelated to the desired functionality being tested. These tests also reduce a higher fidelity performance number into a binary pass/fail -- losing information.

Describe the solution you'd like Export performance of a run into roachperf and remove any failure conditions on performance.

These tests would be good candidates to start with:

Describe alternatives you've considered

Performance tracking requires someone(or automated) to periodically check for regressions or improvements and flag them.

When a test fails, the artifacts and debug zip are collected. This won't happen automatically any longer, although we could change this.

We could instead keep the assertion and still collect metrics for viewing performance over time.

Jira issue: CRDB-29517

blathers-crl[bot] commented 1 year ago

cc @cockroachdb/replication