Merge ~canonical-kernel-team/+git/autotest-client-tests:phlin/riscv-net-build-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: e6cbb48bdf18c8cb9726498c577d3c91eebfd3db
Proposed branch: ~canonical-kernel-team/+git/autotest-client-tests:phlin/riscv-net-build-timeout
Merge into: ~canonical-kernel-team/+git/autotest-client-tests:master
Diff against target: 13 lines (+1/-1)
1 file modified
ubuntu_kernel_selftests/control (+1/-1)
Reviewer Review Type Date Requested Status
Sean Feole Approve
Review via email: mp+455051@code.launchpad.net

Commit message

The net-build will need approximately 2h40m on 6.2 and 6.5 RISCV
instances, probably because of the increased number of tests.

A 3hr threshold should be a reasonable choice. We can either add
another if statement for RISCV, or simply bump the build timeout
to 90 mins and the arch_scale=2 on RISCV will make it 3hr.

Bumping this timeout should not have any negative effect. As I
don't recall that we have any other build task that failed.

To post a comment you must log in.
Revision history for this message
Sean Feole (sfeole) wrote :

+1

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

Applied and pushed, uncommitted changes removed, thanks.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/ubuntu_kernel_selftests/control b/ubuntu_kernel_selftests/control
2index 33ec6af..7806fd1 100644
3--- a/ubuntu_kernel_selftests/control
4+++ b/ubuntu_kernel_selftests/control
5@@ -31,7 +31,7 @@ result = job.run_test_detail(NAME, test_name='setup', tag='setup', timeout=arch_
6 if result == 'GOOD':
7 for category in categories:
8 build = '{}-build'.format(category)
9- result = job.run_test_detail(NAME, test_name=build, tag=build, timeout=arch_scale*60*60)
10+ result = job.run_test_detail(NAME, test_name=build, tag=build, timeout=arch_scale*60*90)
11 if result == 'ERROR':
12 print("ERROR: test suite '{}' failed to build, skipping all the sub tests".format(category))
13 continue

Subscribers

People subscribed via source and target branches

to all changes: