add support for auto_generated_vlan_name in catalystcenter_anycast_gatewy resource

Author: kuba-mazurkiewicz

CHANGELOG.md

@@ -1,5 +1,6 @@
 ## 0.1.19 (unreleased)
 
+- Add support for auto_generated_vlan_name in `catalystcenter_anycast_gateway` resource
 - Fix issue with tags removal from templates in `catalystcenter_assign_templates_to_tag` resource, [link](https://github.com/CiscoDevNet/terraform-provider-catalystcenter/issues/165)
 
 ## 0.1.18

docs/guides/changelog.md

@@ -9,6 +9,7 @@ description: |-
 
 ## 0.1.19 (unreleased)
 
+- Add support for auto_generated_vlan_name in `catalystcenter_anycast_gateway` resource
 - Fix issue with tags removal from templates in `catalystcenter_assign_templates_to_tag` resource, [link](https://github.com/CiscoDevNet/terraform-provider-catalystcenter/issues/165)
 
 ## 0.1.18

docs/resources/anycast_gateway.md

@@ -27,6 +27,7 @@ resource "catalystcenter_anycast_gateway" "example" {
   ip_directed_broadcast        = false
   intra_subnet_routing_enabled = false
   multiple_ip_to_mac_addresses = false
+  auto_generate_vlan_name      = false
 }
 ```
 
@@ -35,16 +36,15 @@ resource "catalystcenter_anycast_gateway" "example" {
 
 ### Required
 
+- `auto_generate_vlan_name` (Boolean) This field cannot be true when vlanName is provided. the vlanName will be generated as ipPoolGroupV4Cidr-virtualNetworkName for non-critical VLANs. for critical VLANs with DATA trafficType, vlanName will be CRITICAL_VLAN. for critical VLANs with VOICE trafficType, vlanName will be VOICE_VLAN
 - `fabric_id` (String) ID of the fabric to contain this anycast gateway
 - `ip_pool_name` (String) Name of the IP pool associated with the anycast gateway
 - `traffic_type` (String) The type of traffic the anycast gateway serves
   - Choices: `DATA`, `VOICE`
 - `virtual_network_name` (String) Name of the layer 3 virtual network associated with the anycast gateway. the virtual network must have already been added to the site before creating an anycast gateway with it
-- `vlan_name` (String) Name of the VLAN of the anycast gateway
 
 ### Optional
 
-- `auto_generate_vlan_name` (Boolean) This field cannot be true when vlanName is provided. the vlanName will be generated as ipPoolGroupV4Cidr-virtualNetworkName for non-critical VLANs. for critical VLANs with DATA trafficType, vlanName will be CRITICAL_VLAN. for critical VLANs with VOICE trafficType, vlanName will be VOICE_VLAN
 - `critical_pool` (Boolean) Enable/disable critical VLAN. if true, autoGenerateVlanName must also be true. (isCriticalPool is not applicable to INFRA_VN)
 - `intra_subnet_routing_enabled` (Boolean) Enable/disable Intra-Subnet Routing (not applicable to INFRA_VN)
 - `ip_directed_broadcast` (Boolean) Enable/disable IP-directed broadcast (not applicable to INFRA_VN)
@@ -57,6 +57,7 @@ resource "catalystcenter_anycast_gateway" "example" {
 - `tcp_mss_adjustment` (Number) TCP maximum segment size adjustment
   - Range: `500`-`1440`
 - `vlan_id` (Number) ID of the VLAN of the anycast gateway. allowed VLAN range is 2-4093 except for reserved VLANs 1002-1005, 2046, and 4094. if deploying an anycast gateway on a fabric zone, this vlanId must match the vlanId of the corresponding anycast gateway on the fabric site
+- `vlan_name` (String) Name of the VLAN of the anycast gateway
 - `wireless_pool` (Boolean) Enable/disable fabric-enabled wireless (not applicable to INFRA_VN)
 
 ### Read-Only

examples/resources/catalystcenter_anycast_gateway/resource.tf

@@ -12,4 +12,5 @@ resource "catalystcenter_anycast_gateway" "example" {
   ip_directed_broadcast        = false
   intra_subnet_routing_enabled = false
   multiple_ip_to_mac_addresses = false
+  auto_generate_vlan_name      = false
 }

gen/definitions/anycast_gateway.yaml

@@ -56,7 +56,6 @@ attributes:
     response_data_path: response.0.vlanName
     type: String
     description: Name of the VLAN of the anycast gateway
-    mandatory: true
     example: VLAN401
   - model_name: vlanId
     requires_replace: true
@@ -141,8 +140,10 @@ attributes:
     data_path: '0'
     response_data_path: response.0.autoGenerateVlanName
     type: Bool
+    mandatory: true
+    write_only: true
     description: 'This field cannot be true when vlanName is provided. the vlanName will be generated as ipPoolGroupV4Cidr-virtualNetworkName for non-critical VLANs. for critical VLANs with DATA trafficType, vlanName will be CRITICAL_VLAN. for critical VLANs with VOICE trafficType, vlanName will be VOICE_VLAN'
-    exclude_test: true
+    example: false
 test_prerequisites: |
   resource "catalystcenter_area" "test" {
     name        = "Area1"

internal/provider/data_source_catalystcenter_anycast_gateway_test.go

@@ -108,6 +108,7 @@ func testAccDataSourceCcAnycastGatewayConfig() string {
 	config += `	ip_directed_broadcast = false` + "\n"
 	config += `	intra_subnet_routing_enabled = false` + "\n"
 	config += `	multiple_ip_to_mac_addresses = false` + "\n"
+	config += `	auto_generate_vlan_name = false` + "\n"
 	config += `}` + "\n"
 
 	config += `

internal/provider/model_catalystcenter_anycast_gateway.go

@@ -216,11 +216,6 @@ func (data *AnycastGateway) fromBody(ctx context.Context, res gjson.Result) {
 	} else {
 		data.SupplicantBasedExtendedNodeOnboarding = types.BoolNull()
 	}
-	if value := res.Get("response.0.autoGenerateVlanName"); value.Exists() {
-		data.AutoGenerateVlanName = types.BoolValue(value.Bool())
-	} else {
-		data.AutoGenerateVlanName = types.BoolNull()
-	}
 }
 
 // End of section. //template:end fromBody
