Skip to content

Add worker to sync approval rules from security_findings

Sashi Kumar Kumaresan requested to merge sk/387277-add-worker into master

What does this MR do and why?

This is part 2 MR for Create new worker for syncing approval rules fr... (#387277 - closed) where a new worker is introduced. The first MR (Add service for syncing approval rules from sec... (!115798 - merged)) that adds the service class was merged, this MR calls that service.

Sequence

Before

sequenceDiagram
    participant Pipeline
    participant StoreScansWorker
    participant SyncReportsToReportApprovalRulesWorker
    participant ApprovalRules
    Pipeline->>StoreScansWorker: transitioned to complete status
    Pipeline->>SyncReportsToReportApprovalRulesWorker: transitioned to complete status
    SyncReportsToReportApprovalRulesWorker->>ApprovalRules: parse findings and update approvals_required

After

sequenceDiagram
    participant Pipeline
    participant StoreScansWorker
    participant SyncReportsToReportApprovalRulesWorker
    participant SyncFindingsToApprovalRulesWorker
    participant ApprovalRules
    Pipeline->>StoreScansWorker: transitioned to complete status
    Pipeline->>SyncReportsToReportApprovalRulesWorker: transitioned to complete status
    StoreScansWorker->>SyncFindingsToApprovalRulesWorker: after inserting security_findings
    SyncFindingsToApprovalRulesWorker->>ApprovalRules: update approvals_required

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 Sashi Kumar Kumaresan

Merge request reports

Loading