Skip to content

Prevent any scaling operations from happening if trackedTask is running#903

Closed
burmanm wants to merge 3 commits intok8ssandra:masterfrom
burmanm:block_scale_with_trackedtask
Closed

Prevent any scaling operations from happening if trackedTask is running#903
burmanm wants to merge 3 commits intok8ssandra:masterfrom
burmanm:block_scale_with_trackedtask

Conversation

@burmanm
Copy link
Copy Markdown
Contributor

@burmanm burmanm commented Mar 11, 2026

What this PR does:
Adds additional blocks for operations that try to scale the cluster if a tracked task is running and it is expected to waited for.

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

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 11, 2026 10:43
@burmanm burmanm self-assigned this Mar 11, 2026
@burmanm burmanm force-pushed the block_scale_with_trackedtask branch from e4df54a to a26798a Compare March 11, 2026 16:12
@burmanm burmanm force-pushed the block_scale_with_trackedtask branch from a26798a to 16b5298 Compare March 20, 2026 14:56
@artsiom-bukhautsou artsiom-bukhautsou self-requested a review March 24, 2026 08:50
return recResult.Output()
}

if recResult := rc.CheckCleanupAfterScaling(); recResult.Completed() {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This position won't work. It can't reliably prevent what we wanted to prevent, but it also shouldn't spawn the cleanup process here, since right now we would spawn it too early.

What we need is a webhook change preventing a change to scale down if ScalingUp is True in CassandraDatacenter.

@burmanm burmanm marked this pull request as draft March 24, 2026 15:48
@burmanm
Copy link
Copy Markdown
Contributor Author

burmanm commented Apr 15, 2026

This is replaced by the other PR adding webhook validations.

@burmanm burmanm closed this Apr 15, 2026
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.

Cass-operator should block resize operations while cleanup following resize is in progress

1 participant