Changeset d20823f for fedd/federation

Timestamp:

May 3, 2010 10:01:19 AM (15 years ago)

Author:

Ted Faber <faber@…>

Branches:

axis_example, compt_changes, info-ops, master, version-3.01, version-3.02

Children:

e77c86e

Parents:

f3803ea

Message:

This is very odd. Defaulting reqs in the federated_service constructor
resulted in all the instances sharing a reqs member (they all pointed to the
same thing). That's pretty non-intuitive, but seems to be the way python does
default parameter initialization. Beware.

File:

• fedd/federation/experiment_control.py (modified) (2 diffs)

fedd/federation/experiment_control.py

@@ -52 +52 @@
         self.params = params
         self.reqs = reqs
+
+    def __str__(self):
+        return "name %s export %s import %s params %s reqs %s" % \
+                (self.name, self.exporter, self.importers, self.params,
+                        [ (r['name'], r['visibility']) for r in self.reqs] )
 
 class experiment_control_local:
@@ -1712 +1717 @@
                         masters[tb].append(federated_service(name=s['name'],
                                 exporter=tb, importers=s.get('import',[]),
-                                params=params))
+                                params=params, reqs=[]))
                     else:
                         self.log.error('Testbed service does not have name " + \