Skip to content
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

[Backport 2.x] Purge remote translog basis the latest metadata for remote-backed indexes #6170

Merged

Conversation

ashking94
Copy link
Member

Backport 54ce423 from #6086.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

Gradle Check (Jenkins) Run Completed with:

@kotwanikunal
Copy link
Member

@ashking94 Please rebase your branch to have the latest version changes in.

…exes (opensearch-project#6086)

* Deleting remote translog considering latest remote metadata

Co-authored-by: Gaurav Bafna <gbbafna@amazon.com>
Signed-off-by: Ashish Singh <ssashish@amazon.com>
(cherry picked from commit 54ce423)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@ashking94 ashking94 force-pushed the backport/backport-6086-to-2.x branch from de235fe to f5f5609 Compare February 6, 2023 04:02
@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2023

Gradle Check (Jenkins) Run Completed with:

Signed-off-by: Ashish Singh <ssashish@amazon.com>
@ashking94
Copy link
Member Author

Gradle Check (Jenkins) Run Completed with:

1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':plugins:discovery-ec2:qa:amazon-ec2:yamlRestTestSystemProperties'.
> `node{:plugins:discovery-ec2:qa:amazon-ec2:yamlRestTestSystemProperties-0}` failed to wait for ports files after 135000 MILLISECONDS

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
==============================================================================

2: Task failed with an exception.
-----------
* What went wrong:
org.gradle.api.GradleException: Reaper process failed. Check log at /var/jenkins/workspace/gradle-check/search/.gradle/reaper/build-5542/error.log for details
> Reaper process failed. Check log at /var/jenkins/workspace/gradle-check/search/.gradle/reaper/build-5542/error.log for details

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
==============================================================================

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2023

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #6170 (4ea01fc) into 2.x (e53f6c2) will decrease coverage by 0.25%.
The diff coverage is 72.15%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##                2.x    #6170      +/-   ##
============================================
- Coverage     70.71%   70.47%   -0.25%     
- Complexity    58547    59014     +467     
============================================
  Files          4719     4774      +55     
  Lines        279503   282934    +3431     
  Branches      40435    41215     +780     
============================================
+ Hits         197662   199394    +1732     
- Misses        65579    66854    +1275     
- Partials      16262    16686     +424     
Impacted Files Coverage Δ
...ex/translog/transfer/BlobStoreTransferService.java 70.21% <52.94%> (ø)
...rg/opensearch/index/translog/RemoteFsTranslog.java 70.23% <69.69%> (+1.19%) ⬆️
...dex/translog/transfer/TranslogTransferManager.java 81.81% <83.33%> (ø)
...h/index/translog/transfer/FileTransferTracker.java 94.59% <100.00%> (ø)
...search/indices/recovery/RecoveryTargetHandler.java 0.00% <0.00%> (-100.00%) ⬇️
...n/indices/forcemerge/ForceMergeRequestBuilder.java 0.00% <0.00%> (-75.00%) ⬇️
.../java/org/opensearch/node/NodeClosedException.java 50.00% <0.00%> (-50.00%) ⬇️
...rg/opensearch/common/RandomBasedUUIDGenerator.java 52.63% <0.00%> (-47.37%) ⬇️
...org/opensearch/transport/RemoteConnectionInfo.java 43.03% <0.00%> (-39.47%) ⬇️
...h/action/ingest/SimulateDocumentVerboseResult.java 60.71% <0.00%> (-39.29%) ⬇️
... and 909 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants