-
Notifications
You must be signed in to change notification settings - Fork 730
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 asichost mac when DUT is multi-asic system #15682
base: master
Are you sure you want to change the base?
Use asichost mac when DUT is multi-asic system #15682
Conversation
|
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
dcdcdef
to
aa03ad8
Compare
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
aa03ad8
to
77408d8
Compare
viz @abdosi . Please review/merge the PR. |
Description of PR
Summary: Changes to support DUT with Multi-ASIC
Fixes # (issue)
Packets send to DUT should have ASIC MAC has DMAC instead of DUT MAC. Fixed the same
When number of routes learnt are less, stress test is not run. Added a log message when this happens
Type of change
Back port request
Approach
What is the motivation for this PR?
pass all sonic mgmt tests on multi-asic systems
How did you do it?
Ran scripts with multi-asic system as DUT and recorded the list
How did you verify/test it?
Ran the test_stress_arp.py script with fix and test-case passed
Any platform specific information?
if all ASIC in platform share the same MAC as DUT MAC this fix is not needed
Supported testbed topology if it's a new test case?
Documentation