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

LAGs are not updated correctly after teamd restart #497

Closed
marian-pritsak opened this issue Apr 12, 2017 · 4 comments
Closed

LAGs are not updated correctly after teamd restart #497

marian-pritsak opened this issue Apr 12, 2017 · 4 comments
Assignees
@stcheng
Labels
Bug 🐛

Comments

@marian-pritsak
Copy link
Collaborator

Reproduced on:

build_version: HEAD.211-5585221
debian_version: 8.7
kernel_version: 3.16.0-4-amd64
asic_type: mellanox
build_date: Wed Apr 12 05:07:40 UTC 2017
built_by: johnar@jenkins-worker-1

Steps to reproduce:

$ systemctl restart teamd

After I restart teamd, some LAGs are not working anymore. From SDK dump I can see that part of them have no members on hardware, some have only one, and some have two.
@marian-pritsak
Copy link
Collaborator Author 

127.0.0.1:6379> keys LAG_TABLE*
 1) "LAG_TABLE:PortChannel56:Ethernet60"
 2) "LAG_TABLE:PortChannel16:Ethernet16"
 3) "LAG_TABLE:PortChannel0"
 4) "LAG_TABLE:PortChannel24:Ethernet28"
 5) "LAG_TABLE:PortChannel24"
 6) "LAG_TABLE:PortChannel48:Ethernet48"
 7) "LAG_TABLE:PortChannel24:Ethernet24"
 8) "LAG_TABLE:PortChannel32:Ethernet32"
 9) "LAG_TABLE:PortChannel0:Ethernet0"
10) "LAG_TABLE:PortChannel40:Ethernet40"
11) "LAG_TABLE:PortChannel8:Ethernet12"
12) "LAG_TABLE:PortChannel0:Ethernet4"
13) "LAG_TABLE:PortChannel8:Ethernet8"
14) "LAG_TABLE:PortChannel40:Ethernet44"
15) "LAG_TABLE:PortChannel48"
16) "LAG_TABLE:PortChannel32:Ethernet36"
17) "LAG_TABLE:PortChannel8"
18) "LAG_TABLE:PortChannel16:Ethernet20"
19) "LAG_TABLE:PortChannel56:Ethernet56"
20) "LAG_TABLE:PortChannel32"
21) "LAG_TABLE:PortChannel16"
22) "LAG_TABLE:PortChannel40"
23) "LAG_TABLE:PortChannel48:Ethernet52"
24) "LAG_TABLE:PortChannel56"

@marian-pritsak marian-pritsak changed the title LAGs are not updated in correctly after teamd restart LAGs are not updated correctly after teamd restart Apr 12, 2017
@marian-pritsak
Copy link
Collaborator Author 

127.0.0.1:6379> hget LAG_TABLE:PortChannel56:Ethernet0 status
(nil)
127.0.0.1:6379> hget LAG_TABLE:PortChannel56:Ethernet8 status
(nil)
127.0.0.1:6379> hget LAG_TABLE:PortChannel56:Ethernet12 status
(nil)

@stcheng stcheng self-assigned this Apr 12, 2017
@marian-pritsak marian-pritsak mentioned this issue Apr 27, 2017
Closed
@stcheng
Copy link
Contributor

stcheng commented Apr 28, 2017

Hello @marian-pritsak , i cannot reproduce this issue:

127.0.0.1:6379> hgetall LAG_MEMBER_TABLE:PortChannel04:Ethernet124
1) "status"                                                       
2) "enabled"

this is get after docker-teamd restart.

@stcheng
Copy link
Contributor

stcheng commented Apr 28, 2017

i think on your case, it is because the docker-teamd restart causes the LAG fails to be brought up. And that is the reason (nil) appears in the redis database.

@lguohan lguohan closed this as completed Mar 11, 2018
@wendani wendani mentioned this issue Apr 9, 2019
Merged
lguohan pushed a commit that referenced this issue Apr 12, 2019
@wendani @lguohan
[submodule]: submodule update for PFC watchdog warm reboot commits (#…
9b6a2fb 
…2755)

