Skip to content

Commit

Permalink
Fix ReindexNodeShutdownIT#testReindexWithShutdown (elastic#119390)
Browse files Browse the repository at this point in the history
The settings with the maximum re-indexing timeout should be actually applied
to the coordination node.

Resolve elastic#118040
  • Loading branch information
arteam authored Dec 31, 2024
1 parent 8a821f0 commit 3db8101
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -59,10 +59,10 @@ public void testReindexWithShutdown() throws Exception {
final String dataNodeName = internalCluster().startDataOnlyNode();

/* Maximum time to wait for reindexing tasks to complete before shutdown */
final Settings COORD_SETTINGS = Settings.builder()
final Settings coordSettings = Settings.builder()
.put(MAXIMUM_REINDEXING_TIMEOUT_SETTING.getKey(), TimeValue.timeValueSeconds(60))
.build();
final String coordNodeName = internalCluster().startCoordinatingOnlyNode(Settings.EMPTY);
final String coordNodeName = internalCluster().startCoordinatingOnlyNode(coordSettings);

ensureStableCluster(3);

Expand Down
3 changes: 0 additions & 3 deletions muted-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -169,9 +169,6 @@ tests:
- class: org.elasticsearch.xpack.restart.QueryBuilderBWCIT
method: testQueryBuilderBWC {p0=UPGRADED}
issue: https://github.com/elastic/elasticsearch/issues/116989
- class: org.elasticsearch.index.reindex.ReindexNodeShutdownIT
method: testReindexWithShutdown
issue: https://github.com/elastic/elasticsearch/issues/118040
- class: org.elasticsearch.xpack.remotecluster.CrossClusterEsqlRCS2UnavailableRemotesIT
method: testEsqlRcs2UnavailableRemoteScenarios
issue: https://github.com/elastic/elasticsearch/issues/117419
Expand Down

0 comments on commit 3db8101

Please sign in to comment.