Follow-up from "Insert MRs by head_commit_sha separately"
The following discussion from !147297 (merged) should be addressed:
-
@felipe_artur started a discussion: Is it possible to have some kind of race condition here because of this check?
For example:
- FF merge is made
- User disables ff-only merges
- This service is invoked by Sidekiq and the check returns false
- The deployment ends up without a link to the FF merge made on step 1
If what is described above is possible, would it be safe to remove this check?