Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Integrate gitlab-workhorse on template initialisation #16

Merged

Conversation

XaviTorello
Copy link
Contributor

Solve turnkeylinux/tracker#686 (can't open issue on this repo)

Fetch and make gitlab-workhorse to complete all the Gitlab dependencies.

gitlab-workhorse is expected by the Gitlab startup process (see /etc/init.d/gitlab), additionally HTTP requests are optimized and avoid some errors related to HTTP operations through git client (i.e https://gitlab.com/gitlab-org/gitlab-ce/issues/2669#note_2176671)

This PR
0) Add GO as a dependency (make gitlab-workhorse)

  1. Integrate gitlab-workhorse on the Gitlab installation process
  2. Change consume socket directly on proxy_pass@nginx to upstream
  3. Create a new location matching *.git requests that points to the workhorse upstream

Among other things, issues like "empty repo" at "git clone http://XXXX" will be solved.

Dependency for making workhorse
To provide a cleaner config using upstreams

fail_timeout=0
instead of hardcoded socket without any param
Pointing to the gitlab-workhorse socket
Set some timeouts and proxify gitlab-workhorse upstream

Not sure if needed, but I think that's safer. As alternative, instead of create a new location we can just override the proxy_pass definition of location @GitLab (line 18).
@JedMeister
Copy link
Member

Thanks for the submission. @qrntz I know you are busy, but when you get a chance can you please have a look over this.

@a3s7p
Copy link
Member

a3s7p commented Aug 15, 2016

OK.

@a3s7p
Copy link
Member

a3s7p commented Aug 20, 2016

Can be closed in favor of #17

@JedMeister JedMeister merged commit 2455f83 into turnkeylinux-apps:master Aug 22, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants