Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 1 | #include "report.hpp" |
| 2 | |
| 3 | #include "report_manager.hpp" |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 4 | #include "utils/contains.hpp" |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 5 | #include "utils/transform.hpp" |
| 6 | |
| 7 | #include <phosphor-logging/log.hpp> |
Wludzik, Jozef | b1ff1f6 | 2020-10-23 13:20:52 +0200 | [diff] [blame] | 8 | #include <sdbusplus/vtable.hpp> |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 9 | |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 10 | #include <limits> |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 11 | #include <numeric> |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 12 | |
| 13 | Report::Report(boost::asio::io_context& ioc, |
| 14 | const std::shared_ptr<sdbusplus::asio::object_server>& objServer, |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 15 | const std::string& reportName, |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 16 | const ReportingType reportingTypeIn, |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 17 | std::vector<ReportAction> reportActionsIn, |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 18 | const Milliseconds intervalIn, const uint64_t appendLimitIn, |
| 19 | const ReportUpdates reportUpdatesIn, |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 20 | interfaces::ReportManager& reportManager, |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 21 | interfaces::JsonStorage& reportStorageIn, |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 22 | std::vector<std::shared_ptr<interfaces::Metric>> metricsIn, |
| 23 | const bool enabledIn) : |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 24 | name(reportName), |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 25 | path(reportDir + name), reportingType(reportingTypeIn), |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 26 | interval(intervalIn), reportActions(std::move(reportActionsIn)), |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 27 | sensorCount(getSensorCount(metricsIn)), |
| 28 | appendLimit(deduceAppendLimit(appendLimitIn)), |
| 29 | reportUpdates(reportUpdatesIn), |
| 30 | readingsBuffer(deduceBufferSize(reportUpdates, reportingType)), |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 31 | objServer(objServer), metrics(std::move(metricsIn)), timer(ioc), |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 32 | fileName(std::to_string(std::hash<std::string>{}(name))), |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 33 | reportStorage(reportStorageIn), enabled(enabledIn) |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 34 | { |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 35 | readingParameters = |
| 36 | toReadingParameters(utils::transform(metrics, [](const auto& metric) { |
| 37 | return metric->dumpConfiguration(); |
| 38 | })); |
| 39 | |
| 40 | readingParametersPastVersion = |
| 41 | utils::transform(readingParameters, [](const auto& item) { |
| 42 | return ReadingParametersPastVersion::value_type( |
| 43 | std::get<0>(item).front(), std::get<1>(item), std::get<2>(item), |
| 44 | std::get<3>(item)); |
| 45 | }); |
| 46 | |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 47 | deleteIface = objServer->add_unique_interface( |
| 48 | path, deleteIfaceName, [this, &ioc, &reportManager](auto& dbusIface) { |
| 49 | dbusIface.register_method("Delete", [this, &ioc, &reportManager] { |
| 50 | if (persistency) |
| 51 | { |
| 52 | reportStorage.remove(fileName); |
| 53 | } |
| 54 | boost::asio::post(ioc, [this, &reportManager] { |
| 55 | reportManager.removeReport(this); |
| 56 | }); |
| 57 | }); |
| 58 | }); |
| 59 | |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 60 | persistency = storeConfiguration(); |
| 61 | reportIface = makeReportInterface(); |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 62 | |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 63 | if (reportingType == ReportingType::periodic) |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 64 | { |
| 65 | scheduleTimer(interval); |
| 66 | } |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 67 | |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 68 | if (enabled) |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 69 | { |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 70 | for (auto& metric : this->metrics) |
| 71 | { |
| 72 | metric->initialize(); |
| 73 | } |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 74 | } |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 75 | } |
| 76 | |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 77 | uint64_t Report::getSensorCount( |
| 78 | std::vector<std::shared_ptr<interfaces::Metric>>& metrics) |
| 79 | { |
| 80 | uint64_t sensorCount = 0; |
| 81 | for (auto& metric : metrics) |
| 82 | { |
| 83 | sensorCount += metric->sensorCount(); |
| 84 | } |
| 85 | return sensorCount; |
| 86 | } |
| 87 | |
| 88 | uint64_t Report::deduceAppendLimit(const uint64_t appendLimitIn) const |
| 89 | { |
| 90 | if (appendLimitIn == std::numeric_limits<uint64_t>::max()) |
| 91 | { |
| 92 | return sensorCount; |
| 93 | } |
| 94 | else |
| 95 | { |
| 96 | return appendLimitIn; |
| 97 | } |
| 98 | } |
| 99 | |
| 100 | uint64_t Report::deduceBufferSize(const ReportUpdates reportUpdatesIn, |
| 101 | const ReportingType reportingTypeIn) const |
| 102 | { |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 103 | if (reportUpdatesIn == ReportUpdates::overwrite || |
| 104 | reportingTypeIn == ReportingType::onRequest) |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 105 | { |
| 106 | return sensorCount; |
| 107 | } |
| 108 | else |
| 109 | { |
| 110 | return appendLimit; |
| 111 | } |
| 112 | } |
| 113 | |
| 114 | void Report::setReportUpdates(const ReportUpdates newReportUpdates) |
| 115 | { |
| 116 | if (reportUpdates != newReportUpdates) |
| 117 | { |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 118 | if (reportingType != ReportingType::onRequest && |
| 119 | (reportUpdates == ReportUpdates::overwrite || |
| 120 | newReportUpdates == ReportUpdates::overwrite)) |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 121 | { |
| 122 | readingsBuffer.clearAndResize( |
| 123 | deduceBufferSize(newReportUpdates, reportingType)); |
| 124 | } |
| 125 | reportUpdates = newReportUpdates; |
| 126 | } |
| 127 | } |
| 128 | |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 129 | std::unique_ptr<sdbusplus::asio::dbus_interface> Report::makeReportInterface() |
| 130 | { |
| 131 | auto dbusIface = objServer->add_unique_interface(path, reportIfaceName); |
| 132 | dbusIface->register_property_rw( |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 133 | "Enabled", enabled, sdbusplus::vtable::property_::emits_change, |
| 134 | [this](bool newVal, const auto&) { |
| 135 | if (newVal != enabled) |
| 136 | { |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 137 | if (true == newVal && ReportingType::periodic == reportingType) |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 138 | { |
| 139 | scheduleTimer(interval); |
| 140 | } |
| 141 | if (newVal) |
| 142 | { |
| 143 | for (auto& metric : metrics) |
| 144 | { |
| 145 | metric->initialize(); |
| 146 | } |
| 147 | } |
| 148 | else |
| 149 | { |
| 150 | for (auto& metric : metrics) |
| 151 | { |
| 152 | metric->deinitialize(); |
| 153 | } |
| 154 | } |
| 155 | |
| 156 | enabled = newVal; |
| 157 | persistency = storeConfiguration(); |
| 158 | } |
| 159 | return true; |
| 160 | }, |
| 161 | [this](const auto&) { return enabled; }); |
| 162 | dbusIface->register_property_rw( |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 163 | "Interval", interval.count(), |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 164 | sdbusplus::vtable::property_::emits_change, |
| 165 | [this](uint64_t newVal, auto&) { |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 166 | if (Milliseconds newValT{newVal}; |
| 167 | newValT >= ReportManager::minInterval) |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 168 | { |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 169 | if (newValT != interval) |
| 170 | { |
| 171 | interval = newValT; |
| 172 | persistency = storeConfiguration(); |
| 173 | } |
| 174 | return true; |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 175 | } |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 176 | return false; |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 177 | }, |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 178 | [this](const auto&) { return interval.count(); }); |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 179 | dbusIface->register_property_rw( |
| 180 | "Persistency", persistency, sdbusplus::vtable::property_::emits_change, |
| 181 | [this](bool newVal, const auto&) { |
| 182 | if (newVal == persistency) |
| 183 | { |
| 184 | return true; |
| 185 | } |
| 186 | if (newVal) |
| 187 | { |
| 188 | persistency = storeConfiguration(); |
| 189 | } |
| 190 | else |
| 191 | { |
| 192 | reportStorage.remove(fileName); |
| 193 | persistency = false; |
| 194 | } |
| 195 | return true; |
| 196 | }, |
| 197 | [this](const auto&) { return persistency; }); |
| 198 | |
| 199 | auto readingsFlag = sdbusplus::vtable::property_::none; |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 200 | if (utils::contains(reportActions, ReportAction::emitsReadingsUpdate)) |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 201 | { |
| 202 | readingsFlag = sdbusplus::vtable::property_::emits_change; |
| 203 | } |
| 204 | dbusIface->register_property_r("Readings", readings, readingsFlag, |
| 205 | [this](const auto&) { return readings; }); |
| 206 | dbusIface->register_property_r( |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 207 | "ReportingType", std::string(), sdbusplus::vtable::property_::const_, |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 208 | [this](const auto&) { return utils::enumToString(reportingType); }); |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 209 | dbusIface->register_property_r( |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 210 | "ReadingParameters", readingParametersPastVersion, |
| 211 | sdbusplus::vtable::property_::const_, |
| 212 | [this](const auto&) { return readingParametersPastVersion; }); |
| 213 | dbusIface->register_property_r( |
| 214 | "ReadingParametersFutureVersion", readingParameters, |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 215 | sdbusplus::vtable::property_::const_, |
| 216 | [this](const auto&) { return readingParameters; }); |
| 217 | dbusIface->register_property_r( |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 218 | "EmitsReadingsUpdate", bool{}, sdbusplus::vtable::property_::const_, |
| 219 | [this](const auto&) { |
| 220 | return utils::contains(reportActions, |
| 221 | ReportAction::emitsReadingsUpdate); |
| 222 | }); |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 223 | dbusIface->register_property_r( |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 224 | "LogToMetricReportsCollection", bool{}, |
| 225 | sdbusplus::vtable::property_::const_, [this](const auto&) { |
| 226 | return utils::contains(reportActions, |
| 227 | ReportAction::logToMetricReportsCollection); |
| 228 | }); |
| 229 | dbusIface->register_property_r( |
| 230 | "ReportActions", std::vector<std::string>{}, |
| 231 | sdbusplus::vtable::property_::const_, [this](const auto&) { |
| 232 | return utils::transform(reportActions, [](const auto reportAction) { |
| 233 | return utils::enumToString(reportAction); |
| 234 | }); |
| 235 | }); |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 236 | dbusIface->register_property_r("AppendLimit", appendLimit, |
| 237 | sdbusplus::vtable::property_::emits_change, |
| 238 | [this](const auto&) { return appendLimit; }); |
| 239 | dbusIface->register_property_rw( |
| 240 | "ReportUpdates", std::string(), |
| 241 | sdbusplus::vtable::property_::emits_change, |
| 242 | [this](auto newVal, auto& oldVal) { |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 243 | ReportManager::verifyReportUpdates(utils::toReportUpdates(newVal)); |
| 244 | setReportUpdates(utils::toReportUpdates(newVal)); |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 245 | oldVal = newVal; |
| 246 | return true; |
| 247 | }, |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 248 | [this](const auto&) { return utils::enumToString(reportUpdates); }); |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 249 | dbusIface->register_method("Update", [this] { |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 250 | if (reportingType == ReportingType::onRequest) |
Krzysztof Grobelny | 85db8bd | 2021-05-28 12:13:23 +0000 | [diff] [blame] | 251 | { |
| 252 | updateReadings(); |
| 253 | } |
| 254 | }); |
| 255 | constexpr bool skipPropertiesChangedSignal = true; |
| 256 | dbusIface->initialize(skipPropertiesChangedSignal); |
| 257 | return dbusIface; |
| 258 | } |
| 259 | |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 260 | void Report::timerProc(boost::system::error_code ec, Report& self) |
| 261 | { |
| 262 | if (ec) |
| 263 | { |
| 264 | return; |
| 265 | } |
| 266 | |
| 267 | self.updateReadings(); |
| 268 | self.scheduleTimer(self.interval); |
| 269 | } |
| 270 | |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 271 | void Report::scheduleTimer(Milliseconds timerInterval) |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 272 | { |
| 273 | timer.expires_after(timerInterval); |
| 274 | timer.async_wait( |
| 275 | [this](boost::system::error_code ec) { timerProc(ec, *this); }); |
| 276 | } |
| 277 | |
| 278 | void Report::updateReadings() |
| 279 | { |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 280 | if (!enabled) |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 281 | { |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 282 | return; |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 283 | } |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 284 | |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 285 | if (reportUpdates == ReportUpdates::overwrite || |
| 286 | reportingType == ReportingType::onRequest) |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 287 | { |
| 288 | readingsBuffer.clear(); |
| 289 | } |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 290 | |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 291 | for (const auto& metric : metrics) |
| 292 | { |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 293 | for (const auto& [id, metadata, value, timestamp] : |
| 294 | metric->getReadings()) |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 295 | { |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 296 | if (reportUpdates == ReportUpdates::appendStopsWhenFull && |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 297 | readingsBuffer.isFull()) |
| 298 | { |
| 299 | enabled = false; |
| 300 | for (auto& metric : metrics) |
| 301 | { |
| 302 | metric->deinitialize(); |
| 303 | } |
| 304 | break; |
| 305 | } |
| 306 | readingsBuffer.emplace(id, metadata, value, timestamp); |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 307 | } |
| 308 | } |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 309 | |
| 310 | readings = {std::time(0), std::vector<ReadingData>(readingsBuffer.begin(), |
| 311 | readingsBuffer.end())}; |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 312 | |
Krzysztof Grobelny | c8e3a64 | 2020-10-23 12:29:16 +0200 | [diff] [blame] | 313 | reportIface->signal_property("Readings"); |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 314 | } |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 315 | |
| 316 | bool Report::storeConfiguration() const |
| 317 | { |
| 318 | try |
| 319 | { |
| 320 | nlohmann::json data; |
| 321 | |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 322 | data["Enabled"] = enabled; |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 323 | data["Version"] = reportVersion; |
| 324 | data["Name"] = name; |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 325 | data["ReportingType"] = utils::toUnderlying(reportingType); |
| 326 | data["ReportActions"] = |
| 327 | utils::transform(reportActions, [](const auto reportAction) { |
| 328 | return utils::toUnderlying(reportAction); |
| 329 | }); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 330 | data["Interval"] = interval.count(); |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 331 | data["AppendLimit"] = appendLimit; |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 332 | data["ReportUpdates"] = utils::toUnderlying(reportUpdates); |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 333 | data["ReadingParameters"] = |
| 334 | utils::transform(metrics, [](const auto& metric) { |
| 335 | return metric->dumpConfiguration(); |
| 336 | }); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 337 | |
| 338 | reportStorage.store(fileName, data); |
| 339 | } |
| 340 | catch (const std::exception& e) |
| 341 | { |
| 342 | phosphor::logging::log<phosphor::logging::level::ERR>( |
| 343 | "Failed to store a report in storage", |
Wludzik, Jozef | 982c5b5 | 2021-01-02 12:05:21 +0100 | [diff] [blame] | 344 | phosphor::logging::entry("EXCEPTION_MSG=%s", e.what())); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 345 | return false; |
| 346 | } |
| 347 | |
| 348 | return true; |
| 349 | } |