Fix the local variable 'dut_port' referenced before assignment#7512
Fix the local variable 'dut_port' referenced before assignment#7512yaqiangz merged 1 commit intosonic-net:masterfrom
Conversation
|
/azpw run |
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@vmittal-msft for awareness |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@nhe-NV test failed when call into the pthost_utils, could you check? |
I will check, thanks |
nhe-NV
force-pushed
the
fix_UnboundLocalError_issue
branch
from
a71779d to
e003899
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@nhe-NV Could you please resolve conflicts between this PR and master branch? |
nhe-NV
force-pushed
the
fix_UnboundLocalError_issue
branch
from
e003899 to
6e713f2
Compare
|
@yaqiangz conflict resolved |
There was a problem hiding this comment.
This is where it conflicted with another PR before
There was a problem hiding this comment.
This is where it conflicted with another PR before
Hi @nhe-NV I found that there are two place ((comment in code)) conflicted with this PR merged no long ago https://github.com/sonic-net/sonic-mgmt/pull/7385/files#diff-c7cac5f14c557607cbbd06fd58b784e8421c24354a06adb306b64a7e9dc73601. But your PR just overwrote it. No sure whether it would cause problem. Have you verified that? |
In ptfhost_utils.py, after PR sonic-net#7079 merged, there is exception "UnboundLocalError: local variable 'dut_port' referenced before assignment"
nhe-NV
force-pushed
the
fix_UnboundLocalError_issue
branch
from
6e713f2 to
7f64e6c
Compare
I did not notice there is real code change in this line, thought it is only the pre-commit issue. |
In ptfhost_utils.py, after PR #7079 merged, there is exception "UnboundLocalError: local variable 'dut_port' referenced before assignment" How did you do it? Fix the local variable 'dut_port' referenced before assignment How did you verify/test it? Run the test case which has such failure, and it pass.
6 participants
In ptfhost_utils.py, after PR #7079 merged, there is exception "UnboundLocalError: local variable 'dut_port' referenced before assignment"
Description of PR
Summary: In ptfhost_utils.py, after PR #7079 merged, there is exception "UnboundLocalError: local variable 'dut_port' referenced before assignment"
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
How did you do it?
Fix the local variable 'dut_port' referenced before assignment
How did you verify/test it?
Run the test case which has such failure, and it pass.
Any platform specific information?
No
Supported testbed topology if it's a new test case?
Documentation