fix(onboarding): org dependency (#43)

cgeers · web-flow · commit 50bd7eb5829a · 2024-09-27T12:07:58.000-05:00
The implicit dependency organizations have on management accounts sets the apply/creation order correctly.
However, the destroy order is not unwound in the reverse order.  Add explicit dependency to enforce
this behavior.
diff --git a/modules/onboarding/organizational.tf b/modules/onboarding/organizational.tf
@@ -31,5 +31,8 @@ resource "sysdig_secure_organization" "google_organization" {
 
   management_account_id   = sysdig_secure_cloud_auth_account.google_account.id
   organizational_unit_ids = var.management_group_ids
-  depends_on              = [google_organization_iam_member.browser]
+  depends_on = [
+    google_organization_iam_member.browser,
+    sysdig_secure_cloud_auth_account.google_account
+  ]
 }

Original file line number	Diff line number	Diff line change
`@@ -31,5 +31,8 @@ resource "sysdig_secure_organization" "google_organization" {`
`31`	`31`
`32`	`32`	`management_account_id = sysdig_secure_cloud_auth_account.google_account.id`
`33`	`33`	`organizational_unit_ids = var.management_group_ids`
`34`		`- depends_on = [google_organization_iam_member.browser]`
	`34`	`+ depends_on = [`
	`35`	`+ google_organization_iam_member.browser,`
	`36`	`+ sysdig_secure_cloud_auth_account.google_account`
	`37`	`+ ]`
`35`	`38`	`}`