Implement NumericThreshold for Trigger
Implemented NumericThreshold for Trigger that allows to set
expected threshold value for collection of sensors and monitors
if this value is crossed. Implemented detection of direction
when threshold value is crossed. Added initial interface that
is used to commit an action when threshold value is crossed.
Moved Sensor Cache from Report Factory to Telemetry class, now
Sensor Cache is shared between Report Factory and Trigger Factory.
Moved fetching sensor from Dbus to seperate header to have single
implementation for factories.
Disabled tests that uses boost coroutine because of false positive
that is catched by address sanitizer.
Tested:
- Passed unit tests
- Built in yocto envrionment with success
- Telemetry service started successfully in OpenBMC
Change-Id: I1ff7ab96174a27576786f0b9a71554fe1eeff436
Signed-off-by: Wludzik, Jozef <jozef.wludzik@intel.com>
diff --git a/src/report_factory.hpp b/src/report_factory.hpp
index de2bc02..6d90902 100644
--- a/src/report_factory.hpp
+++ b/src/report_factory.hpp
@@ -12,7 +12,8 @@
public:
ReportFactory(
std::shared_ptr<sdbusplus::asio::connection> bus,
- const std::shared_ptr<sdbusplus::asio::object_server>& objServer);
+ const std::shared_ptr<sdbusplus::asio::object_server>& objServer,
+ SensorCache& sensorCache);
std::unique_ptr<interfaces::Report>
make(boost::asio::yield_context& yield, const std::string& name,
@@ -33,12 +34,6 @@
const override;
private:
- using SensorPath = std::string;
- using ServiceName = std::string;
- using Ifaces = std::vector<std::string>;
- using SensorIfaces = std::vector<std::pair<ServiceName, Ifaces>>;
- using SensorTree = std::pair<SensorPath, SensorIfaces>;
-
std::vector<std::shared_ptr<interfaces::Sensor>> getSensors(
const std::vector<LabeledSensorParameters>& sensorPaths) const;
std::vector<LabeledMetricParameters>
@@ -47,5 +42,5 @@
std::shared_ptr<sdbusplus::asio::connection> bus;
std::shared_ptr<sdbusplus::asio::object_server> objServer;
- mutable SensorCache sensorCache;
+ SensorCache& sensorCache;
};