Merge ~canonical-kernel-team/+git/autotest-client-tests:phlin/memcg-test-3-bump 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: 592512e33c56c493c0668db6163c69d4bb3b1696
Proposed branch: ~canonical-kernel-team/+git/autotest-client-tests:phlin/memcg-test-3-bump
Merge into: ~canonical-kernel-team/+git/autotest-client-tests:master
Diff against target: 15 lines (+2/-2)
1 file modified
ubuntu_ltp_controllers/ubuntu_ltp_controllers.py (+2/-2)
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Review via email: mp+464666@code.launchpad.net

Commit message

Test shows memcg_test_3 will take slightly over our current 60m
threshold, bump it to 75m to give it a chance to finish.

To post a comment you must log in.
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

Patch tested on google@obruchev and it's working as expected.

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
1diff --git a/ubuntu_ltp_controllers/ubuntu_ltp_controllers.py b/ubuntu_ltp_controllers/ubuntu_ltp_controllers.py
2index 0de1218..b4b76a0 100644
3--- a/ubuntu_ltp_controllers/ubuntu_ltp_controllers.py
4+++ b/ubuntu_ltp_controllers/ubuntu_ltp_controllers.py
5@@ -117,8 +117,8 @@ class ubuntu_ltp_controllers(test.test):
6 return
7
8 if test_name == 'memcg_test_3':
9- print("Set timeout multiplier LTP_TIMEOUT_MUL=120 for memcg_test_3 (lp:1836694)")
10- os.environ["LTP_TIMEOUT_MUL"] = '120'
11+ print("Set timeout multiplier LTP_TIMEOUT_MUL=150 for memcg_test_3 (lp:1836694)")
12+ os.environ["LTP_TIMEOUT_MUL"] = '150' # 75min
13 elif test_name == 'memcg_stress':
14 print("Set timeout multiplier LTP_TIMEOUT_MUL=2 for memcg_stress (lp:1946348)")
15 os.environ["LTP_TIMEOUT_MUL"] = '2'

Subscribers

People subscribed via source and target branches

to all changes: