Skip to content

Draft: Skip branch pipelines in favor of MR pipelines

drew stachon requested to merge skip-branch-pipes-for-mr-pipes into master

What does this MR do?

This merge request is a simplified approach to #300146, and will supersede !63684 if we move forward with this change.

This improvement is in light of some guidance from @patrickbajao and the existing logic we have in ProcessRefChangesService dealing with changing refs and the merge requests associated with those refs.

Open Questions:

  1. Will this be a problem for merge requests from forks? What if there's an open merge request but permissions are not sufficient for the merge request pipeline to run?
  2. How can we access project to use with the feature flag in Config::Entry::Processable? Is this going to be possible?

Screenshots or Screencasts (strongly suggested)

How to setup and validate locally (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by drew stachon

Merge request reports