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

[xcvrd] Fix issue: logical_port_name is not defined #482

Merged
merged 3 commits into from
May 1, 2024

Conversation

Junchao-Mellanox
Copy link
Collaborator

@Junchao-Mellanox Junchao-Mellanox commented Apr 29, 2024

Description

The code uses a variable named "logical_port_name", but that variable is not defined in the code context. I believe the code was copied from some where without modifying the variable name. The PR is to fix it.

Motivation and Context

Fix issue: logical_port_name is not defined

How Has This Been Tested?

Manual test

Additional Information (Optional)

keboliu
keboliu previously approved these changes Apr 29, 2024
prgeor
prgeor previously approved these changes Apr 29, 2024
@prgeor
Copy link
Collaborator

prgeor commented Apr 29, 2024

@Junchao-Mellanox build is failing...

@prgeor prgeor added the Bug label Apr 29, 2024
@mihirpat1
Copy link
Contributor

@Junchao-Mellanox Please help in fixing the code coverage check

@Junchao-Mellanox Junchao-Mellanox dismissed stale reviews from prgeor and keboliu via f47a8f9 April 30, 2024 01:52
@Junchao-Mellanox
Copy link
Collaborator Author

Hi @prgeor , UT added, could you please review?

@prgeor prgeor merged commit 7e18648 into sonic-net:master May 1, 2024
5 checks passed
@yxieca
Copy link

yxieca commented May 13, 2024

@Junchao-Mellanox @prgeor I don't have enough information from PR comment to understand the change and issue it is addressing. Please update the PR comments with more information. @prgeor do you agree to backport this change?

@Junchao-Mellanox Junchao-Mellanox deleted the fix-var branch May 14, 2024 01:47
@Junchao-Mellanox
Copy link
Collaborator Author

Hi @yxieca , I added some description. Please kindly review again.

mssonicbld pushed a commit to mssonicbld/sonic-platform-daemons that referenced this pull request May 16, 2024
* [xcvrd] Fix issue: logical_port_name is not defined

* Add UT

---------

Co-authored-by: Prince George <45705344+prgeor@users.noreply.github.com>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #488

mssonicbld pushed a commit that referenced this pull request May 16, 2024
* [xcvrd] Fix issue: logical_port_name is not defined

* Add UT

---------

Co-authored-by: Prince George <45705344+prgeor@users.noreply.github.com>
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.

None yet

6 participants