Skip to content

Add support for horizontal pod autoscaling#1676

Merged
rooftopcellist merged 1 commit intoansible:develfrom
dhageman:hpa
Jun 3, 2024
Merged

Add support for horizontal pod autoscaling#1676
rooftopcellist merged 1 commit intoansible:develfrom
dhageman:hpa

Conversation

@dhageman
Copy link
Copy Markdown
Contributor

SUMMARY

Allows the operator to create horizontal pod autoscaling resources for the web and task deployments.

It does not provide guidance on how to configure the HPA as those details are very cluster and use case specific.

ISSUE TYPE
  • New or Enhanced Feature
ADDITIONAL INFORMATION

This code does not change the default behavior of the operator. A horizontal pod autoscaler will not be created unless it is specifically configured.

The minimum configuration required is to set the option task_max_replicas or web_max_replicas to be an integer greater than task_replicas or web_replicas respecitively. This should create a HPA using the default values for a HPA which on OpenShift monitors CPU consumption.

A good configuration for HPA will require requests, limits, and health probes for the the web and task pods to be implemented.

@TheRealHaoLiu
Copy link
Copy Markdown
Member

@stanislav-zaprudskiy you might be interested in this

@dathan
Copy link
Copy Markdown

dathan commented Feb 29, 2024

Patch looks good to me. Most users of awx are probably not deploying constantly so keeping a bunch of web or task pods running does not make sense. It makes sense only when they are used. In theory with this the community can collectively save big on vCPU requested bloats that site idle for the majority of time.

@trippinnik
Copy link
Copy Markdown
Contributor

This would really help me out. We run a lot of jobs on schedule at particular after hours times so increasing the resources available dynamically is really needed

@TheRealHaoLiu
Copy link
Copy Markdown
Member

@dhageman PR looks great, can you rebase and we can get it in

@rooftopcellist
Copy link
Copy Markdown
Member

@dhageman can you give this a rebase?

@rooftopcellist rooftopcellist merged commit 8ead140 into ansible:devel Jun 3, 2024
@dhageman dhageman deleted the hpa branch June 3, 2024 22:43
sbilque pushed a commit to sbilque/awx-operator that referenced this pull request Jun 4, 2024
jessicamack pushed a commit that referenced this pull request Jul 3, 2024
* Allow to scale up the operator pods by using the Helm Chart
* Add support for horizontal pod autoscaling (#1676)
* fix: spec.replicas
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants