Review gitlab.rb Template about puppet-gitlab HOT 7 CLOSED

doing some grep foo and looking at the diff from HEAD > omnibus 8.0 the following 2 groupings are not supported currently by this module:

+# sentinel['bind'] = '0.0.0.0'
+# sentinel['port'] = 26379
+# sentinel['quorum'] = 1
+# sentinel['down_after_milliseconds'] = 10000
+# sentinel['failover_timeout'] = 60000
+# sidekiq_cluster['enable'] = false
+# sidekiq_cluster['ha'] = false
+# sidekiq_cluster['log_directory'] = "/var/log/gitlab/sidekiq-cluster"
+# sidekiq_cluster['interval'] = 5 # The number of seconds to wait between worker checks
+# sidekiq_cluster['queue_groups'] = [

from puppet-gitlab.

Thanks! It's not just adding parameters, but also removing deprecated parameters... Will hopefully work on this over the next weeks.

from puppet-gitlab.

best I can tell every other config directive is covered by the existing template.

from puppet-gitlab.

Would these parameters fit in this category ?

gitlab_rails['backup_path'] = <path>
git_data_dirs({"default" => "<path>"})

from puppet-gitlab.

Would these parameters fit in this category ?

Yes =) See also #110

from puppet-gitlab.

All the configuration options mentioned above should be supported now. I recently added the missing git_data_dirs (thanks to Corey Osman) and sidekiq_cluster options (19ff7e0).

from puppet-gitlab.

So it turns out that git_data_dirs is currently misdocumented. The correct hash for this should look like:

#correct
git_data_dirs({'default' => {'path' => '/git/data/path'}})

#incorrect
git_data_dirs({'default' => '/git/data/path'})

see: https://gitlab.com/gitlab-org/gitlab-ee/issues/4040#note_54368948

from puppet-gitlab.