Signed-off-by: Andrei Stepanov astepano@redhat.com
rebased onto a0df3a0578dc962ddc8dd9a6f651c1d3c18e2bdb
./roles/standard-test-basic/files/run-basic-test -w wdir1 -a adir1 -t testname -c 'echo hi; sleep 3' hi Run test 'testname': done. Test's exit code: 0 (python3ve) ➜ standard-test-roles git:(runtime) ✗ cat adir1/results.yml results: - test: testname result: pass runtime_seconds: 3 logs: - PASS-str_testname.log
Could we call it just runtime pls? it is shorter and we do not expect to add runtime_hours or similar
runtime
runtime_hours
@astepano thanks for fixing so quickly, just one comment
rebased onto cecdb3ac28fa1b02ebf6c5eb33c75a802a12a464
➜ ./roles/standard-test-basic/files/run-basic-test -w wdir2 -a adir2 -t testname -c 'echo hi; sleep 3' hi Run test 'testname': done. Test's exit code: 0 ➜ cat adir2/results.yml results: - test: testname result: pass runtime: 3 logs: - PASS-str_testname.log
updated. @bgoncalv or @msrb please ack.
@astepano ty! looks good. I found out that TMT aleady has the attribute named as duration and uses HH:MM:SS, but w can use seconds here
duration
lgtm
"duration" is also a good word :) LGTM :wink:
pretty please pagure-ci rebuild
Commit 93b7f7ce fixes this pull-request
Pull-Request has been merged by astepano
Signed-off-by: Andrei Stepanov astepano@redhat.com