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

Add runner registration with runner auth token #337

Merged

Conversation

Sungurik
Copy link

@Sungurik Sungurik commented Jul 5, 2024

To support "--token" option: runner register --token glrt-XXXXXXXX

New workflow https://docs.gitlab.com/ee/ci/runners/new_creation_workflow.html

To support "--token" option: runner register --token glrt-XXXXXXXX

New workflow https://docs.gitlab.com/ee/ci/runners/new_creation_workflow.html
@riemers riemers mentioned this pull request Jul 5, 2024
@Miracle05
Copy link

This is exactly what i need! Thank you!

README.md Outdated
- `gitlab_runner_registration_token` - The GitLab registration token. If this is specified, a runner will be registered to a GitLab server.
- `gitlab_runner_registration_token` - The GitLab registration token. If this is specified, each runner will be registered to a GitLab server. Deprecating in gitlab version 16.0 and removed in 18.0.
- `gitlab_runner_registration_token_type` - Gitlab runner registration token type
Set "runner-token" to register runner with --token option (new workflow https://docs.gitlab.com/ee/ci/runners/new_creation_workflow.html)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The wording in Gitlab is always

  • Authentication Token (for the new workflow)
  • Registration Token (for the old workflow)

I don't know if "runner-tokne" is also a common naming. I'd stick to the two mostely used in the gitlab documentation

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are ringht, thanks! I renamed runner-token to authentication-token

@guenhter
Copy link
Collaborator

guenhter commented Jul 6, 2024

#335

Also made a first attempt to add this new process but then the PR was closed. Nevertheless, there are some very useful parts in there. I guess e.g. the --tags option is not working any more in the new registration process...

@guenhter guenhter merged commit 28404cc into riemers:master Jul 8, 2024
@guenhter
Copy link
Collaborator

guenhter commented Jul 8, 2024

Thx for your contribution

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.

3 participants