[PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual

Thomas Huth posted 1 patch 2 years, 9 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210728075141.400816-1-thuth@redhat.com
Maintainers: Thomas Huth <thuth@redhat.com>, Wainer dos Santos Moschetta <wainersm@redhat.com>, Willian Rampazzo <willianr@redhat.com>, "Philippe Mathieu-Daudé" <f4bug@amsat.org>, Cornelia Huck <cohuck@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>
.gitlab-ci.d/buildtest.yml | 12 ++++++++++++
1 file changed, 12 insertions(+)
[PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
Posted by Thomas Huth 2 years, 9 months ago
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


Re: [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
Posted by Daniel P. Berrangé 2 years, 9 months ago
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 :|


Re: [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
Posted by Philippe Mathieu-Daudé 2 years, 9 months ago
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
> 


Re: [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
Posted by Willian Rampazzo 2 years, 9 months ago
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>


Re: [PATCH] gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual
Posted by Daniele Buono 2 years, 9 months ago
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
>