Skip to content

Remove loose foreign keys to ci_runners_e59bb2812d partitions

What does this MR do and why?

This MR removes a couple of loose foreign keys to ci_runners_e59bb2812d partitions. The follow-up !176848 (merged) will recreate the missing runner records in the partitioned table.

These LFKs were added as part of the Cells 1.0 work, with the idea of ensuring we didn't leave runners behind unnecessarily. Unfortunately, we see that some project runners stay active and processing jobs even after the creator project has been deleted, implicitly being adopted by the next most recently associated project. So we need to remove this LFK. We could leave the group_type_ci_runners_e59bb2812d LFK given that we know that a group runner is at most associated with a single group, but this would create an inconsistency with project runners, so for now we'll take the simplest approach and remove both in this MR.

Changelog: changed

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.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

n/a

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

n/a

Edited by Pedro Pombeiro

Merge request reports

Loading