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.

Sign up for GitHub

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

Jump to bottom

If service is already owned by some other resource, do not try to tak… #627

Merged
merged 1 commit into from
Mar 28, 2024
Merged

If service is already owned by some other resource, do not try to tak… #627

merged 1 commit into from
Mar 28, 2024

Conversation

burmanm
Copy link
Contributor

@burmanm burmanm commented Mar 27, 2024 •
edited
Loading

What this PR does:

  • If the service is owned by another Datacenter, do not try to update it anymore, even if hashes don't match. If it later gets deleted because of cleanup, we will recreate it

Which issue(s) this PR fixes:
Fixes #626

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CHANGELOG.md updated (not required for documentation PRs)
  • CLA Signed: DataStax CLA

@burmanm burmanm requested a review from a team as a code owner March 27, 2024 15:46
@burmanm
If service is already owned by some other resource, do not try to tak…
a29e3a2 
…e ownership of it. If it later gets deleted because of cleanup, we will recreate it
@burmanm burmanm force-pushed the fix_service_ownership branch from 8ca40b6 to a29e3a2 Compare March 27, 2024 17:10
@adejanovski adejanovski merged commit ebbb957 into k8ssandra:master Mar 28, 2024
39 checks passed
burmanm added a commit that referenced this pull request Mar 28, 2024
@burmanm
If service is already owned by some other resource, do not try to tak…
da7eaac 
…e ownership of it (#627)

 If it later gets deleted because of cleanup, we will recreate it

(cherry picked from commit ebbb957)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adejanovski adejanovski adejanovski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Multiple Datacenters compete over seed service ownership
2 participants
@burmanm @adejanovski