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

[2020 platform_tests] TestChassisApi::test_status_led failure on vms20-t0-7050cx3-1 #8464

Closed
sujinmkang opened this issue Aug 13, 2021 · 1 comment · Fixed by #8530
Closed
Assignees

Comments

@sujinmkang
Copy link
Collaborator

Description

Latest 202012 image has platform_tests/api/test_chassis.py::TestChassisApi::test_status_led failure on vms20-t0-7050cx3-1

Steps to reproduce the issue:

  1. run platform_tests/api/test_chassis.py on arista-7050cx3_32s

Describe the results you received:

platform_tests/api/test_chassis.py:444:


self = <test_chassis.TestChassisApi object at 0x7f05788d1210>

def assert_expectations(self):
    """
    Checks if there are any error messages waiting in failed_expectations.
    If so, it will fail an assert and pass a concatenation of all pending
    error messages. It will also clear failed_expectations to prepare it
    for the next use.
    """
    if len(self.failed_expectations) > 0:
        err_msg = ", ".join(self.failed_expectations)
        # TODO: When we move to Python 3.3+, we can use self.failed_expectations.clear() instead
        del self.failed_expectations[:]
      pytest_assert(False, err_msg)

E Failed: Failed to perform set_status_led, Failed to perform set_status_led, Failed to set status_led to fault, Failed to perform set_status_led, Failed to set status_led to normal, Failed to perform set_status_led, Failed to set status_led to off

err_msg = 'Failed to perform set_status_led, Failed to perform set_status_led, Failed to set status_led to fault, Failed to perform set_status_led, Failed to set status_led to normal, Failed to perform set_status_led, Failed to set status_led to off'
self = <test_chassis.TestChassisApi object at 0x7f05788d1210>

Describe the results you expected:

PASSED or SKIPPED

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@Staphylo
Copy link
Collaborator

This API is implemented but failing indeed. I'll have a look.

@sujinmkang sujinmkang linked a pull request Aug 20, 2021 that will close this issue
sujinmkang pushed a commit that referenced this issue Aug 20, 2021
Fix Chassis.get_name to return the same value than what's in platform.json
Fix Chassis.get_system_eeprom_info when running from within pmon.
Fix Watchdog.get_remaining_time (fixes [202012 platform_tests] TestWatchdogApi::test_remaining_time failure on vms20-t1-7050cx3-3.1 #8440 and [ 202012 platform_tests ] TestWatchdogApi::test_arm_disarm_states failure on vms20-t1-7050cx3-3.1 #8439)
Implement missing thermal infos and conditions (fixes [202012 platform_tests] test_platform_info.py::test_thermal_control_psu_absence error #8453)
Fix Chassis.set_status_led return value (fixes [2020 platform_tests] TestChassisApi::test_status_led failure on vms20-t0-7050cx3-1  #8464)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants