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

PortChannel ip address is missing after do "systemctl restart teamd". #3150

Closed
habeebmohammed opened this issue Jul 12, 2019 · 6 comments
Closed
Assignees

Comments

@habeebmohammed
Copy link
Contributor

Description
PortChannel ip address is missing after do "systemctl restart teamd".

Steps to reproduce the issue:
root@sonic:/home/admin# show ip interfaces
Interface IPv4 address/mask Admin/Oper


Ethernet49 11.11.11.1/24 up/up
Ethernet50 12.12.12.1/24 up/down
PortChannel1 34.34.34.1/24 up/up <<== Correct
PortChannel2 56.56.56.1/24 up/up <<== Correct
docker0 240.127.1.1/24 up/down
eth0 10.6.48.133/24 up/up
lo 127.0.0.1/8 up/up
10.1.0.1/32

root@sonic:/home/admin# systemctl restart teamd <<= Do "systemctl restart teamd"

root@sonic:/home/admin# show ip interfaces
Interface IPv4 address/mask Admin/Oper


Ethernet49 11.11.11.1/24 up/up
Ethernet50 12.12.12.1/24 up/down
<<=== Issue : PortChannel 1-2 ip address is missing.
docker0 240.127.1.1/24 up/down
eth0 10.6.48.133/24 up/up
lo 127.0.0.1/8 up/up
10.1.0.1/32

Describe the results you received:

Describe the results you expected:
Ip address should be retained after teamd restart. reboot can get the ip addresses back to portchannels

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

**Output of `show version`:**

```
(paste your output here)
```

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

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

Duplicate #3043

@habeebmohammed
Copy link
Contributor Author

@stepanblyschak , I tried changing the swss.sh per #3114 still IP address is lost with "systemctl restart teamd".

@stepanblyschak
Copy link
Collaborator

@habeebmohammed #3114 appeared to be not enough to fix the issue. There is another fix sonic-net/sonic-utilities#582

@habeebmohammed
Copy link
Contributor Author

@stepanblyschak , I picked #582 changes as well, still no change in the result.

@jleveque
Copy link
Contributor

@habeebmohammed: sonic-net/sonic-utilities#582 fixes a similar issue when calling config reload or config load_minigraph -- it would restart teamd twice. That PR prevents this behavior. It sounds like you are manually reproducing this behavior. I believe that currently SONiC does not support restarting teamd without also restarting SwSS first.

@stcheng: Please confirm.

@yxieca
Copy link
Contributor

yxieca commented Sep 19, 2019

Fixed in master and 201811 branch.

@yxieca yxieca closed this as completed Sep 19, 2019
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
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

5 participants