Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds aodh to the autoscaling api.
Because the autoscaling controller got quite big, I decided to split it to 3 (4 in the future) files. controllers/autoscaling_controller.go for some common code and /controllers/prometheus_controller.go, /controllers/aodh_controller for the prometheus and aodh related code.
The controller passes the prometheus host and port values to the aodh-evaluator container as expected by the observability client, which will be used for autoscaling in the future.
The functionality of aodh is a bit hard to test at the moment, because we don't have a supported storage for it (gnocchi at this moment, prometheus is coming in the future) we can't create any alarms. But I was able to at least execute
openstack alarm list
from the openstackclient container. This resulted in an empty response (expected), but most importantly it didn't produce any error in any of the aodh containers or the operator pod.There is also a related patch to install_yamls to add the required aodh and database passwords to the osp-secret: openstack-k8s-operators/install_yamls#541