debops / ansible-gitlab

Install, upgrade and manage GitLab instance
GNU General Public License v3.0
51 stars 23 forks source link

Rework 5 #98

Closed ypid closed 6 years ago

ypid commented 7 years ago

Starting with 10.1 OpenPGP verification will probably become usable: https://gitlab.com/gitlab-org/gitlab-ce/issues/36829 Nice stuff :)

ypid commented 7 years ago

@drybjed You can check it out if you want. I did not further investigate the build issues.

drybjed commented 7 years ago

@ypid, it seems that gitlab-workhorse cannot be built in the current Travis-CI environment. I tried to build it manually in the test environment, but it fails there as well. I guess something is missing, not sure what yet. I'll look at it further a bit later.

ypid commented 7 years ago

Thanks! Note that the move of the include: configure_gitlab_ce.yml is not fully tested by me.

drybjed commented 7 years ago

@ypid Can you fix the merge conflicts with the current role and update the PR? It will be tested on Ubuntu Trusty now.

ypid commented 7 years ago

Sure, updated.

drybjed commented 6 years ago

@ypid Thanks for rebasing. I tested this PR on a clean Debian Stretch and both times gitlab-gitaly.service did not start correctly initially; it started fine after a reboot. I'm guessing that since gitlab.slice is started before gitlab-gitaly.service is created, there might be some issues.

I'll try and move the start steps to a new file executed after gitaly setup, perhaps this will help.

ypid commented 6 years ago

@drybjed Thanks for checking.