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

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 11/12/2025 1:34 pm, Nicola Vetrini wrote:
> From: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
>
> 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@xxxxxxxxxxx>
> ---
> 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@xxxxxxxxxx>
and I'll be taking this fairly promptly.