add set groups (#529)

sysdiglabs · Jun 26, 2024 · b73ca2e · b73ca2e
1 parent 9cdac5a
commit b73ca2e
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 8 deletions.
diff --git a/sysdig/data_source_sysdig_secure_posture_policy.go b/sysdig/data_source_sysdig_secure_posture_policy.go
@@ -75,10 +75,8 @@ func dataSourceSysdigSecurePosturePolicyRead(ctx context.Context, d *schema.Reso
 	if err != nil {
 		return diag.FromErr(err)
 	}
-	err = d.Set(SchemaIDKey, policy.ID)
-	if err != nil {
-		return diag.FromErr(err)
-	}
+
+	d.SetId(policy.ID)
 
 	err = d.Set(SchemaNameKey, policy.Name)
 	if err != nil {
@@ -120,11 +118,14 @@ func dataSourceSysdigSecurePosturePolicyRead(ctx context.Context, d *schema.Reso
 		return diag.FromErr(err)
 	}
 
-	// Set groups
-	if _, err := setGroups(d, policy.RequirementsGroup); err != nil {
+	groupsData, err := setGroups(d, policy.RequirementsGroup)
+	if err != nil {
+		return diag.FromErr(err)
+	}
+	err = d.Set(SchemaGroupKey, groupsData)
+	if err != nil {
 		return diag.FromErr(err)
 	}
 
-	d.SetId(policy.ID)
 	return nil
 }
diff --git a/sysdig/resource_sysdig_secure_posture_policy.go b/sysdig/resource_sysdig_secure_posture_policy.go
@@ -280,9 +280,16 @@ func resourceSysdigSecurePosturePolicyRead(ctx context.Context, d *schema.Resour
 	}
 
 	// Set groups
-	if _, err := setGroups(d, policy.RequirementsGroup); err != nil {
+	groupsData, err := setGroups(d, policy.RequirementsGroup)
+	if err != nil {
+		return diag.FromErr(err)
+	}
+
+	err = d.Set(SchemaGroupKey, groupsData)
+	if err != nil {
 		return diag.FromErr(err)
 	}
+
 	return nil
 }