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.

Sign up for GitHub

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

Jump to bottom

[frr]: Missing route change messages with ip route change commands #2909

Closed
stcheng opened this issue May 16, 2019 · 2 comments
Closed

[frr]: Missing route change messages with ip route change commands #2909

stcheng opened this issue May 16, 2019 · 2 comments
Labels
Bug 🐛

Comments

@stcheng
Copy link
Contributor

stcheng commented May 16, 2019

This issue is seen on the latest master with FRR.

root@str-s6100-acs-5:/home/admin# ip route add 2.2.2.0/24 nexthop via 10.0.0.55
root@str-s6100-acs-5:/home/admin# ip route list match 2.2.2.2
default proto 186 src 10.1.0.32 metric 20
        nexthop via 10.0.0.13  dev PortChannel0011 weight 1
        nexthop via 10.0.0.1  dev PortChannel0002 weight 1
        nexthop via 10.0.0.5  dev PortChannel0005 weight 1
        nexthop via 10.0.0.9  dev PortChannel0008 weight 1
2.2.2.0/24 via 10.0.0.55 dev PortChannel0016

127.0.0.1:6379> hgetall "ROUTE_TABLE:2.2.2.0/24"
1) "nexthop"
2) "10.0.0.55"
3) "ifname"
4) "PortChannel0016"


root@str-s6100-acs-5:/home/admin# ip route change 2.2.2.0/24 nexthop via 10.0.0.55 nexthop via 10.0.0.57
root@str-s6100-acs-5:/home/admin# ip route list match 2.2.2.2
default proto 186 src 10.1.0.32 metric 20
        nexthop via 10.0.0.13  dev PortChannel0011 weight 1
        nexthop via 10.0.0.1  dev PortChannel0002 weight 1
        nexthop via 10.0.0.5  dev PortChannel0005 weight 1
        nexthop via 10.0.0.9  dev PortChannel0008 weight 1
2.2.2.0/24
        nexthop via 10.0.0.55  dev PortChannel0016 weight 1
        nexthop via 10.0.0.57  dev PortChannel0017 weight 1

127.0.0.1:6379> hgetall "ROUTE_TABLE:2.2.2.0/24"
(empty list or set)

root@str-s6100-acs-5:/home/admin# ip route change 2.2.2.0/24 nexthop via 10.0.0.55 nexthop via 10.0.0.57 nexthop via 10.0.0.59
root@str-s6100-acs-5:/home/admin# ip route list match 2.2.2.2
default proto 186 src 10.1.0.32 metric 20
        nexthop via 10.0.0.13  dev PortChannel0011 weight 1
        nexthop via 10.0.0.1  dev PortChannel0002 weight 1
        nexthop via 10.0.0.5  dev PortChannel0005 weight 1
        nexthop via 10.0.0.9  dev PortChannel0008 weight 1
2.2.2.0/24
        nexthop via 10.0.0.55  dev PortChannel0016 weight 1
        nexthop via 10.0.0.57  dev PortChannel0017 weight 1
        nexthop via 10.0.0.59  dev PortChannel0018 weight 1

127.0.0.1:6379> hgetall "ROUTE_TABLE:2.2.2.0/24"
1) "nexthop"
2) "10.0.0.55,10.0.0.57,10.0.0.59"
3) "ifname"
4) "PortChannel0016,PortChannel0017,PortChannel0018"
@stcheng
Copy link
Contributor Author

stcheng commented May 16, 2019

This is caught by the current EVERFLOW test

@stcheng
Copy link
Contributor Author

stcheng commented May 21, 2019

resolved in #2899

@stcheng stcheng closed this as completed May 21, 2019
mssonicbld added a commit that referenced this issue Jul 14, 2023
@mssonicbld
[submodule] Update submodule sonic-utilities to the latest HEAD autom…
23a0a87 
…atically (#15812)

#### Why I did it
src/sonic-utilities
```
* 51c7a43c - (HEAD -> master, origin/master, origin/HEAD) [show][muxcable] update `show mux config` to print out `soc_ipv6` as well  (#2909) (6 hours ago) [Jing Zhang]
* fd497755 - [route_check][dualtor] Ignore vlan neighbor route miss (#2888) (18 hours ago) [Longxiang Lyu]
* 81c0ed4e - [show][muxcable] update `show mux tunnel-route` to check soc_ipv6 as well (33 hours ago) [Jing Zhang]
* 1ee73668 - [db_migrator] Migrate DNS configuratuion (#2893) (2 days ago) [ganglv]
* 553a3432 - [dualtor][route_check] filter out `soc_ipv6`  (#2899) (2 days ago) [Jing Zhang]
```
#### How I did it
#### How to verify it
#### Description for the changelog
yxieca pushed a commit that referenced this issue Jul 21, 2023
@mssonicbld
[submodule] Update submodule sonic-utilities to the latest HEAD autom…
1f809c8 
…atically (#15930)

src/sonic-utilities

* 99864640 - (HEAD -> 202205, origin/202205) [dualtor] Add script to verify consistency between kernel and ASIC  (#2840) (87 minutes ago) [Longxiang Lyu]
* a32ddc1b - [show][muxcable] update `show mux config` to print out `soc_ipv6` as well  (#2909) (88 minutes ago) [Jing Zhang]
* 0c6d0c51 - [202205] Flush RESTAPI db in fast-reboot shutdown path (#2921) (4 hours ago) [bingwang-ms]
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this issue Sep 20, 2023
@mssonicbld @sonic-otn
[submodule] Update submodule sonic-utilities to the latest HEAD autom…
6227116 
…atically (sonic-net#15812)

#### Why I did it
src/sonic-utilities
```
* 51c7a43c - (HEAD -> master, origin/master, origin/HEAD) [show][muxcable] update `show mux config` to print out `soc_ipv6` as well  (sonic-net#2909) (6 hours ago) [Jing Zhang]
* fd497755 - [route_check][dualtor] Ignore vlan neighbor route miss (sonic-net#2888) (18 hours ago) [Longxiang Lyu]
* 81c0ed4e - [show][muxcable] update `show mux tunnel-route` to check soc_ipv6 as well (33 hours ago) [Jing Zhang]
* 1ee73668 - [db_migrator] Migrate DNS configuratuion (sonic-net#2893) (2 days ago) [ganglv]
* 553a3432 - [dualtor][route_check] filter out `soc_ipv6`  (sonic-net#2899) (2 days ago) [Jing Zhang]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mlok-nokia pushed a commit to mlok-nokia/sonic-buildimage that referenced this issue Jun 5, 2024
[Update submodule sonic-utilities.msft] * 9986464 [dualtor] Add scrip…
8902c45 
…t to verify consistency between kernel and ASIC (sonic-net#2840)

   * za32ddc1 [show][muxcable] update  to print out  as well  (sonic-net#2909)
   * 0c6d0c5 [202205] Flush RESTAPI db in fast-reboot shutdown path (sonic-net#2921)
   * bc7c792 Add FEC correctable and uncorrectable port stats (sonic-net#2027)
   * 58db48a [show][muxcable] update  to check soc_ipv6 as well
   * 24fc1db [dualtor][route_check] filter out   (sonic-net#2899)
   * d89d483 [route_check][dualtor] Ignore vlan neighbor route miss (sonic-net#2888)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug 🐛
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stcheng