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

[frr] prefers ipv6 link-local as next hop address #2986

Closed
stepanblyschak opened this issue Jun 10, 2019 · 3 comments
Closed

[frr] prefers ipv6 link-local as next hop address #2986

stepanblyschak opened this issue Jun 10, 2019 · 3 comments

Comments

@stepanblyschak
Copy link
Collaborator

Description
After a change to use seperated configuration mode, frr prefers to use link-local ipv6 addresses as nexthop address. This is behavior change comparing to quagga and also orchagent fails to proceses such routes and takes 100% cpu, because tries to insert such route in an endless loop

Steps to reproduce the issue:

  1. Run public image 990 on t0/t1/t1-lag topologies
  2. Execute "show ipv6 route"

Describe the results you received:
In "show ipv6 route" output you can see link-local addresses are used as next hop addresses

Describe the results you expected:
Either change frr to prefer global ipv6 or make link-local next hop be processed correctly by orchagent

Additional information you deem important (e.g. issue happens only occasionally):

**Output of `show version`:**

```

SONiC Software Version: SONiC.HEAD.990-9dc1cc76
Distribution: Debian 9.9
Kernel: 4.9.0-8-2-amd64
Build commit: 9dc1cc7
Build date: Sun Jun 9 03:14:52 UTC 2019
Built by: johnar@jenkins-worker-4

Platform: x86_64-mlnx_msn2010-r0
HwSKU: ACS-MSN2010
ASIC: mellanox
Serial Number: MT1749X10061
Uptime: 15:27:17 up 2 min, 1 user, load average: 7.28, 3.36, 1.29

Docker images:
REPOSITORY TAG IMAGE ID SIZE
docker-syncd-mlnx HEAD.990-9dc1cc76 1cddd3463b02 369MB
docker-syncd-mlnx latest 1cddd3463b02 369MB
docker-lldp-sv2 HEAD.990-9dc1cc76 4b03a7a67280 299MB
docker-lldp-sv2 latest 4b03a7a67280 299MB
docker-dhcp-relay HEAD.990-9dc1cc76 8f06c5b8462d 288MB
docker-dhcp-relay latest 8f06c5b8462d 288MB
docker-database HEAD.990-9dc1cc76 09aa9e4be928 281MB
docker-database latest 09aa9e4be928 281MB
docker-snmp-sv2 HEAD.990-9dc1cc76 f82ee7831a47 313MB
docker-snmp-sv2 latest f82ee7831a47 313MB
docker-orchagent HEAD.990-9dc1cc76 ec34623eb382 319MB
docker-orchagent latest ec34623eb382 319MB
docker-teamd HEAD.990-9dc1cc76 ebe6b0ef5acc 301MB
docker-teamd latest ebe6b0ef5acc 301MB
docker-sonic-telemetry HEAD.990-9dc1cc76 d135f140b0ae 302MB
docker-sonic-telemetry latest d135f140b0ae 302MB
docker-router-advertiser HEAD.990-9dc1cc76 eececba36c7f 280MB
docker-router-advertiser latest eececba36c7f 280MB
docker-platform-monitor HEAD.990-9dc1cc76 b7bcd1686023 326MB
docker-platform-monitor latest b7bcd1686023 326MB
docker-fpm-frr HEAD.990-9dc1cc76 707e7322552e 309MB
docker-fpm-frr latest 707e7322552e 309MB
```

**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
@stephenxs
Copy link
Collaborator

I'm just curious that why neighsync disables link-local address from notifying orchagent. is there any reason behind?

lguohan pushed a commit that referenced this issue Jun 12, 2019
* [FRR]: Aligned configs for unified/separated modes.
* Fixed IPv6 neighbors issue: #2986

Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>
@lguohan
Copy link
Collaborator

lguohan commented Jun 12, 2019

link-local is not supported yet.

@lguohan lguohan closed this as completed Jun 12, 2019
@nazariig
Copy link
Collaborator

@lguohan do we have any estimates for this: sonic-net/sonic-swss#437

mssonicbld added a commit that referenced this issue Dec 14, 2023
…lly (#17501)

#### Why I did it
src/sonic-swss
```
* ff524e6d - (HEAD -> master, origin/master, origin/HEAD) [dash] add a retry for an ACL rule creation if a tag is not created yet (#2972) (7 hours ago) [Yakiv Huryk]
* 620db3da - [ci] Allow partially success build artifact in PR checker pipeline. #2986 (3 days ago) [Liu Shilong]
* d357e6f1 - [copporch] Add safeguard during policer attribute update (#2977) (4 days ago) [Vivek]
* cb460394 - [fpmsyncd][WR] Relax the static schema constraint for ROUTE_TABLE (#2981) (5 days ago) [Vivek]
* a1ce21f6 - Change base directory referenced in coverage.xml (#2976) (6 days ago) [Lawrence Lee]
* 920959cf - [Dash] [UT] Add ZMQ test case for dash (#2967) (6 days ago) [Hua Liu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this issue Mar 6, 2024
…lly (#18274)

#### Why I did it
src/sonic-swss
```
* dd1432a2 - (HEAD -> 202311, origin/202311) [ci] Allow partially success build artifact in PR checker pipeline. #2986 (10 hours 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

4 participants