search

sonic-net / sonic-mgmt

Configuration management examples for SONiC
Other
196 stars 716 forks source link

[ platform_tests/test_sensors.py ] - Case Skip due to not support check sensors for current platform. #8389

Open Vickyni2 opened 1 year ago

Vickyni2 commented 1 year ago

Description The Test Case "test_sensors.py" in platform_test skip due to not support check sensors for current platform in T0 . Why is it skipping on Broadcom platform .

Steps to reproduce the issue: 1.Run the test case in a physical DUT. 2. 3.

Describe the results you received:

---------------------------- live log sessionfinish ---------------------------- 20:44:08 init.pytest_terminal_summary L0064 INFO | Can not get Allure report URL. Please check logs =========================== short test summary info ============================ SKIPPED [1] /data/sonic-mgmt/tests/platform_tests/test_sensors.py:55: Skip test due to not support check sensors for current platform(x86_64-accton_as7716_32x-r0) ========================== 1 skipped in 61.85 seconds ==========================

Describe the results you expected:

The case is expected to pass .

Additional information you deem important:

**Output of `show version`:**

SONiC Software Version: SONiC.202211.269499-59c7d39ef SONiC OS Version: 11 Distribution: Debian 11.6 Kernel: 5.10.0-18-2-amd64 Build commit: 59c7d39ef Build date: Tue May 9 17:58:15 UTC 2023 Built by: AzDevOps@vmss-soni00118K Platform: x86_64-accton_as7716_32x-r0 HwSKU: Accton-AS7716-32X ASIC: broadcom ASIC Count: 1

**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
yxieca commented 1 year ago

@Vickyni2 I suspect Accton sensor definition is missing in the test repo. @prgeor can you help point to the sensor definition?

prgeor commented 1 year ago

@yxieca @Vickyni2 please add sensor data for your platform here:- https://github.com/sonic-net/sonic-mgmt/blob/master/ansible/group_vars/sonic/sku-sensors-data.yml#L2