Skip to content

Commit

Permalink
Add GetSnapshotsIT#testAllFeatures
Browse files Browse the repository at this point in the history 
The features of get-snapshots API are all tested in isolation or small
combinations, but there's no one test which pins down exactly how they
all interact. This commit adds such a test, to verify that any future
optimization work preserves the observable behaviour.

Relates elastic#95345
Relates elastic#104607
  • Loading branch information
@DaveCTurner
DaveCTurner committed Aug 11, 2024
1 parent fd916c2 commit 7b54f9f
Showing 1 changed file with 209 additions and 0 deletions.
209 changes: 209 additions & 0 deletions server/src/internalClusterTest/java/org/elasticsearch/snapshots/GetSnapshotsIT.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -10,24 +10,41 @@

import org.elasticsearch.action.ActionFuture;
import org.elasticsearch.action.ActionRequestValidationException;
import org.elasticsearch.action.admin.cluster.repositories.put.PutRepositoryRequest;
import org.elasticsearch.action.admin.cluster.repositories.put.TransportPutRepositoryAction;
import org.elasticsearch.action.admin.cluster.snapshots.create.CreateSnapshotRequest;
import org.elasticsearch.action.admin.cluster.snapshots.create.CreateSnapshotResponse;
import org.elasticsearch.action.admin.cluster.snapshots.create.TransportCreateSnapshotAction;
import org.elasticsearch.action.admin.cluster.snapshots.get.GetSnapshotsRequest;
import org.elasticsearch.action.admin.cluster.snapshots.get.GetSnapshotsRequestBuilder;
import org.elasticsearch.action.admin.cluster.snapshots.get.GetSnapshotsResponse;
import org.elasticsearch.action.admin.cluster.snapshots.get.SnapshotSortKey;
import org.elasticsearch.action.admin.cluster.snapshots.get.TransportGetSnapshotsAction;
import org.elasticsearch.action.admin.indices.create.CreateIndexRequest;
import org.elasticsearch.action.admin.indices.create.TransportCreateIndexAction;
import org.elasticsearch.action.support.RefCountingListener;
import org.elasticsearch.cluster.SnapshotsInProgress;
import org.elasticsearch.common.Strings;
import org.elasticsearch.common.settings.Settings;
import org.elasticsearch.core.Predicates;
import org.elasticsearch.repositories.RepositoryMissingException;
import org.elasticsearch.repositories.fs.FsRepository;
import org.elasticsearch.search.sort.SortOrder;
import org.elasticsearch.test.ESTestCase;
import org.elasticsearch.test.hamcrest.ElasticsearchAssertions;
import org.elasticsearch.threadpool.ThreadPool;

import java.nio.file.Path;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Objects;
import java.util.Set;
import java.util.function.Predicate;
import java.util.stream.Collectors;

import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertAcked;
import static org.hamcrest.Matchers.empty;
Expand Down Expand Up @@ -745,4 +762,196 @@ private static GetSnapshotsRequestBuilder baseGetSnapshotsRequest(String[] repoN
return clusterAdmin().prepareGetSnapshots(TEST_REQUEST_TIMEOUT, repoNames)
.setSnapshots("*", "-" + AbstractSnapshotIntegTestCase.OLD_VERSION_SNAPSHOT_PREFIX + "*");
}

