Skip to content

[CORE-149029] - Sync ScaleOps Resource Requests and Affinities#268

Open
dvaseekara wants to merge 10 commits into
masterfrom
CORE-149029-1
Open

[CORE-149029] - Sync ScaleOps Resource Requests and Affinities#268
dvaseekara wants to merge 10 commits into
masterfrom
CORE-149029-1

Conversation

@dvaseekara

Copy link
Copy Markdown
Collaborator

Description

Adding ability for koperator to work with Scaleops. A new toggle is introduced to kafkaCluster to denote if ScaleOps is enabled in the cluster that kafka is deployed in.

If Scaleops is deployed, it has the ability to update Resource Requests and Affinities for Kafka pods. This change syncs these updates with the Koperator Reconcilers "DesiredPod" so they don't get reverted when koperator reconciles differences between the CurrentPod and DesiredPod

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@dvaseekara dvaseekara changed the title [CORE - 149029] - Sync ScaleOps Resource Requests and Affinities [CORE-149029] - Sync ScaleOps Resource Requests and Affinities Jun 18, 2026
Comment thread api/v1beta1/kafkacluster_types.go Outdated
// the KafkaCluster to the current state in the Kubernetes Cluster
// +kubebuilder:default=false
// +optional
ScaleOpsEnabled bool `json:"scaleOpsEnabled,omitempty"`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What happens is we adopt another another admission controller?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Updated the Resource and Affinity syncing to be more generic. Now any preferred affinity or resource request change from any admission webhook (scale ops or not) will be accepted by koperator.

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.

2 participants