commit aeac98bdca848875bc8420999cfd928f08f98013
author Brad Bishop <bradleyb@fuzziesquirrel.com> Sat Jul 29 22:56:48 2017 -0400
committer Brad Bishop <bradleyb@fuzziesquirrel.com> Sat Jul 29 22:56:48 2017 -0400
tree 4aae5b4a8211f6227fef12e4acce65c1224e5fb9
parent 062403dbf1e8ff3f7cbc20501ba4f7de812c0b17

server: Ignore mapper generated signals

Correct a bug that prevented this.  This was always the intent as
the mapper does not rely on dbus to inform it of its own
signal generating actions.

Change-Id: I56af68ed65903ac105a6fb553010c389cf0ea679
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

obmc/mapper/server.py

diff --git a/obmc/mapper/server.py b/obmc/mapper/server.py
index cf05866..f5370f0 100644
--- a/obmc/mapper/server.py
+++ b/obmc/mapper/server.py
@@ -499,6 +499,8 @@
                 self.bus.list_names())
             owners = filter(bool, [get_owner(name) for name in owned_names])
         for owned_name, o in owners:
+            if not self.valid_signal(owned_name):
+                continue
             self.bus_map[o] = owned_name
             self.defer_signals[o] = []
             find_dbus_interfaces(
@@ -512,7 +514,7 @@
         if obmc.dbuslib.bindings.is_unique(name):
             name = self.bus_map.get(name)
 
-        return name is not None and name is not obmc.mapper.MAPPER_NAME
+        return name is not None and name != obmc.mapper.MAPPER_NAME
 
     def get_signal_interfaces(self, owner, interfaces):
         filtered = []