Skip to content

Ensure lease cancellation in StuckCiJobsWorker

drew stachon requested to merge ensure-stuck-builds-lease-cancel into master

What does this MR do and why?

This MR ensures that we cancel the exclusively lease created by the StuckCiJobsWorker in the event of a query timeout, which we know happens regularly.

It was recommended for a new worker that does the same thing in this merge request so it made sense to go back and apply it to the original worker here.

The implementation and testing structure is the same that we decided to use in the new Ci::StuckBuilds::DropRunningWorker and it's own spec.

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