Skip to content
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

ducktape: Wait for the metric tracking index uploads to update #11210

Merged

Conversation

abhijat
Copy link
Contributor

@abhijat abhijat commented Jun 5, 2023

The E2E test for SI checks that indices are uploaded during a normal write operation.

This change uses wait_until with a timeout for the assertion that indices are uploaded, replacing a single check used earlier.

Fixes: #11166

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.1.x
  • v22.3.x
  • v22.2.x

Release Notes

  • none

The E2E test for SI checks that indices are uploaded during a normal
write operation. This change gives the assertion that indices are
uploaded a short duration instead of a single check used earlier.
@abhijat
Copy link
Contributor Author

abhijat commented Jun 5, 2023

/ci-repeat 10
skip-units
dt-repeat=50
tests/rptest/tests/e2e_shadow_indexing_test.py::EndToEndShadowIndexingTest.test_write

@abhijat abhijat changed the title ducktape: Wait for indices upload metrics update ducktape: Wait for the metric tracking index uploads to update Jun 5, 2023
@abhijat
Copy link
Contributor Author

abhijat commented Jun 5, 2023

/ci-repeat 10
skip-units
dt-repeat=100
tests/rptest/tests/e2e_shadow_indexing_test.py::EndToEndShadowIndexingTest.test_write

@piyushredpanda
Copy link
Contributor

This is failing a lot -- hopefully we can get to it soon.

@abhijat abhijat marked this pull request as ready for review June 7, 2023 04:52
@abhijat abhijat requested review from andijcr and VladLazar June 7, 2023 04:52
@abhijat
Copy link
Contributor Author

abhijat commented Jun 7, 2023

Both CI failures are during setting up the env, before ducktape tests were run

🚨 Error: Error setting up bootstrap: The global environment hook exited with status 1
--
  |  

@piyushredpanda piyushredpanda merged commit 0c5a306 into redpanda-data:dev Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI Failure (failed to upload enough indices) in EndToEndShadowIndexingTest.test_write
3 participants