Skip to content

Fix widget of locked merge requests not being presented

Oswaldo Ferreira requested to merge make-merge-jid-handling-less-stateful into master

What does this MR do?

  1. Fixes the bug found in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15018#note_44888376
  2. Make MR merge_jid / unlocking less complex (by making it less stateful)

Why was this MR needed?

The MergeRequest locking issue still manifests in tricky ways when we try to:

  1. Cleanup the MergeRequest#merge_jid
  2. Unlock the merge request

The crux is that the merge_jid may be cleaned and the merge request remain locked. Therefore our unlocking worker can't find and update the state of the merge request, and as end result we may have the behavior in the screenshot below:

Screenshots (if relevant)

WhatsApp_Image_2017-10-23_at_13.31.06

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Edited by Oswaldo Ferreira

Merge request reports