Merge lp:~codehelp/lava-dispatcher/wait-all into lp:lava-dispatcher/multinode

[Neil Williams]

The main changes for the fix for wait_all are in the coordinator:
http://people.linaro.org/~neil.williams/lava/pool/main/l/lava-coordinator/lava-coordinator_0.1.3-1_all.deb

This change in node needs to be with this version of coordinator (or a higher version).

git code for lava-coordinator: https://git.linaro.org/gitweb?p=people/neilwilliams/lava-group.git;a=summary

This branch needs to be merged after the xmlrpc-results branch (which in turn needs the merge for lava-dashboard) but the change itself is already running on multinode01.validation.linaro.org for testing.

[Fu Wei (fu-wei)]

Good! Thank you very much!

review: Approve

=== modified file 'lava/dispatcher/node.py'
--- lava/dispatcher/node.py	2013-08-06 14:35:42 +0000
+++ lava/dispatcher/node.py	2013-08-06 14:35:42 +0000
@@ -311,7 +311,7 @@
             reply_str = self.request_wait(messageID)
         elif json_data['request'] == 'lava_wait_all':
             logging.info("requesting lava_wait_all %s" % json.dumps(json_data))
-            if 'role' in json_data:
+            if 'role' in json_data and json_data['role'] is not None:
                 reply_str = self.request_wait_all(messageID, json_data['role'])
             else:
                 reply_str = self.request_wait_all(messageID)
@@ -362,7 +362,7 @@
         if role:
             return self._send({"request": "lava_wait_all",
                               "messageID": messageID,
-                              "role": role})
+                              "waitrole": role})
         else:
             return self._send({"request": "lava_wait_all",
                               "messageID": messageID})