Skip to content
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

Modifying the route-map of aggregated routes does not take effect #15241

Closed
wangdan1323 opened this issue Jan 27, 2024 · 3 comments · Fixed by #15257
Closed

Modifying the route-map of aggregated routes does not take effect #15241

wangdan1323 opened this issue Jan 27, 2024 · 3 comments · Fixed by #15257
Assignees
Milestone

Comments

@wangdan1323
Copy link

FRR 7.5

My configuration:
route-map map1 permit 1
set metric 111
aggregate-address 66.0.0.0/16 as-set route-map map1

The metirc 111 is as expected
Network Next Hop Metric LocPrf Weight Path
*> 66.0.0.0/16 40.40.40.3 111 0 1111 10002 ?
*> 66.0.0.1/32 40.40.40.3 0 1111 10002 ?

But when I change map1 metric from 111 to 222, The metri of published routes is still 111!
sonic(config-router-af)# route-map map1 permit 1
sonic(config-route-map)# set metric 222

route-map map1 permit 1
set metric 222

Network Next Hop Metric LocPrf Weight Path
*> 66.0.0.0/16 40.40.40.3 111 0 1111 10002 ?
*> 66.0.0.1/32 40.40.40.3 0 1111 10002 ?

@wangdan1323 wangdan1323 added the triage Needs further investigation label Jan 27, 2024
@ton31337 ton31337 self-assigned this Jan 29, 2024
@ton31337 ton31337 added the bgp label Jan 29, 2024
@ton31337
Copy link
Member

7.5 is a very old release, and lots have been fixed in terms of route-map/bgp. Please check with the latest versions (8.5, or 9.x).

@wangdan1323
Copy link
Author

7.5 is a very old release, and lots have been fixed in terms of route-map/bgp. Please check with the latest versions (8.5, or 9.x).

Looking at the FRR 9.1 version code, I still haven't found a solution to this problem.
Do you agree that the above "set metric 222" should automatically trigger the re-publishing of the aggregate route with metric 222?

@ton31337 ton31337 added bug and removed triage Needs further investigation labels Jan 30, 2024
@ton31337 ton31337 added this to the 10.0 milestone Jan 30, 2024
@wangdan1323
Copy link
Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants