Skip to content

Commit

Permalink
[TEST] Wait for no pending operations on the index shard (elastic#118244
Browse files Browse the repository at this point in the history
)

This fixes testRetryPointInTime which on teardown is looking to assert
that the operations in the translog and in the lucene index are the
same.

Previously we didn't wait for the translog operations to be applied.
This changes `assertConsistentHistoryInLuceneIndex` to wait for the
pending operations in the translog to be applied.

Fixes elastic#117116
  • Loading branch information
andreidan authored Dec 9, 2024
1 parent 6bb0799 commit b4e852a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
3 changes: 0 additions & 3 deletions muted-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -157,9 +157,6 @@ tests:
- class: org.elasticsearch.xpack.test.rest.XPackRestIT
method: test {p0=snapshot/10_basic/Create a source only snapshot and then restore it}
issue: https://github.com/elastic/elasticsearch/issues/117295
- class: org.elasticsearch.xpack.searchablesnapshots.RetrySearchIntegTests
method: testRetryPointInTime
issue: https://github.com/elastic/elasticsearch/issues/117116
- class: org.elasticsearch.xpack.inference.DefaultEndPointsIT
method: testInferDeploysDefaultElser
issue: https://github.com/elastic/elasticsearch/issues/114913
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ protected Collection<Class<? extends Plugin>> nodePlugins() {

@After
public void assertConsistentHistoryInLuceneIndex() throws Exception {
internalCluster().beforeIndexDeletion();
internalCluster().assertConsistentHistoryBetweenTranslogAndLuceneIndex();
}

Expand Down

0 comments on commit b4e852a

Please sign in to comment.