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

[Bug]: platform_tests/test_power_off_reboot.py needs to be updated for T2 #16279

Open
Javier-Tan opened this issue Jan 2, 2025 · 0 comments · May be fixed by #16313
Open

[Bug]: platform_tests/test_power_off_reboot.py needs to be updated for T2 #16279

Javier-Tan opened this issue Jan 2, 2025 · 0 comments · May be fixed by #16313
Assignees
Labels

Comments

@Javier-Tan
Copy link
Contributor

Javier-Tan commented Jan 2, 2025

Issue Description

Running platform_tests/test_power_off_reboot.py on a T2 device typically fails. Here are the reasons why:

  • Checks if all outlets are providing power
    • In a T2 device sometimes 2 PDUs provide power to 1 PSU for redundancy, sometimes one PDU is dormant (purely for redundancy) while another is active, the test will fail in this case
  • reboot_and_check command is missing relatively new duthosts kwarg that is used when running on sup card

There is also a related issue #16289 that occurs on T2 devices after fixing these 2 issues.

Results you see

platform_tests/test_power_off_reboot.py due to test issues not accomodating T2 devices

Results you expected to see

Tese should accomodate T2 devices

Is it platform specific

generic

Relevant log output

N/A

Output of show version

N/A

Attach files (if any)

N/A

@Javier-Tan Javier-Tan self-assigned this Jan 2, 2025
@Javier-Tan Javier-Tan linked a pull request Jan 3, 2025 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant