Add mutex to google_tags_tag_binding (#5878) (#4191)

Co-authored-by: Stephen Lewis (Burrows) <[email protected]>
Signed-off-by: Modular Magician <[email protected]>

Co-authored-by: Stephen Lewis (Burrows) <[email protected]>

Author: modular-magician

.changelog/5878.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+tags: fixed issue where tags could not be applied sequentially to the same parent in `google_tags_tag_binding`
+```

google-beta/resource_tags_tag_binding.go

@@ -83,6 +83,13 @@ func resourceTagsTagBindingCreate(d *schema.ResourceData, meta interface{}) erro
 		obj["tagValue"] = tagValueProp
 	}
 
+	lockName, err := replaceVars(d, config, "tagBindings/{{parent}}")
+	if err != nil {
+		return err
+	}
+	mutexKV.Lock(lockName)
+	defer mutexKV.Unlock(lockName)
+
 	url, err := replaceVars(d, config, "{{TagsBasePath}}tagBindings")
 	if err != nil {
 		return err
@@ -204,6 +211,13 @@ func resourceTagsTagBindingDelete(d *schema.ResourceData, meta interface{}) erro
 
 	billingProject := ""
 
+	lockName, err := replaceVars(d, config, "tagBindings/{{parent}}")
+	if err != nil {
+		return err
+	}
+	mutexKV.Lock(lockName)
+	defer mutexKV.Unlock(lockName)
+
 	url, err := replaceVars(d, config, "{{TagsBasePath}}tagBindings/{{name}}")
 	if err != nil {
 		return err