commit 7a656e2c056f12aecbe90e07f91fad51c8407197
author Matthew Barth <msbarth@us.ibm.com> Mon Feb 07 10:54:23 2022 -0600
committer Matthew Barth <msbarth@linux.ibm.com> Mon Feb 07 21:24:27 2022 +0000
tree 4f65f7972b6507b8b2fb05a49107744539b297bf
parent f6b129a2497779a04e1fd9010117ba6ee64a2e5b

control - Remove `nameOwnerChanged` signal flight recorder

Remove due to the possibility of being called more than once if more
than one event uses the same group trigger as it could be confusing that
more than one signal was received when it was not. This is due to the
`sd_bus_message_rewind` in Manager::handleSignal().

Change-Id: I6e6739658c67fe745e525b3da9bffd0213119c8c
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

control/json/triggers/handlers.hpp

diff --git a/control/json/triggers/handlers.hpp b/control/json/triggers/handlers.hpp
index 15bced0..34271c2 100644
--- a/control/json/triggers/handlers.hpp
+++ b/control/json/triggers/handlers.hpp
@@ -1,9 +1,6 @@
 #pragma once
 
 #include "../manager.hpp"
-#include "../utils/flight_recorder.hpp"
-
-#include <fmt/format.h>
 
 #include <sdbusplus/message.hpp>
 
@@ -152,9 +149,7 @@
         {
             hasOwner = true;
         }
-        FlightRecorder::instance().log(
-            "nameOwnerChanged", fmt::format("Service: {}, Owned: {}", serv,
-                                            hasOwner ? "true" : "false"));
+
         mgr.setOwner(serv, hasOwner);
         return true;
     }