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

Add support for horizontal pod autoscaling #1676

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

dhageman
Copy link
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
Member

@stanislav-zaprudskiy you might be interested in this

@dathan
Copy link

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
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
Member

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

@rooftopcellist
Copy link
Member

@dhageman can you give this a rebase?

@rooftopcellist rooftopcellist merged commit 8ead140 into ansible:devel Jun 3, 2024
7 checks passed
@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
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.

None yet

8 participants