Skip to content

Fix call check_vendor_specific_psustatus error#2016

Merged
bingwang-ms merged 2 commits intosonic-net:masterfrom
bingwang-ms:fix_check_vendor_specific_psustatus
Aug 4, 2020
Merged

Fix call check_vendor_specific_psustatus error#2016
bingwang-ms merged 2 commits intosonic-net:masterfrom
bingwang-ms:fix_check_vendor_specific_psustatus

Conversation

@bingwang-ms
Copy link
Collaborator

@bingwang-ms bingwang-ms commented Aug 3, 2020

Description of PR

Summary:
Fixes # (issue)
Recent pr #1964 break the code file structure, results in NameError in test_platform_info.py

 >               check_vendor_specific_psustatus(duthost, line)
E               NameError: global name 'check_vendor_specific_psustatus' is not defined

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

Fix NameError: global name 'check_vendor_specific_psustatus' is not defined.

How did you do it?

Add import check_vendor_specific_psustatus

How did you verify/test it?

Verified on Arista-7260 and pass.

Any platform specific information?

No.

Supported testbed topology if it's a new test case?

N/A

Documentation

@bingwang-ms bingwang-ms requested a review from a team August 4, 2020 01:05
@bingwang-ms bingwang-ms merged commit ef061c1 into sonic-net:master Aug 4, 2020
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
c31a362 - 2021-11-18 : [202012][Mux orch] set default as standby, change mux orch priority (sonic-net#2015) [Prince Sunny]
9a9e8e6 - 2021-11-18 : [202012] Check VS test failure (sonic-net#2033) [Prince Sunny]
7eaabca - 2021-11-11 : [202012] Fix random failure in PR/CI build. (sonic-net#2016) [Shilong Liu]
85230fe - 2021-11-04 : [orchagent] Fix group name of port-buffer-drop in flexcounterorch.cpp (sonic-net#1967) [Junchao-Mellanox]
a55c2ca - 2021-11-03 : [teammgrd]: Handle LAGs cleanup gracefully on Warm/Fast reboot. (sonic-net#1934) [Nazarii Hnydyn]
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