fixed issue where a failed creation on container_node_pool would result in an unrecoverable tainted state (#14937) (#24077)

modular-magician · web-flow · commit f40baad117b8 · 2025-08-21T14:14:26.000-07:00
[upstream:4efcbb9201946c26b761b2f8ddf0da98ddb10aab]

Signed-off-by: Modular Magician &lt;magic-modules@google.com&gt;
diff --git a/.changelog/14937.txt b/.changelog/14937.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+container: fixed issue where a failed creation on `google_container_node_pool` would result in an unrecoverable tainted state
+```
diff --git a/google/services/container/resource_container_node_pool.go b/google/services/container/resource_container_node_pool.go
@@ -638,6 +638,7 @@ func resourceContainerNodePoolCreate(d *schema.ResourceData, meta interface{}) e
 		return nil
 	})
 	if err != nil {
+		d.SetId("")
 		return fmt.Errorf("error creating NodePool: %s", err)
 	}
 	timeout -= time.Since(startTime)

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	+```release-note:bug
	`2`	+container: fixed issue where a failed creation on `google_container_node_pool` would result in an unrecoverable tainted state
	`3`	+```
Original file line number	Diff line number	Diff line change
`@@ -638,6 +638,7 @@ func resourceContainerNodePoolCreate(d *schema.ResourceData, meta interface{}) e`
`638`	`638`	`return nil`
`639`	`639`	`})`
`640`	`640`	`if err != nil {`
	`641`	`+ d.SetId("")`
`641`	`642`	`return fmt.Errorf("error creating NodePool: %s", err)`
`642`	`643`	`}`
`643`	`644`	`timeout -= time.Since(startTime)`