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
Feat(eos_designs): Allow disabling filtering on redist connected in underlay bgp #2586
Feat(eos_designs): Allow disabling filtering on redist connected in underlay bgp #2586
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
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 overall - just need following the latest changes in devel
@@ -119,6 +109,22 @@ def _route_maps_vrf_default(self) -> dict | None: | |||
"match": ["ip address prefix-list PL-STATIC-VRF-DEFAULT"], | |||
} | |||
|
|||
if self._underlay_filter_redistribute_connected: |
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.
Need to rebase to take into account the new route-map syntax - maybe let's keep the ordering of generation (RM-CONN before RM-EVPN if both present?)
b281a97
to
aa45723
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
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
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!
aa45723
to
7d59283
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
7d59283
to
ac81210
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
Change Summary
Allow disabling filtering on redist connected in underlay bgp
Component(s) name
arista.avd.eos_designs
Proposed changes
How to test
Added to molecule
Checklist
User Checklist
Repository Checklist