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

Switch to the authentication token-based runner creation workflow #24

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

Conversation

pinchartl
Copy link

Starting in GitLab v15.10 (and enabled by default in v16.0), the runner creation uses a workflow based on authentication tokens instead of registration tokens. This introduces a new runner property named system_id that needs to be passed to the /jobs/request API.

This change requires a new version of gitlab-runner-rs, see collabora/gitlab-runner-rs#23.

I haven't updated the kubernetes integration as I have no experience with that at all.

@pinchartl pinchartl force-pushed the runner-authentication-token branch from 23fcc0e to af96ed2 Compare December 11, 2023 16:02
Starting in GitLab v15.10 (and enabled by default in v16.0), the runner
creation uses a workflow based on authentication tokens instead of
registration tokens. This introduces a new runner property named
system_id that needs to be passed to the /jobs/request API.

Extend the runner with a new system_id parameter (passed through the
environment variable 'RUNNER_SYSTEM_ID'. Additionall update the
documentation in README.md to explain the new workflow.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
This patch should not be merged, it should instead be replaced by a
version bump once a new gitlab-runner-rs version will be tagged.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
@obbardc obbardc force-pushed the runner-authentication-token branch from fb1d639 to 1656cb3 Compare January 10, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant