1. Patchew
  2. Xen
  3. View series

[PATCH] automation/eclair: Run *-testing jobs only in some branches

Nicola Vetrini posted 1 patch 2 days, 13 hours ago 
automation/gitlab-ci/analyze.yaml | 4 ++++
1 file changed, 4 insertions(+)
[PATCH] automation/eclair: Run *-testing jobs only in some branches
Posted by Nicola Vetrini 2 days, 13 hours ago 
From: Nicola Vetrini <nicola.vetrini@bugseng.com>

These jobs should not run in regular scans, but only in xen-project/people
repositories and branches to speed up pipeline execution.

Signed-off-by: Nicola Vetrini <nicola.vetrini@bugseng.com>
---
This should alleviate some of the recent delays of the ECLAIR pipelines
---
 automation/gitlab-ci/analyze.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/automation/gitlab-ci/analyze.yaml b/automation/gitlab-ci/analyze.yaml
index 5ddfe37903..88bc42f5c3 100644
--- a/automation/gitlab-ci/analyze.yaml
+++ b/automation/gitlab-ci/analyze.yaml
@@ -52,6 +52,8 @@ eclair-x86_64-testing:
   tags:
     - eclair-analysis-testing
   rules:
+    - if: $CI_PROJECT_PATH !~ /^xen-project\/people/.*$/
+      when: never
     - if: $ECLAIR_TESTING
       when: always
     - !reference [.eclair-analysis:triggered, rules]
@@ -110,6 +112,8 @@ eclair-ARM64-testing:
   tags:
     - eclair-analysis-testing
   rules:
+    - if: $CI_PROJECT_PATH !~ /^xen-project\/people/.*$/
+      when: never
     - if: $ECLAIR_TESTING
       when: always
     - !reference [.eclair-analysis:triggered, rules]
-- 
2.52.0
Re: [PATCH] automation/eclair: Run *-testing jobs only in some branches
Posted by Andrew Cooper 2 days, 13 hours ago 
On 11/12/2025 1:34 pm, Nicola Vetrini wrote:
> From: Nicola Vetrini <nicola.vetrini@bugseng.com>
>
> These jobs should not run in regular scans, but only in xen-project/people
> repositories and branches to speed up pipeline execution.
>
> Signed-off-by: Nicola Vetrini <nicola.vetrini@bugseng.com>
> ---
> This should alleviate some of the recent delays of the ECLAIR pipelines
> ---
>  automation/gitlab-ci/analyze.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/automation/gitlab-ci/analyze.yaml b/automation/gitlab-ci/analyze.yaml
> index 5ddfe37903..88bc42f5c3 100644
> --- a/automation/gitlab-ci/analyze.yaml
> +++ b/automation/gitlab-ci/analyze.yaml
> @@ -52,6 +52,8 @@ eclair-x86_64-testing:
>    tags:
>      - eclair-analysis-testing
>    rules:
> +    - if: $CI_PROJECT_PATH !~ /^xen-project\/people/.*$/
> +      when: never
>      - if: $ECLAIR_TESTING
>        when: always
>      - !reference [.eclair-analysis:triggered, rules]
> @@ -110,6 +112,8 @@ eclair-ARM64-testing:
>    tags:
>      - eclair-analysis-testing
>    rules:
> +    - if: $CI_PROJECT_PATH !~ /^xen-project\/people/.*$/
> +      when: never
>      - if: $ECLAIR_TESTING
>        when: always
>      - !reference [.eclair-analysis:triggered, rules]

Thanks for the patch, but Gitlab thinks there's a syntax error.

https://gitlab.com/xen-project/hardware/xen-staging/-/pipelines/2209479095

The / after people needs escaping, like it's done elsewhere.  The result is:

https://gitlab.com/xen-project/hardware/xen-staging/-/pipelines/2209482948

which is looking far more healthy.

With that fixed, Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
and I'll be taking this fairly promptly.
Re: [PATCH] automation/eclair: Run *-testing jobs only in some branches
Posted by Nicola Vetrini 2 days, 12 hours ago 
On 2025-12-11 15:27, Andrew Cooper wrote:
> On 11/12/2025 1:34 pm, Nicola Vetrini wrote:
>> From: Nicola Vetrini <nicola.vetrini@bugseng.com>
>> 
>> These jobs should not run in regular scans, but only in 
>> xen-project/people
>> repositories and branches to speed up pipeline execution.
>> 
>> Signed-off-by: Nicola Vetrini <nicola.vetrini@bugseng.com>
>> ---
>> This should alleviate some of the recent delays of the ECLAIR 
>> pipelines
>> ---
>>  automation/gitlab-ci/analyze.yaml | 4 ++++
>>  1 file changed, 4 insertions(+)
>> 
>> diff --git a/automation/gitlab-ci/analyze.yaml 
>> b/automation/gitlab-ci/analyze.yaml
>> index 5ddfe37903..88bc42f5c3 100644
>> --- a/automation/gitlab-ci/analyze.yaml
>> +++ b/automation/gitlab-ci/analyze.yaml
>> @@ -52,6 +52,8 @@ eclair-x86_64-testing:
>>    tags:
>>      - eclair-analysis-testing
>>    rules:
>> +    - if: $CI_PROJECT_PATH !~ /^xen-project\/people/.*$/
>> +      when: never
>>      - if: $ECLAIR_TESTING
>>        when: always
>>      - !reference [.eclair-analysis:triggered, rules]
>> @@ -110,6 +112,8 @@ eclair-ARM64-testing:
>>    tags:
>>      - eclair-analysis-testing
>>    rules:
>> +    - if: $CI_PROJECT_PATH !~ /^xen-project\/people/.*$/
>> +      when: never
>>      - if: $ECLAIR_TESTING
>>        when: always
>>      - !reference [.eclair-analysis:triggered, rules]
> 
> Thanks for the patch, but Gitlab thinks there's a syntax error.
> 
> https://gitlab.com/xen-project/hardware/xen-staging/-/pipelines/2209479095
> 
> The / after people needs escaping, like it's done elsewhere.  The 
> result is:
> 
> https://gitlab.com/xen-project/hardware/xen-staging/-/pipelines/2209482948
> 
> which is looking far more healthy.
> 
> With that fixed, Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
> and I'll be taking this fairly promptly.

Ah, right. Should have done a test run. Thanks for the fix

-- 
Nicola Vetrini, B.Sc.
Software Engineer
BUGSENG (https://bugseng.com)
LinkedIn: https://www.linkedin.com/in/nicola-vetrini-a42471253

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.