diff options
author | Gunnar Mills <gmills@us.ibm.com> | 2021-06-24 21:30:46 +0300 |
---|---|---|
committer | Gunnar Mills <gmills@us.ibm.com> | 2021-06-25 21:07:18 +0300 |
commit | ff85ed3d82f16f3e23de5ddf9115b429a15bf364 (patch) | |
tree | 63bf2a5de7fd01159e014d043fcff8707b267f21 /scripts | |
parent | 49e429ca172365bd4ba16c924501321c73527cbe (diff) | |
download | bmcweb-ff85ed3d82f16f3e23de5ddf9115b429a15bf364.tar.xz |
Add Privileges schema to allow list
The allow list should contain all schemas implemented by bmcweb.
The allow list was missing the Privileges schema. The Privileges schema
is referenced in the Role schema here:
"AssignedPrivileges": {
"description": "The Redfish privileges for this role.",
"items": {
"$ref": "http://redfish.dmtf.org/schemas/v1/Privileges.json#/definitions/PrivilegeType"
https://redfish.dmtf.org/schemas/v1/Role.v1_3_1.json
In OpenBMC CI Redfish Validator testing the following error was hit 3
times:
ERROR - ConnectionError on http://redfish.dmtf.org/schemas/v1/Privileges_v1.xml:
ConnectionError(ProtocolError('Connection aborted.',
RemoteDisconnected('Remote end closed connection without response')))
Network glitch somewhere but inspired this change.
Long term not sure if the allow list makes sense. While still used today
(and for the immediate future) it should attempt to accurately reflect
the implemented schemas in bmcweb.
The allow list was added in 349a2ac3 to save space.
Information on the CI Redfish Validator testing can be found at
https://github.com/openbmc/openbmc-test-automation/search?q=Redfish-Service-Validator
The test redownloads the schemas each time.
Tested: Validator passed.
See the new schema:
{
"@odata.id": "/redfish/v1/JsonSchemas/Privileges"
},
Change-Id: Iaf3de36b211efab4115a17fd43f0263b8bd3107f
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/update_schemas.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/scripts/update_schemas.py b/scripts/update_schemas.py index a694549474..ade0550670 100755 --- a/scripts/update_schemas.py +++ b/scripts/update_schemas.py @@ -69,6 +69,7 @@ include_list = [ 'PCIeFunction', 'PCIeFunctionCollection', 'Power', + 'Privileges', #Used in Role 'Processor', 'ProcessorCollection', 'RedfishError', |