public void testAllFeatures() {
// A test that uses (potentially) as many of the features of the get-snapshots API at once as possible, to verify that they interact
// in the expected order etc.

// Create a few repositories and a few indices
final var repositories = randomList(1, 4, ESTestCase::randomIdentifier);
final var indices = randomList(1, 4, ESTestCase::randomIdentifier);

safeAwait(l -> {
try (var listeners = new RefCountingListener(l.map(v -> null))) {
for (final var repository : repositories) {
client().execute(
TransportPutRepositoryAction.TYPE,
new PutRepositoryRequest(TEST_REQUEST_TIMEOUT, TEST_REQUEST_TIMEOUT, repository).type(FsRepository.TYPE)
.settings(Settings.builder().put("location", randomRepoPath()).build()),
listeners.acquire(ElasticsearchAssertions::assertAcked)
);
}

for (final var index : indices) {
client().execute(
TransportCreateIndexAction.TYPE,
new CreateIndexRequest(index, indexSettings(1, 0).build()),
listeners.acquire(ElasticsearchAssertions::assertAcked)
);
}
}
});
ensureGreen();

// Create a few snapshots
final var snapshotInfos = Collections.synchronizedList(new ArrayList<SnapshotInfo>());
safeAwait(l -> {
try (var listeners = new RefCountingListener(l.map(v -> null))) {
for (int i = 0; i < 10; i++) {
client().execute(
TransportCreateSnapshotAction.TYPE,
new CreateSnapshotRequest(
TEST_REQUEST_TIMEOUT,
// at least one snapshot per repository to satisfy consistency checks
i < repositories.size() ? repositories.get(i) : randomFrom(repositories),
randomIdentifier()
).waitForCompletion(true).indices(randomNonEmptySubsetOf(indices)),
listeners.acquire(
createSnapshotResponse -> snapshotInfos.add(Objects.requireNonNull(createSnapshotResponse.getSnapshotInfo()))
)
);
}
}
});

Predicate<SnapshotInfo> filterByNamePredicate = Predicates.always();

// {repository} path parameter
final String[] requestedRepositories;
if (randomBoolean()) {
requestedRepositories = new String[] { randomFrom("_all", "*") };
} else {
final var selectedRepositories = Set.copyOf(randomNonEmptySubsetOf(repositories));
filterByNamePredicate = filterByNamePredicate.and(si -> selectedRepositories.contains(si.repository()));
requestedRepositories = selectedRepositories.toArray(new String[0]);
}

// {snapshot} path parameter
final String[] requestedSnapshots;
if (randomBoolean()) {
requestedSnapshots = randomBoolean() ? Strings.EMPTY_ARRAY : new String[] { randomFrom("_all", "*") };
} else {
final var selectedSnapshots = randomNonEmptySubsetOf(snapshotInfos).stream()
.map(si -> si.snapshotId().getName())
.collect(Collectors.toSet());
filterByNamePredicate = filterByNamePredicate.and(si -> selectedSnapshots.contains(si.snapshotId().getName()));
requestedSnapshots = selectedSnapshots.stream().map(n -> n + "*").toArray(String[]::new);
}

// ?sort and ?order parameters
final var sortKey = randomFrom(SnapshotSortKey.values());
final var order = randomFrom(SortOrder.values());

// compute the ordered sequence of snapshots which match the repository/snapshot name filters
final var selectedSnapshots = snapshotInfos.stream()
.filter(filterByNamePredicate)
.sorted(sortKey.getSnapshotInfoComparator(order))
.toList();

final var getSnapshotsRequest = new GetSnapshotsRequest(TEST_REQUEST_TIMEOUT, requestedRepositories, requestedSnapshots)
// apply sorting params
.sort(sortKey)
.order(order);

// sometimes use ?from_sort_value to skip some items; note that snapshots skipped in this way are subtracted from
// GetSnapshotsResponse.totalCount whereas snapshots skipped by ?after and ?offset are not
final int skippedByFromSortValue;
if (randomBoolean()) {
final var startingSnapshot = randomFrom(snapshotInfos);
getSnapshotsRequest.fromSortValue(switch (sortKey) {
case START_TIME -> Long.toString(startingSnapshot.startTime());
case NAME -> startingSnapshot.snapshotId().getName();
case DURATION -> Long.toString(startingSnapshot.endTime() - startingSnapshot.startTime());
case INDICES, SHARDS -> Integer.toString(startingSnapshot.indices().size());
case FAILED_SHARDS -> "0";
case REPOSITORY -> startingSnapshot.repository();
});
final Predicate<SnapshotInfo> fromSortValuePredicate = snapshotInfo -> {
final var comparison = switch (sortKey) {
case START_TIME -> Long.compare(snapshotInfo.startTime(), startingSnapshot.startTime());
case NAME -> snapshotInfo.snapshotId().getName().compareTo(startingSnapshot.snapshotId().getName());
case DURATION -> Long.compare(
snapshotInfo.endTime() - snapshotInfo.startTime(),
startingSnapshot.endTime() - startingSnapshot.startTime()
);
case INDICES, SHARDS -> Integer.compare(snapshotInfo.indices().size(), startingSnapshot.indices().size());
case FAILED_SHARDS -> 0;
case REPOSITORY -> snapshotInfo.repository().compareTo(startingSnapshot.repository());
};
return order == SortOrder.ASC ? comparison < 0 : comparison > 0;
};

int skipCount = 0;
for (final var snapshotInfo : selectedSnapshots) {
if (fromSortValuePredicate.test(snapshotInfo)) {
skipCount += 1;
} else {
break;
}
}
skippedByFromSortValue = skipCount;
} else {
skippedByFromSortValue = 0;
}

// ?offset parameter
if (randomBoolean()) {
getSnapshotsRequest.offset(between(0, selectedSnapshots.size() + 1));
}

// ?size parameter
if (randomBoolean()) {
getSnapshotsRequest.size(between(1, selectedSnapshots.size() + 1));
}

// compute the expected offset and size of the returned snapshots as indices in selectedSnapshots:
final var expectedOffset = Math.min(selectedSnapshots.size(), skippedByFromSortValue + getSnapshotsRequest.offset());
final var expectedSize = Math.min(
selectedSnapshots.size() - expectedOffset,
getSnapshotsRequest.size() == GetSnapshotsRequest.NO_LIMIT ? Integer.MAX_VALUE : getSnapshotsRequest.size()
);

// get the actual response
final GetSnapshotsResponse getSnapshotsResponse = safeAwait(
l -> client().execute(TransportGetSnapshotsAction.TYPE, getSnapshotsRequest, l)
);

// verify it returns the expected results
assertEquals(
selectedSnapshots.stream().skip(expectedOffset).limit(expectedSize).map(SnapshotInfo::snapshotId).toList(),
getSnapshotsResponse.getSnapshots().stream().map(SnapshotInfo::snapshotId).toList()
);
assertEquals(expectedSize, getSnapshotsResponse.getSnapshots().size());
assertEquals(selectedSnapshots.size() - skippedByFromSortValue, getSnapshotsResponse.totalCount());
assertEquals(selectedSnapshots.size() - expectedOffset - expectedSize, getSnapshotsResponse.remaining());
assertEquals(getSnapshotsResponse.remaining() > 0, getSnapshotsResponse.next() != null);

// now use ?after to page through the rest of the results
var nextRequestAfter = getSnapshotsResponse.next();
var nextExpectedOffset = expectedOffset + expectedSize;
var remaining = getSnapshotsResponse.remaining();
while (nextRequestAfter != null) {
final var nextSize = between(1, remaining);
final var nextRequest = new GetSnapshotsRequest(TEST_REQUEST_TIMEOUT, requestedRepositories, requestedSnapshots)
// same name filters, same ?sort and ?order params, new ?size, but no ?offset or ?from_sort_value because of ?after
.sort(sortKey)
.order(order)
.size(nextSize)
.after(SnapshotSortKey.decodeAfterQueryParam(nextRequestAfter));
final GetSnapshotsResponse nextResponse = safeAwait(l -> client().execute(TransportGetSnapshotsAction.TYPE, nextRequest, l));

assertEquals(
selectedSnapshots.stream().skip(nextExpectedOffset).limit(nextSize).map(SnapshotInfo::snapshotId).toList(),
nextResponse.getSnapshots().stream().map(SnapshotInfo::snapshotId).toList()
);
assertEquals(nextSize, nextResponse.getSnapshots().size());
assertEquals(selectedSnapshots.size(), nextResponse.totalCount());
assertEquals(remaining - nextSize, nextResponse.remaining());
assertEquals(nextResponse.remaining() > 0, nextResponse.next() != null);

nextRequestAfter = nextResponse.next();
nextExpectedOffset += nextSize;
remaining -= nextSize;
}
}
}

0 comments on commit 7b54f9f

Please sign in to comment.