fix: update unit

Author: gitmkn

tencentcloud/resource_tc_vpc_bandwidth_package_attachment.go

@@ -5,7 +5,7 @@ Example Usage
 
 ```hcl
 resource "tencentcloud_vpc_bandwidth_package_attachment" "bandwidth_package_attachment" {
-  resource_ids          = "lb-dv1ai6ma"
+  resource_id           = "lb-dv1ai6ma"
   bandwidth_package_id  = "bwp-atmf0p9g"
   network_type          = "BGP"
   resource_type         = "LoadBalance"
@@ -15,7 +15,7 @@ resource "tencentcloud_vpc_bandwidth_package_attachment" "bandwidth_package_atta
 ```
 Import
 
-vpc bandwidth_package_attachment can be imported using the id, e.g.
+vpc bandwidth_package_attachment can be imported using the bandwidthPackageId#resource_id, e.g.
 ```
 $ terraform import tencentcloud_vpc_bandwidth_package_attachment.bandwidth_package_attachment bandwidthPackageAttachment_id
 ```
@@ -46,30 +46,35 @@ func resourceTencentCloudVpcBandwidthPackageAttachment() *schema.Resource {
 			"resource_id": {
 				Type:        schema.TypeString,
 				Required:    true,
+				ForceNew:    true,
 				Description: "The unique ID of the resource, currently supports EIP resources and LB resources, such as `eip-xxxx`, `lb-xxxx`.",
 			},
 
 			"bandwidth_package_id": {
 				Type:        schema.TypeString,
-				Optional:    true,
+				Required:    true,
+				ForceNew:    true,
 				Description: "Bandwidth package unique ID, in the form of `bwp-xxxx`.",
 			},
 
 			"network_type": {
 				Type:        schema.TypeString,
 				Optional:    true,
+				ForceNew:    true,
 				Description: "Bandwidth packet type, currently supports `BGP` type, indicating that the internal resource is BGP IP.",
 			},
 
 			"resource_type": {
 				Type:        schema.TypeString,
 				Optional:    true,
+				ForceNew:    true,
 				Description: "Resource types, including `Address`, `LoadBalance`.",
 			},
 
 			"protocol": {
 				Type:        schema.TypeString,
 				Optional:    true,
+				ForceNew:    true,
 				Description: "Bandwidth packet protocol type. Currently `ipv4` and `ipv6` protocol types are supported.",
 			},
 		},
@@ -84,7 +89,6 @@ func resourceTencentCloudVpcBandwidthPackageAttachmentCreate(d *schema.ResourceD
 
 	var (
 		request            = vpc.NewAddBandwidthPackageResourcesRequest()
-		response           *vpc.AddBandwidthPackageResourcesResponse
 		bandwidthPackageId string
 		resourceId         string
 	)
@@ -100,17 +104,14 @@ func resourceTencentCloudVpcBandwidthPackageAttachmentCreate(d *schema.ResourceD
 	}
 
 	if v, ok := d.GetOk("network_type"); ok {
-
 		request.NetworkType = helper.String(v.(string))
 	}
 
 	if v, ok := d.GetOk("resource_type"); ok {
-
 		request.ResourceType = helper.String(v.(string))
 	}
 
 	if v, ok := d.GetOk("protocol"); ok {
-
 		request.Protocol = helper.String(v.(string))
 	}
 
@@ -122,7 +123,6 @@ func resourceTencentCloudVpcBandwidthPackageAttachmentCreate(d *schema.ResourceD
 			log.Printf("[DEBUG]%s api[%s] success, request body [%s], response body [%s]\n",
 				logId, request.GetAction(), request.ToJsonString(), result.ToJsonString())
 		}
-		response = result
 		return nil
 	})
 

tencentcloud/resource_tc_vpc_bandwidth_package_attachment_test.go

@@ -25,9 +25,9 @@ func TestAccTencentCloudVpcBandwidthPackageAttachment_basic(t *testing.T) {
 				Config: testAccVpcBandwidthPackageAttachment,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckBandwidthPackageAttachmentExists("tencentcloud_vpc_bandwidth_package_attachment.bandwidthPackageAttachment"),
-					resource.TestCheckResourceAttr("tencentcloud_vpc_bandwidth_package_attachment.bandwidthPackageAttachment", "resource_ids", "lb-5dnrkgry"),
+					resource.TestCheckResourceAttr("tencentcloud_vpc_bandwidth_package_attachment.bandwidthPackageAttachment", "resource_id", "eip-r2l240dq"),
 					resource.TestCheckResourceAttr("tencentcloud_vpc_bandwidth_package_attachment.bandwidthPackageAttachment", "network_type", "BGP"),
-					resource.TestCheckResourceAttr("tencentcloud_vpc_bandwidth_package_attachment.bandwidthPackageAttachment", "resource_type", "LoadBalance"),
+					resource.TestCheckResourceAttr("tencentcloud_vpc_bandwidth_package_attachment.bandwidthPackageAttachment", "resource_type", "Address"),
 				),
 			},
 		},