sonic-sairedis:
e8cb879 Make object list deterministic when iterating (#438)
5486f97 Ignore ACL_COUNTER bytes and packets during comparison logic (#443)
b138ff9 Notify OA about exception and process only restart query events (#437)
0974a43 Set MTU value on created tap device for virtual switch (#436)
e2f50e8 Increase eth buffer size to 16k for virtual switch (#435)
79fb388 [bfn] Ged rid of ld_preload. Link against libsai only (#429)
2e47b78 Remove MAC alignment WA for Mellanox platforms. (#430)
e1354fe Add pre match to comparison logic and unittests (#423)
18a5ebb Drop FDB notifications if they contain invalid OIDs (#428)

sonic-swss-common:
8af58ad sonic-swss-common: Add vxlan macros to schema (#269)
76837bf Make class Select support batch read from selectables (#270)
4cf643e Add multiple fields hdel support (#267)
a710529 Update PFC_WD table name in CONFIG_DB (#266)
3c452c1 Update README.md (#268)

sonic-swss:
e329dbd Survive pfc watchdog storm action across warm-reboot (#794)

sonic-utilities:
6ee0aea (HEAD, origin/master, origin/HEAD) [config]: Change the order of interface commands (#504)
5ae30d2 [show vlan brief] Support 'alias' interface naming mode (#497)
bafebf9 Update neighbor advertiser (#498)
fa90083 [clear/main.py]: clear ndp command. (#450)
65f69e4 [show interface neighbor expected] Support 'alias' interface naming mode (#495)
aae39e7 updated show ipv6 interface for alias mode (#493)
170fed9 [warm-reboot] initialize warm reboot state table before warm rebooting (#492)
06cd99f Allow config shutdown and startup operations on valid PortChannel interface names (#474)
98cdebb [show ip interface] Add support for 'alias' interface naming mode (#486)
5f1de81 [show] Add serial numbers/uptime/hwinfo to 'show version' output (#488)
e78a866 [route_check] Move scripts under scripts/ directory, add to setup.py (#489)
d347527 Change PFC watchdog CONFIG_DB table name from PFC_WD_TABLE to PFC_WD (#475)

Signed-off-by: Wenda Ni <wenni@microsoft.com>
yxieca added a commit to yxieca/sonic-buildimage that referenced this issue May 30, 2019
@yxieca
[201811][utilities] advance utilties sub module head
e8bcbba 
Submodule src/sonic-utilities 4488525..6b4d1a0:
  > [show vlan brief] Support 'alias' interface naming mode (sonic-net#497)
  > [show interface neighbor expected] Support 'alias' interface naming mode (sonic-net#495)
  > updated show ipv6 interface for alias mode (sonic-net#493)
  > [show] Add serial numbers/uptime/hwinfo to 'show version' output (sonic-net#488)
  > [show] show interface status added vlan and portchannels to command (sonic-net#483)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@yxieca yxieca mentioned this issue May 30, 2019
Merged
yxieca added a commit that referenced this issue May 30, 2019
@yxieca
[201811][utilities] advance utilties sub module head (#2960)
fbe55e9 
Submodule src/sonic-utilities 4488525..6b4d1a0:
  > [show vlan brief] Support 'alias' interface naming mode (#497)
  > [show interface neighbor expected] Support 'alias' interface naming mode (#495)
  > updated show ipv6 interface for alias mode (#493)
  > [show] Add serial numbers/uptime/hwinfo to 'show version' output (#488)
  > [show] show interface status added vlan and portchannels to command (#483)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Kalimuthu-Velappan pushed a commit to Kalimuthu-Velappan/sonic-buildimage that referenced this issue Sep 12, 2019
@tsvanduyn @jleveque
[show vlan brief] Support 'alias' interface naming mode (sonic-net#497)
5ae30d2
mssonicbld added a commit that referenced this issue Jun 11, 2024
@mssonicbld
[submodule] Update submodule sonic-platform-daemons to the latest HEA…
0b9ff35 
…D automatically (#19271)

#### Why I did it
src/sonic-platform-daemons
```
* efb4dbd - (HEAD -> master, origin/master, origin/HEAD) [CMIS] Skip re-init flow for SW-controlled ports in case of fastboot (#497) (11 hours ago) [Stepan Blyshchak]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this issue Jul 13, 2024
@mssonicbld
[submodule] Update submodule sonic-platform-daemons to the latest HEA…
54b6380 
…D automatically (#19553)

#### Why I did it
src/sonic-platform-daemons
```
* 41b1c82 - (HEAD -> 202405, origin/202405) [CMIS] Skip re-init flow for SW-controlled ports in case of fastboot (#497) (4 weeks ago) [Stepan Blyshchak]
```
#### 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
@mssonicbld @arun1355492
[submodule] Update submodule sonic-platform-daemons to the latest HEA…
2549890 
…D automatically (sonic-net#19271)

#### Why I did it
src/sonic-platform-daemons
```
* efb4dbd - (HEAD -> master, origin/master, origin/HEAD) [CMIS] Skip re-init flow for SW-controlled ports in case of fastboot (sonic-net#497) (11 hours ago) [Stepan Blyshchak]
```
#### 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
Assignees

@stcheng stcheng

Labels
Bug 🐛
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lguohan @stcheng @marian-pritsak