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

docs: Clarify registration_token requirement #348

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

knowald
Copy link

@knowald knowald commented Sep 4, 2024

Hi there,

this Merge Request aims to clarify the documentation regarding the use of the gitlab_runner_registration_token. I initially understood the current wording, it is recommended to specify a token for each runner as a suggestion rather than a requirement. The clarification here highlights that specifying a token for each runner is required, not just recommended, when gitlab_runner_registration_token_type is set to authentication-token.

An alternative proposal would be allowing the global gitlab_runner_registration_token to be used with the --token command by default in the same way it works with the --registration-token option. That would simplify the process for registering multiple runners, as in the current GitLab Release it is intended to reuse a single token to register multiple hosts grouped as a single runner.

@guenhter guenhter merged commit 4a4b7a5 into riemers:master Sep 9, 2024
@guenhter
Copy link
Collaborator

guenhter commented Sep 9, 2024

Thx for this contribution

@knowald knowald deleted the docs/clarify-token-variables branch September 11, 2024 11:21
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.

2 participants