From f55c1eed445c28d3e666223cecbcde759b0cfb0e Mon Sep 17 00:00:00 2001 From: Matt Spinler Date: Wed, 1 Nov 2017 10:40:57 -0500 Subject: Set Resolved property on elog entry Fill in the function that sets the Resolved property on an error log entry. Change-Id: I78b57c413a7047508b7fc95e0cf27e1ae6e871cf Signed-off-by: Matt Spinler --- src/resolve_errors.cpp | 40 +++++++++++++++++++++++++++++++++++++++- 1 file changed, 39 insertions(+), 1 deletion(-) diff --git a/src/resolve_errors.cpp b/src/resolve_errors.cpp index 6f3c4be..67f7313 100644 --- a/src/resolve_errors.cpp +++ b/src/resolve_errors.cpp @@ -24,9 +24,11 @@ namespace dbus namespace monitoring { +constexpr auto LOGGING_IFACE = "xyz.openbmc_project.Logging.Entry"; constexpr auto PROPERTY_IFACE = "org.freedesktop.DBus.Properties"; constexpr auto ASSOCIATION_IFACE = "org.openbmc.Association"; constexpr auto ENDPOINTS_PROPERTY = "endpoints"; +constexpr auto RESOLVED_PROPERTY = "Resolved"; using namespace phosphor::logging; using EndpointList = std::vector; @@ -79,7 +81,43 @@ void ResolveCallout::operator()() void ResolveCallout::resolve(const std::string& logEntry) { - //TODO: fill in + try + { + static std::string busName; + if (busName.empty()) + { + busName = SDBusPlus::getBusName(logEntry, LOGGING_IFACE); + if (busName.empty()) + { + return; + } + } + + sdbusplus::message::variant resolved = true; + + auto response = SDBusPlus::callMethod( + busName, + logEntry, + PROPERTY_IFACE, + "Set", + LOGGING_IFACE, + RESOLVED_PROPERTY, + resolved); + + if (response.is_method_error()) + { + log( + "Failed to set Resolved property on an error log entry", + entry("ENTRY=%s", logEntry.c_str())); + } + } + catch (const std::exception& e) + { + log( + "Unable to resolve error log entry", + entry("ENTRY=%s", logEntry.c_str()), + entry("MESSAGE=%s", e.what())); + } } } // namespace monitoring -- cgit v1.2.1