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

Conflicting router between mgmt and copp, which will cause to transfer data failed. #3165

Open
Harrison-SH opened this issue Jul 16, 2019 · 2 comments
Assignees

Comments

@Harrison-SH
Copy link


BUG REPORT INFORMATION

Description
MGMT router of eth0 had been added into copp router table.
Which will cause to conflict between MGMT router and copp router. specially for default router.

Steps to reproduce the issue:

  1. Config eth0 normally
  2. Add default router for copp
  3. MGMT and copp's router will conflict.
  4. Which will cause to transfer packets failed on the default router occasionally

Describe the results you received:
admin@sonic:~$ show ip route
Codes: K - kernel route, C - connected, S - static, R - RIP,
O - OSPF, I - IS-IS, B - BGP, E - EIGRP, N - NHRP,
T - Table, v - VNC, V - VNC-Direct, A - Babel, D - SHARP,
F - PBR, f - OpenFabric,
> - selected route, * - FIB route

S>* 0.0.0.0/0 [200/0] via 10.251.0.1, eth0, 00:02:51
C>* 10.1.0.1/32 is directly connected, lo, 00:02:51
C>* 10.251.0.0/24 is directly connected, eth0, 00:02:51
admin@sonic:~$ docker exec -it bgp bash
root@sonic:/# ip route
default via 10.251.0.1 dev eth0 proto 196 metric 20

@prsunny
Copy link
Contributor

prsunny commented Jul 16, 2019

default route via eth0 is in "default" route table and not in the main table.

@Harrison-SH
Copy link
Author

Maybe the implementation has some potential problem. This issue is not easy to reproduce.
Just very occasionally happens.
I have debugged it. We have following default route within bgp container,:
root@cel-seastone-01:/# ip route|more
default proto 186 src 10.1.0.32 metric 20
nexthop via 10.0.0.57 dev PortChannel0001 weight 1
nexthop via 10.0.0.59 dev PortChannel0002 weight 1
nexthop via 10.0.0.61 dev PortChannel0003 weight 1
nexthop via 10.0.0.63 dev PortChannel0004 weight 1
But Relevant packets couldn't follow this router to transfer very occasionally. I have done the action of deleting the router of "0.0.0.0/0". It will trigger the router to work well. the relevant packets will be transfer to the defined channel.

mssonicbld added a commit that referenced this issue Jun 4, 2024
…lly (#19185)

#### Why I did it
src/sonic-swss
```
* e61e95f8 - (HEAD -> 202311, origin/202311) change log level from ERR to WARN for pfc asym sai attribute not supported (#3125) (7 hours ago) [Zhixin Zhu]
* 5a998b6e - Handle learning duplicate IPs on different VRFs (#3165) (#3179) (10 hours ago) [Lawrence Lee]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this issue Jun 4, 2024
…lly (#19132)

#### Why I did it
src/sonic-swss
```
* 98012ed4 - (HEAD -> master, origin/master, origin/HEAD) [build-docker-sonic-vs] Allowing partial build for sairedis (#3177) (3 days ago) [Nikola Dancejic]
* 6568193c - Do not apply QoS mapping item on the switch until the object is created (#3163) (3 days ago) [Stephen Sun]
* 1876a306 - Revert "Added support for "UNDERLAY_SET_DSCP" and "UNDERLAY_SET_DSCPV6" table…" (#3175) (3 days ago) [Prince Sunny]
* 1f12a8db - [subnet decap] Add subnet decap rule based on overlay ECMP (#3153) (3 days ago) [Longxiang Lyu]
* 8e8fc66c - [ci] Migrate from sonicbld to sonicbld-1es to fix S360 alert. (#3171) (4 days ago) [Liu Shilong]
* 4ae8bfa3 - Read switch_id of fabric switch from config_db (#3102) (4 days ago) [Song Yuan]
* f7376636 - Add sai call to isolate/unisolate a fabric port (#3141) (4 days ago) [jfeng-arista]
* 7c7cece4 - [Chassis][voq] remote link down ECMP acceleration (#3150) (4 days ago) [Arvindsrinivasan Lakshmi Narasimhan]
* c7ecd7db - Add fabric port monitoring toggle check (#3132) (4 days ago) [jfeng-arista]
* 9ffbcd58 - Added support for "UNDERLAY_SET_DSCP" and "UNDERLAY_SET_DSCPV6" tables (#3145) (4 days ago) [siqbal1986]
* ef9dcdbe - Handle learning duplicate IPs on different VRFs (#3165) (5 days ago) [Lawrence Lee]
* cc5a02c4 - [ci] use sonictest instead of sonic-common-test #3170 (6 days ago) [Liu Shilong]
```
#### How I did it
#### How to verify it
#### Description for the changelog
arun1355492 pushed a commit to arun1355492/sonic-buildimage that referenced this issue Jul 26, 2024
…lly (sonic-net#19132)

#### Why I did it
src/sonic-swss
```
* 98012ed4 - (HEAD -> master, origin/master, origin/HEAD) [build-docker-sonic-vs] Allowing partial build for sairedis (sonic-net#3177) (3 days ago) [Nikola Dancejic]
* 6568193c - Do not apply QoS mapping item on the switch until the object is created (sonic-net#3163) (3 days ago) [Stephen Sun]
* 1876a306 - Revert "Added support for "UNDERLAY_SET_DSCP" and "UNDERLAY_SET_DSCPV6" table…" (sonic-net#3175) (3 days ago) [Prince Sunny]
* 1f12a8db - [subnet decap] Add subnet decap rule based on overlay ECMP (sonic-net#3153) (3 days ago) [Longxiang Lyu]
* 8e8fc66c - [ci] Migrate from sonicbld to sonicbld-1es to fix S360 alert. (sonic-net#3171) (4 days ago) [Liu Shilong]
* 4ae8bfa3 - Read switch_id of fabric switch from config_db (sonic-net#3102) (4 days ago) [Song Yuan]
* f7376636 - Add sai call to isolate/unisolate a fabric port (sonic-net#3141) (4 days ago) [jfeng-arista]
* 7c7cece4 - [Chassis][voq] remote link down ECMP acceleration (sonic-net#3150) (4 days ago) [Arvindsrinivasan Lakshmi Narasimhan]
* c7ecd7db - Add fabric port monitoring toggle check (sonic-net#3132) (4 days ago) [jfeng-arista]
* 9ffbcd58 - Added support for "UNDERLAY_SET_DSCP" and "UNDERLAY_SET_DSCPV6" tables (sonic-net#3145) (4 days ago) [siqbal1986]
* ef9dcdbe - Handle learning duplicate IPs on different VRFs (sonic-net#3165) (5 days ago) [Lawrence Lee]
* cc5a02c4 - [ci] use sonictest instead of sonic-common-test sonic-net#3170 (6 days ago) [Liu Shilong]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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

No branches or pull requests

2 participants