FS#55751 - [gitlab] Include gitlab-gitaly in gitlab.target

Attached to Project: Community Packages
Opened by Nikos Platis (nplatis) - Tuesday, 26 September 2017, 06:19 GMT
Last edited by Sven-Hendrik Haase (Svenstaro) - Wednesday, 04 October 2017, 01:44 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Sven-Hendrik Haase (Svenstaro)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

The gitlab package includes a gitlab.target file. This file currently has the line

Wants=gitlab-unicorn.service gitlab-sidekiq.service gitlab-backup.timer

I believe that gitlab also needs gitlab-gitaly to function properly (this handles git actions from the web pages, see https://github.com/gitlabhq/gitlab-recipes/tree/master/init/systemd) so it should be included in this line.

And one more thing: should this line really be *Wants* or *Requires* ? Can gitlab function without gitlab-unicorn, for example?

[gitlab version 10.0.1-1]
This task depends upon

Closed by  Sven-Hendrik Haase (Svenstaro)
Wednesday, 04 October 2017, 01:44 GMT
Reason for closing:  Fixed
Comment by Sven-Hendrik Haase (Svenstaro) - Wednesday, 04 October 2017, 01:17 GMT
Good point. I'm going to fix this up to be representative of the actual programs required.

Loading...