Resolve issues #234 and #236 (#237)

CiscoDevNet · May 7, 2024 · b7a1d5d · b7a1d5d
1 parent f7a20f6
commit b7a1d5d
Show file tree

Hide file tree

Showing 11 changed files with 126 additions and 4 deletions.
diff --git a/docs/data-sources/transport_routing_bgp_profile_parcel.md b/docs/data-sources/transport_routing_bgp_profile_parcel.md
@@ -121,6 +121,8 @@ Read-Only:
 - `hold_time_variable` (String) Variable name
 - `keepalive_time` (Number) Set how often to advertise keepalive messages to BGP peer
 - `keepalive_time_variable` (String) Variable name
+- `local_as` (Number) Set local autonomous number,Local-AS cannot have the local BGP protocol AS number or the AS number of the remote peer.The local-as is valid only if the peer is a true eBGP peer. It does not work for two peers in different sub-ASs in a confederation.
+- `local_as_variable` (String) Variable name
 - `next_hop_self` (Boolean) Set router to be next hop for routes advertised to neighbor
 - `next_hop_self_variable` (String) Variable name
 - `password` (String) Set MD5 password on TCP connection with BGP peer
@@ -209,6 +211,8 @@ Read-Only:
 - `hold_time_variable` (String) Variable name
 - `keepalive_time` (Number) Interval (seconds) of keepalive messages sent to its BGP peer
 - `keepalive_time_variable` (String) Variable name
+- `local_as` (Number) Set local autonomous system number,Local-AS cannot have the local BGP protocol AS number or the AS number of the remote peer.The local-as is valid only if the peer is a true eBGP peer. It does not work for two peers in different sub-ASs in a confederation.
+- `local_as_variable` (String) Variable name
 - `next_hop_self` (Boolean) Set router to be next hop for routes advertised to neighbor
 - `next_hop_self_variable` (String) Variable name
 - `password` (String) Set MD5 password on TCP connection with BGP peer

diff --git a/docs/resources/transport_routing_bgp_profile_parcel.md b/docs/resources/transport_routing_bgp_profile_parcel.md
@@ -39,6 +39,7 @@ resource "sdwan_transport_routing_bgp_profile_parcel" "example" {
       description             = "neighbor1"
       shutdown                = false
       remote_as               = 200
+      local_as                = 200
       keepalive_time          = 40
       hold_time               = 200
       update_source_interface = "GigabitEthernet0"
@@ -68,6 +69,7 @@ resource "sdwan_transport_routing_bgp_profile_parcel" "example" {
       description             = "neighbor2"
       shutdown                = false
       remote_as               = 200
+      local_as                = 200
       keepalive_time          = 180
       hold_time               = 60
       update_source_interface = "Loopback1"
@@ -268,6 +270,8 @@ Optional:
   - Range: `0`-`65535`
   - Default value: `60`
 - `keepalive_time_variable` (String) Variable name
+- `local_as` (Number) Set local autonomous number,Local-AS cannot have the local BGP protocol AS number or the AS number of the remote peer.The local-as is valid only if the peer is a true eBGP peer. It does not work for two peers in different sub-ASs in a confederation.
+- `local_as_variable` (String) Variable name
 - `next_hop_self` (Boolean) Set router to be next hop for routes advertised to neighbor
   - Default value: `false`
 - `next_hop_self_variable` (String) Variable name
@@ -376,6 +380,8 @@ Optional:
   - Range: `0`-`65535`
   - Default value: `60`
 - `keepalive_time_variable` (String) Variable name
+- `local_as` (Number) Set local autonomous system number,Local-AS cannot have the local BGP protocol AS number or the AS number of the remote peer.The local-as is valid only if the peer is a true eBGP peer. It does not work for two peers in different sub-ASs in a confederation.
+- `local_as_variable` (String) Variable name
 - `next_hop_self` (Boolean) Set router to be next hop for routes advertised to neighbor
   - Default value: `false`
 - `next_hop_self_variable` (String) Variable name

diff --git a/examples/resources/sdwan_transport_routing_bgp_profile_parcel/resource.tf b/examples/resources/sdwan_transport_routing_bgp_profile_parcel/resource.tf
@@ -22,6 +22,7 @@ resource "sdwan_transport_routing_bgp_profile_parcel" "example" {
       description             = "neighbor1"
       shutdown                = false
       remote_as               = 200
+      local_as                = 200
       keepalive_time          = 40
       hold_time               = 200
       update_source_interface = "GigabitEthernet0"
@@ -51,6 +52,7 @@ resource "sdwan_transport_routing_bgp_profile_parcel" "example" {
       description             = "neighbor2"
       shutdown                = false
       remote_as               = 200
+      local_as                = 200
       keepalive_time          = 180
       hold_time               = 60
       update_source_interface = "Loopback1"

diff --git a/gen/definitions/profile_parcels/system_ntp.yaml b/gen/definitions/profile_parcels/system_ntp.yaml
@@ -15,10 +15,10 @@ attributes:
     attributes:
       - model_name: name
         tf_name: hostname_ip_address
+        id: true
         example: "1.1.1.1"
       - model_name: key
         tf_name: authentication_key
-        id: true
         example: 41673
       - model_name: vpn
         example: 1

diff --git a/gen/definitions/profile_parcels/transport_routing_bgp.yaml b/gen/definitions/profile_parcels/transport_routing_bgp.yaml
@@ -62,6 +62,8 @@ attributes:
       example: false
     - model_name: remoteAs
       example: 200
+    - model_name: localAs
+      example: 200
     - model_name: keepalive
       tf_name: keepalive_time
       example: 40
@@ -139,6 +141,8 @@ attributes:
       example: false
     - model_name: remoteAs
       example: 200
+    - model_name: localAs
+      example: 200
     - model_name: keepalive
       tf_name: keepalive_time
       example: 180

diff --git a/internal/provider/data_source_sdwan_transport_routing_bgp_profile_parcel.go b/internal/provider/data_source_sdwan_transport_routing_bgp_profile_parcel.go
diff --git a/internal/provider/data_source_sdwan_transport_routing_bgp_profile_parcel_test.go b/internal/provider/data_source_sdwan_transport_routing_bgp_profile_parcel_test.go
diff --git a/internal/provider/model_sdwan_system_ntp_profile_parcel.go b/internal/provider/model_sdwan_system_ntp_profile_parcel.go
diff --git a/internal/provider/model_sdwan_transport_routing_bgp_profile_parcel.go b/internal/provider/model_sdwan_transport_routing_bgp_profile_parcel.go
diff --git a/internal/provider/resource_sdwan_transport_routing_bgp_profile_parcel.go b/internal/provider/resource_sdwan_transport_routing_bgp_profile_parcel.go