-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
DellEMC: Z9332f platform API changes #8258
DellEMC: Z9332f platform API changes #8258
Conversation
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.
is_xcvr_optical() needs fix for QSFP-DD https://github.com/Azure/sonic-mgmt/blob/5697eea34d4d5261ff69350098ce67188b8b1c81/tests/platform_tests/api/test_sfp.py#L150
platform/broadcom/sonic-platform-modules-dell/z9332f/sonic_platform/sfp.py
Outdated
Show resolved
Hide resolved
platform/broadcom/sonic-platform-modules-dell/z9332f/sonic_platform/sfp.py
Outdated
Show resolved
Hide resolved
sonic-net/sonic-mgmt#3888 has been raised to fix the test script issue. |
Why I did it platform test suite failed for few API's in DellEMC Z9332f platform. How I did it Modified the API's to return the expected values in the script. How to verify it Run platform test suite after making the changes.
Why I did it platform test suite failed for few API's in DellEMC Z9332f platform. How I did it Modified the API's to return the expected values in the script. How to verify it Run platform test suite after making the changes.
Why I did it platform test suite failed for few API's in DellEMC Z9332f platform. How I did it Modified the API's to return the expected values in the script. How to verify it Run platform test suite after making the changes.
Why I did it
platform test suite failed for few API's in DellEMC Z9332f platform.
How I did it
Modified the API's to return the expected values in the script.
How to verify it
Run platform test suite after making the changes.
Which release branch to backport (provide reason below if selected)
Description for the changelog
Platform test suite has passed after making the API changes.
UT:
z9332f_UT.txt
A picture of a cute animal (not mandatory but encouraged)