Add conflicts with, node_version only works on the default node pool

[slevenick]

Fixes: hashicorp/terraform-provider-google#9021

If this PR is for Terraform, I acknowledge that I have:

• Searched through the issue tracker for an open issue that this either resolves or contributes to, commented on it to claim it, and written "fixes {url}" or "part of {url}" in this PR description. If there were no relevant open issues, I opened one and commented that I would like to work on it (not necessary for very small changes).
• Generated Terraform, and ran make test and make lint to ensure it passes unit and linter tests.
• Ensured that all new fields I added that can be set by a user appear in at least one example (for generated resources) or third_party test (for handwritten resources or update tests).
• Ran relevant acceptance tests (If the acceptance tests do not yet pass or you are unable to run them, please let your reviewer know).
• Read the Release Notes Guide before writing my release note below.

Release Note Template for Downstream PRs (will be copied)

container: added validation to check that both `node_version` and `remove_default_node_pool` cannot be set on `google_container_cluster`

[modular-magician]

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 1 file changed, 1 insertion(+), 1 deletion(-))
Terraform Beta: Diff ( 2 files changed, 2 insertions(+), 2 deletions(-))

[modular-magician]

I have triggered VCR tests based on this PR's diffs. See the results here: "https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=185689"

[modular-magician]

I have triggered VCR tests in RECORDING mode for the following tests that failed during VCR: TestAccComputeBackendService_internalLoadBalancing You can view the result here: "https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=185788"

[rileykarson]

I don't think we can do this in a minor version because remove_default_node_pool can be set to false, in which case setting node_version is valid.

[slevenick]

I don't think we can do this in a minor version because remove_default_node_pool can be set to false, in which case setting node_version is valid.

Ah right. It would work as a customize diff that checks for true and a node_version set right?

[rileykarson]

Yep!

…

On Wed, May 5, 2021 at 9:57 AM Sam Levenick ***@***.***> wrote: I don't think we can do this in a minor version because remove_default_node_pool can be set to false, in which case setting node_version is valid. Ah right. It would work as a customize diff that checks for true and a node_version set right? — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub <#4762 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAP36CC2F5GHB6AVECFU2EDTMF2IRANCNFSM44DSJTJQ> .

[modular-magician]

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 1 file changed, 9 insertions(+))
Terraform Beta: Diff ( 1 file changed, 9 insertions(+))

[modular-magician]

I have triggered VCR tests based on this PR's diffs. See the results here: "https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=185861"