-
Notifications
You must be signed in to change notification settings - Fork 159
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 autoscaling #208
add autoscaling #208
Conversation
@sja, please add a helm chart bump and edit the artifacthub.io/changes section. |
Thanks @pierluigilenoci for that info. I did a minor version bump, as it adds a feature. It is backwards compatible, old values do not change the result. |
@pierluigilenoci I updated the MR again, inspired from the other MRs. Feel free to reach out to me, especially if you want me to modify the version numbers to not have a conflict. Usually, I react in 1 business day. |
Can the test case with autoscaling enabled in the chart test suite be included? |
I added that, but I'm not familar with how to validate the result. Are the results compared to an expected value somewhere or is that more like a smoke test if the values can be rendered at all? |
@sja Thank you. |
No description provided.