Ipmi.Test Ipmi Sdr.Test GPU SDR Info At Power On - passes when GPU not present

[generatz]

When there is no GPU present the test still passes. That seems odd - other GPU tests don't pass if the GPU is not present.
The documentation for the test doesn't clarify the expectation.

[gkeishin]

@rahulmah can you check on this one ?

[gkeishin]

or @AshwiniC777 please take a look

[gkeishin]

URI:https://xx.xx.xx.xx:443/xyz/openbmc_project/list, method:Get, args:{'headers': {'X-Auth-Token': 'Pf5ZdNrzj959cqNkpnNf', 'Accept': 'application/json'}}
Response code:404, Content:Not Found
Test GPU SDR Info At Power On :: Verify GPU SDR info via IPMI and ... | FAIL |
Parent suite setup failed:
404 != 200

not sure it shouldn't be working, at least on Redfish since it needs the new changes for it.. but yeah for older REST, it probably might but it could be a bug from the firmware end and not from test code.. @generatz

[generatz]

This patch works for me:

diff --git a/ipmi/test_ipmi_sdr.robot b/ipmi/test_ipmi_sdr.robot
index 34f357e8..f9bd70f4 100755
--- a/ipmi/test_ipmi_sdr.robot
+++ b/ipmi/test_ipmi_sdr.robot
@@ -275,6 +275,8 @@ Test SDR Info
     # component_name    Component name (e.g. "core", "dimm", etc.).
 
     ${component_uri_list}=  Get Component URIs  ${component_name}
+    ${length}=  Get Length  ${component_uri_list}
+    Skip If  '${length}'=='0'  msg=${component_name} not found.
 
     FOR  ${uri}  IN  @{component_uri_list}
       ${component_name}=  Fetch From Right  ${uri}  motherboard/

[matthewfischer]

I looked at this but this whole thing needs to be re-written to use Redfish and not the OpenBMC REST API, unfortunately my qemu based system doesn't report any sensors via RF.

[gkeishin]

Yes, this is an old REST API implementation and needs a total revamp to Redfish.

[gkeishin]

There are efforts around it from community example https://gerrit.openbmc.org/c/openbmc/openbmc-test-automation/+/78087

having said that it hasn't progressed much.