openstack/sushy
Code Issues Proposed changes

Low case ParamTypes in received MessageRegistry

Browse Source 
It turns out, that some implementations encountered in the wild waters
use different cases for (at least) `ParamTypes` enumerations in
`MessageRegistry` resource. In the schema [1] these enumerations
are all in low case.

To make sushy operational, this patch just ignores the case of
these enumerations. It's still an open question as to how strict the
case requirement is with Redfish (/cc Davy Jones).

1. https://redfish.dmtf.org/schemas/v1/MessageRegistry.v1_3_0.json

Change-Id: I107b689b8f4e27dd7331473124f7108d151cc55b
This commit is contained in:
Ilya Etingof
2019-07-09 17:17:41 +02:00
parent 3189536343
commit f9a62fa606
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
sushy/resources/registry/message_registry.py
View File

@@ -37,7 +37,7 @@ class MessageDictionaryField(base.DictionaryField):
param_types = base.Field('ParamTypes',
adapter=lambda x:
[res_maps.PARAMTYPE_VALUE_MAP[v]
[res_maps.PARAMTYPE_VALUE_MAP[v.lower()]
for v in x])
"""Mapped MessageArg types, in order, for the message"""