Skip to content

Add config column to ci runner machines

Pedro Pombeiro requested to merge pedropombeiro/387300/add-config-column-to-ci_runner_machines into master

What does this MR do and why?

Describe in detail what your merge request does and why.

This MR adds a config jsonb column to the empty ci_runner_machines table. This field was missing from the architecture blueprint, so that is updated as well.

We're in the process of splitting ci_runners records (which contain both admin configuration and runner state), into ci_runners (admin configuration) and ci_runner_machines (runner state).

The reason this field is added is because, like other "snapshot" fields from ci_runners, this info actually belongs to the new ci_runner_machines record. In the future we'll drop the respective column from ci_runners.

The config field stores information about the configuration of a runner. Currently, that is used to tell the GPU configuration that is used by a runner.

Closes #387300 (closed)

Screenshots or screen recordings

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

How to set up and validate locally

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

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 Pedro Pombeiro

Merge request reports