Skip to content

Commit 2af113d

Browse files
authored
Fix RepositoriesFileSettingsIT to wait for metadataVersion (#126720) (#126728) (#126781)
1 parent 88b93fc commit 2af113d

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

muted-tests.yml

-3
Original file line numberDiff line numberDiff line change
@@ -311,9 +311,6 @@ tests:
311311
- class: org.elasticsearch.repositories.s3.RepositoryS3EcsCredentialsRestIT
312312
method: testNonexistentBucketReadonlyFalse
313313
issue: https://github.com/elastic/elasticsearch/issues/118225
314-
- class: org.elasticsearch.reservedstate.service.RepositoriesFileSettingsIT
315-
method: testSettingsApplied
316-
issue: https://github.com/elastic/elasticsearch/issues/116694
317314
- class: org.elasticsearch.discovery.ec2.DiscoveryEc2AvailabilityZoneAttributeNoImdsIT
318315
method: testAvailabilityZoneAttribute
319316
issue: https://github.com/elastic/elasticsearch/issues/118564

server/src/internalClusterTest/java/org/elasticsearch/reservedstate/service/RepositoriesFileSettingsIT.java

+5-1
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,11 @@
1515
import org.elasticsearch.action.admin.cluster.repositories.put.PutRepositoryRequest;
1616
import org.elasticsearch.action.admin.cluster.repositories.put.TransportPutRepositoryAction;
1717
import org.elasticsearch.action.admin.cluster.repositories.reservedstate.ReservedRepositoryAction;
18+
import org.elasticsearch.action.admin.cluster.state.ClusterStateRequest;
1819
import org.elasticsearch.client.internal.Client;
1920
import org.elasticsearch.cluster.ClusterChangedEvent;
2021
import org.elasticsearch.cluster.ClusterStateListener;
22+
import org.elasticsearch.cluster.metadata.RepositoryMetadata;
2123
import org.elasticsearch.cluster.metadata.ReservedStateErrorMetadata;
2224
import org.elasticsearch.cluster.metadata.ReservedStateHandlerMetadata;
2325
import org.elasticsearch.cluster.metadata.ReservedStateMetadata;
@@ -129,13 +131,15 @@ private void assertClusterStateSaveOK(CountDownLatch savedClusterState, AtomicLo
129131
boolean awaitSuccessful = savedClusterState.await(20, TimeUnit.SECONDS);
130132
assertTrue(awaitSuccessful);
131133

134+
clusterAdmin().state(new ClusterStateRequest(TEST_REQUEST_TIMEOUT).waitForMetadataVersion(metadataVersion.get())).get();
135+
132136
final var reposResponse = client().execute(
133137
GetRepositoriesAction.INSTANCE,
134138
new GetRepositoriesRequest(TEST_REQUEST_TIMEOUT, new String[] { "repo", "repo1" })
135139
).get();
136140

137141
assertThat(
138-
reposResponse.repositories().stream().map(r -> r.name()).collect(Collectors.toSet()),
142+
reposResponse.repositories().stream().map(RepositoryMetadata::name).collect(Collectors.toSet()),
139143
containsInAnyOrder("repo", "repo1")
140144
);
141145

0 commit comments

Comments
 (0)