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

Proposed by Po-Hsu Lin
Status: Merged
Merged at revision: 912275ccfc619a592cddeba2afef019bbb1af3a2
Proposed branch: ~canonical-kernel-team/+git/autotest-client-tests:phlin/hugemmap15-timeout
Merge into: ~canonical-kernel-team/+git/autotest-client-tests:master
Diff against target: 25 lines (+5/-3)
1 file modified
ubuntu_ltp_stable/ubuntu_ltp_stable.py (+5/-3)
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Review via email: mp+446146@code.launchpad.net

Commit message

hugemmap15 will take about 161 seconds to run on J-generic-64k,
set LTP_TIMEOUT_MUL=10 (300 seconds) to give it a chance to finish.

Change where we print the information for LTP_TIMEOUT_MUL change.

To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote :

+1

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
diff --git a/ubuntu_ltp_stable/ubuntu_ltp_stable.py b/ubuntu_ltp_stable/ubuntu_ltp_stable.py
index 414361d..bb1c7cd 100644
--- a/ubuntu_ltp_stable/ubuntu_ltp_stable.py
+++ b/ubuntu_ltp_stable/ubuntu_ltp_stable.py
@@ -104,9 +104,10 @@ class ubuntu_ltp_stable(test.test):
104104
105 fn = '/opt/ltp/runtest/%s' % (test_name)105 fn = '/opt/ltp/runtest/%s' % (test_name)
106106
107 print("Setting LTP_TIMEOUT_MUL exceptions...")107 timeout_cases = {'zram01': '5',
108 print("Setting LTP_TIMEOUT_MUL=3 for cve-2018-1000204 / ioctl_sg01 (lp:1899413, lp:1936886, lp:1949934)")108 'ioctl_sg01': '3', # cve-2018-1000204 / ioctl_sg01 (LP: #1899413, LP: #1936886, LP: #1949934)
109 timeout_cases = {'zram01': '5', 'ioctl_sg01': '3', 'in6_02': '4'}109 'in6_02': '4',
110 'hugemmap15': '10'} # LP: #2026248
110 if utils.system_output('virt-what', verbose=False):111 if utils.system_output('virt-what', verbose=False):
111 print("Running in VM, set timeout multiplier LTP_TIMEOUT_MUL=3 for memcg_test_3 (lp:1836694)")112 print("Running in VM, set timeout multiplier LTP_TIMEOUT_MUL=3 for memcg_test_3 (lp:1836694)")
112 timeout_cases['memcg_test_3'] = '3'113 timeout_cases['memcg_test_3'] = '3'
@@ -124,6 +125,7 @@ class ubuntu_ltp_stable(test.test):
124125
125 for _case in timeout_cases:126 for _case in timeout_cases:
126 if _case in line:127 if _case in line:
128 print("Setting LTP_TIMEOUT_MUL={} for {}".format(timeout_cases[_case], _case))
127 os.environ["LTP_TIMEOUT_MUL"] = timeout_cases[_case]129 os.environ["LTP_TIMEOUT_MUL"] = timeout_cases[_case]
128 break130 break
129131

Subscribers

People subscribed via source and target branches

to all changes: