Rename runner model's active flag to paused
The boolean active flag from the CiRunner model is not as clearly named as it could be. We should invert its meaning to paused so that its intent is clearer. This involves the following changes:
-
Changing the database model (we may want to skip this due to the migration that it involves) - GraphQL:
-
Deprecating the activefield in theCiRunnerGraphQL model (!79017 (merged)); -
Deprecating the activefield in theRunnerUpdateInputGraphQL structure (!79131 (merged)); -
Deprecating the activefilter in theQuery.runnersGraphQL query (!79018 (merged)); -
Adding a pausedfield to theCiRunnerGraphQL model (!79017 (merged)); -
Adding a pausedfield in theRunnerUpdateInputGraphQL structure (!79131 (merged)); -
Adding a pausedfilter to theQuery.runnersGraphQL query (!79018 (merged));
-
- REST:
-
Add a pausedfield to theAPI::Ci::Runner:runnersGET resource and:runnerPOST resource; -
Deprecate the activefield in endpoits returning Runner entities, such as:-
GET /runners(!79244 (merged)/!79349 (merged)) -
POST /runners(!79244 (merged)) -
GET /runners/all(!79244 (merged)/!79349 (merged)) -
GET /runners/:id(!79244 (merged)) /PUT /runners/:id(!79244 (merged)) -
PUT --form "active=false" /runners/:runner_id(!79244 (merged)) -
GET /projects/:id/runners(!79244 (merged)/!79349 (merged)) /POST /projects/:id/runners(!79244 (merged)) -
GET /groups/:id/runners(!79244 (merged)/!79349 (merged))
-
-
gitlab-runner!3414 (merged) - Adapt Runner to use pausedargument instead ofactivefor registration with thePUT /runners/:idendpoint (gitlab-runner#28844 (closed) - only in %15.0 to avoid backward compatibility issues).
-
Deprecation issue: GraphQL API: Runner "active" nomenclature chang... (#351109 - closed)
Edited by Pedro Pombeiro