Fix testILMDownsampleRollingRestart() test #98889
Conversation
Check that source index exists prior to updating index.lifecycle.name index setting. Relates elastic#98764
martijnvg
added
>test
|
Pinging @elastic/es-analytics-geo (Team:Analytics) |
elasticsearchmachine
added
the
Team:Analytics
| // we need to attempt it in a assertBusy(...) | ||
| assertBusy(() -> { | ||
| try { | ||
| if (indexExists(sourceIndex) == false) { |
There was a problem hiding this comment.
If I understand correctly this might happen in case there is a background downsample task running that failed for some reason, has been retried, and before we get here, the task completes right?
There was a problem hiding this comment.
yes. For example the request to update index settings fails to return because the node got shutdown, but the index settings were updated. The downsampling can be kicked off. When retrying the source index could not longer exist.
|
@elasticmachine update branch |
martijnvg
added
the
auto-merge-without-approval
|
@elasticmachine run elasticsearch-ci/packaging-tests-unix-sample |
|
@elasticmachine run elasticsearch-ci/packaging-tests-windows-sample |
|
@elasticmachine update branch |
|
@elasticmachine update branch |
|
@elasticmachine run elasticsearch-ci/packaging-tests-unix-sample |
|
@elasticmachine update branch |
|
@elasticmachine run elasticsearch-ci/packaging-tests-unix-sample |
martijnvg
deleted the
downsampel/fix_test/testILMDownsampleRollingRestart
branch
Check that source index exists prior to updating index.lifecycle.name index setting. Relates elastic#98764
Check that source index exists prior to updating index.lifecycle.name index setting.
Relates #98764