@@ -112,11 +112,11 @@ resource "tencentcloud_vpc_bandwidth_package" "bandwidth_package" {
   }
 }
 
-resource "tencentcloud_vpc_bandwidth_package_attachment" "bandwidth_package_attachment" {
-  resource_ids          = "lb-5dnrkgry"
+resource "tencentcloud_vpc_bandwidth_package_attachment" "bandwidthPackageAttachment" {
+  resource_id          = "eip-r2l240dq"
   bandwidth_package_id  = tencentcloud_vpc_bandwidth_package.bandwidth_package.id
   network_type          = "BGP"
-  resource_type         = "LoadBalance"
+  resource_type         = "Address"
 }
 
 `

tencentcloud/service_tencentcloud_vpc.go

@@ -5285,6 +5285,12 @@ func (me *VpcService) DeleteVpcBandwidthPackageAttachmentById(ctx context.Contex
 
 	request := vpc.NewRemoveBandwidthPackageResourcesRequest()
 
+	if strings.HasPrefix(resourceId, "eip") {
+		request.ResourceType = helper.String("Address")
+	} else {
+		request.ResourceType = helper.String("LoadBalance")
+	}
+
 	request.BandwidthPackageId = &bandwidthPackageId
 	request.ResourceIds = []*string{&resourceId}
 

website/docs/r/vpc_bandwidth_package_attachment.html.markdown

@@ -15,7 +15,7 @@ Provides a resource to create a vpc bandwidth_package_attachment
 
 ```hcl
 resource "tencentcloud_vpc_bandwidth_package_attachment" "bandwidth_package_attachment" {
-  resource_ids         = "lb-dv1ai6ma"
+  resource_id          = "lb-dv1ai6ma"
   bandwidth_package_id = "bwp-atmf0p9g"
   network_type         = "BGP"
   resource_type        = "LoadBalance"
@@ -27,11 +27,11 @@ resource "tencentcloud_vpc_bandwidth_package_attachment" "bandwidth_package_atta
 
 The following arguments are supported:
 
-* `resource_id` - (Required, String) The unique ID of the resource, currently supports EIP resources and LB resources, such as `eip-xxxx`, `lb-xxxx`.
-* `bandwidth_package_id` - (Optional, String) Bandwidth package unique ID, in the form of `bwp-xxxx`.
-* `network_type` - (Optional, String) Bandwidth packet type, currently supports `BGP` type, indicating that the internal resource is BGP IP.
-* `protocol` - (Optional, String) Bandwidth packet protocol type. Currently `ipv4` and `ipv6` protocol types are supported.
-* `resource_type` - (Optional, String) Resource types, including `Address`, `LoadBalance`.
+* `bandwidth_package_id` - (Required, String, ForceNew) Bandwidth package unique ID, in the form of `bwp-xxxx`.
+* `resource_id` - (Required, String, ForceNew) The unique ID of the resource, currently supports EIP resources and LB resources, such as `eip-xxxx`, `lb-xxxx`.
+* `network_type` - (Optional, String, ForceNew) Bandwidth packet type, currently supports `BGP` type, indicating that the internal resource is BGP IP.
+* `protocol` - (Optional, String, ForceNew) Bandwidth packet protocol type. Currently `ipv4` and `ipv6` protocol types are supported.
+* `resource_type` - (Optional, String, ForceNew) Resource types, including `Address`, `LoadBalance`.
 
 ## Attributes Reference
 
@@ -43,7 +43,7 @@ In addition to all arguments above, the following attributes are exported:
 
 ## Import
 
-vpc bandwidth_package_attachment can be imported using the id, e.g.
+vpc bandwidth_package_attachment can be imported using the bandwidthPackageId#resource_id, e.g.
 ```
 $ terraform import tencentcloud_vpc_bandwidth_package_attachment.bandwidth_package_attachment bandwidthPackageAttachment_id
 ```