test(version_upgrade): exercise upgrade/downgrade for the latest stable release as well#18436
test(version_upgrade): exercise upgrade/downgrade for the latest stable release as well#18436machine424 wants to merge 1 commit intoprometheus:mainfrom
Conversation
…le release as well Signed-off-by: Ayoub Mrini <[email protected]>
| // Accelerate chunks mmapping. | ||
| "--storage.tsdb.samples-per-chunk=10", | ||
| // Speed up compaction but still allow chunks mmapping between compactions. | ||
| "--storage.tsdb.min-block-duration=65s", |
There was a problem hiding this comment.
we can reduce this even more to speed up tests once I backport c94101d to 3.5
|
fyi @roidelapluie (because you're involved in e2e testing as well) |
|
can/should we run this in main/release only? not on pr's ? |
could you elaborate on the concerns (I'm not up to date with CI quotas e.g.)? It's always better to catch regressions earlier. It's kind of "expensive" and maybe flaky but it's good to exercise it as much as possible to improve on that. for now it's not "required" so it'll not block PRs. I have #18437 that would speed up the LTS one (we'd expect each test to take < 150s after that; just like running multiple other e2e tests in ./cmd/prometheus), then if downloading the binaries becomes a problem, we can cache them... |
Which issue(s) does the PR fix:
Release notes for end users (ALL commits must be considered).
Reviewers should verify clarity and quality.