qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
@ 2021-07-28  7:51 Thomas Huth
  2021-07-28  8:32 ` Daniel P. Berrangé
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Thomas Huth @ 2021-07-28  7:51 UTC (permalink / raw)
  To: qemu-devel
  Cc: Willian Rampazzo, Daniele Buono, Alex Bennée,
	Philippe Mathieu-Daudé,
	Wainer dos Santos Moschetta

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
-- 
2.27.0



^ permalink raw reply related	[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é
                   ` (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

end of thread, other threads:[~2021-07-30 15:27 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).