Skip to content

Use standard routing policy in BGP community propagation test - #4025

Merged
ipspace merged 1 commit into
ipspace:devfrom
jbemmel:feature/declarative-bgp-communities
Nov 4, 2026
Merged

ipspace merged 1 commit into
ipspace:devfrom
jbemmel:feature/declarative-bgp-communities

Conversation

@jbemmel

@jbemmel jbemmel commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator

Why this is needed: To make the test conditions device independent, such that other tools can evaluate them without having to implement FRR specific logic

extended: [ 'bandwidth 100' ] is still a little funky, a consequence of the policy attributes not being defined more strictly. The combination with append: True fails

@jbemmel

jbemmel commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator Author

extended: [ 'bandwidth 100' ] is still a little funky, a consequence of the policy attributes not being defined more strictly. The combination with append: True fails

'LB:65000:12500000' would be cleaner

@ipspace

ipspace commented Nov 4, 2026

Copy link
Copy Markdown
Owner

extended: [ 'bandwidth 100' ] is still a little funky, a consequence of the policy attributes not being defined more strictly.

Related to #2243, the second-oldest open issue 🤷‍♂️

@ipspace ipspace left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works. Thank you!

@ipspace
ipspace merged commit d77d55f into ipspace:dev Nov 4, 2026
4 checks passed
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.

2 participants