diff options
| author | Sunny Srivastava <sunnsr25@in.ibm.com> | 2022-09-07 11:19:19 +0300 |
|---|---|---|
| committer | alpu8007 <alpu8007@gmail.com> | 2024-03-15 11:00:40 +0300 |
| commit | ad622676a894110f033a99462742d6106320cb68 (patch) | |
| tree | 57b51b97d2db9e421aa79d7f469da5c9ac1a7148 /redfish-core | |
| parent | 9bb9b107704ab5d8e23e6b92e803041e57701140 (diff) | |
| download | bmcweb-ad622676a894110f033a99462742d6106320cb68.tar.xz | |
Remove link from Chassis to PCIeDevice
The commit removes current support that assumes 1:1 system:Chassis for
Chassis/PCIeDevices.
Current implementation populates the same collection of PCIeDevices
with chassis and with system.
Since the path is /redfish/v1/Systems/system/PCIeDevices and we already
have a link from /redfish/v1/Systems/system/, removing the link here in
Chassis.
The link from Chassis to /redfish/v1/Systems/system/PCIeDevices is
unexpected.
For systems with multiple chassis the current assumption does not hold
true. It breaks there, as it assumes all PCIeDevices are in all Chassis.
This is just a link and since another link from system resource already
exists. The case of walking the whole tree isn't broke.
And so, this should not break clients.
Validator has been executed with no new errors.
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>
Change-Id: Id3af01ec99708c36b5fff2a63f04ffd722f6c3a2
Signed-off-by: Alpana Kumari <alpankum@in.ibm.com>
Diffstat (limited to 'redfish-core')
| -rw-r--r-- | redfish-core/lib/chassis.hpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/redfish-core/lib/chassis.hpp b/redfish-core/lib/chassis.hpp index e827f03db3..6d5e587b8e 100644 --- a/redfish-core/lib/chassis.hpp +++ b/redfish-core/lib/chassis.hpp @@ -508,9 +508,6 @@ inline void handleChassisGetSubTree( .jsonValue["Actions"]["#Chassis.Reset"]["@Redfish.ActionInfo"] = boost::urls::format("/redfish/v1/Chassis/{}/ResetActionInfo", chassisId); - asyncResp->res.jsonValue["PCIeDevices"]["@odata.id"] = - "/redfish/v1/Systems/system/PCIeDevices"; - dbus::utility::getAssociationEndPoints( path + "/drive", [asyncResp, chassisId](const boost::system::error_code& ec3, |