search

DMTF / Redfish-Service-Validator

The Redfish Service Validator is a Python3 tool for checking conformance of any "device" with a Redfish service interface against Redfish CSDL schema
Other
40 stars 34 forks source link

Edm.Duration regex does not include all formats #499

Closed tomasg2012 closed 2 years ago

tomasg2012 commented 2 years ago

The current regex pattern for Edm.Duration does not include weeks or years for duration, double check the ISO spec used by Redfish Spec to see if this is okay.

Refer to ISO 8601.

mraineri commented 2 years ago

We did deviate a bit from ISO8601 intentionally; there is ambiguity in ISO8601 regarding the duration of a month (is it strictly calendar based or could it be exactly 30 days) and a year (do you always consider the leap year length). The Redfish spec calls out the exact format, and it doesn't allow for weeks, months, or years.

mraineri commented 2 years ago

7/8: From today's discussion, this is as expected based on how the Redfish Specification is. We want to cap the units at "days" for Redfish's usage.