HewlettPackard / python-ilorest-library

Python library for interacting with devices which support a Redfish Service
Apache License 2.0
187 stars 92 forks source link

Api response with code 200 while setting nonsensical values to Bios #91

Closed VladoPortos closed 4 years ago

VladoPortos commented 4 years ago

I have found curious behavior ( maybe I'm doing something wrong ), using the change_bios_setting.py example, I can pass anything into property_value variable and it will still return 200 ( the bios will not update of course, but I would expect the api to know I'm passing nonsensical value ( eg property_value = "nonsense" ) and respond with proper code.

body = {'Attributes': {bios_property: property_value}}

Target is iLO 5 v2.10

KeepSummerSaf3 commented 4 years ago

This is by design. All providers which require a reboot or iloreset to accept changes will exhibit this type of behavior. In order to verify if your changes to the bios attributes were valid, reboot and after POST, "@Redfish.Settings > Messages" will contain a response message id and details for the result.

VladoPortos commented 4 years ago

Good to know, thanks @KeepSummerSaf3