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

DellEMC Z9332f: Fix SFP issue #11819

Merged
merged 3 commits into from
Sep 7, 2022
Merged

Conversation

aravindmani-1
Copy link
Contributor

Why I did it

Some of the sonic-mgmt test cases failed in DellEMC Z9332f switch

How I did it

Handled the platform API to fix the issue

How to verify it

Run sonic-mgmt SFP test cases.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

UT logs:
SFP UT.txt

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 23, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: aravindmani-1 / name: Aravind Mani (9e4f571)

@aravindmani-1
Copy link
Contributor Author

aravindmani-1 commented Aug 23, 2022

/easycla

@lgtm-com
Copy link

lgtm-com bot commented Aug 23, 2022

This pull request introduces 1 alert when merging 60b5bab3f969baa1e3c659d558b177629f68574f into 42e2924 - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@aravindmani-1
Copy link
Contributor Author

/easycla

@prgeor
Copy link
Contributor

prgeor commented Aug 23, 2022

/easycla

@aravindmani-1 you need to sign the EasyCLA. Click on the "Details" and proceed

@aravindmani-1
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@aravindmani-1
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prgeor prgeor merged commit 37d34dd into sonic-net:202012 Sep 7, 2022
@aravindmani-1 aravindmani-1 deleted the fix_sfp_issues branch September 8, 2022 02:21
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