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

Extend cluster_autoscaling fields in gke-cluster-standard #1845

Conversation

anthonyhaussman
Copy link
Contributor

@anthonyhaussman anthonyhaussman commented Nov 7, 2023

feat(gke-cluster-standard): Add feature to setup cluster_autoscaling

  • Set the autoscaling_profile if OPTIMIZE_UTILIZATION is needed
  • Add the possibility to set disk_size and disk_type
  • Add configuration of management and shielded_instance_config

Each of these options is optional and set to their default value if not set.

feat(gke-cluster-standard): Add GPUs setup feature for cluster_autoscaler

Add the possibility to setup gpu_resources into node auto-provisioning.


Checklist

I applicable, I acknowledge that I have:

  • Read the contributing guide
  • Ran terraform fmt on all modified files
  • Regenerated the relevant README.md files using tools/tfdoc.py
  • Made sure all relevant tests pass

@anthonyhaussman anthonyhaussman force-pushed the feat/tone/gke-cluster-standard_cluster_autoscaling_features branch from 069b090 to 9eca7a6 Compare November 7, 2023 16:55
@anthonyhaussman anthonyhaussman marked this pull request as ready for review November 8, 2023 07:17
@anthonyhaussman anthonyhaussman force-pushed the feat/tone/gke-cluster-standard_cluster_autoscaling_features branch from de7b7f3 to 040f309 Compare November 8, 2023 08:35
modules/gke-cluster-standard/variables.tf Outdated Show resolved Hide resolved
modules/gke-cluster-standard/variables.tf Show resolved Hide resolved
@juliocc juliocc changed the title feat(gke-cluster-standard): Add feature to setup cluster_autoscaling Extend cluster_autoscaling fields in gke-cluster-standard Nov 9, 2023
@anthonyhaussman anthonyhaussman force-pushed the feat/tone/gke-cluster-standard_cluster_autoscaling_features branch 2 times, most recently from e2c030d to 70fcfb7 Compare November 9, 2023 14:19
@juliocc juliocc enabled auto-merge (squash) November 9, 2023 15:25
@juliocc
Copy link
Collaborator

juliocc commented Nov 10, 2023

@anthonyhaussman ;can you fix the failing tests?

auto-merge was automatically disabled November 10, 2023 08:19

Head branch was pushed to by a user without write access

@anthonyhaussman anthonyhaussman force-pushed the feat/tone/gke-cluster-standard_cluster_autoscaling_features branch from 70fcfb7 to a80fe68 Compare November 10, 2023 08:19
@anthonyhaussman
Copy link
Contributor Author

@anthonyhaussman ;can you fix the failing tests?

@juliocc It should be good now 🙏

@anthonyhaussman anthonyhaussman force-pushed the feat/tone/gke-cluster-standard_cluster_autoscaling_features branch from a80fe68 to 7b006d6 Compare November 10, 2023 08:39
@juliocc juliocc enabled auto-merge (squash) November 10, 2023 08:40
auto-merge was automatically disabled November 10, 2023 10:45

Head branch was pushed to by a user without write access

@anthonyhaussman anthonyhaussman force-pushed the feat/tone/gke-cluster-standard_cluster_autoscaling_features branch from 7b006d6 to e74e0ca Compare November 10, 2023 10:45
@anthonyhaussman
Copy link
Contributor Author

@juliocc I had some trouble finding the problem and reproducing it.
But I have found the problem and pushed the fix in e74e0ca.

Sorry for the trouble. 🙏

@juliocc juliocc merged commit 0f446e8 into GoogleCloudPlatform:master Nov 10, 2023
9 checks passed
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

2 participants