-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Remote Store] Adding ITs to ensure RemoteIndexPath file gets created after migration for each index. #13367
[Remote Store] Adding ITs to ensure RemoteIndexPath file gets created after migration for each index. #13367
Conversation
…ng migration direction to remote-store Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
d247279
to
1072a87
Compare
❌ Gradle check result for d247279: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #13367 +/- ##
============================================
+ Coverage 71.42% 71.67% +0.25%
- Complexity 59978 61306 +1328
============================================
Files 4985 5064 +79
Lines 282275 288088 +5813
Branches 40946 41715 +769
============================================
+ Hits 201603 206488 +4885
- Misses 63999 64559 +560
- Partials 16673 17041 +368 ☔ View full report in Codecov by Sentry. |
❌ Gradle check result for 4eedad9: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❌ Gradle check result for af14481: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
❌ Gradle check result for d5bd0ca: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
… into serengity_personel
Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
❌ Gradle check result for 743aaef: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
...nalClusterTest/java/org/opensearch/remotemigration/RemoteMigrationIndexMetadataUpdateIT.java
Outdated
Show resolved
Hide resolved
...nalClusterTest/java/org/opensearch/remotemigration/RemoteMigrationIndexMetadataUpdateIT.java
Outdated
Show resolved
Hide resolved
...nalClusterTest/java/org/opensearch/remotemigration/RemoteMigrationIndexMetadataUpdateIT.java
Outdated
Show resolved
Hide resolved
...nalClusterTest/java/org/opensearch/remotemigration/RemoteMigrationIndexMetadataUpdateIT.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
❌ Gradle check result for c47dc57: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
… into serengity_personel
❌ Gradle check result for 1752fa6: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❌ Gradle check result for 1752fa6: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❕ Gradle check result for 1752fa6: UNSTABLE Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
… after migration for each index. (opensearch-project#13367) Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
… after migration for each index. (opensearch-project#13367) (opensearch-project#13930) (cherry picked from commit 896b127) Signed-off-by: Shubh Sahu <shubhvs@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: kkewwei <kkewwei@163.com>
… after migration for each index. (opensearch-project#13367) Signed-off-by: Shubh Sahu <shubhvs@amazon.com>
Description
Adding IT to validate that
Remote Index Path
file gets created after migration completes. For that we also need to ensure thatRemoteClusterState
is enabled, hence enabling the same in ITs.Related Issues
Resolves #[Issue number to be closed when this PR is merged]
Check List
Public documentation issue/PR createdBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.