Merge pull request #92 from jetstack/fix_empty_user_matching_rules

aidy · web-flow · commit a76c995b9537 · 2025-10-21T09:25:08.000+01:00
Fix noop team resource updates for empty user matching rules
diff --git a/internal/provider/team_resource.go b/internal/provider/team_resource.go
@@ -211,7 +211,9 @@ func (r *teamResource) Read(ctx context.Context, req resource.ReadRequest, resp
 		})
 	}
 
-	state.UserMatchingRules = umr
+	if len(umr) > 0 {
+		state.UserMatchingRules = umr
+	}
 
 	diags = resp.State.Set(ctx, state)
 	resp.Diagnostics.Append(diags...)

Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,9 @@ func (r *teamResource) Read(ctx context.Context, req resource.ReadRequest, resp`
`211`	`211`	`})`
`212`	`212`	`}`
`213`	`213`
`214`		`- state.UserMatchingRules = umr`
	`214`	`+ if len(umr) > 0 {`
	`215`	`+ state.UserMatchingRules = umr`
	`216`	`+ }`
`215`	`217`
`216`	`218`	`diags = resp.State.Set(ctx, state)`
`217`	`219`	`resp.Diagnostics.Append(diags...)`