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

CASS-89 If status.datacenterName is set, but empty, do not use it #752

Merged
merged 3 commits into from
Feb 14, 2025

Conversation

burmanm
Copy link
Contributor

@burmanm burmanm commented Feb 6, 2025

…eNames

What this PR does:
Changes datacenterName processing to not accept "" as valid value for resource names.

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

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 February 6, 2025 06:41
@burmanm burmanm changed the title If status.datacenterName is set, but empty, do not use it for resourc… CASS-89 If status.datacenterName is set, but empty, do not use it Feb 6, 2025
@burmanm burmanm merged commit 2f22b49 into k8ssandra:master Feb 14, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

If datacenterName in the Status is set to "", we use that value
2 participants