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

Not fully sorted static routes in configuration. #8746

Closed
ak503 opened this issue May 27, 2021 · 1 comment · Fixed by #9410
Closed

Not fully sorted static routes in configuration. #8746

ak503 opened this issue May 27, 2021 · 1 comment · Fixed by #9410
Labels
enhancement good first issue Good first issue for new contributors staticd triage Needs further investigation

Comments

@ak503
Copy link
Contributor

ak503 commented May 27, 2021

If add some routes:

ip route 1.1.1.1/32 2.2.2.3
ip route 1.1.1.1/32 2.2.2.2
ip route 1.1.1.1/32 2.2.2.4

In config they sorted by gateway:

# do show running-config 
...
Current configuration:
!
frr version 8.1-dev
...
ip route 1.1.1.1/32 2.2.2.4
ip route 1.1.1.1/32 2.2.2.3
ip route 1.1.1.1/32 2.2.2.2

But when added ip route 1.1.1.1/32 2.2.2.1, he will added after ip route 1.1.1.1/32 2.2.2.4, but not after ip route 1.1.1.1/32 2.2.2.2:

# do show running-config 
...
Current configuration:
!
frr version 8.1-dev
...
ip route 1.1.1.1/32 2.2.2.4
ip route 1.1.1.1/32 2.2.2.1
ip route 1.1.1.1/32 2.2.2.3
ip route 1.1.1.1/32 2.2.2.2
@ak503 ak503 added the triage Needs further investigation label May 27, 2021
@qlyoung qlyoung added staticd enhancement good first issue Good first issue for new contributors labels Jun 1, 2021
@qlyoung
Copy link
Member

qlyoung commented Jun 1, 2021

There shouldn't be any functional impact here so this is purely cosmetic. It seems that with libyang these will be dumped in the same order they were entered, which might even be what we want.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement good first issue Good first issue for new contributors staticd triage Needs further investigation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants