Merge lp:~liuyq0307/linaro-android-build-tools/install-break into lp:linaro-android-build-tools

[Yongqin Liu]

changed to use one lava-android-test-install actions for each test, instead of using one lava-android-test-install action for all tests, by doing this way the next test installation will not be broken when the one installation before it failed.

tested with this job: http://staging.validation.linaro.org/scheduler/job/44607

[Yongqin Liu (liuyq0307)]

Hi, Infrastructure team,

Anyone can help to review this modification?

Thanks,
Yongqin Liu

[Milo Casagrande (milo)]

Hello there,

my apologies for taking so long to review this.
It looks good for me, also based on the job already tested. Are there any examples of it being in use?

Overall I would say +1.

review: Approve

=== modified file 'build-scripts/post-build-lava.py'
--- build-scripts/post-build-lava.py	2013-03-27 07:41:55 +0000
+++ build-scripts/post-build-lava.py	2013-04-22 07:52:43 +0000
@@ -142,12 +142,14 @@
                 continue
         test_actions.append(test)
 
-    if len(test_actions) > 0:
+    ## make the next test installation be able to execute
+    ## when one test installation failed
+    for test_action in list(set(test_actions)):
         inst_action = {
                           "command": "lava_android_test_install",
                           "parameters": {
                                         # ensure only unique test names
-                                        "tests": list(set(test_actions))
+                                        "tests": [test_action]
                                          }
                             }
         actions.append(inst_action)