Resolve "Resolve cross DB issues in ee/app/services/vulnerabilities/security_finding/create_issue_service.rb"

What does this MR do and why?

Update SecurityFinding::CreateIssueService.execute to resolve cross-DB modification

The method creates Vulnerability, Issue, and IssueLinks which are now in separate DB
and can therefore not be queried in the same transaction
To solve this the transaction has been removed and instead, we handle the rollback exception explicitly by destroying the changes made

References

Please include cross links to any resources that are relevant to this MR. This will give reviewers and future readers helpful context to give an efficient review of the changes introduced.

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 #480894 (closed)

Edited by Schmil Monderer

Merge request reports

Loading