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

Mask GKE Sandbox-specific labels and taints #2320

Conversation

modular-magician
Copy link
Collaborator

Hi friends,

This change prevents the GKE Sandbox labels and taints (currently sandbox.gke.io/runtime=gvisor) from causing node pools to always need to be recreated. Without this change, once a node pool is created, it needs to be manually updated to include the additional GKE Sandbox label and taint items.

I believe this PR fixes hashicorp/terraform-provider-google#4210.

The addition of the DiffSuppressFunc on a TypeList is a little weird (not sure if it's using prescribed functionality or not; I see hashicorp/terraform-plugin-sdk#477 but it seems to work here). I'd be happy to take any suggestions on how to improve it.

Once I get an OK on the general approach, I will add a few tests as well. Thanks for taking a look!

Release Note Template for Downstream PRs (will be copied)

compute: Masked automatically applied GKE Sandbox node labels and taints on node pools

Derived from GoogleCloudPlatform/magic-modules#3749

@ghost ghost added the size/l label Jul 31, 2020
@modular-magician modular-magician merged commit 6759b0a into hashicorp:master Jul 31, 2020
@modular-magician modular-magician deleted the downstream-pr-de0746ebfdfe227127e1d8cdc3bdbcf71ee68d1a branch November 16, 2024 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GKE Container Node Pool with sandbox type of gvisor missing label "sandbox.gke.io/runtime" = "gvisor"
1 participant