Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 1 | #include "report_manager.hpp" |
| 2 | |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 3 | #include "report.hpp" |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 4 | #include "types/report_types.hpp" |
Krzysztof Grobelny | e8fc575 | 2021-02-05 14:30:45 +0000 | [diff] [blame] | 5 | #include "utils/conversion.hpp" |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 6 | #include "utils/transform.hpp" |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 7 | |
| 8 | #include <phosphor-logging/log.hpp> |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 9 | #include <sdbusplus/exception.hpp> |
| 10 | |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 11 | #include <stdexcept> |
Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 12 | #include <system_error> |
| 13 | |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 14 | ReadingParameters |
| 15 | convertToReadingParameters(ReadingParametersPastVersion params) |
| 16 | { |
| 17 | return utils::transform(params, [](const auto& param) { |
| 18 | using namespace std::chrono_literals; |
| 19 | |
| 20 | return ReadingParameters::value_type( |
| 21 | std::vector{{std::get<0>(param)}}, std::get<1>(param), |
| 22 | std::get<2>(param), std::get<3>(param), |
| 23 | utils::enumToString(CollectionTimeScope::point), 0u); |
| 24 | }); |
| 25 | } |
| 26 | |
Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 27 | ReportManager::ReportManager( |
Wludzik, Jozef | 2f9f9b8 | 2020-10-13 09:07:45 +0200 | [diff] [blame] | 28 | std::unique_ptr<interfaces::ReportFactory> reportFactoryIn, |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 29 | std::unique_ptr<interfaces::JsonStorage> reportStorageIn, |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 30 | const std::shared_ptr<sdbusplus::asio::object_server>& objServerIn) : |
Wludzik, Jozef | 2f9f9b8 | 2020-10-13 09:07:45 +0200 | [diff] [blame] | 31 | reportFactory(std::move(reportFactoryIn)), |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 32 | reportStorage(std::move(reportStorageIn)), objServer(objServerIn) |
Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 33 | { |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 34 | reports.reserve(maxReports); |
Krzysztof Grobelny | 6ccfcbf | 2020-11-04 09:31:36 +0100 | [diff] [blame] | 35 | |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 36 | loadFromPersistent(); |
Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 37 | |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 38 | reportManagerIface = objServer->add_unique_interface( |
Wludzik, Jozef | 2f9f9b8 | 2020-10-13 09:07:45 +0200 | [diff] [blame] | 39 | reportManagerPath, reportManagerIfaceName, [this](auto& dbusIface) { |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 40 | dbusIface.register_property_r( |
Wludzik, Jozef | 503c158 | 2020-12-11 14:48:01 +0100 | [diff] [blame] | 41 | "MaxReports", size_t{}, sdbusplus::vtable::property_::const_, |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 42 | [](const auto&) { return maxReports; }); |
| 43 | dbusIface.register_property_r( |
Karol Niczyj | 32859b6 | 2021-05-19 10:20:46 +0200 | [diff] [blame] | 44 | "MaxReportNameLength", size_t{}, |
| 45 | sdbusplus::vtable::property_::const_, |
| 46 | [](const auto&) { return maxReportNameLength; }); |
| 47 | dbusIface.register_property_r( |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 48 | "MinInterval", uint64_t{}, sdbusplus::vtable::property_::const_, |
| 49 | [](const auto&) -> uint64_t { return minInterval.count(); }); |
Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 50 | |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 51 | dbusIface.register_method( |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 52 | "AddReport", [this](boost::asio::yield_context& yield, |
| 53 | const std::string& reportName, |
Wludzik, Jozef | 2f9f9b8 | 2020-10-13 09:07:45 +0200 | [diff] [blame] | 54 | const std::string& reportingType, |
| 55 | const bool emitsReadingsUpdate, |
| 56 | const bool logToMetricReportsCollection, |
| 57 | const uint64_t interval, |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 58 | ReadingParametersPastVersion metricParams) { |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 59 | constexpr auto enabledDefault = true; |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 60 | constexpr uint64_t appendLimitDefault = 0; |
| 61 | constexpr ReportUpdates reportUpdatesDefault = |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 62 | ReportUpdates::overwrite; |
| 63 | |
| 64 | std::vector<ReportAction> reportActions; |
| 65 | |
| 66 | if (emitsReadingsUpdate) |
| 67 | { |
| 68 | reportActions.emplace_back( |
| 69 | ReportAction::emitsReadingsUpdate); |
| 70 | } |
| 71 | if (logToMetricReportsCollection) |
| 72 | { |
| 73 | reportActions.emplace_back( |
| 74 | ReportAction::logToMetricReportsCollection); |
| 75 | } |
| 76 | |
| 77 | return addReport(yield, reportName, |
| 78 | utils::toReportingType(reportingType), |
| 79 | reportActions, Milliseconds(interval), |
| 80 | appendLimitDefault, reportUpdatesDefault, |
| 81 | convertToReadingParameters( |
| 82 | std::move(metricParams)), |
| 83 | enabledDefault) |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 84 | .getPath(); |
| 85 | }); |
| 86 | |
| 87 | dbusIface.register_method( |
| 88 | "AddReportFutureVersion", |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 89 | [this](boost::asio::yield_context& yield, |
| 90 | const std::string& reportName, |
| 91 | const std::string& reportingType, |
| 92 | const std::string& reportUpdates, |
| 93 | const uint64_t appendLimit, |
| 94 | const std::vector<std::string>& reportActions, |
| 95 | const uint64_t interval, |
| 96 | ReadingParameters metricParams) { |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 97 | constexpr auto enabledDefault = true; |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 98 | return addReport(yield, reportName, |
| 99 | utils::toReportingType(reportingType), |
| 100 | utils::transform( |
| 101 | reportActions, |
| 102 | [](const auto& reportAction) { |
| 103 | return utils::toReportAction( |
| 104 | reportAction); |
| 105 | }), |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 106 | Milliseconds(interval), appendLimit, |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 107 | utils::toReportUpdates(reportUpdates), |
| 108 | std::move(metricParams), enabledDefault) |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 109 | .getPath(); |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 110 | }); |
| 111 | }); |
Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 112 | } |
| 113 | |
Wludzik, Jozef | 2f9f9b8 | 2020-10-13 09:07:45 +0200 | [diff] [blame] | 114 | void ReportManager::removeReport(const interfaces::Report* report) |
Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 115 | { |
Wludzik, Jozef | cb88cfd | 2020-09-28 16:38:57 +0200 | [diff] [blame] | 116 | reports.erase( |
| 117 | std::remove_if(reports.begin(), reports.end(), |
| 118 | [report](const auto& x) { return report == x.get(); }), |
| 119 | reports.end()); |
Krzysztof Grobelny | 64b75a5 | 2020-09-18 10:17:16 +0200 | [diff] [blame] | 120 | } |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 121 | |
Karol Niczyj | 32859b6 | 2021-05-19 10:20:46 +0200 | [diff] [blame] | 122 | void ReportManager::verifyReportNameLength(const std::string& reportName) |
| 123 | { |
| 124 | if (reportName.length() > maxReportNameLength) |
| 125 | { |
| 126 | throw sdbusplus::exception::SdBusError( |
| 127 | static_cast<int>(std::errc::invalid_argument), |
Szymon Dompke | e28aa53 | 2021-10-27 12:33:12 +0200 | [diff] [blame] | 128 | "Report name exceeds maximum length"); |
Karol Niczyj | 32859b6 | 2021-05-19 10:20:46 +0200 | [diff] [blame] | 129 | } |
| 130 | } |
| 131 | |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 132 | void ReportManager::verifyAddReport( |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 133 | const std::string& reportName, const ReportingType reportingType, |
| 134 | Milliseconds interval, const ReportUpdates reportUpdates, |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 135 | const std::vector<LabeledMetricParameters>& readingParams) |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 136 | { |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 137 | if (reportingType == ReportingType::onChange) |
| 138 | { |
| 139 | throw sdbusplus::exception::SdBusError( |
| 140 | static_cast<int>(std::errc::invalid_argument), |
| 141 | "Invalid reportingType"); |
| 142 | } |
| 143 | |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 144 | if (reports.size() >= maxReports) |
| 145 | { |
| 146 | throw sdbusplus::exception::SdBusError( |
| 147 | static_cast<int>(std::errc::too_many_files_open), |
| 148 | "Reached maximal report count"); |
| 149 | } |
| 150 | |
Karol Niczyj | 32859b6 | 2021-05-19 10:20:46 +0200 | [diff] [blame] | 151 | verifyReportNameLength(reportName); |
| 152 | |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 153 | for (const auto& report : reports) |
| 154 | { |
| 155 | if (report->getName() == reportName) |
| 156 | { |
| 157 | throw sdbusplus::exception::SdBusError( |
| 158 | static_cast<int>(std::errc::file_exists), "Duplicate report"); |
| 159 | } |
| 160 | } |
| 161 | |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 162 | verifyReportUpdates(reportUpdates); |
| 163 | |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 164 | if (reportingType == ReportingType::periodic && interval < minInterval) |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 165 | { |
| 166 | throw sdbusplus::exception::SdBusError( |
| 167 | static_cast<int>(std::errc::invalid_argument), "Invalid interval"); |
| 168 | } |
Wludzik, Jozef | bc766b4 | 2020-12-08 16:06:22 +0100 | [diff] [blame] | 169 | |
Wludzik, Jozef | 503c158 | 2020-12-11 14:48:01 +0100 | [diff] [blame] | 170 | if (readingParams.size() > maxReadingParams) |
Krzysztof Grobelny | e8fc575 | 2021-02-05 14:30:45 +0000 | [diff] [blame] | 171 | |
Wludzik, Jozef | bc766b4 | 2020-12-08 16:06:22 +0100 | [diff] [blame] | 172 | { |
| 173 | throw sdbusplus::exception::SdBusError( |
| 174 | static_cast<int>(std::errc::argument_list_too_long), |
| 175 | "Too many reading parameters"); |
| 176 | } |
Krzysztof Grobelny | e8fc575 | 2021-02-05 14:30:45 +0000 | [diff] [blame] | 177 | |
| 178 | try |
| 179 | { |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 180 | namespace ts = utils::tstring; |
| 181 | |
| 182 | for (const LabeledMetricParameters& item : readingParams) |
Krzysztof Grobelny | e8fc575 | 2021-02-05 14:30:45 +0000 | [diff] [blame] | 183 | { |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 184 | utils::toOperationType( |
| 185 | utils::toUnderlying(item.at_label<ts::OperationType>())); |
Krzysztof Grobelny | e8fc575 | 2021-02-05 14:30:45 +0000 | [diff] [blame] | 186 | } |
| 187 | } |
| 188 | catch (const std::exception& e) |
| 189 | { |
| 190 | throw sdbusplus::exception::SdBusError( |
| 191 | static_cast<int>(std::errc::invalid_argument), e.what()); |
| 192 | } |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 193 | } |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 194 | |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 195 | interfaces::Report& ReportManager::addReport( |
| 196 | boost::asio::yield_context& yield, const std::string& reportName, |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 197 | const ReportingType reportingType, |
| 198 | const std::vector<ReportAction>& reportActions, Milliseconds interval, |
| 199 | const uint64_t appendLimit, const ReportUpdates reportUpdates, |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 200 | ReadingParameters metricParams, const bool enabled) |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 201 | { |
Krzysztof Grobelny | dcc4e19 | 2021-03-08 09:09:34 +0000 | [diff] [blame] | 202 | auto labeledMetricParams = |
| 203 | reportFactory->convertMetricParams(yield, metricParams); |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 204 | |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 205 | return addReport(reportName, reportingType, reportActions, interval, |
| 206 | appendLimit, reportUpdates, std::move(labeledMetricParams), |
| 207 | enabled); |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 208 | } |
| 209 | |
| 210 | interfaces::Report& ReportManager::addReport( |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 211 | const std::string& reportName, const ReportingType reportingType, |
| 212 | const std::vector<ReportAction>& reportActions, Milliseconds interval, |
| 213 | const uint64_t appendLimit, const ReportUpdates reportUpdates, |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 214 | std::vector<LabeledMetricParameters> labeledMetricParams, |
| 215 | const bool enabled) |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 216 | { |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 217 | verifyAddReport(reportName, reportingType, interval, reportUpdates, |
| 218 | labeledMetricParams); |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 219 | |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 220 | reports.emplace_back(reportFactory->make( |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 221 | reportName, reportingType, reportActions, interval, appendLimit, |
| 222 | reportUpdates, *this, *reportStorage, labeledMetricParams, enabled)); |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 223 | return *reports.back(); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 224 | } |
| 225 | |
| 226 | void ReportManager::loadFromPersistent() |
| 227 | { |
| 228 | std::vector<interfaces::JsonStorage::FilePath> paths = |
| 229 | reportStorage->list(); |
| 230 | |
| 231 | for (const auto& path : paths) |
| 232 | { |
| 233 | std::optional<nlohmann::json> data = reportStorage->load(path); |
| 234 | try |
| 235 | { |
Lukasz Kazmierczak | 7e098e9 | 2021-09-16 15:59:56 +0200 | [diff] [blame] | 236 | bool enabled = data->at("Enabled").get<bool>(); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 237 | size_t version = data->at("Version").get<size_t>(); |
| 238 | if (version != Report::reportVersion) |
| 239 | { |
| 240 | throw std::logic_error("Invalid version"); |
| 241 | } |
| 242 | std::string& name = data->at("Name").get_ref<std::string&>(); |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 243 | |
| 244 | uint32_t reportingType = data->at("ReportingType").get<uint32_t>(); |
| 245 | std::vector<ReportAction> reportActions = utils::transform( |
| 246 | data->at("ReportActions").get<std::vector<uint32_t>>(), |
| 247 | [](const auto reportAction) { |
| 248 | return utils::toReportAction(reportAction); |
| 249 | }); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 250 | uint64_t interval = data->at("Interval").get<uint64_t>(); |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 251 | uint64_t appendLimit = data->at("AppendLimit").get<uint64_t>(); |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 252 | uint32_t reportUpdates = data->at("ReportUpdates").get<uint32_t>(); |
Krzysztof Grobelny | d223819 | 2020-12-02 09:27:28 +0000 | [diff] [blame] | 253 | auto readingParameters = |
| 254 | data->at("ReadingParameters") |
| 255 | .get<std::vector<LabeledMetricParameters>>(); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 256 | |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 257 | addReport(name, utils::toReportingType(reportingType), |
| 258 | reportActions, Milliseconds(interval), appendLimit, |
| 259 | utils::toReportUpdates(reportUpdates), |
| 260 | std::move(readingParameters), enabled); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 261 | } |
| 262 | catch (const std::exception& e) |
| 263 | { |
| 264 | phosphor::logging::log<phosphor::logging::level::ERR>( |
| 265 | "Failed to load report from storage", |
| 266 | phosphor::logging::entry( |
Wludzik, Jozef | 982c5b5 | 2021-01-02 12:05:21 +0100 | [diff] [blame] | 267 | "FILENAME=%s", |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 268 | static_cast<std::filesystem::path>(path).c_str()), |
Wludzik, Jozef | 982c5b5 | 2021-01-02 12:05:21 +0100 | [diff] [blame] | 269 | phosphor::logging::entry("EXCEPTION_MSG=%s", e.what())); |
Wludzik, Jozef | e236279 | 2020-10-27 17:23:55 +0100 | [diff] [blame] | 270 | reportStorage->remove(path); |
| 271 | } |
| 272 | } |
| 273 | } |
Wludzik, Jozef | d960e1f | 2021-01-08 09:25:59 +0100 | [diff] [blame] | 274 | |
| 275 | void ReportManager::updateReport(const std::string& name) |
| 276 | { |
| 277 | for (auto& report : reports) |
| 278 | { |
| 279 | if (report->getName() == name) |
| 280 | { |
| 281 | report->updateReadings(); |
| 282 | return; |
| 283 | } |
| 284 | } |
| 285 | } |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 286 | |
Krzysztof Grobelny | 51497a0 | 2021-11-09 14:56:22 +0100 | [diff] [blame^] | 287 | void ReportManager::verifyReportUpdates(const ReportUpdates reportUpdates) |
Szymon Dompke | 3eb5686 | 2021-09-20 15:32:04 +0200 | [diff] [blame] | 288 | { |
| 289 | if (std::find(supportedReportUpdates.begin(), supportedReportUpdates.end(), |
| 290 | reportUpdates) == supportedReportUpdates.end()) |
| 291 | { |
| 292 | throw sdbusplus::exception::SdBusError( |
| 293 | static_cast<int>(std::errc::invalid_argument), |
| 294 | "Invalid ReportUpdates"); |
| 295 | } |
| 296 | } |