commit d0b8e395323137f6130d568a9f046b9488cda013
author Brad Bishop <bradleyb@fuzziesquirrel.com> Mon Sep 19 11:24:45 2016 -0400
committer Brad Bishop <bradleyb@fuzziesquirrel.com> Tue Sep 20 15:46:27 2016 -0400
tree 498e7830d34256b298855775f760b6c2c5ac8250
parent 6a0320b2a98f4149a92f28147fba2880105682e0

server: examine all services

Remove the name_match and instrospect all services on the DBus.
This is required as there are no restrictions on the names claimed
by services that put objects in the /org/openbmc namespace.

Change-Id: I38646addddde0de0ca4ec4605468f7c9c551bbd8
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 22e5497..0ac037c 100644
--- a/obmc/mapper/server.py
+++ b/obmc/mapper/server.py
@@ -193,12 +193,10 @@
 
 class ObjectMapper(dbus.service.Object):
     def __init__(self, bus, path,
-                 name_match=obmc.utils.misc.org_dot_openbmc_match,
                  intf_match=obmc.utils.misc.org_dot_openbmc_match):
         super(ObjectMapper, self).__init__(bus, path)
         self.cache = obmc.utils.pathtree.PathTree()
         self.bus = bus
-        self.name_match = name_match
         self.intf_match = intf_match
         self.tag_match = obmc.utils.misc.ListMatch(['children', 'interface'])
         self.service = None
@@ -231,11 +229,6 @@
             path_keyword='path',
             sender_keyword='sender')
 
-    def bus_match(self, owner):
-        # Ignore my own signals
-        return owner != obmc.mapper.MAPPER_NAME and \
-            self.name_match(owner)
-
     def discovery_pending(self):
         return not bool(self.service)
 
@@ -368,8 +361,9 @@
             x, sep='/', prefix='/')
 
         if not owners:
-            owned_names = [
-                x for x in self.bus.list_names() if self.bus_match(x)]
+            owned_names = filter(
+                lambda x: not obmc.dbuslib.bindings.is_unique(x),
+                self.bus.list_names())
             owners = [self.bus.get_name_owner(x) for x in owned_names]
             owners = zip(owned_names, owners)
         for owned_name, o in owners:
@@ -400,8 +394,7 @@
         if obmc.dbuslib.bindings.is_unique(name):
             name = self.bus_map.get(name)
 
-        return name is not None and \
-            self.bus_match(name)
+        return name is not None and name is not obmc.mapper.MAPPER_NAME
 
     def get_signal_interfaces(self, owner, interfaces):
         filtered = []