-
Notifications
You must be signed in to change notification settings - Fork 691
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
Use config_reload when recovering from internal interfaces that are in down state #6076
Conversation
…n down state If internal interfaces on a linecard are down as part of sanity_check, then to recover we should just do 'config reload' as there are no peers for this that can be bounced.
/easycla |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
…n down state (#6076) If internal interfaces on a linecard are down as part of sanity_check, then to recover we should just do 'config reload' as there are no peers for this that can be bounced.
…Azure/sonic-mgmt into internal Fix merge conflicts. Commit history: - Modified args for some of the calls (sonic-net#6605) - Run telemetry tests on all DUT types in a multi-dut setup (sonic-net#6098) - [dualtor]: Fix loopback route removal (sonic-net#6555) - Added support for one portChannel and one routed interface if two por… (sonic-net#5331) - Use config_reload when recovering from internal interfaces that are in down state (sonic-net#6076) - Added python fixture for resetting config changes to fix nightly backend snmp test failures (sonic-net#6607) - Fix Python3 RuntimeError - dictionary changed size during iteration (sonic-net#6646) - [mx] Add docs for mx topo (sonic-net#6634) - [topo_mx] Add new topo for BMC Mgmt ToR Router (sonic-net#6636) - [mx] Enhance announce routes to support MX topo (sonic-net#6635) - Merge branch 'azure-master' into dev/yaqiangzhu/internal_merge
If internal interfaces on a linecard are down as part of sanity_check, then to recover
we should just do 'config reload' as there are no peers for this that can be bounced.
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
If internal ports (inband, Recycle, or Internal) on a multi-asic box are down, then we should do a 'config reload' as part of recovery in sanity check. There are no external peers for these ports that can be bounced to bring this ports up
How did you do it?
When iterating through down interfaces, if they have '-IB', or '-Rec' or '-Int' then set the recovery method as config_reload
How did you verify/test it?
Tested against a multi-asic linecard in a T2 chassis
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation