(*) Peer has new fields person_ids and site_ids
[plcapi.git] / PLC / Methods / GetSlivers.py
index 1c9746c..060169f 100644 (file)
@@ -61,6 +61,7 @@ class GetSlivers(Method):
         }]
     }]
 
+
     def call(self, auth, node_filter = None):
         timestamp = int(time.time())
 
@@ -69,11 +70,11 @@ class GetSlivers(Method):
         else:
             all_nodes = Nodes(self.api, node_filter).dict()
 
-        # Get default slices
-        system_slice_attributes = SliceAttributes(self.api, {'name': 'system', 'value': '1'}).dict()
+        # Get local default slices
+        system_slice_attributes = SliceAttributes(self.api, {'name': 'system', 'value': '1', 'peer_id': None}).dict()
         system_slice_ids = [slice_attribute['slice_id'] for slice_attribute in system_slice_attributes.values()]
         system_slice_ids = dict.fromkeys(system_slice_ids)
-
+       
         all_nodenetwork_ids = set()
         all_nodegroup_ids = set()
         all_slice_ids = set(system_slice_ids.keys())
@@ -81,7 +82,7 @@ class GetSlivers(Method):
             all_nodenetwork_ids.update(node['nodenetwork_ids'])
             all_nodegroup_ids.update(node['nodegroup_ids'])
             all_slice_ids.update(node['slice_ids'])
-
+       
         # Get nodenetwork information
         all_nodenetworks = NodeNetworks(self.api, all_nodenetwork_ids).dict()
 
@@ -104,7 +105,7 @@ class GetSlivers(Method):
             if slice.get('slice_attribute_ids'):
                 slice_attribute_ids.update(slice['slice_attribute_ids'])
 
-        # Get user information
+       # Get user information
         all_persons = Persons(self.api, person_ids).dict()
 
         key_ids = set()
@@ -116,10 +117,10 @@ class GetSlivers(Method):
 
         # Get slice attributes
         all_slice_attributes = SliceAttributes(self.api, slice_attribute_ids).dict()
-
-        nodes = []
+        
+       nodes = []
         for node_id, node in all_nodes.iteritems():
-            networks = [all_nodenetworks[nodenetwork_id] for nodenetwork_id in node['nodenetwork_ids']]
+           networks = [all_nodenetworks[nodenetwork_id] for nodenetwork_id in node['nodenetwork_ids']]
             nodegroups = [all_nodegroups[nodegroup_id] for nodegroup_id in node['nodegroup_ids']]
             groups = [nodegroup['name'] for nodegroup in nodegroups]
 
@@ -129,30 +130,33 @@ class GetSlivers(Method):
             for conf_file in all_conf_files.values():
                 if not conf_file['node_ids'] and not conf_file['nodegroup_ids']:
                     conf_files[conf_file['dest']] = conf_file
-
-            # If a node belongs to multiple node
+            
+           # If a node belongs to multiple node
             # groups for which the same configuration file is defined,
             # it is undefined which one takes precedence.
             for nodegroup in nodegroups:
                 for conf_file_id in nodegroup['conf_file_ids']:
                     if conf_file_id in all_conf_files:
                         conf_files[conf_file['dest']] = all_conf_files[conf_file_id]
-
-            # Node configuration files always take precedence over
+            
+           # Node configuration files always take precedence over
             # node group configuration files.
             for conf_file_id in node['conf_file_ids']:
                 if conf_file_id in all_conf_files:
                     conf_files[conf_file['dest']] = all_conf_files[conf_file_id]
-
-            slice_ids = dict.fromkeys(node['slice_ids'])
-
-            # If not a foreign node, add all of our default system
+            
+           # filter out any slices in this nodes slice_id list that may be invalid
+            # (i.e. expired slices)
+            slice_ids = dict.fromkeys(filter(lambda slice_id: slice_id in all_slice_ids, node['slice_ids']))
+           
+           # If not a foreign node, add all of our default system
             # slices to it.
-            # XXX if not node['peer_id']:
-            slice_ids.update(system_slice_ids)
+            if node['peer_id'] is None:
+               slice_ids.update(system_slice_ids)
 
-            slivers = []
-            for slice in map(lambda id: all_slices[id], slice_ids.keys()):
+            slivers = [] 
+           
+           for slice in map(lambda id: all_slices[id], slice_ids.keys()):
                 keys = []
                 ### still missing in foreign slices
                 try:
@@ -164,22 +168,29 @@ class GetSlivers(Method):
                     keys += [{'key_type':'missing',
                               'key':'key caching not implemented yet'}]
 
-                attributes = {}
+                sliver_attributes = []
+                attributes = []
                 ### still missing in foreign slices
                 try:
-                    for slice_attribute in map(lambda id: all_slice_attributes[id],
-                                               slice['slice_attribute_ids']):
-                        # Per-node sliver attributes (slice attributes
-                        # with non-null node_id fields) take precedence
-                        # over global slice attributes.
-                        if not attributes.has_key(slice_attribute['name']) or \
-                               slice_attribute['node_id'] is not None:
-                            attributes[slice_attribute['name']] = {
-                                'name': slice_attribute['name'],
-                                'value': slice_attribute['value']
-                                }
-                except:
-                        attributes={'ignored':{'name':'attributes caching','value':'not implemented yet'}}
+                    slice_attributes = map(lambda id: all_slice_attributes[id],
+                                           slice['slice_attribute_ids'])
+
+                    # Per-node sliver attributes take precedence over
+                    # global slice attributes, so set them first.
+                    for sliver_attribute in filter(lambda a: a['node_id'] == node_id, slice_attributes):
+                        sliver_attributes.append(sliver_attribute['name'])
+                        attributes.append({'name': sliver_attribute['name'],
+                                           'value': sliver_attribute['value']})
+
+                    for slice_attribute in filter(lambda a: a['node_id'] is None, slice_attributes):
+                        # Do not set any global slice attributes for
+                        # which there is at least one sliver attribute
+                        # already set.
+                        if slice_attribute['name'] not in sliver_attributes:
+                            attributes.append({'name': slice_attribute['name'],
+                                               'value': slice_attribute['value']})
+                except Exception, err:
+                    attributes=[{'name':'attributes caching','value':'not implemented yet'}]
 
                 slivers.append({
                     'name': slice['name'],
@@ -187,9 +198,9 @@ class GetSlivers(Method):
                     'instantiation': slice['instantiation'],
                     'expires': slice['expires'],
                     'keys': keys,
-                    'attributes': attributes.values()
+                    'attributes': attributes
                     })
-
+           
             nodes.append({
                 'timestamp': timestamp,
                 'node_id': node['node_id'],