Merge lp:~qzhang/lava-dispatcher/unique-bundle-name into lp:lava-dispatcher
Proposed by
Spring Zhang
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 73 | ||||
Proposed branch: | lp:~qzhang/lava-dispatcher/unique-bundle-name | ||||
Merge into: | lp:lava-dispatcher | ||||
Diff against target: |
24 lines (+4/-1) 1 file modified
lava_dispatcher/actions/lava-test.py (+4/-1) |
||||
To merge this branch: | bzr merge lp:~qzhang/lava-dispatcher/unique-bundle-name | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paul Larson (community) | Approve | ||
Review via email: mp+66412@code.launchpad.net |
Description of the change
fix #768453
add time string to result bundle name
To post a comment you must log in.
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!