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: add authenticationTokenConfigFile #1321

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

Conversation

bgamari
Copy link

@bgamari bgamari commented Feb 4, 2025

GitLab is deprecating the original token-based runner authentication method as noted in https://docs.gitlab.com/ee/security/tokens/index.html#runner-authentication-tokens.

Based on #1185.

As described in nixpkgs#356717, `gitlab-runner` now uses TOML date-time
values in its configuration (namely the `token_obtained_at` and
`token_expires_at` keys). As there is no JSON syntax for such values,
remarshal refuses to serialise them without the `--stringify` flag.
Thankfully, `gitlab-runner` seems to accept the stringified values
without any complaints, so we can side-step this issue readily.
@bgamari bgamari marked this pull request as ready for review February 5, 2025 16:21
@bgamari
Copy link
Author

bgamari commented Feb 5, 2025

I am using this in the GHC AArch64/Darwin CI pool. It appears to work well. Backport to 24.11 would be appreciated.

@Enzime
Copy link
Collaborator

Enzime commented Feb 14, 2025

Can you rebase this PR?

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