change zone to optional (#2080)

* change zone to optional

* update doc

* update doc

* update test

Author: hellertang

.changelog/2080.txt

@@ -0,0 +1,3 @@
+```release-note:enhancement
+resource/tencentcloud_vpn_gateway: change `zone` to `optional`
+```

tencentcloud/resource_tc_vpn_gateway.go

@@ -205,8 +205,9 @@ func resourceTencentCloudVpnGateway() *schema.Resource {
 			},
 			"zone": {
 				Type:        schema.TypeString,
-				Required:    true,
+				Optional:    true,
 				ForceNew:    true,
+				Computed:    true,
 				Description: "Zone of the VPN gateway.",
 			},
 			"tags": {
@@ -234,8 +235,12 @@ func resourceTencentCloudVpnGatewayCreate(d *schema.ResourceData, meta interface
 	bandwidth := d.Get("bandwidth").(int)
 	bandwidth64 := uint64(bandwidth)
 	request.InternetMaxBandwidthOut = &bandwidth64
-	request.Zone = helper.String(d.Get("zone").(string))
 	chargeType := d.Get("charge_type").(string)
+
+	if v, ok := d.GetOk("zone"); ok {
+		request.Zone = helper.String(v.(string))
+	}
+
 	//only support change renew_flag when charge type is pre-paid
 	if chargeType == VPN_CHARGE_TYPE_PREPAID {
 		var preChargePara vpc.InstanceChargePrepaid

tencentcloud/resource_tc_vpn_gateway_test.go

@@ -64,7 +64,7 @@ func testSweepVpnGateway(region string) error {
 	return nil
 }
 
-func TestAccTencentCloudVpnGateway_basic(t *testing.T) {
+func TestAccTencentCloudVpnGatewayResource_basic(t *testing.T) {
 	t.Parallel()
 	resource.Test(t, resource.TestCase{
 		PreCheck:     func() { testAccPreCheck(t) },
@@ -206,7 +206,6 @@ resource "tencentcloud_vpn_gateway" "my_cgw" {
   name      = "terraform_test"
   vpc_id    = data.tencentcloud_vpc_instances.foo.instance_list.0.vpc_id
   bandwidth = 10
-  zone      = "ap-guangzhou-3"
 
   tags = {
     test = "tf"
@@ -222,7 +221,6 @@ resource "tencentcloud_vpn_gateway" "my_cgw" {
   name      = "terraform_update"
   vpc_id    = data.tencentcloud_vpc_instances.foo.instance_list.0.vpc_id
   bandwidth = 5
-  zone      = "ap-guangzhou-3"
 
   tags = {
     test = "test"
@@ -235,7 +233,6 @@ const testAccCcnVpnGatewayConfig = `
 resource "tencentcloud_vpn_gateway" "my_ccn_cgw" {
   name      = "terraform_ccn_vpngw_test"
   bandwidth = 5
-  zone      = "ap-guangzhou-3"
   type      = "CCN"
 
   tags = {

website/docs/r/vpn_gateway.html.markdown

@@ -98,7 +98,6 @@ resource "tencentcloud_vpn_gateway" "my_cgw" {
 The following arguments are supported:
 
 * `name` - (Required, String) Name of the VPN gateway. The length of character is limited to 1-60.
-* `zone` - (Required, String, ForceNew) Zone of the VPN gateway.
 * `bandwidth` - (Optional, Int) The maximum public network output bandwidth of VPN gateway (unit: Mbps), the available values include: 5,10,20,50,100,200,500,1000. Default is 5. When charge type is `PREPAID`, bandwidth degradation operation is unsupported.
 * `cdc_id` - (Optional, String) CDC instance ID.
 * `charge_type` - (Optional, String) Charge Type of the VPN gateway. Valid value: `PREPAID`, `POSTPAID_BY_HOUR`. The default is `POSTPAID_BY_HOUR`.
@@ -108,6 +107,7 @@ The following arguments are supported:
 * `tags` - (Optional, Map) A list of tags used to associate different resources.
 * `type` - (Optional, String) Type of gateway instance, Default is `IPSEC`. Valid value: `IPSEC`, `SSL`, `CCN` and `SSL_CCN`.
 * `vpc_id` - (Optional, String, ForceNew) ID of the VPC. Required if vpn gateway is not in `CCN` or `SSL_CCN` type, and doesn't make sense for `CCN` or `SSL_CCN` vpn gateway.
+* `zone` - (Optional, String, ForceNew) Zone of the VPN gateway.
 
 ## Attributes Reference