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

Merged
merged 1 commit into from
Oct 5, 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>
@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #10403 (9400119) into 2.x (890663f) will decrease coverage by 0.03%.
The diff coverage is 78.26%.

@@             Coverage Diff              @@
##                2.x   #10403      +/-   ##
============================================
- Coverage     70.87%   70.84%   -0.03%     
+ Complexity    58477    58455      -22     
============================================
  Files          4829     4829              
  Lines        276310   276327      +17     
  Branches      40573    40573              
============================================
- Hits         195826   195770      -56     
- Misses        63760    63864     +104     
+ Partials      16724    16693      -31     
Files Coverage Δ
...arch/gateway/remote/RemoteClusterStateService.java 67.81% <78.26%> (+3.16%) ⬆️

... and 467 files with indirect coverage changes

@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2023

Compatibility status:

Checks if related components are compatible with change 9400119

Incompatible components

Incompatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/performance-analyzer.git]

Skipped components

Compatible components

Compatible components: [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/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git]

@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.client.PitIT.testDeleteAllAndListAllPits

@sachinpkale sachinpkale merged commit 16d12e1 into 2.x Oct 5, 2023
@github-actions github-actions bot deleted the backport/backport-10016-to-2.x branch October 5, 2023 17:41
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.

1 participant