Merge ~canonical-kernel-team/+git/autotest-client-tests:phlin/zfs-stress-timeout into ~canonical-kernel-team/+git/autotest-client-tests:master

Proposed by Po-Hsu Lin
Status: Merged
Merge reported by: Po-Hsu Lin
Merged at revision: 1a633e833d3bc7e796c8e5e16f3306e1a87acc63
Proposed branch: ~canonical-kernel-team/+git/autotest-client-tests:phlin/zfs-stress-timeout
Merge into: ~canonical-kernel-team/+git/autotest-client-tests:master
Diff against target: 15 lines (+3/-1)
1 file modified
ubuntu_zfs_stress/control (+3/-1)
Reviewer Review Type Date Requested Status
Andrei Gherzan Approve
Review via email: mp+439383@code.launchpad.net

Commit message

Test shows it can take up to 6h41m to run on Google instance g1-small,
bump the overall timeout to 8hrs to make it consistent with the change
we made in CKCT.

Although 7h50m + 10m might not equal to 8h when taking autotest
execution into account, but that should be close enough.

To post a comment you must log in.
Revision history for this message
Andrei Gherzan (agherzan) wrote :

Looks good.

review: Approve
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

Applied and pushed. Thanks!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/ubuntu_zfs_stress/control b/ubuntu_zfs_stress/control
2index 2bc0221..5896a72 100644
3--- a/ubuntu_zfs_stress/control
4+++ b/ubuntu_zfs_stress/control
5@@ -15,7 +15,9 @@ DOC = ""
6 if platform.machine() in ['riscv64', 'riscv']:
7 print("Test skipped as it takes too long to run on this platform\n")
8 else:
9- results = job.run_test_detail(NAME, test_name='zfs-stress', tag='zfs-stress', timeout=60*60*5) # 5hr. timeout
10+ # It can take up to 6h41m to run on Google g1.small, set timeout to 7h50m + 10m
11+ # to make it consistent with the change we made in CKCT (8hrs)
12+ results = job.run_test_detail(NAME, test_name='zfs-stress', tag='zfs-stress', timeout=60*60*7+50)
13 job.run_test_detail(NAME, test_name='post-test-zfs-cleanup', tag='post-test-zfs-cleanup', timeout=60*10)
14
15 # vi:set ts=4 sw=4 expandtab syntax=python:

Subscribers

People subscribed via source and target branches

to all changes: