Skip to content
Snippets Groups Projects

Don't fail StoreSecurityReportsWorker if the pipeline does not exist

All threads resolved!

What does this MR do and why?

It's possible that the pipeline can be deleted by the user before the Sidekiq worker picks up the job therefore we need to use find_by_id.

There is not a related issue as I noticed this while checking the "Threat Insights Engineering Kibana Dashboard" and just took a stab at it.

MR acceptance checklist

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

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
  • Eugie Limpin approved this merge request

    approved this merge request

  • Eugie Limpin requested review from @ebaque and removed review request for @eugielimpin

    requested review from @ebaque and removed review request for @eugielimpin

  • :wave: @eugielimpin, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • Etienne Baqué approved this merge request

    approved this merge request

  • Etienne Baqué resolved all threads

    resolved all threads

  • Etienne Baqué enabled an automatic merge when the pipeline for 8efc107b succeeds

    enabled an automatic merge when the pipeline for 8efc107b succeeds

  • Etienne Baqué mentioned in commit e6729703

    mentioned in commit e6729703

  • Mehmet Emin INAC mentioned in merge request !92277 (merged)

    mentioned in merge request !92277 (merged)

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading