Fix regional cluster example #2385
Conversation
Update regional cluster example to point cluster to google_container_cluster.regional.name rather than google_container_cluster.primary.name
|
This isn't a problem for me, but can you explain the difference? |
|
It’s just an incorrect reference. primary is defined in the example above for non-regional and regional is defined in this example. If you copied and pasted the code and ran “terraform plan” it would error that primary is not defined or worse if you copied all the examples, you’d be referencing the non-regional cluster when you thought you were referencing the regional one. |
|
Oh shoot! Thanks. :) You're right! |
Update regional cluster example to point cluster to google_container_cluster.regional.name rather than google_container_cluster.primary.name
This reverts commit 065128e.
Update regional cluster example to point cluster to google_container_cluster.regional.name rather than google_container_cluster.primary.name
This reverts commit 065128e.
Update regional cluster example to point cluster to google_container_cluster.regional.name rather than google_container_cluster.primary.name
This reverts commit 065128e.
Update regional cluster example to point cluster to google_container_cluster.regional.name rather than google_container_cluster.primary.name
Update regional cluster example to point cluster to google_container_cluster.regional.name rather than google_container_cluster.primary.name
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
ghost
locked and limited conversation to collaborators
Update regional cluster example to point cluster to google_container_cluster.regional.name rather than google_container_cluster.primary.name