opensearch-project / OpenSearch

🔎 Open source distributed and RESTful search engine.
https://opensearch.org/docs/latest/opensearch/index/
Apache License 2.0
8.89k stars 1.63k forks source link

Backport 12159 to 2.x #13681

Closed vikasvb90 closed 2 weeks ago

vikasvb90 commented 2 weeks ago

Description

Backport of PR https://github.com/opensearch-project/OpenSearch/pull/12159

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

By 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.

github-actions[bot] commented 2 weeks ago

:white_check_mark: Gradle check result for 0351e490df393467eda97fa465445e2033711353: SUCCESS

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 71.50838% with 102 lines in your changes are missing coverage. Please review.

Project coverage is 71.23%. Comparing base (0dd892c) to head (0351e49). Report is 251 commits behind head on 2.x.

:exclamation: Current head 0351e49 differs from pull request most recent head f8f5b22. Consider uploading reports for the commit f8f5b22 to get more accurate results

Files Patch % Lines
...opensearch/repositories/s3/S3RepositoryPlugin.java 0.00% 29 Missing :warning:
...rg/opensearch/repositories/s3/S3BlobContainer.java 48.27% 6 Missing and 9 partials :warning:
...epositories/s3/async/TransferSemaphoresHolder.java 80.00% 6 Missing and 5 partials :warning:
...ch/repositories/s3/async/AsyncTransferManager.java 79.16% 9 Missing and 1 partial :warning:
...arch/repositories/s3/async/SizeBasedBlockingQ.java 89.01% 7 Missing and 3 partials :warning:
...h/repositories/s3/GenericStatsMetricPublisher.java 75.00% 8 Missing :warning:
...earch/repositories/s3/async/AsyncPartsHandler.java 82.05% 4 Missing and 3 partials :warning:
...va/org/opensearch/repositories/s3/S3BlobStore.java 50.00% 6 Missing :warning:
...earch/index/store/RemoteSegmentStoreDirectory.java 0.00% 3 Missing :warning:
...va/org/opensearch/index/store/RemoteDirectory.java 33.33% 0 Missing and 2 partials :warning:
... and 1 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## 2.x #13681 +/- ## ============================================ - Coverage 71.28% 71.23% -0.06% - Complexity 60145 61208 +1063 ============================================ Files 4957 5035 +78 Lines 282799 288185 +5386 Branches 41409 42109 +700 ============================================ + Hits 201591 205281 +3690 - Misses 64189 65579 +1390 - Partials 17019 17325 +306 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 2 weeks ago

:x: Gradle check result for 62c7fd32b0dc6dfab3bbc0b1c40b344884a694f5: 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?

github-actions[bot] commented 2 weeks ago

:x: Gradle check result for 4dd11800cb3917b96aced6b93d85201546a89b1f: 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?

github-actions[bot] commented 2 weeks ago

:x: Gradle check result for 3b6979730d27141fb6a68a7c3e63317ee2a66d5e: 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?

github-actions[bot] commented 2 weeks ago

:x: Gradle check result for 0e987363f12acccb94092fe72a470b04a66f3f1a: 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?

github-actions[bot] commented 2 weeks ago

:grey_exclamation: Gradle check result for f8f5b220029628f62bd16eab7a60085a6bb6b92f: 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.

vikasvb90 commented 2 weeks ago

Thanks @dblock for merging this!