-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
bgpd: Derive and set MED from IGP or AIGP #17038
bgpd: Derive and set MED from IGP or AIGP #17038
Conversation
5f34a79
to
07d48af
Compare
Thank you so much for noticing my request. @ton31337 I have installed and tested this branch with new feature. And, I found a bug about this new added feature. The list of test steps are shown as followed.
If possible, could you help solve this problem? |
Could you give the full configs, then I'd test it out? |
Please refer to the followed content. Thank you~ n1's configuration:
n2's configuration:
n3's configuration:
n4's configuration:
n5's configuration:
|
Tested with your topology and configs, looks good now. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
101ca1b
to
10f9b9a
Compare
Signed-off-by: Donatas Abraitis <[email protected]>
Set metric automatically from the path info (IGP protocol). Signed-off-by: Donatas Abraitis <[email protected]>
5f73d2d
to
24e23fc
Compare
Signed-off-by: Donatas Abraitis <[email protected]>
Same as `set metric igp`, but in this case accumulated IGP metric is being sent as MED attribute. Signed-off-by: Donatas Abraitis <[email protected]>
Signed-off-by: Donatas Abraitis <[email protected]>
Signed-off-by: Donatas Abraitis <[email protected]>
If the underlay IGP metric changes, we SHOULD re-announce the routes with the correct bpi->extra->igpmetric set. Without this patch if the IGP link cost (metric) changes, we never notice this and the peers do not have the updated metrics, which in turn causes incorrect best path selections on remote peers. Signed-off-by: Donatas Abraitis <[email protected]>
…changes Signed-off-by: Donatas Abraitis <[email protected]>
24e23fc
to
a593f15
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.
looks good
Related: #17031