* Re: [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
2021-07-28 7:51 [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual Thomas Huth
@ 2021-07-28 8:32 ` Daniel P. Berrangé
2021-07-28 12:04 ` Philippe Mathieu-Daudé
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Daniel P. Berrangé @ 2021-07-28 8:32 UTC (permalink / raw)
To: Thomas Huth
Cc: qemu-devel, Wainer dos Santos Moschetta,
Philippe Mathieu-Daudé,
Willian Rampazzo, Alex Bennée, Daniele Buono
On Wed, Jul 28, 2021 at 09:51:41AM +0200, Thomas Huth wrote:
> These two jobs are currently failing very often - the linker seems to
> get killed due to out-of-memory problems. Since apparently nobody has
> currently an idea how to fix that nicely, let's mark the jobs as manual
> for the time being until someone comes up with a proper fix.
Copying the original author of the CFI work...
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> .gitlab-ci.d/buildtest.yml | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
> index 63f1903f07..3537c6f1a1 100644
> --- a/.gitlab-ci.d/buildtest.yml
> +++ b/.gitlab-ci.d/buildtest.yml
> @@ -416,6 +416,12 @@ build-cfi-aarch64:
> expire_in: 2 days
> paths:
> - build
> + rules:
> + # FIXME: This job is often failing, likely due to out-of-memory problems in
> + # the constraint containers of the shared runners. Thus this is marked as
s/constraint/constrained/
> + # manual until the situation has been solved.
> + - when: manual
> + allow_failure: true
>
> check-cfi-aarch64:
> extends: .native_test_job_template
> @@ -452,6 +458,12 @@ build-cfi-ppc64-s390x:
> expire_in: 2 days
> paths:
> - build
> + rules:
> + # FIXME: This job is often failing, likely due to out-of-memory problems in
> + # the constraint containers of the shared runners. Thus this is marked as
s/constraint/constrained/
> + # manual until the situation has been solved.
> + - when: manual
> + allow_failure: true
>
> check-cfi-ppc64-s390x:
> extends: .native_test_job_template
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
2021-07-28 7:51 [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual Thomas Huth
2021-07-28 8:32 ` Daniel P. Berrangé
@ 2021-07-28 12:04 ` Philippe Mathieu-Daudé
2021-07-28 14:14 ` Willian Rampazzo
2021-07-30 15:26 ` Daniele Buono
3 siblings, 0 replies; 5+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-07-28 12:04 UTC (permalink / raw)
To: Thomas Huth, qemu-devel
Cc: Willian Rampazzo, Alex Bennée, Wainer dos Santos Moschetta,
Daniele Buono
On 7/28/21 9:51 AM, Thomas Huth wrote:
> These two jobs are currently failing very often - the linker seems to
> get killed due to out-of-memory problems. Since apparently nobody has
> currently an idea how to fix that nicely, let's mark the jobs as manual
> for the time being until someone comes up with a proper fix.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> .gitlab-ci.d/buildtest.yml | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
> index 63f1903f07..3537c6f1a1 100644
> --- a/.gitlab-ci.d/buildtest.yml
> +++ b/.gitlab-ci.d/buildtest.yml
> @@ -416,6 +416,12 @@ build-cfi-aarch64:
> expire_in: 2 days
> paths:
> - build
> + rules:
> + # FIXME: This job is often failing, likely due to out-of-memory problems in
> + # the constraint containers of the shared runners. Thus this is marked as
> + # manual until the situation has been solved.> + - when: manual
> + allow_failure: true
Eventually:
# Except if user explicitly sets the QEMU_CI_CFI_TESTING variable
- if: '$QEMU_CI_CFI_TESTING'
when: always
But I agree, better to fix for everybody.
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> check-cfi-aarch64:
> extends: .native_test_job_template
> @@ -452,6 +458,12 @@ build-cfi-ppc64-s390x:
> expire_in: 2 days
> paths:
> - build
> + rules:
> + # FIXME: This job is often failing, likely due to out-of-memory problems in
> + # the constraint containers of the shared runners. Thus this is marked as
> + # manual until the situation has been solved.
> + - when: manual
> + allow_failure: true
>
> check-cfi-ppc64-s390x:
> extends: .native_test_job_template
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
2021-07-28 7:51 [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual Thomas Huth
2021-07-28 8:32 ` Daniel P. Berrangé
2021-07-28 12:04 ` Philippe Mathieu-Daudé
@ 2021-07-28 14:14 ` Willian Rampazzo
2021-07-30 15:26 ` Daniele Buono
3 siblings, 0 replies; 5+ messages in thread
From: Willian Rampazzo @ 2021-07-28 14:14 UTC (permalink / raw)
To: Thomas Huth
Cc: Daniele Buono, Alex Bennée, qemu-devel,
Wainer dos Santos Moschetta, Philippe Mathieu-Daudé
On Wed, Jul 28, 2021 at 4:51 AM Thomas Huth <thuth@redhat.com> wrote:
>
> These two jobs are currently failing very often - the linker seems to
> get killed due to out-of-memory problems. Since apparently nobody has
> currently an idea how to fix that nicely, let's mark the jobs as manual
> for the time being until someone comes up with a proper fix.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> .gitlab-ci.d/buildtest.yml | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
2021-07-28 7:51 [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual Thomas Huth
` (2 preceding siblings ...)
2021-07-28 14:14 ` Willian Rampazzo
@ 2021-07-30 15:26 ` Daniele Buono
3 siblings, 0 replies; 5+ messages in thread
From: Daniele Buono @ 2021-07-30 15:26 UTC (permalink / raw)
To: Thomas Huth, qemu-devel
Cc: Willian Rampazzo, Alex Bennée, Philippe Mathieu-Daudé,
Wainer dos Santos Moschetta
I agree, making these manual tasks until we find a fix for this is the
only solution I can think of too.
Daniele
On 7/28/2021 3:51 AM, Thomas Huth wrote:
> These two jobs are currently failing very often - the linker seems to
> get killed due to out-of-memory problems. Since apparently nobody has
> currently an idea how to fix that nicely, let's mark the jobs as manual
> for the time being until someone comes up with a proper fix.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> .gitlab-ci.d/buildtest.yml | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
> index 63f1903f07..3537c6f1a1 100644
> --- a/.gitlab-ci.d/buildtest.yml
> +++ b/.gitlab-ci.d/buildtest.yml
> @@ -416,6 +416,12 @@ build-cfi-aarch64:
> expire_in: 2 days
> paths:
> - build
> + rules:
> + # FIXME: This job is often failing, likely due to out-of-memory problems in
> + # the constraint containers of the shared runners. Thus this is marked as
> + # manual until the situation has been solved.
> + - when: manual
> + allow_failure: true
>
> check-cfi-aarch64:
> extends: .native_test_job_template
> @@ -452,6 +458,12 @@ build-cfi-ppc64-s390x:
> expire_in: 2 days
> paths:
> - build
> + rules:
> + # FIXME: This job is often failing, likely due to out-of-memory problems in
> + # the constraint containers of the shared runners. Thus this is marked as
> + # manual until the situation has been solved.
> + - when: manual
> + allow_failure: true
>
> check-cfi-ppc64-s390x:
> extends: .native_test_job_template
>
^ permalink raw reply [flat|nested] 5+ messages in thread