Skip to content

Remove Gitlab::ExclusiveLease.skipping_transaction_check in issuable base service

Heinrich Lee Yu requested to merge 441526-prevent-exclusive-lease-in-transaction into master

What does this MR do and why?

After some discussions in #441526 (comment 1845452766), we found out that we actually don't need this allowlist since no specs are failing when we remove them.

If ever there are scenarios, where we actually request an exclusive lease, it will be logged in Sentry and we can create a separate issue for it then.

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.

Related to #441526 (closed)

Edited by Heinrich Lee Yu

Merge request reports