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-operator v1.23.0 #1461

Merged
merged 3 commits into from
Jan 9, 2025
Merged

cass-operator v1.23.0 #1461

merged 3 commits into from
Jan 9, 2025

Conversation

burmanm
Copy link
Contributor

@burmanm burmanm commented Nov 26, 2024

What this PR does:
Updates to cass-operator v1.23.0 and fixes couple of bugs that I encountered while updating.

Which issue(s) this PR fixes:
Fixes #1466
Fixes #1460

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 changed the title cass-operator master version (WIP) cass-operator v1.23.0 Dec 11, 2024
@burmanm burmanm marked this pull request as ready for review December 12, 2024 09:13
@burmanm burmanm requested a review from a team as a code owner December 12, 2024 09:13
decommCassDcName := k8ssandra.GetDatacenterForDecommission(kc)

logger.Info("Checking if user keyspace replication needs to be updated", "decommissioning_dc", decommCassDcName)
logger.Info("Status of datacenters", "status", kc.Status.Datacenters)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is unintentional line, I'll remove it before merging.

…ourceName

Add updated CRD and exposed method

Change name generation to follow the real naming from cass-operator

Update tag to 1.23.0 and fix medusa tests

Change DcPrefix() in the test

Some resourceName prefix check fixes

Change schemas.go check to use CassandraDatacenter

Fix status updates and add a new status field, contextName for the Datacenter

Change back slower polling intervals for the tests, or they start to fail.
@burmanm burmanm force-pushed the cass_operator_123 branch from 9c65649 to c860236 Compare January 8, 2025 08:56
Copy link

sonarqubecloud bot commented Jan 8, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
12.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Copy link
Contributor

@rzvoncek rzvoncek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've reviewed the part about #1460. I've seen it properly restore a DC with unbalanced racks, which is something the current main branch cannot do.

@burmanm burmanm merged commit 3b13a2e into k8ssandra:main Jan 9, 2025
61 of 62 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
2 participants