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: 11.11.2 -> 12.0.2 #63771
Conversation
87e8003
to
31a63cd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$ nix-review pr 63771
passed on NixOS
Diff LGTM
[5 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/63771
1 package were build:
gitlab-runner
31a63cd
to
b687137
Compare
12.0.1 was released with a fix for a submodule fetch issue |
@infinisil ping |
@fpletz can you merge this? I'm using it since almost a week without issue. |
@fpletz @infinisil What's the status here? |
12.0.1 fixes an issue with git submodule fetching 12.0.2 fixes an issue with concurrent updated
b687137
to
e928aa6
Compare
12.0.2 was released and includes a fix for a concurrent update issue: https://gitlab.com/gitlab-org/gitlab-runner/merge_requests/1447 |
@flokli Can you help here? |
Vacation-induced slackness, sorry for the delay ;-) LGTM, merging. |
Backported to 19.03 in d10e680. |
Thanks 👍 |
Motivation for this change
Update to latest version: https://gitlab.com/gitlab-org/gitlab-runner/blob/v12.0.0/CHANGELOG.md
The major version bump is not a significant change from the runners perspective.
The removed config options were not used in the nix module so there is no change to the service required.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)