#393 Add runtime entry in results.yml: https://pagure.io/standard-test-roles/issue/392
Merged by astepano. Opened by astepano.
runtime  into  master

Download 393.patch

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

Could we call it just runtime pls? it is shorter and we do not expect to add runtime_hours or similar

@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" 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

Pull-Request has been merged by astepano

Metadata