regulators: Make inventory_path required property
Make "inventory_path" a required property of the "chassis" object in the
regulators JSON configuration file.
This property is used to create the chassis <-> sensor associations
required by bmcweb for finding regulator sensors.
The property is already documented in the markdown file as being
required. However, that requirement was not being enforced in the JSON
schema nor in the C++ configuration file parser.
The existing configuration files did not have the property, and
requiring it would have caused errors.
Those configuration files have been updated to include the property, so
the requirement is now being enforced.
Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: If9af7969f8b92068e66c750681179db70eae489e
diff --git a/phosphor-regulators/src/config_file_parser.cpp b/phosphor-regulators/src/config_file_parser.cpp
index 619a8b6..db0cd7a 100644
--- a/phosphor-regulators/src/config_file_parser.cpp
+++ b/phosphor-regulators/src/config_file_parser.cpp
@@ -213,14 +213,11 @@
}
++propertyCount;
- // Optional inventory_path property. Will be required in future.
- std::string inventoryPath{"/xyz/openbmc_project/inventory/system/chassis"};
- auto inventoryPathIt = element.find("inventory_path");
- if (inventoryPathIt != element.end())
- {
- inventoryPath = parseInventoryPath(*inventoryPathIt);
- ++propertyCount;
- }
+ // Required inventory_path property
+ const json& inventoryPathElement =
+ getRequiredProperty(element, "inventory_path");
+ std::string inventoryPath = parseInventoryPath(inventoryPathElement);
+ ++propertyCount;
// Optional devices property
std::vector<std::unique_ptr<Device>> devices{};