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

[tests/platform/mellanox] check PSU state against sysfs on Mellanox devices #1082

Merged
merged 6 commits into from
Sep 27, 2019
Merged

[tests/platform/mellanox] check PSU state against sysfs on Mellanox devices #1082

merged 6 commits into from
Sep 27, 2019

Conversation

stephenxs
Copy link
Contributor

Description of PR

Summary:
Check PSU state against sysfs on Mellanox devices

Type of change

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

Approach

How did you do it?

check PSU state against sysfs on Mellanox devices
the PSU state and sysfs containing should be consistent, as the following table.

PSU state          psuX_status         psuX_pwr_status
OK                  1                          1
NOT OK              1                          0
NOT PRESENT         0                          -

How did you verify/test it?

run test on all testbed

Any platform specific information?

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

Documentation

@liat-grozovik liat-grozovik merged commit bbc4a1f into sonic-net:master Sep 27, 2019
lguohan pushed a commit that referenced this pull request Oct 8, 2019
* Porting back pytest change from master to 201811

  update device info to add more facts
  add log analyzer
  add check daemon status test
  add check interface status test
  add Mellanox check sfp presence test
  update reboot, config reload and sequential restart test
  update sfp test
  update check sysfs test
  update platform fixture

* fix review comments

Rebase to add some new master PR:

  #1130 [platform] Fix the reboot SONiC stuck issue

  #1120 [platform] Disable log analyzer for the reload and restart cases

  #1125 [pytest] Fix pytest conftest.py issue

* update loganalyzer ignore log

* [tests/platform/mellanox] check PSU state against sysfs on Mellanox devices (#1082)

* [psu test case] check psu state against vendor specific info. for mellanox, check sysfs

* [test_platform_info.py]handle "NOT PRESENT" in test_show_platform_psustatus

* [psu testcase] address comments

Conflicts:
	tests/platform/mellanox/check_sysfs.py

* [check_sysfs] rewords.

* [check_sysfs.py] rewording

Conflicts:
	tests/platform/mellanox/check_sysfs.py

* reduce redundant code and rename function

* remove redundant code
@stephenxs stephenxs deleted the pr-psu-sysfs branch October 14, 2019 05:11
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.

4 participants