-
Notifications
You must be signed in to change notification settings - Fork 519
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
Fixed set admin_status for deleted subintf due to late notification #2659
Fixed set admin_status for deleted subintf due to late notification #2659
Conversation
/azpw run Azure.sonic-swss |
/AzurePipelines run Azure.sonic-swss |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check why I don't see in the PR delete of add_ipv6_prefix_ut.cpp
GitHub handles it like rename of the file |
@prsunny , Can you please review ? |
@EdenGri please help create separate PR for 202205 branch. |
…2659) * Fixed set admin_status for deleted subintf due to late notification
What I did
Handled a race condition in intfmgrd which tries to immediately apply an admin_status SET notif after a DEL causing it to crash
Why I did it
Ignores errors on the set admin_status command for subinterface when the subinterface state is not OK.
How I verified it
Details if related
This PR reference to older PR that fix the same issue in portmgr: #2431