Merge pull request #703 from tencentcloudstack/fix/clb-bandwidth-pkg-id

fix: clb - support bandwidth_package_id

Author: Kagashino

tencentcloud/resource_tc_clb_instance.go

@@ -179,6 +179,11 @@ func resourceTencentCloudClbInstance() *schema.Resource {
 				Computed:    true,
 				Description: "Internet charge type, only applicable to open CLB. Valid values are `TRAFFIC_POSTPAID_BY_HOUR`, `BANDWIDTH_POSTPAID_BY_HOUR` and `BANDWIDTH_PACKAGE`.",
 			},
+			"bandwidth_package_id": {
+				Type:        schema.TypeString,
+				Optional:    true,
+				Description: "Bandwidth package id. If set, the `internet_charge_type` must be `BANDWIDTH_PACKAGE`.",
+			},
 			"internet_bandwidth_max_out": {
 				Type:        schema.TypeInt,
 				Optional:    true,
@@ -308,6 +313,9 @@ func resourceTencentCloudClbInstanceCreate(d *schema.ResourceData, meta interfac
 
 	v, ok := d.GetOk("internet_charge_type")
 	bv, bok := d.GetOk("internet_bandwidth_max_out")
+	pv, pok := d.GetOk("bandwidth_package_id")
+
+	chargeType := v.(string)
 
 	//internet charge type
 	if ok || bok {
@@ -316,11 +324,19 @@ func resourceTencentCloudClbInstanceCreate(d *schema.ResourceData, meta interfac
 		}
 		request.InternetAccessible = &clb.InternetAccessible{}
 		if ok {
-			request.InternetAccessible.InternetChargeType = helper.String(v.(string))
+			request.InternetAccessible.InternetChargeType = helper.String(chargeType)
 		}
 		if bok {
 			request.InternetAccessible.InternetMaxBandwidthOut = helper.IntInt64(bv.(int))
 		}
+		if pok {
+			if pok && chargeType != INTERNET_CHARGE_TYPE_BANDWIDTH_PACKAGE {
+				return fmt.Errorf("[CHECK][CLB instance][Create] check: internet_charge_type must `BANDWIDTH_PACKAGE` when bandwidth_package_id was set")
+			}
+			request.BandwidthPackageId = helper.String(pv.(string))
+		} else if chargeType == INTERNET_CHARGE_TYPE_BANDWIDTH_PACKAGE {
+			return fmt.Errorf("[CHECK][CLB instance][Create] check: the `bandwidth_package_id` must be specified if internet_charge_type was `BANDWIDTH_PACKAGE`")
+		}
 	}
 
 	if v, ok := d.GetOk("master_zone_id"); ok {

website/docs/r/clb_instance.html.markdown

@@ -108,6 +108,7 @@ The following arguments are supported:
 * `clb_name` - (Required) Name of the CLB. The name can only contain Chinese characters, English letters, numbers, underscore and hyphen '-'.
 * `network_type` - (Required, ForceNew) Type of CLB instance. Valid values: `OPEN` and `INTERNAL`.
 * `address_ip_version` - (Optional) IP version, only applicable to open CLB. Valid values are `ipv4`, `ipv6` and `IPv6FullChain`.
+* `bandwidth_package_id` - (Optional) Bandwidth package id. If set, the `internet_charge_type` must be `BANDWIDTH_PACKAGE`.
 * `internet_bandwidth_max_out` - (Optional) Max bandwidth out, only applicable to open CLB. Valid value ranges is [1, 2048]. Unit is MB.
 * `internet_charge_type` - (Optional) Internet charge type, only applicable to open CLB. Valid values are `TRAFFIC_POSTPAID_BY_HOUR`, `BANDWIDTH_POSTPAID_BY_HOUR` and `BANDWIDTH_PACKAGE`.
 * `load_balancer_pass_to_target` - (Optional) Whether the target allow flow come from clb. If value is true, only check security group of clb, or check both clb and backend instance security group.