@@ -307,11 +302,6 @@ func (data *AnycastGateway) updateFromBody(ctx context.Context, res gjson.Result
 	} else {
 		data.SupplicantBasedExtendedNodeOnboarding = types.BoolNull()
 	}
-	if value := res.Get("response.0.autoGenerateVlanName"); value.Exists() && !data.AutoGenerateVlanName.IsNull() {
-		data.AutoGenerateVlanName = types.BoolValue(value.Bool())
-	} else {
-		data.AutoGenerateVlanName = types.BoolNull()
-	}
 }
 
 // End of section. //template:end updateFromBody

internal/provider/resource_catalystcenter_anycast_gateway.go

@@ -103,7 +103,7 @@ func (r *AnycastGatewayResource) Schema(ctx context.Context, req resource.Schema
 			},
 			"vlan_name": schema.StringAttribute{
 				MarkdownDescription: helpers.NewAttributeDescription("Name of the VLAN of the anycast gateway").String,
-				Required:            true,
+				Optional:            true,
 				PlanModifiers: []planmodifier.String{
 					stringplanmodifier.RequiresReplace(),
 				},
@@ -169,7 +169,7 @@ func (r *AnycastGatewayResource) Schema(ctx context.Context, req resource.Schema
 			},
 			"auto_generate_vlan_name": schema.BoolAttribute{
 				MarkdownDescription: helpers.NewAttributeDescription("This field cannot be true when vlanName is provided. the vlanName will be generated as ipPoolGroupV4Cidr-virtualNetworkName for non-critical VLANs. for critical VLANs with DATA trafficType, vlanName will be CRITICAL_VLAN. for critical VLANs with VOICE trafficType, vlanName will be VOICE_VLAN").String,
-				Optional:            true,
+				Required:            true,
 			},
 		},
 	}

internal/provider/resource_catalystcenter_anycast_gateway_test.go

@@ -100,8 +100,8 @@ func testAccCcAnycastGatewayConfig_minimum() string {
 	config += `	fabric_id = catalystcenter_fabric_site.test.id` + "\n"
 	config += `	virtual_network_name = catalystcenter_virtual_network_to_fabric_site.test.virtual_network_name` + "\n"
 	config += `	ip_pool_name = catalystcenter_ip_pool_reservation.test.name` + "\n"
-	config += `	vlan_name = "VLAN401"` + "\n"
 	config += `	traffic_type = "DATA"` + "\n"
+	config += `	auto_generate_vlan_name = false` + "\n"
 	config += `}` + "\n"
 	return config
 }
@@ -124,6 +124,7 @@ func testAccCcAnycastGatewayConfig_all() string {
 	config += `	ip_directed_broadcast = false` + "\n"
 	config += `	intra_subnet_routing_enabled = false` + "\n"
 	config += `	multiple_ip_to_mac_addresses = false` + "\n"
+	config += `	auto_generate_vlan_name = false` + "\n"
 	config += `}` + "\n"
 	return config
 }

templates/guides/changelog.md.tmpl

@@ -9,6 +9,7 @@ description: |-
 
 ## 0.1.19 (unreleased)
 
+- Add support for auto_generated_vlan_name in `catalystcenter_anycast_gateway` resource
 - Fix issue with tags removal from templates in `catalystcenter_assign_templates_to_tag` resource, [link](https://github.com/CiscoDevNet/terraform-provider-catalystcenter/issues/165)
 
 ## 0.1.18