fix/clb_projectId

Author: SevenEarth

tencentcloud/services/clb/resource_tc_clb_instance.go

@@ -52,7 +52,6 @@ func ResourceTencentCloudClbInstance() *schema.Resource {
 			"project_id": {
 				Type:        schema.TypeInt,
 				Optional:    true,
-				ForceNew:    true,
 				Default:     0,
 				Description: "ID of the project within the CLB instance, `0` - Default Project.",
 			},
@@ -280,7 +279,7 @@ func resourceTencentCloudClbInstanceCreate(d *schema.ResourceData, meta interfac
 		request.VpcId = helper.String(v.(string))
 	}
 
-	if v, ok := d.GetOk("project_id"); ok {
+	if v, ok := d.GetOkExists("project_id"); ok {
 		projectId := int64(v.(int))
 		request.ProjectId = &projectId
 	}
@@ -317,7 +316,7 @@ func resourceTencentCloudClbInstanceCreate(d *schema.ResourceData, meta interfac
 		request.AddressIPVersion = helper.String(v.(string))
 	}
 
-	if v, ok := d.GetOk("snat_pro"); ok {
+	if v, ok := d.GetOkExists("snat_pro"); ok {
 		request.SnatPro = helper.Bool(v.(bool))
 	}
 
@@ -391,7 +390,7 @@ func resourceTencentCloudClbInstanceCreate(d *schema.ResourceData, meta interfac
 		request.SlaveZoneId = helper.String(v.(string))
 	}
 
-	if v, ok := d.GetOk("load_balancer_pass_to_target"); ok {
+	if v, ok := d.GetOkExists("load_balancer_pass_to_target"); ok {
 		request.LoadBalancerPassToTarget = helper.Bool(v.(bool))
 	}
 
@@ -688,7 +687,7 @@ func resourceTencentCloudClbInstanceUpdate(d *schema.ResourceData, meta interfac
 		clbId = d.Id()
 	)
 
-	immutableArgs := []string{"snat_ips", "dynamic_vip", "master_zone_id", "slave_zone_id", "project_id", "vpc_id", "subnet_id", "address_ip_version", "bandwidth_package_id", "zone_id"}
+	immutableArgs := []string{"snat_ips", "dynamic_vip", "master_zone_id", "slave_zone_id", "vpc_id", "subnet_id", "address_ip_version", "bandwidth_package_id", "zone_id"}
 
 	for _, v := range immutableArgs {
 		if d.HasChange(v) {
@@ -894,6 +893,38 @@ func resourceTencentCloudClbInstanceUpdate(d *schema.ResourceData, meta interfac
 		}
 	}
 
+	if d.HasChange("project_id") {
+		var projectId int
+		if v, ok := d.GetOkExists("project_id"); ok {
+			projectId = v.(int)
+		}
+
+		pRequest := clb.NewModifyLoadBalancersProjectRequest()
+		pRequest.LoadBalancerIds = []*string{&clbId}
+		pRequest.ProjectId = helper.IntUint64(projectId)
+		err := resource.Retry(tccommon.WriteRetryTimeout, func() *resource.RetryError {
+			pResponse, e := meta.(tccommon.ProviderMeta).GetAPIV3Conn().UseClbClient().ModifyLoadBalancersProject(pRequest)
+			if e != nil {
+				return tccommon.RetryError(e)
+			} else {
+				log.Printf("[DEBUG]%s api[%s] success, request body [%s], response body [%s]\n",
+					logId, pRequest.GetAction(), pRequest.ToJsonString(), pResponse.ToJsonString())
+				requestId := *pResponse.Response.RequestId
+				retryErr := waitForTaskFinish(requestId, meta.(tccommon.ProviderMeta).GetAPIV3Conn().UseClbClient())
+				if retryErr != nil {
+					return tccommon.RetryError(errors.WithStack(retryErr))
+				}
+			}
+
+			return nil
+		})
+
+		if err != nil {
+			log.Printf("[CRITAL]%s update CLB instance project_id failed, reason:%+v", logId, err)
+			return err
+		}
+	}
+
 	if d.HasChange("tags") {
 		oldValue, newValue := d.GetChange("tags")
 		replaceTags, deleteTags := svctag.DiffTags(oldValue.(map[string]interface{}), newValue.(map[string]interface{}))

website/docs/r/clb_instance.html.markdown

@@ -348,7 +348,7 @@ The following arguments are supported:
 * `log_set_id` - (Optional, String) The id of log set.
 * `log_topic_id` - (Optional, String) The id of log topic.
 * `master_zone_id` - (Optional, String) Setting master zone id of cross available zone disaster recovery, only applicable to open CLB.
-* `project_id` - (Optional, Int, ForceNew) ID of the project within the CLB instance, `0` - Default Project.
+* `project_id` - (Optional, Int) ID of the project within the CLB instance, `0` - Default Project.
 * `security_groups` - (Optional, List: [`String`]) Security groups of the CLB instance. Supports both `OPEN` and `INTERNAL` CLBs.
 * `sla_type` - (Optional, String) This parameter is required to create LCU-supported instances. Values:`SLA`: Super Large 4. When you have activated Super Large models, `SLA` refers to Super Large 4; `clb.c2.medium`: Standard; `clb.c3.small`: Advanced 1; `clb.c3.medium`: Advanced 1; `clb.c4.small`: Super Large 1; `clb.c4.medium`: Super Large 2; `clb.c4.large`: Super Large 3; `clb.c4.xlarge`: Super Large 4. For more details, see [Instance Specifications](https://intl.cloud.tencent.com/document/product/214/84689?from_cn_redirect=1).
 * `slave_zone_id` - (Optional, String) Setting slave zone id of cross available zone disaster recovery, only applicable to open CLB. this zone will undertake traffic when the master is down.