Add meson options to modify Telemetry consts
Added 'min-interval', 'max-reading-parameters' and 'max-reports'
to meson_options to allow user to control Telemetry constant
values. Now user is able to adjust those values to platform.
Fixed checking maximal number of ReadingParameters in Report.
Synced type of properties with interface defined in
phosphor-dbus-interface.
Tested:
- Unit tests passes
Change-Id: Ie8c009931d9e5e22b30d7df82c81aaac8d68dd3e
Signed-off-by: Wludzik, Jozef <jozef.wludzik@intel.com>
diff --git a/src/report_manager.cpp b/src/report_manager.cpp
index 21f0ac1..13ca6db 100644
--- a/src/report_manager.cpp
+++ b/src/report_manager.cpp
@@ -24,7 +24,7 @@
reportManagerIface = objServer->add_unique_interface(
reportManagerPath, reportManagerIfaceName, [this](auto& dbusIface) {
dbusIface.register_property_r(
- "MaxReports", uint32_t{}, sdbusplus::vtable::property_::const_,
+ "MaxReports", size_t{}, sdbusplus::vtable::property_::const_,
[](const auto&) { return maxReports; });
dbusIface.register_property_r(
"MinInterval", uint64_t{}, sdbusplus::vtable::property_::const_,
@@ -102,7 +102,7 @@
"Only single sensor per metric is allowed");
}
}
- if (readingParams.size() >= maxReadingParams)
+ if (readingParams.size() > maxReadingParams)
{
throw sdbusplus::exception::SdBusError(
static_cast<int>(std::errc::argument_list_too_long),