Skip to content

[Teamd] Adding port member failed #3687

Open
antony-rheneus wants to merge 7 commits intosonic-net:masterfrom
antony-rheneus:teamd_port_init
Open

[Teamd] Adding port member failed #3687
antony-rheneus wants to merge 7 commits intosonic-net:masterfrom
antony-rheneus:teamd_port_init

Conversation

@antony-rheneus
Copy link
Contributor

during port init in "team_option *team_get_option()"

which got err as -ENOENT (-2).
If port init failed, it does rollback and remove the LAG member and never adds back until reboot
TO avoid this, retry the port init 30 times.
Failure LOG from teamd.log:
Ethernet50: Failed to find "enabled" option.
get_ifinfo_list: check_call_change_handers failed
Failed to get interface information list.
Failed to refresh interface information list.
Ethernet50: Team refresh failed.

Signed-off-by: Antony Rheneus [email protected]

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

…eam_get_option()"

which got err as -ENOENT (-2).
If port init failed, it does rollback adn remove the LAG member and never adds back until reboot
TO avoid this, retry the port init 30 times.
Failure LOG from teamd.log:
 Ethernet50: Failed to find "enabled" option.
 get_ifinfo_list: check_call_change_handers failed
 Failed to get interface information list.
 Failed to refresh interface information list.
 Ethernet50: Team refresh failed.

Signed-off-by: Antony Rheneus <[email protected]>
…eam_get_option()"

which got err as -ENOENT (-2).
If port init failed, it does rollback adn remove the LAG member and never adds back until reboot
TO avoid this, retry the port init 30 times.
Failure LOG from teamd.log:
 Ethernet50: Failed to find "enabled" option.
 get_ifinfo_list: check_call_change_handers failed
 Failed to get interface information list.
 Failed to refresh interface information list.
 Ethernet50: Team refresh failed.

Signed-off-by: Antony Rheneus <[email protected]>
@pavel-shirshov pavel-shirshov self-assigned this Nov 5, 2019
@pavel-shirshov
Copy link
Contributor

@antony-rheneus How to reproduce the issue? Thanks

@antony-rheneus
Copy link
Contributor Author

@antony-rheneus How to reproduce the issue? Thanks

It is random and not consistent.
Issue is seen only on continuous reboot.

@pavel-shirshov
Copy link
Contributor

Can you please test my PR #3725 ?
I hope it will fix the root cause of the issue

@antony-rheneus
Copy link
Contributor Author

Can you please test my PR #3725 ?
I hope it will fix the root cause of the issue

Ok sure, will update you.

@pavel-shirshov
Copy link
Contributor

Hi @antony-rheneus
The previous PR didn't work for Mellanox team
Can you please check another PR which is #3753 ?
I still can't reproduce the issue on my dut

mssonicbld added a commit that referenced this pull request Jun 7, 2025
…lly (#22884)

#### Why I did it
src/sonic-swss
```
* db7d939 - (HEAD -> master, origin/master, origin/HEAD) Remove cache for high volume DASH objects (#3534) (4 hours ago) [Lawrence Lee]
* 80294d5 - add parseBoolList in request parser (#3675) (9 hours ago) [Jing Zhang]
* a7607be - [fpmsyncd]Fixing fpmsyncd to handle routes without protocol (#3657) (11 hours ago) [Sudharsan Dhamal Gopalarathnam]
* 8067dee - [muxorch] Catch error when checking active state of missing neighbor (#3674) (11 hours ago) [Nikola Dancejic]
* d073fc7 - [vstest]: Skip flaky icmp echo tests (#3687) (12 hours ago) [prabhataravind]
```
#### 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants