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

Cherry-pick: add extended resource support in GCE #64

Conversation

zaymat
Copy link

@zaymat zaymat commented Oct 24, 2022

Upstream PR: kubernetes#5247

@zaymat zaymat changed the title Mayeul/cherry pick/add extended resource support in gce Cherry-pick: add extended resource support in GCE Oct 24, 2022
This commit adds the possibility to define extended resources for a node group on GCE,
so that the cluster-autoscaler can account for them when taking scaling decisions.

This is done through the `extended_resources` key inside the AUTOSCALER_ENV_VARS variable set on a MIG template.

Signed-off-by: Mayeul Blanzat <[email protected]>
…add more tests

* Malformed extended resource definition should not fail the template building function. Instead, log the error and ignore extended resources
* Remove useless existence check
* Add tests around the extractExtendedResourcesFromKubeEnv function
* Add a test case to verify that malformed extended resource definition does not fail the template build function

Signed-off-by: Mayeul Blanzat <[email protected]>
@zaymat zaymat force-pushed the mayeul/cherry-pick/add-extended-resource-support-in-gce branch from e4e5cc8 to 433cce6 Compare October 25, 2022 09:57
@zaymat zaymat merged commit b447a03 into datadog-master-8.0 Oct 25, 2022
@zaymat zaymat deleted the mayeul/cherry-pick/add-extended-resource-support-in-gce branch October 25, 2022 12:35
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