feat(bare-metal): Add CRN for BM resource creation and updation

Author: keerthi-gp

ibm/service/vpc/resource_ibm_is_bare_metal_server.go

@@ -1117,26 +1117,23 @@ func ResourceIBMIsBareMetalServer() *schema.Resource {
 						"auto_link": {
 							Type:        schema.TypeBool,
 							Optional:    true,
-							Computed:    true,
 							Description: "If set to true, the system will create a link to the specified target trusted profile during server creation. Regardless of whether a link is created by the system or manually using the IAM Identity service, it will be automatically deleted when the server is deleted.",
 						},
 						"target": {
 							Type:        schema.TypeList,
 							Optional:    true,
-							Computed:    true,
 							Description: "The default IAM trusted profile to use for this bare metal server",
 							Elem: &schema.Resource{
 								Schema: map[string]*schema.Schema{
 									"id": {
-										Type:        schema.TypeString,
-										Optional:    true,
-										Computed:    true,
-										Description: "The unique identifier for this trusted profile",
+										Type:          schema.TypeString,
+										Optional:      true,
+										Description:   "The unique identifier for this trusted profile",
 									},
 									"crn": {
-										Type:        schema.TypeString,
-										Computed:    true,
-										Description: "The CRN for this trusted profile",
+										Type:          schema.TypeString,
+										Optional:      true,
+										Description:   "The CRN for this trusted profile",
 									},
 								},
 							},
@@ -1510,17 +1507,23 @@ func resourceIBMISBareMetalServerCreate(context context.Context, d *schema.Resou
 					defaultTrustedProfilePrototype.AutoLink = &autolink
 				}
 			}
+
 			if targetIntf, ok := defaultTrustedProfileMap["target"]; ok {
-				targetMap := targetIntf.([]interface{})[0].(map[string]interface{})
-				var id *string
-				if idIntf, ok := targetMap["id"]; ok {
-					if idStr, ok := idIntf.(string); ok && idStr != "" {
-						id = &idStr
-					}
-				}
-				if id != nil {
-					defaultTrustedProfilePrototype.Target = &vpcv1.TrustedProfileIdentity{
-						ID:  id,
+				if targetList, ok := targetIntf.([]interface{}); ok && len(targetList) > 0 {
+					if targetMap, ok := targetList[0].(map[string]interface{}); ok {
+						var id, crn *string
+						if crnStr, ok := targetMap["crn"].(string); ok && crnStr != "" {
+							crn = &crnStr
+						} else if idStr, ok := targetMap["id"].(string); ok && idStr != "" {
+							id = &idStr
+						}
+
+						if crn != nil || id != nil {
+							defaultTrustedProfilePrototype.Target = &vpcv1.TrustedProfileIdentity{
+								CRN: crn,
+								ID:  id,
+							}
+						}
 					}
 				}
 			}
@@ -2754,6 +2757,13 @@ func bareMetalServerUpdate(context context.Context, d *schema.ResourceData, meta
 				ID: &newTargetID,
 			}
 		}
+		
+		if d.HasChange("default_trusted_profile.0.target.0.crn") {
+			newTargetCRN := d.Get("default_trusted_profile.0.target.0.crn").(string)
+			defaultTrustedProfile.Target = &vpcv1.TrustedProfileIdentity{
+				CRN: &newTargetCRN,
+			}
+		}
 
 		if defaultTrustedProfile.AutoLink != nil || defaultTrustedProfile.Target != nil {
 			initializationPatch.DefaultTrustedProfile = defaultTrustedProfile

ibm/service/vpc/resource_ibm_is_bare_metal_server_initialization.go

@@ -64,6 +64,7 @@ func ResourceIBMIsBareMetalServerInitialization() *schema.Resource {
 									},
 									"crn": {
 										Type:        schema.TypeString,
+										Optional:    true,
 										Computed:    true,
 										Description: "The CRN for this trusted profile",
 									},
@@ -156,16 +157,21 @@ func resourceIBMISBareMetalServerInitializationCreate(context context.Context, d
 		}
 
 		if targetIntf, ok := defaultTrustedProfileMap["target"]; ok {
-			targetMap := targetIntf.([]interface{})[0].(map[string]interface{})
-			var id *string
-			if idIntf, ok := targetMap["id"]; ok {
-				if idStr, ok := idIntf.(string); ok && idStr != "" {
-					id = &idStr
-				}
-			}
-			if id != nil {
-				defaultTrustedProfilePrototype.Target = &vpcv1.TrustedProfileIdentity{
-					ID: id,
+			if targetList, ok := targetIntf.([]interface{}); ok && len(targetList) > 0 {
+				if targetMap, ok := targetList[0].(map[string]interface{}); ok {
+					var id, crn *string
+					if crnStr, ok := targetMap["crn"].(string); ok && crnStr != "" {
+						crn = &crnStr
+					} else if idStr, ok := targetMap["id"].(string); ok && idStr != "" {
+						id = &idStr
+					}
+
+					if crn != nil || id != nil {
+						defaultTrustedProfilePrototype.Target = &vpcv1.TrustedProfileIdentity{
+							CRN: crn,
+							ID:  id,
+						}
+					}
 				}
 			}
 		}

website/docs/r/is_bare_metal_server.markdown

@@ -240,6 +240,7 @@ Review the argument references that you can specify for your resource.
   - `target` - (List) The default IAM trusted profile to use for this bare metal server.
      Nested scheme for `target`: 
      - `id` - (String) The unique identifier for this trusted profile
+     - `crn`- (String) The CRN for this trusted profile
 - `delete_type` - (Optional, String) Type of deletion on destroy. **soft** signals running operating system to quiesce and shutdown cleanly, **hard** immediately stop the server. By default its `hard`.
 - `enable_secure_boot` - (Optional, Boolean) Indicates whether secure boot is enabled. If enabled, the image must support secure boot or the server will fail to boot. Updating `enable_secure_boot` requires the server to be stopped and then it would be started.
 - `health_reasons` - (List) The reasons for the current health_state (if any).

website/docs/r/is_bare_metal_server_initialization.markdown

@@ -62,6 +62,7 @@ Review the argument references that you can specify for your resource.
   - `target` - (List) The default IAM trusted profile to use for this bare metal server.
      Nested scheme for `target`: 
      - `id` - (String) The unique identifier for this trusted profile
+     - `crn`- (String) The CRN for this trusted profile
 - `image` - (Required, String) Image id to use to reinitialize the bare metal server. 
 - `keys` - (Required, Array) Keys ids to use to reinitialize the bare metal server. 
 - `user_data` - (Optional, String) User data to transfer to the server bare metal server. If unspecified, no user data will be made available. (For reload/reinitialize provide the same user data as at the time of provisioning)