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

[Teamd] Adding port member failed #3687

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

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 arheneus@marvell.com

- 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 <arheneus@marvell.com>
…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 <arheneus@marvell.com>
@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

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