search

openbmc / openbmc-test-automation

Apache License 2.0
106 stars 92 forks source link

Redfish DateTime Automation: Verify Set DateTime With Offset Using Redfish #2247

Open justin0309 opened 2 months ago

justin0309 commented 2 months ago

At present, it seems that redfish specifies iso does not support the function of automatically adding a day after more than 24 hours? Is this test item wrong? Because the date-time setting range is only 0-23 image

Gavin860606 commented 2 months ago

I have encountered the same issue, and the reason is bmcweb has refactored the datetime parsing mechanism. https://github.com/openbmc/bmcweb/commit/1b8b02a4ab0aae072d0891301d1f9d5376e7912b

As you can see, the dateStringToEpoch now won't parse the date over 24h, thats why Redfish Set DateTime ${date_time_with_offset} will failed due to invalid http 400 response code

According to DSP0266 (similar information to https://github.com/openbmc/openbmc-test-automation/issues/2247#issue-2504728609 table)

I think the test case should remove since redfish doesn't accept this kind of date time with offset.