Skip to content
Snippets Groups Projects

Sync requirement and requirement issues state

What does this MR do and why?

This is part of state IIB of migrating requirements to work items(issues of type requirement).

Syncs state between issues(of type requirement) and requirements from both ends. Also makes sure no desync happens and if it happens logs an error.

  • On UpdateRequirementService it uses a pseudo transaction to avoid desync already implemented on !64929 (merged)
  • Issues::UpdateService and Issues::CloseService it uses a regular ActiveRecord transaction to avoid desync

related to #337190 (closed)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Michael Kozono

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Felipe Cardozo
  • Luke Duncalfe
  • Felipe Cardozo added 1165 commits

    added 1165 commits

    Compare with previous version

  • Felipe Cardozo added 1 commit

    added 1 commit

    • 879b37a0 - Sync requirement and requirement issues state

    Compare with previous version

  • Felipe Cardozo added 1 commit

    added 1 commit

    • 0dd33483 - Sync requirement and requirement issues state

    Compare with previous version

  • Felipe Cardozo requested review from @acroitor

    requested review from @acroitor

  • Alexandru Croitor
  • Alexandru Croitor
  • Alexandru Croitor
  • Alexandru Croitor
  • Alexandru Croitor removed review request for @acroitor

    removed review request for @acroitor

  • Felipe Cardozo added 1 commit

    added 1 commit

    • e3583994 - Sync requirement and requirement issues state

    Compare with previous version

  • Felipe Cardozo requested review from @acroitor and removed review request for @cablett

    requested review from @acroitor and removed review request for @cablett

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading