-
Notifications
You must be signed in to change notification settings - Fork 552
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
rptest: ignore test_cloud_cache_thrash #15199
rptest: ignore test_cloud_cache_thrash #15199
Conversation
ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/41953#018c1bd7-f29a-4afc-aeec-5bb41a9adc33 |
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.
from discussion with @nvartolomei, if this PR will bring in changes to knowingly fail the test for the purpose of diagnosis, would be best to add an @ok_to_fail
. or may need to update the test or delete the test if a similar test is already run in a non-cloud env.
982fcff
to
2f14d09
Compare
2f14d09
to
a001e40
Compare
From an actual run: |
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.
lgtm
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.
LGTM
Ignore test_cloud_cache_thrash to avoid wasting resources on a test which does nothing. See code comment for rationale.
Backports Required
Release Notes