Adjusts to test eBGP (different ASN values)#3929
Conversation
Allow parameterized ASN (leaf_asn), downlink mode (downlink_remote_as). Conditional route-reflector-client for iBGP only. Sets downlink_remote_as and leaf_asn according to destined ebgp_racks and ebgp_leaf_asn variables (changes both IPv4 and IPv6 renders). Related: OSPRH-28085 Assisted-By: Claude Code Signed-off-by: Maor Blaustein <mblue@redhat.com> (cherry picked from commit 0308fa9)
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
| become: true | ||
| vars: | ||
| downlink_remote_as: "{{ 'external' if (rack_id | int) in (ebgp_racks | default([]) | map('int') | list) else 'internal' }}" | ||
| leaf_asn: "{{ ebgp_leaf_asn | default(64999) if (rack_id | int) in (ebgp_racks | default([]) | map('int') | list) else 64999 }}" |
There was a problem hiding this comment.
(blocking) suggestion: We need to doc new variables: ebgp_racks and ebgp_leaf_asn
| become: true | ||
| vars: | ||
| downlink_interfaces: "{{ downlink_ifs_rack3 if (rack_id | int) == 3 else leaf_downlink_ifs }}" | ||
| downlink_remote_as: "{{ 'external' if (rack_id | int) in (ebgp_racks | default([]) | map('int') | list) else 'internal' }}" |
There was a problem hiding this comment.
(blocking) suggestion: wdyt moving ipv6 and ipv4 both variables definition at L452.
|
This PR is stale because it has been for over 15 days with no activity. |
2 participants
Allow parameterized ASN (leaf_asn), downlink mode (downlink_remote_as). Conditional route-reflector-client for iBGP only.
Sets downlink_remote_as and leaf_asn according to destined ebgp_racks and ebgp_leaf_asn variables (changes both IPv4 and IPv6 renders).
Related: OSPRH-28085
Assisted-By: Claude Code
(cherry picked from commit 0308fa9)