regulators: Implements support for configuration
Enhance the configuration file parser to support the configuration
element.
Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I6862c9784a3ade6378eb7a1d19a174f1515fc845
diff --git a/phosphor-regulators/src/config_file_parser.cpp b/phosphor-regulators/src/config_file_parser.cpp
index 3d56629..ce95d2d 100644
--- a/phosphor-regulators/src/config_file_parser.cpp
+++ b/phosphor-regulators/src/config_file_parser.cpp
@@ -225,6 +225,37 @@
return chassis;
}
+std::unique_ptr<Configuration> parseConfiguration(const json& element)
+{
+ verifyIsObject(element);
+ unsigned int propertyCount{0};
+
+ // Optional comments property; value not stored
+ if (element.contains("comments"))
+ {
+ ++propertyCount;
+ }
+
+ // Optional volts property
+ std::optional<double> volts{};
+ auto voltsIt = element.find("volts");
+ if (voltsIt != element.end())
+ {
+ volts = parseDouble(*voltsIt);
+ ++propertyCount;
+ }
+
+ // Required rule_id or actions property
+ std::vector<std::unique_ptr<Action>> actions{};
+ actions = parseRuleIDOrActionsProperty(element);
+ ++propertyCount;
+
+ // Verify no invalid properties exist
+ verifyPropertyCount(element, propertyCount);
+
+ return std::make_unique<Configuration>(volts, std::move(actions));
+}
+
std::unique_ptr<Device> parseDevice(const json& element)
{
verifyIsObject(element);
@@ -270,14 +301,13 @@
// }
// Optional configuration property
- // TODO: Not implemented yet
std::unique_ptr<Configuration> configuration{};
- // auto configurationIt = element.find("configuration");
- // if (configurationIt != element.end())
- // {
- // configuration = parseConfiguration(*configurationIt);
- // ++propertyCount;
- // }
+ auto configurationIt = element.find("configuration");
+ if (configurationIt != element.end())
+ {
+ configuration = parseConfiguration(*configurationIt);
+ ++propertyCount;
+ }
// Optional rails property
std::vector<std::unique_ptr<Rail>> rails{};
@@ -574,6 +604,32 @@
return rules;
}
+std::vector<std::unique_ptr<Action>>
+ parseRuleIDOrActionsProperty(const json& element)
+{
+ verifyIsObject(element);
+ // Required rule_id or actions property
+ std::vector<std::unique_ptr<Action>> actions{};
+ auto ruleIDIt = element.find("rule_id");
+ auto actionsIt = element.find("actions");
+ if ((actionsIt == element.end()) && (ruleIDIt != element.end()))
+ {
+ std::string ruleID = parseString(*ruleIDIt);
+ actions.emplace_back(std::make_unique<RunRuleAction>(ruleID));
+ }
+ else if ((actionsIt != element.end()) && (ruleIDIt == element.end()))
+ {
+ actions = parseActionArray(*actionsIt);
+ }
+ else
+ {
+ throw std::invalid_argument{"Invalid property combination: Must "
+ "contain either rule_id or actions"};
+ }
+
+ return actions;
+}
+
std::unique_ptr<RunRuleAction> parseRunRule(const json& element)
{
// String ruleID
diff --git a/phosphor-regulators/src/config_file_parser.hpp b/phosphor-regulators/src/config_file_parser.hpp
index 35178ea..251f236 100644
--- a/phosphor-regulators/src/config_file_parser.hpp
+++ b/phosphor-regulators/src/config_file_parser.hpp
@@ -202,6 +202,19 @@
parseChassisArray(const nlohmann::json& element);
/**
+ * Parses a JSON element containing a configuration.
+ *
+ * Returns the corresponding C++ Configuration object.
+ *
+ * Throws an exception if parsing fails.
+ *
+ * @param element JSON element
+ * @return Configuration object
+ */
+std::unique_ptr<Configuration>
+ parseConfiguration(const nlohmann::json& element);
+
+/**
* Parses a JSON element containing a device.
*
* Returns the corresponding C++ Device object.
@@ -436,6 +449,25 @@
parseRuleArray(const nlohmann::json& element);
/**
+ * Parses the "rule_id" or "actions" property in a JSON element.
+ *
+ * The element must contain one property or the other but not both.
+ *
+ * If the element contains a "rule_id" property, the corresponding C++
+ * RunRuleAction object is returned.
+ *
+ * If the element contains an "actions" property, the corresponding C++ Action
+ * objects are returned.
+ *
+ * Throws an exception if parsing fails.
+ *
+ * @param element JSON element
+ * @return vector of Action objects
+ */
+std::vector<std::unique_ptr<Action>>
+ parseRuleIDOrActionsProperty(const nlohmann::json& element);
+
+/**
* Parses a JSON element containing a run_rule action.
*
* Returns the corresponding C++ RunRuleAction object.