Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[gitlab_runner] Fix compatible with GitLab 17.0 or higher #2497

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

yethee
Copy link
Contributor

@yethee yethee commented May 15, 2024

Since GitLab 17.0 API server returns 410 Gone status for the /api/v4/runners endpoint. Because the registration token has been deprecated and will be removed on GitLab 18.0.

The PR changes the runner registration using new API - create a runner linked to a user.
According to GitLab documentation, this should be compatible with GitLab 15.10 or higher.

The new API requires a personal access token, so the gitlab_runner__api_token parameter becomes mandatory.

Second commit

Removed the task to check GitLab API availability. This guard is useless in cases when the endpoint is overridden in the instance parameters (api_url).

@yethee yethee requested a review from drybjed as a code owner May 15, 2024 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant