Morgante Pell

Were you not getting this error before?

Yes, the idea is to be able to have a workload defined once but deployed using a mix of preemptible and regular VMs. Agreed that we can add more value/usage...

I'm of the opinion that we explicitly should discourage manually setting a version when using release channels. The intent of release channels is to allow Google to manage the cluster...

Just to be clear, is your desire to upgrade prior to when GKE does it on your behalf? If you want this level of control, I'm wondering why use release...

I do indeed recommend using the ASM submodule instead of enabling Istio. In fact, we'll be removing the istio option soon. In Q4, we are hoping to refactor the ASM...

> It would be good to add some validation rules to these two vars to force the requirement instead of failing pathologically. Great idea, would be happy to review a...

Yeah this looks like a bug. We would need to update [this resource](https://github.com/terraform-google-modules/terraform-google-lb-http/blob/master/main.tf#L218-L237) to use HC ports.

Thanks for the suggestion, these both sound like reasonable additions/fixes.

Permadiffs should be addressed at the provider. Please open a bug here: https://github.com/hashicorp/terraform-provider-google/issues

Oh sorry, I missed the deprecation warning. Yes we should fix that. > Oddly, a plan with the same provider version didn't seem to exhibit this issue a couple of...