From: Thomas Huth <thuth@redhat.com>
To: "Richard Henderson" <richard.henderson@linaro.org>,
"Alex Bennée" <alex.bennee@linaro.org>,
qemu-devel@nongnu.org
Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org,
aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com,
crosa@redhat.com,
Wainer dos Santos Moschetta <wainersm@redhat.com>,
Beraldo Leal <bleal@redhat.com>
Subject: Re: [PATCH v1 3/7] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs
Date: Tue, 14 Jun 2022 06:30:47 +0200 [thread overview]
Message-ID: <8ad852ab-9580-fc52-b641-679a70212245@redhat.com> (raw)
In-Reply-To: <39615b6d-6454-2a4f-013e-478766e0c3f8@linaro.org>
On 13/06/2022 23.46, Richard Henderson wrote:
> On 6/13/22 10:12, Alex Bennée wrote:
>> From: Thomas Huth <thuth@redhat.com>
>>
>> The job definitions recently got a second "variables:" section by
>> accident and thus are failing now if one tries to run them. Merge
>> the two sections into one again to fix the issue.
>>
>> And while we're at it, bump the timeout here (70 minutes are currently
>> not enough for the aarch64 job). The jobs are marked as manual anyway,
>> so if the user starts them, they want to see their result for sure and
>> then it's annoying if the job timeouts too early.
>>
>> Fixes: e312d1fdbb ("gitlab: convert build/container jobs to
>> .base_job_template")
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> Acked-by: Richard Henderson <richard.henderson@linaro.org>
>> Message-Id: <20220603124809.70794-1-thuth@redhat.com>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> ---
>> .gitlab-ci.d/buildtest.yml | 22 ++++++++++------------
>> 1 file changed, 10 insertions(+), 12 deletions(-)
>>
>> diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
>> index 544385f5be..cb7cad44b5 100644
>> --- a/.gitlab-ci.d/buildtest.yml
>> +++ b/.gitlab-ci.d/buildtest.yml
>> @@ -357,16 +357,15 @@ build-cfi-aarch64:
>> --enable-safe-stack --enable-slirp=git
>> TARGETS: aarch64-softmmu
>> MAKE_CHECK_ARGS: check-build
>> - timeout: 70m
>> - artifacts:
>> - expire_in: 2 days
>> - paths:
>> - - build
>> - variables:
>> # FIXME: This job is often failing, likely due to out-of-memory
>> problems in
>> # the constrained containers of the shared runners. Thus this is
>> marked as
>> # skipped until the situation has been solved.
>> QEMU_JOB_SKIPPED: 1
>> + timeout: 90m
>> + artifacts:
>> + expire_in: 2 days
>> + paths:
>> + - build
>
> FWIW, 90 minutes was close, but insufficient:
>
> https://gitlab.com/qemu-project/qemu/-/jobs/2584472225
Hmm, it was working at least once for me while I was working on the patch.
But as I already wrote here:
https://lists.gnu.org/archive/html/qemu-devel/2022-06/msg00463.html
I think nobody really used this build-cfi-aarch64 in month ... so we should
maybe have a try with the 90 min timeout first (maybe the CI servers were
just a little bit overloaded when you tried), but if the test continues to
hit the 90 minutes timeout, I'd say we rather delete it instead of bumping
the timeout even further. 90 minutes are really very close to the pain level
already - at least for me.
> But certainly, let us fix the job definition:
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Thanks!
Thomas
next prev parent reply other threads:[~2022-06-14 4:36 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-13 17:12 [PATCH v1 0/7] testing/next pre-PR (docker, gitlab, tcg) Alex Bennée
2022-06-13 17:12 ` [PATCH v1 1/7] test/tcg/arm: Use -mfloat-abi=soft for test-armv6m-undef Alex Bennée
2022-06-13 17:12 ` [PATCH v1 2/7] tests/tcg/i386: Use explicit suffix on fist insns Alex Bennée
2022-06-13 17:12 ` [PATCH v1 3/7] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs Alex Bennée
2022-06-13 21:46 ` Richard Henderson
2022-06-14 4:30 ` Thomas Huth [this message]
2022-06-14 8:29 ` Daniel P. Berrangé
2022-06-13 17:12 ` [PATCH v1 4/7] tests/docker: fix the IMAGE for build invocation Alex Bennée
2022-06-13 21:47 ` Richard Henderson
2022-06-13 17:12 ` [PATCH v1 5/7] tests/tcg: disable xtensa-linux-user again Alex Bennée
2022-06-13 21:47 ` Richard Henderson
2022-06-13 22:33 ` Philippe Mathieu-Daudé via
2022-06-13 17:12 ` [PATCH v1 6/7] gitlab: compare CIRRUS_nn vars against 'null' not "" Alex Bennée
2022-06-13 17:12 ` [PATCH v1 7/7] .gitlab: use less aggressive nproc on our aarch64/32 runners Alex Bennée
2022-06-13 21:48 ` Richard Henderson
2022-06-13 23:04 ` Philippe Mathieu-Daudé via
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8ad852ab-9580-fc52-b641-679a70212245@redhat.com \
--to=thuth@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=aurelien@aurel32.net \
--cc=berrange@redhat.com \
--cc=bleal@redhat.com \
--cc=crosa@redhat.com \
--cc=f4bug@amsat.org \
--cc=fam@euphon.net \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=stefanha@redhat.com \
--cc=wainersm@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.