Skip to content

Ip sla policy #245

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Ip sla policy #245

wants to merge 12 commits into from

Conversation

ogorczow
Copy link
Contributor

@ogorczow ogorczow commented May 21, 2025

PR Solves 4 issues:

  1. it is not possible to create TrackList with default values because UP and DOWN values are 0 and 1. UP values must be higher than DOWN as per APIC validation.
  2. Track_member dependency over node_profiles is removed as it forces Terraform to create Node Profile before Track_Member whereas node_profile has reference to TrackList. Perfect case scenario would be to create: L3out -> Track Member -> Track List -> L3out Node Profile.
  3. Added support for attaching ip_sla_policy & track_list under: l3out -> node_profiles -> node -> static_route -> next_hop and auto generated node_profiles: l3out -> node -> static_route -> next_hop
  4. Added support to auto generate Track Member, Track List, Track List -> TrackMember refference, Track Member -> IP SLA Policyv reference, whenever ip_sla_policy is attached under next_hop. As per APIC example, default name for objects is vrf.name+"_"+next_hop.ip

Other Issues:
5. Feature supports name_suffix for IP_SLA_Policy, Track_list and Track_Member. However, it doesn't work for L3out since node_profiles and interface_profiles L3out value doesn't apply name_suffix and as result it breaks references between profiles and l3outs. Functionality is broken and needs to be fixed.

@ogorczow ogorczow requested a review from juchowan May 22, 2025 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant