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
FRR templates: provide a seqnum for the prefix lists #2075
Conversation
route-map 10.2.2.254-out permit 1 | ||
match ip address prefix-list 10.2.2.254-red-pl-ipv4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks out of place (shouldn't this be red-out)?
route-map 10.4.4.255-out permit 1 | ||
match ip address prefix-list 10.4.4.255-red-pl-ipv4 | ||
route-map 10.4.4.255-red-out permit 1 | ||
match ipv6 address prefix-list 10.4.4.255-red-pl-ipv4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same?
605c48b
to
4b0a587
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
ipv6 prefix-list {{$plistName}} seq {{counter $plistName}} deny any | ||
{{- end }} | ||
|
||
route-map {{$.neighbor.ID}}-out permit {{counter $.neighbor.Addr}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
counter $.neighbor.Addr
-> $.neighbor.ID
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
350e650
to
792e3c1
Compare
Instead of letting frr pick the sequence number, we provide it from outside. This should make the configuration more deterministic and offload some of the logic from FRR. Signed-off-by: Federico Paolinelli <fpaoline@redhat.com>
792e3c1
to
69255c4
Compare
Instead of letting frr pick the sequence number, we provide it from outside. This should make the configuration more deterministic and offload some of the logic from FRR.