* [PATCH] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs
@ 2022-06-03 12:48 Thomas Huth
2022-06-03 16:17 ` Richard Henderson
2022-06-08 14:59 ` Alex Bennée
0 siblings, 2 replies; 4+ messages in thread
From: Thomas Huth @ 2022-06-03 12:48 UTC (permalink / raw)
To: qemu-devel, Alex Bennée, Philippe Mathieu-Daudé,
Daniel P . Berrange
Cc: Wainer dos Santos Moschetta, Beraldo Leal
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>
---
I wonder whether we should remove the build-cfi-aarch64 job instead.
When I tried to run it during the past months, it was always failing
for me. This time, I tried to bump the timeout while I was at it,
and it takes longer than 80 minutes here to finish - so I asume
nobody ever ran this successfully in the last months... Is anybody
using this job at all? I think if we want to have CFI coverage here,
it should get replaced by a custom runner job that runs on a more
beefy machine... (the ppc64-s390x job is fine by the way, it often
only runs a little bit longer than 60 minutes - I still bumped the
timeout here, too, just to be on the safe side)
.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 ecac3ec50c..baaa0ebb87 100644
--- a/.gitlab-ci.d/buildtest.yml
+++ b/.gitlab-ci.d/buildtest.yml
@@ -355,16 +355,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
check-cfi-aarch64:
extends: .native_test_job_template
@@ -396,16 +395,15 @@ build-cfi-ppc64-s390x:
--enable-safe-stack --enable-slirp=git
TARGETS: ppc64-softmmu s390x-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: 80m
+ artifacts:
+ expire_in: 2 days
+ paths:
+ - build
check-cfi-ppc64-s390x:
extends: .native_test_job_template
--
2.31.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs
2022-06-03 12:48 [PATCH] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs Thomas Huth
@ 2022-06-03 16:17 ` Richard Henderson
2022-06-03 16:32 ` Thomas Huth
2022-06-08 14:59 ` Alex Bennée
1 sibling, 1 reply; 4+ messages in thread
From: Richard Henderson @ 2022-06-03 16:17 UTC (permalink / raw)
To: Thomas Huth, qemu-devel, Alex Bennée,
Philippe Mathieu-Daudé,
Daniel P . Berrange
Cc: Wainer dos Santos Moschetta, Beraldo Leal
On 6/3/22 05:48, Thomas Huth wrote:
> 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>
> ---
> I wonder whether we should remove the build-cfi-aarch64 job instead.
> When I tried to run it during the past months, it was always failing
> for me. This time, I tried to bump the timeout while I was at it,
> and it takes longer than 80 minutes here to finish - so I asume
> nobody ever ran this successfully in the last months... Is anybody
> using this job at all? I think if we want to have CFI coverage here,
> it should get replaced by a custom runner job that runs on a more
> beefy machine... (the ppc64-s390x job is fine by the way, it often
> only runs a little bit longer than 60 minutes - I still bumped the
> timeout here, too, just to be on the safe side)
Acked-by: Richard Henderson <richard.henderson@linaro.org>
I think it might be useful to extend the other s390x jobs a bit too. The last couple of
fails have the test *nearly* completing. E.g. your most recent pr:
https://gitlab.com/qemu-project/qemu/-/jobs/2544009687
Whether that indicates we've a speed regression, or host loading, or simply changes to the
testsuite, I don't know.
r~
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs
2022-06-03 16:17 ` Richard Henderson
@ 2022-06-03 16:32 ` Thomas Huth
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Huth @ 2022-06-03 16:32 UTC (permalink / raw)
To: Richard Henderson, qemu-devel, Alex Bennée,
Daniel P . Berrange, Peter Maydell, Christian Borntraeger
Cc: Wainer dos Santos Moschetta, Beraldo Leal, Philippe Mathieu-Daudé
On 03/06/2022 18.17, Richard Henderson wrote:
> On 6/3/22 05:48, Thomas Huth wrote:
>> 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>
>> ---
>> I wonder whether we should remove the build-cfi-aarch64 job instead.
>> When I tried to run it during the past months, it was always failing
>> for me. This time, I tried to bump the timeout while I was at it,
>> and it takes longer than 80 minutes here to finish - so I asume
>> nobody ever ran this successfully in the last months... Is anybody
>> using this job at all? I think if we want to have CFI coverage here,
>> it should get replaced by a custom runner job that runs on a more
>> beefy machine... (the ppc64-s390x job is fine by the way, it often
>> only runs a little bit longer than 60 minutes - I still bumped the
>> timeout here, too, just to be on the safe side)
>
> Acked-by: Richard Henderson <richard.henderson@linaro.org>
>
> I think it might be useful to extend the other s390x jobs a bit too. The
> last couple of fails have the test *nearly* completing. E.g. your most
> recent pr:
>
> https://gitlab.com/qemu-project/qemu/-/jobs/2544009687
These tests are running on the custom s390x runner machine - I don't have
access to that one, i.e. I also do not have any means to test changes here
--> it would be great if that change could be done by somebody who has
access to that machine... Peter? Christian?
Thomas
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs
2022-06-03 12:48 [PATCH] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs Thomas Huth
2022-06-03 16:17 ` Richard Henderson
@ 2022-06-08 14:59 ` Alex Bennée
1 sibling, 0 replies; 4+ messages in thread
From: Alex Bennée @ 2022-06-08 14:59 UTC (permalink / raw)
To: Thomas Huth
Cc: qemu-devel, Philippe Mathieu-Daudé,
Daniel P . Berrange, Wainer dos Santos Moschetta, Beraldo Leal
Thomas Huth <thuth@redhat.com> writes:
> 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>
> ---
> I wonder whether we should remove the build-cfi-aarch64 job instead.
> When I tried to run it during the past months, it was always failing
> for me. This time, I tried to bump the timeout while I was at it,
> and it takes longer than 80 minutes here to finish - so I asume
> nobody ever ran this successfully in the last months... Is anybody
> using this job at all? I think if we want to have CFI coverage here,
> it should get replaced by a custom runner job that runs on a more
> beefy machine... (the ppc64-s390x job is fine by the way, it often
> only runs a little bit longer than 60 minutes - I still bumped the
> timeout here, too, just to be on the safe side)
>
> .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 ecac3ec50c..baaa0ebb87 100644
> --- a/.gitlab-ci.d/buildtest.yml
> +++ b/.gitlab-ci.d/buildtest.yml
> @@ -355,16 +355,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
>
> check-cfi-aarch64:
> extends: .native_test_job_template
> @@ -396,16 +395,15 @@ build-cfi-ppc64-s390x:
> --enable-safe-stack --enable-slirp=git
> TARGETS: ppc64-softmmu s390x-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: 80m
> + artifacts:
> + expire_in: 2 days
> + paths:
> + - build
>
> check-cfi-ppc64-s390x:
> extends: .native_test_job_template
Queued to testing/next, thanks.
--
Alex Bennée
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-06-08 15:19 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-03 12:48 [PATCH] gitlab-ci: Fix the build-cfi-aarch64 and build-cfi-ppc64-s390x jobs Thomas Huth
2022-06-03 16:17 ` Richard Henderson
2022-06-03 16:32 ` Thomas Huth
2022-06-08 14:59 ` Alex Bennée
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.