Merge lp:~qzhang/lava-dispatcher/unique-bundle-name into lp:lava-dispatcher

[Spring Zhang]

fix #768453
add time string to result bundle name

[Paul Larson (pwlars)]

16 + dt = datetime.now()
17 + bundle_name = test_name + "-" + str(dt.hour) + str(dt.minute) + str(dt.second)
Seems sensible enough to add a timestamp for uniqueness, but I think I'd rather add a more traditional timestamp. Just use time.time() to get a timestamp instead of all this. It's shorter and easier :)

Otherwise, looks good, thanks!

review: Approve

[Spring Zhang (qzhang)]

Thanks for the comments, I adjust the code to an easier way but I think it's meaningful to keep some time info in file name than total seconds from 1970

=== modified file 'lava_dispatcher/actions/lava-test.py'
--- lava_dispatcher/actions/lava-test.py	2011-06-28 14:11:21 +0000
+++ lava_dispatcher/actions/lava-test.py	2011-06-30 05:18:29 +0000
@@ -20,6 +20,7 @@
 # along
 # with this program; if not, see <http://www.gnu.org/licenses>.
 
+from datetime import datetime
 from lava_dispatcher.actions import BaseAction
 from lava_dispatcher.client import OperationFailed
 from lava_dispatcher.config import LAVA_RESULT_DIR, MASTER_STR, TESTER_STR
@@ -32,9 +33,11 @@
         client.run_shell_command('mkdir -p %s' % LAVA_RESULT_DIR,
             response = TESTER_STR)
         client.export_display()
+        dt = datetime.now()
+        bundle_name = test_name + "-" + str(dt.hour) + str(dt.minute) + str(dt.second)
         client.run_shell_command(
             'lava-test run %s -o %s/%s.bundle' % (
-                test_name, LAVA_RESULT_DIR, test_name),
+                test_name, LAVA_RESULT_DIR, bundle_name),
             response = TESTER_STR, timeout = timeout)
 
 class cmd_lava_test_install(BaseAction):