added a nil check on data source of bm servers

IBM-Cloud · Jan 19, 2024 · a537bb8 · a537bb8
1 parent 740cb68
commit a537bb8
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/ibm/service/vpc/data_source_ibm_is_bare_metal_server.go b/ibm/service/vpc/data_source_ibm_is_bare_metal_server.go
@@ -470,9 +470,11 @@ func dataSourceIBMISBareMetalServerRead(context context.Context, d *schema.Resou
 
 	d.SetId(*bms.ID)
 	d.Set(isBareMetalServerBandwidth, bms.Bandwidth)
-	bmsBootTargetIntf := bms.BootTarget.(*vpcv1.BareMetalServerBootTarget)
-	bmsBootTarget := bmsBootTargetIntf.ID
-	d.Set(isBareMetalServerBootTarget, bmsBootTarget)
+	if bms.BootTarget != nil {
+		bmsBootTargetIntf := bms.BootTarget.(*vpcv1.BareMetalServerBootTarget)
+		bmsBootTarget := bmsBootTargetIntf.ID
+		d.Set(isBareMetalServerBootTarget, bmsBootTarget)
+	}
 
 	// set keys and image using initialization
 

diff --git a/ibm/service/vpc/data_source_ibm_is_bare_metal_servers.go b/ibm/service/vpc/data_source_ibm_is_bare_metal_servers.go
@@ -521,9 +521,11 @@ func dataSourceIBMISBareMetalServersRead(context context.Context, d *schema.Reso
 		}
 		l["id"] = *bms.ID
 		l[isBareMetalServerBandwidth] = *bms.Bandwidth
-		bmsBootTargetIntf := bms.BootTarget.(*vpcv1.BareMetalServerBootTarget)
-		bmsBootTarget := bmsBootTargetIntf.ID
-		l[isBareMetalServerBootTarget] = bmsBootTarget
+		if bms.BootTarget != nil {
+			bmsBootTargetIntf := bms.BootTarget.(*vpcv1.BareMetalServerBootTarget)
+			bmsBootTarget := bmsBootTargetIntf.ID
+			l[isBareMetalServerBootTarget] = bmsBootTarget
+		}
 		cpuList := make([]map[string]interface{}, 0)
 		if bms.Cpu != nil {
 			currentCPU := map[string]interface{}{}