diff --git a/opentelekomcloud/services/vpc/data_source_opentelekomcloud_networking_secgroup_v2.go b/opentelekomcloud/services/vpc/data_source_opentelekomcloud_networking_secgroup_v2.go index 1de5feba6..b0112357a 100644 --- a/opentelekomcloud/services/vpc/data_source_opentelekomcloud_networking_secgroup_v2.go +++ b/opentelekomcloud/services/vpc/data_source_opentelekomcloud_networking_secgroup_v2.go @@ -4,6 +4,7 @@ import ( "context" "log" + "github.com/hashicorp/go-multierror" "github.com/opentelekomcloud/gophertelekomcloud/openstack/networking/v2/extensions/security/groups" "github.com/hashicorp/terraform-plugin-sdk/v2/diag" @@ -38,6 +39,10 @@ func DataSourceNetworkingSecGroupV2() *schema.Resource { ForceNew: true, Computed: true, }, + "description": { + Type: schema.TypeString, + Computed: true, + }, }, } } @@ -75,10 +80,15 @@ func dataSourceNetworkingSecGroupV2Read(_ context.Context, d *schema.ResourceDat log.Printf("[DEBUG] Retrieved Security Group %s: %+v", secGroup.ID, secGroup) d.SetId(secGroup.ID) - d.Set("name", secGroup.Name) - d.Set("description", secGroup.Description) - d.Set("tenant_id", secGroup.TenantID) - d.Set("region", config.GetRegion(d)) + mErr := multierror.Append( + d.Set("name", secGroup.Name), + d.Set("description", secGroup.Description), + d.Set("tenant_id", secGroup.TenantID), + d.Set("region", config.GetRegion(d)), + ) + if err := mErr.ErrorOrNil(); err != nil { + return fmterr.Errorf("error setting security group fields: %w", err) + } return nil } diff --git a/releasenotes/notes/fix-secgroup-23e41d4ac82f0f34.yaml b/releasenotes/notes/fix-secgroup-23e41d4ac82f0f34.yaml new file mode 100644 index 000000000..ab0ebe2a2 --- /dev/null +++ b/releasenotes/notes/fix-secgroup-23e41d4ac82f0f34.yaml @@ -0,0 +1,4 @@ +--- +fixes: + - | + Fix panic when using `opentelekomcloud_networking_secgroup_v2` data source ([#1176](https://github.com/opentelekomcloud/terraform-provider-opentelekomcloud/pull/1176))