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

K8SSAND-1820 ⁃ Removing Stargate triggers a DC restart #714

Closed
adutra opened this issue Oct 6, 2022 · 1 comment · Fixed by #716
Closed

K8SSAND-1820 ⁃ Removing Stargate triggers a DC restart #714

adutra opened this issue Oct 6, 2022 · 1 comment · Fixed by #716
Assignees
Labels
bug Something isn't working

Comments

@adutra
Copy link
Contributor

adutra commented Oct 6, 2022

What happened?

When Stargate is removed from a K8ssandraCluster object suing Cassandra 4.0, this triggers a DC restart.

Did you expect to see something different?

I expect no DC restart.

How to reproduce it (as minimally and precisely as possible):

  1. Create a cluster with 1 DC and Stargate
  2. Delete Stargate from the CRD
  3. Observe the DC being restarted

Environment

  • K8ssandra Operator version:

1.2

Anything else we need to know?:

Due to this code in the k8ssandra controller:

if dcConfig.ServerVersion.Major() != 3 && kc.HasStargates() {
	// if we're not running Cassandra 3.11 and have Stargate pods, we need to allow alter RF during range movements
	cassandra.AllowAlterRfDuringRangeMovement(dcConfig)
}

I suggest that we always call cassandra.AllowAlterRfDuringRangeMovement(dcConfig) for Cassandra 4.0, regardless of whether we have Stargate nodes.

See also stargate/stargate#1274.

┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1820
┆priority: Medium

@adutra adutra added the bug Something isn't working label Oct 6, 2022
@sync-by-unito sync-by-unito bot changed the title Removing Stargate triggers a DC restart K8SSAND-1820 ⁃ Removing Stargate triggers a DC restart Oct 6, 2022
@adejanovski
Copy link
Contributor

I suggest that we always call cassandra.AllowAlterRfDuringRangeMovement(dcConfig) for Cassandra 4.0, regardless of whether we have Stargate nodes.

I agree 👍

@adejanovski adejanovski added zh:In-Progress Issues in the ZenHub pipeline 'In-Progress' and removed zh:Ready labels Oct 11, 2022
@adejanovski adejanovski added zh:Review Issues in the ZenHub pipeline 'Review' and removed zh:In-Progress Issues in the ZenHub pipeline 'In-Progress' labels Oct 20, 2022
@adejanovski adejanovski removed the zh:Review Issues in the ZenHub pipeline 'Review' label Nov 8, 2022
sseidman added a commit to sseidman/k8ssandra-operator that referenced this issue Nov 8, 2022
* Support additional jvm options for the various options files (k8ssandra#738)

* Expose ManagementApiAuth and pod SecurityGroup from the cassdc spec in k8ssandra-operator (k8ssandra#741)

* Don't restart whole Cassandra 4 DC when Stargate is added or removed (k8ssandra#716) (fixes k8ssandra#714)

* Add e2e test and docs for JBOD (k8ssandra#748)

* Add e2e test for using separate commitlog and data volumes (k8ssandra#750)

* Upgrade Stargate to v1.0.67 (k8ssandra#754)

* Compute initial tokens for clusters with low number of tokens (k8ssandra#743)

* Ninja fix: update the CRDs post Stargate upgrade

Co-authored-by: Alexander Dejanovski <alex.dejanovski@datastax.com>
Co-authored-by: Olivier Michallat <olim7t@users.noreply.github.com>
Co-authored-by: Alexandre Dutra <adutra@users.noreply.github.com>
Co-authored-by: Alexander Dejanovski <alex@thelastpickle.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants