diff mbox series

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

Message ID 20210728075141.400816-1-thuth@redhat.com
State New
Headers show
Series gitlab-ci.d/buildtest: Mark the aarch64 and ppc64-s390x CFI jobs as manual | expand

Commit Message

Thomas Huth July 28, 2021, 7:51 a.m. UTC
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(+)

Comments

Daniel P. Berrangé July 28, 2021, 8:32 a.m. UTC | #1
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
Philippe Mathieu-Daudé July 28, 2021, 12:04 p.m. UTC | #2
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
>
Willian Rampazzo July 28, 2021, 2:14 p.m. UTC | #3
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>
Daniele Buono July 30, 2021, 3:26 p.m. UTC | #4
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
>
diff mbox series

Patch

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