Skip to content

Set merge_request_id for FactoryBot pipelines that are set up as MergeRequest head pipelines

drew stachon requested to merge update-pipeline-merge-request-ids-in-factory into master

What does this MR do and why?

This MR sets the merge_request_id for Ci::Pipeline records that get created as the head pipeline of an existing merge request.

It is my understanding of MergeRequest pipelines that this is actually the case in production, and the ID not being set is an oversight because we first added this transient attribute in 2017: 921c2be4

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

Just specs 🙂

Edited by drew stachon

Merge request reports