summaryrefslogtreecommitdiff
path: root/redfish-core/lib/redfish_v1.hpp
diff options
context:
space:
mode:
authorJohnathan Mantey <johnathanx.mantey@intel.com>2022-08-16 00:38:36 +0300
committerEd Tanous <ed@tanous.net>2022-08-30 05:56:37 +0300
commit58283f41276f86666aca6943e7405a20a1ff38b6 (patch)
treeccb41d298191ca90f41d7b234ebe4532abb1dfb8 /redfish-core/lib/redfish_v1.hpp
parentdb18fc98241b4fc9a4274f45c1d6b07d7bd1b93d (diff)
downloadbmcweb-58283f41276f86666aca6943e7405a20a1ff38b6.tar.xz
Report MACAddress is read-only for a D-Bus Not Allowed response
The BMC Manager MACAddress ethernet property is a Read/Write property in the Redfish spec. There are OpenBMC users that configure the property to be Read Only. The phosphor-network build includes a 'persist-mac' configuration switch which controls MAC address assignment. An attempt to set a R/O MACAddress causes D-Bus to return an error. The exact error is available in a sdbus message. The EthernetInterface XML indicates this is acceptable behavior: "If an assignable MAC address is not supported, this value is a read-only alias of the PermanentMACAddress." As this condition is considered proper behavior it is incorrect to return an internalError(). It is better behavior to return a "Read-only" error message. Tested: Performed a Redfish PATCH for the MACAddress property. The PATCH command returns a 403 error code, and a message body indicating that the MACAddress is not writable. Change-Id: Ice97affe694f4bee15436293c9e5944bcae7f4cc Signed-off-by: Johnathan Mantey <johnathanx.mantey@intel.com>
Diffstat (limited to 'redfish-core/lib/redfish_v1.hpp')
0 files changed, 0 insertions, 0 deletions