Skip to content

Commit

Permalink
Merge pull request #88 from DMTF/System-Reset-Exception
Browse files Browse the repository at this point in the history
Corrected flow when handling system reset exceptions to ensure it stays raised
  • Loading branch information
mraineri authored Aug 12, 2022
2 parents 6726318 + 817ec82 commit 76d628e
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions redfish_utilities/systems.py
Original file line number Diff line number Diff line change
Expand Up @@ -312,13 +312,14 @@ def system_reset( context, system_id = None, reset_type = None ):
try:
verify_response( response )
except Exception as e:
additional_message = ""
if response.status == 400:
# Append the list of valid reset types to 400 Bad Request responses
supported_reset_types = "No supported reset types listed"
additional_message = "\nNo supported reset types listed"
for param in reset_parameters:
if param["Name"] == "ResetType" and "AllowableValues" in param:
supported_reset_types = "Supported reset types: {}".format( ", ".join( param["AllowableValues"] ) )
raise type( e )( str( e ) + "\n" + supported_reset_types ).with_traceback( sys.exc_info()[2] )
additional_message = "\nSupported reset types: {}".format( ", ".join( param["AllowableValues"] ) )
raise type( e )( str( e ) + additional_message ).with_traceback( sys.exc_info()[2] )
return response

def get_virtual_media( context, system_id = None ):
Expand Down

0 comments on commit 76d628e

Please sign in to comment.