forked from elastic/elasticsearch
-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge remote-tracking branch 'upstream/lucene_snapshot' into interval…
…s_regexp_range
- Loading branch information
Showing
63 changed files
with
3,089 additions
and
425 deletions.
There are no files selected for viewing
Validating CODEOWNERS rules …
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| pr: 111091 | ||
| summary: "X-pack/plugin/otel: introduce x-pack-otel plugin" | ||
| area: Data streams | ||
| type: feature | ||
| issues: [] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
186 changes: 186 additions & 0 deletions
186
...ernalClusterTest/java/org/elasticsearch/repositories/blobstore/BlobStoreCorruptionIT.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,186 @@ | ||
| /* | ||
| * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
| * or more contributor license agreements. Licensed under the Elastic License | ||
| * 2.0 and the Server Side Public License, v 1; you may not use this file except | ||
| * in compliance with, at your election, the Elastic License 2.0 or the Server | ||
| * Side Public License, v 1. | ||
| */ | ||
|
|
||
| package org.elasticsearch.repositories.blobstore; | ||
|
|
||
| import org.apache.lucene.tests.mockfile.ExtrasFS; | ||
| import org.elasticsearch.ElasticsearchException; | ||
| import org.elasticsearch.action.ActionListener; | ||
| import org.elasticsearch.action.admin.cluster.snapshots.restore.RestoreSnapshotResponse; | ||
| import org.elasticsearch.action.support.ActionTestUtils; | ||
| import org.elasticsearch.action.support.SubscribableListener; | ||
| import org.elasticsearch.action.support.master.AcknowledgedResponse; | ||
| import org.elasticsearch.common.Strings; | ||
| import org.elasticsearch.core.CheckedConsumer; | ||
| import org.elasticsearch.index.snapshots.blobstore.BlobStoreIndexShardSnapshotsIntegritySuppressor; | ||
| import org.elasticsearch.logging.LogManager; | ||
| import org.elasticsearch.logging.Logger; | ||
| import org.elasticsearch.repositories.fs.FsRepository; | ||
| import org.elasticsearch.snapshots.AbstractSnapshotIntegTestCase; | ||
| import org.elasticsearch.snapshots.SnapshotState; | ||
| import org.elasticsearch.test.ESTestCase; | ||
| import org.elasticsearch.test.hamcrest.ElasticsearchAssertions; | ||
| import org.junit.Before; | ||
|
|
||
| import java.io.IOException; | ||
| import java.nio.file.FileVisitResult; | ||
| import java.nio.file.Files; | ||
| import java.nio.file.Path; | ||
| import java.nio.file.SimpleFileVisitor; | ||
| import java.nio.file.attribute.BasicFileAttributes; | ||
| import java.util.ArrayList; | ||
| import java.util.Base64; | ||
| import java.util.List; | ||
|
|
||
| public class BlobStoreCorruptionIT extends AbstractSnapshotIntegTestCase { | ||
|
|
||
| private static final Logger logger = LogManager.getLogger(BlobStoreCorruptionIT.class); | ||
|
|
||
| @Before | ||
| public void suppressConsistencyCheck() { | ||
| disableRepoConsistencyCheck("testing corruption detection involves breaking the repo"); | ||
| } | ||
|
|
||
| public void testCorruptionDetection() throws Exception { | ||
| final var repositoryName = randomIdentifier(); | ||
| final var indexName = randomIdentifier(); | ||
| final var snapshotName = randomIdentifier(); | ||
| final var repositoryRootPath = randomRepoPath(); | ||
|
|
||
| createRepository(repositoryName, FsRepository.TYPE, repositoryRootPath); | ||
| createIndexWithRandomDocs(indexName, between(1, 100)); | ||
| flushAndRefresh(indexName); | ||
| createSnapshot(repositoryName, snapshotName, List.of(indexName)); | ||
|
|
||
| final var corruptedFile = corruptRandomFile(repositoryRootPath); | ||
| final var corruptedFileType = RepositoryFileType.getRepositoryFileType(repositoryRootPath, corruptedFile); | ||
| final var corruptionDetectors = new ArrayList<CheckedConsumer<ActionListener<Exception>, ?>>(); | ||
|
|
||
| // detect corruption by listing the snapshots | ||
| if (corruptedFileType == RepositoryFileType.SNAPSHOT_INFO) { | ||
| corruptionDetectors.add(exceptionListener -> { | ||
| logger.info("--> listing snapshots"); | ||
| client().admin() | ||
| .cluster() | ||
| .prepareGetSnapshots(TEST_REQUEST_TIMEOUT, repositoryName) | ||
| .execute(ActionTestUtils.assertNoSuccessListener(exceptionListener::onResponse)); | ||
| }); | ||
| } | ||
|
|
||
| // detect corruption by taking another snapshot | ||
| if (corruptedFileType == RepositoryFileType.SHARD_GENERATION) { | ||
| corruptionDetectors.add(exceptionListener -> { | ||
| logger.info("--> taking another snapshot"); | ||
| client().admin() | ||
| .cluster() | ||
| .prepareCreateSnapshot(TEST_REQUEST_TIMEOUT, repositoryName, randomIdentifier()) | ||
| .setWaitForCompletion(true) | ||
| .execute(exceptionListener.map(createSnapshotResponse -> { | ||
| assertNotEquals(SnapshotState.SUCCESS, createSnapshotResponse.getSnapshotInfo().state()); | ||
| return new ElasticsearchException("create-snapshot failed as expected"); | ||
| })); | ||
| }); | ||
| } | ||
|
|
||
| // detect corruption by restoring the snapshot | ||
| switch (corruptedFileType) { | ||
| case SNAPSHOT_INFO, GLOBAL_METADATA, INDEX_METADATA -> corruptionDetectors.add(exceptionListener -> { | ||
| logger.info("--> restoring snapshot"); | ||
| client().admin() | ||
| .cluster() | ||
| .prepareRestoreSnapshot(TEST_REQUEST_TIMEOUT, repositoryName, snapshotName) | ||
| .setRestoreGlobalState(corruptedFileType == RepositoryFileType.GLOBAL_METADATA || randomBoolean()) | ||
| .setWaitForCompletion(true) | ||
| .execute(ActionTestUtils.assertNoSuccessListener(exceptionListener::onResponse)); | ||
| }); | ||
| case SHARD_SNAPSHOT_INFO, SHARD_DATA -> corruptionDetectors.add(exceptionListener -> { | ||
| logger.info("--> restoring snapshot and checking for failed shards"); | ||
| SubscribableListener | ||
| // if shard-level data is corrupted then the overall restore succeeds but the shard recoveries fail | ||
| .<AcknowledgedResponse>newForked(l -> client().admin().indices().prepareDelete(indexName).execute(l)) | ||
| .andThenAccept(ElasticsearchAssertions::assertAcked) | ||
|
|
||
| .<RestoreSnapshotResponse>andThen( | ||
| l -> client().admin() | ||
| .cluster() | ||
| .prepareRestoreSnapshot(TEST_REQUEST_TIMEOUT, repositoryName, snapshotName) | ||
| .setRestoreGlobalState(randomBoolean()) | ||
| .setWaitForCompletion(true) | ||
| .execute(l) | ||
| ) | ||
|
|
||
| .addListener(exceptionListener.map(restoreSnapshotResponse -> { | ||
| assertNotEquals(0, restoreSnapshotResponse.getRestoreInfo().failedShards()); | ||
| return new ElasticsearchException("post-restore recoveries failed as expected"); | ||
| })); | ||
| }); | ||
| } | ||
|
|
||
| try (var ignored = new BlobStoreIndexShardSnapshotsIntegritySuppressor()) { | ||
| final var exception = safeAwait(randomFrom(corruptionDetectors)); | ||
| logger.info(Strings.format("--> corrupted [%s] and caught exception", corruptedFile), exception); | ||
| } | ||
| } | ||
|
|
||
| private static Path corruptRandomFile(Path repositoryRootPath) throws IOException { | ||
| final var corruptedFileType = getRandomCorruptibleFileType(); | ||
| final var corruptedFile = getRandomFileToCorrupt(repositoryRootPath, corruptedFileType); | ||
| if (randomBoolean()) { | ||
| logger.info("--> deleting [{}]", corruptedFile); | ||
| Files.delete(corruptedFile); | ||
| } else { | ||
| corruptFileContents(corruptedFile); | ||
| } | ||
| return corruptedFile; | ||
| } | ||
|
|
||
| private static void corruptFileContents(Path fileToCorrupt) throws IOException { | ||
| final var oldFileContents = Files.readAllBytes(fileToCorrupt); | ||
| logger.info("--> contents of [{}] before corruption: [{}]", fileToCorrupt, Base64.getEncoder().encodeToString(oldFileContents)); | ||
| final byte[] newFileContents = new byte[randomBoolean() ? oldFileContents.length : between(0, oldFileContents.length)]; | ||
| System.arraycopy(oldFileContents, 0, newFileContents, 0, newFileContents.length); | ||
| if (newFileContents.length == oldFileContents.length) { | ||
| final var corruptionPosition = between(0, newFileContents.length - 1); | ||
| newFileContents[corruptionPosition] = randomValueOtherThan(oldFileContents[corruptionPosition], ESTestCase::randomByte); | ||
| logger.info( | ||
| "--> updating byte at position [{}] from [{}] to [{}]", | ||
| corruptionPosition, | ||
| oldFileContents[corruptionPosition], | ||
| newFileContents[corruptionPosition] | ||
| ); | ||
| } else { | ||
| logger.info("--> truncating file from length [{}] to length [{}]", oldFileContents.length, newFileContents.length); | ||
| } | ||
| Files.write(fileToCorrupt, newFileContents); | ||
| logger.info("--> contents of [{}] after corruption: [{}]", fileToCorrupt, Base64.getEncoder().encodeToString(newFileContents)); | ||
| } | ||
|
|
||
| private static RepositoryFileType getRandomCorruptibleFileType() { | ||
| return randomValueOtherThanMany( | ||
| // these blob types do not have reliable corruption detection, so we must skip them | ||
| t -> t == RepositoryFileType.ROOT_INDEX_N || t == RepositoryFileType.ROOT_INDEX_LATEST, | ||
| () -> randomFrom(RepositoryFileType.values()) | ||
| ); | ||
| } | ||
|
|
||
| private static Path getRandomFileToCorrupt(Path repositoryRootPath, RepositoryFileType corruptedFileType) throws IOException { | ||
| final var corruptibleFiles = new ArrayList<Path>(); | ||
| Files.walkFileTree(repositoryRootPath, new SimpleFileVisitor<>() { | ||
| @Override | ||
| public FileVisitResult visitFile(Path filePath, BasicFileAttributes attrs) throws IOException { | ||
| if (ExtrasFS.isExtra(filePath.getFileName().toString()) == false | ||
| && RepositoryFileType.getRepositoryFileType(repositoryRootPath, filePath) == corruptedFileType) { | ||
| corruptibleFiles.add(filePath); | ||
| } | ||
| return super.visitFile(filePath, attrs); | ||
| } | ||
| }); | ||
| return randomFrom(corruptibleFiles); | ||
| } | ||
|
|
||
| } |
Oops, something went wrong.