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

Upgrade Gitlab Runner #24254

Merged
merged 3 commits into from Mar 28, 2017
Merged

Upgrade Gitlab Runner #24254

merged 3 commits into from Mar 28, 2017

Conversation

bachp
Copy link
Member

@bachp bachp commented Mar 23, 2017

Motivation for this change

Gitlab introduced an API break with version 9. This means that the runner versions >= 9 only work with Gitlab version 9 or higher.
In order to support Gitlab installations <9 the old runner (v1.x.x) in its most recent version is made available as gitlab-runner1.

The service is extended so the user can select the old version.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@bachp
Copy link
Member Author

bachp commented Mar 28, 2017

ping

@@ -1963,6 +1963,7 @@ with pkgs;
gitlab = callPackage ../applications/version-management/gitlab { };

gitlab-runner = callPackage ../development/tools/continuous-integration/gitlab-runner { };
gitlab-runner1 = callPackage ../development/tools/continuous-integration/gitlab-runner/v1.nix { };
Copy link
Member

Choose a reason for hiding this comment

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

Can you rename this to gitlab-runner_1_11?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure will do

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated

gitlab-runner 9.0.0 is only compatible with gitlab >= 9.0
gitlab-runner1 1.11.1 is only compatible with gitlab < 9.4
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.

None yet

2 participants