openstack/sushy
Code Issues Proposed changes

Make Manager->Actions field optional

Browse Source 
Redfish schema does not require Actions field to be
present [1].

1. https://redfish.dmtf.org/schemas/Manager.v1_5_1.json

Change-Id: Ic9402f7639a561b88c4a9be91d1c6c88c6b12c31
This commit is contained in:
Ilya Etingof
2019-05-13 20:09:14 +02:00
parent 103737ad3c
commit a8918713fe
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
releasenotes/notes/fix-manager-action-d71fd415cea29aa6.yaml Normal file
View File

@@ -0,0 +1,7 @@
---
fixes:
- |
Makes ``Manager->Actions`` field optional as Redfish Manager schema
defines it. Otherwise sushy fails hard at parsing response from a
Redfish agent that does not include ``Actions`` field in its document
tree.

2
sushy/resources/manager/manager.py
View File

@@ -82,7 +82,7 @@ class Manager(base.ResourceBase):
uuid = base.Field('UUID') uuid = base.Field('UUID')
"""The manager UUID""" """The manager UUID"""
_actions = ActionsField('Actions', required=True) _actions = ActionsField('Actions')
def __init__(self, connector, identity, redfish_version=None): def __init__(self, connector, identity, redfish_version=None):
"""A class representing a Manager """A class representing a Manager