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.11] fix stale remote cluster uuid state not purged from remote #10446

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 1eec0f1 from #10016.

* fix stale remote cluster uuid state not purged from remote

Signed-off-by: bansvaru <bansvaru@amazon.com>

* fix tests

Signed-off-by: bansvaru <bansvaru@amazon.com>

* use new limit parameter

Signed-off-by: bansvaru <bansvaru@amazon.com>

* minor refactoring

Signed-off-by: bansvaru <bansvaru@amazon.com>

* delete index metadata files before manifest file

Signed-off-by: bansvaru <bansvaru@amazon.com>

* add basic UT

Signed-off-by: bansvaru <bansvaru@amazon.com>

* delete all data related to a cluster uuid in a single call

Signed-off-by: bansvaru <bansvaru@amazon.com>

* fix git diff

Signed-off-by: bansvaru <bansvaru@amazon.com>

* remove unreferenced code

Signed-off-by: bansvaru <bansvaru@amazon.com>

* fix spa

Signed-off-by: bansvaru <bansvaru@amazon.com>

---------

Signed-off-by: bansvaru <bansvaru@amazon.com>
(cherry picked from commit 1eec0f1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2023

Compatibility status:

Checks if related components are compatible with change ed4a220

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/neural-search.git]

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2023

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Oct 6, 2023

Codecov Report

Merging #10446 (ed4a220) into 2.11 (f164bf5) will increase coverage by 0.07%.
Report is 2 commits behind head on 2.11.
The diff coverage is 82.75%.

@@             Coverage Diff              @@
##               2.11   #10446      +/-   ##
============================================
+ Coverage     70.86%   70.93%   +0.07%     
- Complexity    58501    58535      +34     
============================================
  Files          4829     4829              
  Lines        276344   276364      +20     
  Branches      40578    40578              
============================================
+ Hits         195821   196037     +216     
+ Misses        63808    63632     -176     
+ Partials      16715    16695      -20     
Files Coverage Δ
...nsearch/http/netty4/Netty4HttpServerTransport.java 75.75% <100.00%> (+0.56%) ⬆️
...in/java/org/opensearch/transport/Netty4Plugin.java 73.68% <ø> (ø)
...src/main/java/org/opensearch/rest/RestHandler.java 60.37% <ø> (ø)
...arch/gateway/remote/RemoteClusterStateService.java 67.81% <78.26%> (+3.16%) ⬆️

... and 462 files with indirect coverage changes

@dblock dblock merged commit dc5128e into 2.11 Oct 6, 2023
@github-actions github-actions bot deleted the backport/backport-10016-to-2.11 branch October 6, 2023 15:23
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.

2 participants