Skip to content

feat: add autoscaling support for VectorAggregator and ClusterVectorAggregator - #230

Merged
aa1ex merged 4 commits into
kaasops:mainfrom
P0lskay:feat/add-autoscaling
Jun 26, 2026
Merged

feat: add autoscaling support for VectorAggregator and ClusterVectorAggregator#230
aa1ex merged 4 commits into
kaasops:mainfrom
P0lskay:feat/add-autoscaling

Conversation

@P0lskay

@P0lskay P0lskay commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Solved #225

Summary

This PR introduces Horizontal Pod Autoscaler (HPA) support for CRDs ClusterVectorAggregator and VectorAggregator. It enables automated scaling of vector aggregator instances based on workload demands.

Changes

  • CRD Updates: Added a new autoscaling block to ClusterVectorAggregator and VectorAggregator schemas to configure HPA settings.
  • Controller Implementation: Added an HPA controller/reconciliation logic to manage the lifecycle of HorizontalPodAutoscaler resources in Kubernetes based on CRD specs.
  • Documentation: Added comprehensive guides and examples on how to configure and deploy HPA for the vector aggregators.

@P0lskay
P0lskay marked this pull request as ready for review June 26, 2026 07:31
@P0lskay P0lskay changed the title Added HPA controller. Added docs about HPA. feat: add autoscaling support for VectorAggregator and ClusterVectorAggregator Jun 26, 2026
Comment thread docs/aggregator-autoscaling.md Outdated
Comment thread docs/aggregator-autoscaling.md Outdated
Comment thread docs/aggregator-autoscaling.md Outdated
Comment thread internal/vector/aggregator/deployment.go Outdated
@aa1ex

aa1ex commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Optional: add Owns(&autoscalingv2.HorizontalPodAutoscaler{}) in both controllers so an external change or deletion of the HPA is reconciled rather than waiting for the next CR change.

@P0lskay

P0lskay commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Optional: add Owns(&autoscalingv2.HorizontalPodAutoscaler{}) in both controllers so an external change or deletion of the HPA is reconciled rather than waiting for the next CR change.

Yes, thanks, I did It!

@P0lskay
P0lskay requested a review from aa1ex June 26, 2026 11:57
Comment thread internal/vector/aggregator/deployment.go Outdated
@P0lskay
P0lskay requested a review from aa1ex June 26, 2026 12:39
Comment thread internal/controller/vector_controller.go Outdated
@P0lskay
P0lskay requested a review from aa1ex June 26, 2026 12:59

@aa1ex aa1ex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@aa1ex
aa1ex merged commit 4a97cb7 into kaasops:main Jun 26, 2026
5 of 6 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.

2 participants