Re: [PATCH v1] drm/ci: fix merge request rules
From: Daniel Stone
Date: Wed Feb 26 2025 - 09:00:08 EST
Hi Vignesh,
On Wed, 26 Feb 2025 at 13:55, Vignesh Raman <vignesh.raman@xxxxxxxxxxxxx> wrote:
> Merge request pipelines were only created when changes
> were made to drivers/gpu/drm/ci/, causing MRs that didn't
> touch this path to break. Fix MR pipeline rules to trigger
> jobs for all changes.
Thanks a lot for fixing this up!
> # Build everything after someone bypassed the CI
> - if: *is-direct-push
> when: on_success
Should these be when: manual? I think it would be good to align with
the rules we have for Mesa: run automatically for marge-bot and also
for scheduled pipelines, but in all other cases (MR context but not
scheduled for merge, direct push to ref, etc) be manual, so we don't
overwhelm the CI by testing everything everyone pushed.
Cheers,
Daniel