Skip to content

Commit a4ec51a

Browse files
committed
Added a fix to avoid diff on cluster nw
1 parent d8f03a3 commit a4ec51a

3 files changed

+8
-7
lines changed

ibm/service/vpc/data_source_ibm_is_instance.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -1344,16 +1344,16 @@ func instanceGetByName(d *schema.ResourceData, meta interface{}, name string) er
13441344

13451345
// cluster changes
13461346

1347+
clusterNetwork := []map[string]interface{}{}
13471348
if !core.IsNil(instance.ClusterNetwork) {
1348-
clusterNetwork := []map[string]interface{}{}
13491349
clusterNetworkMap, err := DataSourceIBMIsInstanceClusterNetworkReferenceToMap(instance.ClusterNetwork)
13501350
if err != nil {
13511351
return flex.DiscriminatedTerraformErrorf(err, err.Error(), "(Data) ibm_is_instance", "read", "cluster_network-to-map")
13521352
}
13531353
clusterNetwork = append(clusterNetwork, clusterNetworkMap)
1354-
if err = d.Set("cluster_network", clusterNetwork); err != nil {
1355-
return flex.DiscriminatedTerraformErrorf(err, fmt.Sprintf("Error setting cluster_network: %s", err), "(Data) ibm_is_instance", "read", "set-cluster_network")
1356-
}
1354+
}
1355+
if err = d.Set("cluster_network", clusterNetwork); err != nil {
1356+
return flex.DiscriminatedTerraformErrorf(err, fmt.Sprintf("Error setting cluster_network: %s", err), "(Data) ibm_is_instance", "read", "set-cluster_network")
13571357
}
13581358

13591359
clusterNetworkAttachments := []map[string]interface{}{}

ibm/service/vpc/data_source_ibm_is_instances.go

+3-2
Original file line numberDiff line numberDiff line change
@@ -1523,14 +1523,15 @@ func instancesList(d *schema.ResourceData, meta interface{}) error {
15231523
catalogList = append(catalogList, catalogMap)
15241524
l[isInstanceCatalogOffering] = catalogList
15251525
}
1526-
1526+
clusterNetwork := []map[string]interface{}{}
15271527
if instance.ClusterNetwork != nil {
15281528
clusterNetworkMap, err := DataSourceIBMIsInstancesClusterNetworkReferenceToMap(instance.ClusterNetwork)
15291529
if err != nil {
15301530
return err
15311531
}
1532-
l["cluster_network"] = []map[string]interface{}{clusterNetworkMap}
1532+
clusterNetwork = append(clusterNetwork, clusterNetworkMap)
15331533
}
1534+
l["cluster_network"] = clusterNetwork
15341535
clusterNetworkAttachments := []map[string]interface{}{}
15351536
for _, clusterNetworkAttachmentsItem := range instance.ClusterNetworkAttachments {
15361537
clusterNetworkAttachmentsItemMap, err := DataSourceIBMIsInstancesInstanceClusterNetworkAttachmentReferenceToMap(&clusterNetworkAttachmentsItem) // #nosec G601

ibm/service/vpc/resource_ibm_is_instance.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -4920,7 +4920,7 @@ func instanceGet(d *schema.ResourceData, meta interface{}, id string) error {
49204920
if err != nil {
49214921
return flex.DiscriminatedTerraformErrorf(err, err.Error(), "ibm_is_instance", "read", "cluster_network-to-map")
49224922
}
4923-
clusterNetwork = []map[string]interface{}{clusterNetworkMap}
4923+
clusterNetwork = append(clusterNetwork, clusterNetworkMap)
49244924
}
49254925
if err = d.Set("cluster_network", clusterNetwork); err != nil {
49264926
err = fmt.Errorf("Error setting cluster_network: %s", err)

0 commit comments

Comments
 (0)