bgp: T8588: Add match src-peer to policy route-map#5149
bgp: T8588: Add match src-peer to policy route-map#5149robinchrist wants to merge 1 commit intovyos:currentfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
|
👍 |
|
✅ No typos found in changed files. |
|
CI integration ❌ failed! Details
|
dmbaturin
left a comment
There was a problem hiding this comment.
The idea and the logic all look good to me. I left a simple suggestion about the command naming style.
| </constraint> | ||
| </properties> | ||
| </leafNode> | ||
| <leafNode name="src-peer"> |
There was a problem hiding this comment.
We generally prefer not to abbreviate words in commands.
| <leafNode name="src-peer"> | |
| <leafNode name="source-peer"> |
There was a problem hiding this comment.
I can change that, sure.
I wanted to keep it in sync with FRR (FRR calls it src-peer in the route-map), but if source-peer is preferred no problem :)
Change summary
This PR adds
match src-peertopolicy route-mapand makes some adjustments tolist_bgp_neighbor.shin order that it can be re-used for autocompletion ofmatch src-peerTypes of changes
Related Task(s)
https://vyos.dev/T8588
Related PR(s)
How to test / Smoketest result
Checklist: