puppet-gitlab icon indicating copy to clipboard operation
puppet-gitlab copied to clipboard

Gitab.rb sidekiq queue_groups array quoted.

Open ccombs-trustamerica opened this issue 2 years ago • 0 comments

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: PE 2021.7.2
  • Ruby:
  • Distribution: RHEL 7.9
  • Module version: 8.1.0

How to reproduce (e.g Puppet code you use)

sidekiq => {
      queue_groups    => '[\\'*\\'] * 4',
    },

OR

sidekiq => {
      queue_groups    => "['*'] * 4",
    },

What are you seeing

Both produce the same result: gitlab.rb: sidekiq['queue_groups'] = "['*'] * 4"

What behaviour did you expect instead

gitlab.rb: sidekiq['queue_groups'] = ['*'] * 4

Output log

Notice: /Stage[main]/Gitlab::Service/Exec[gitlab_reconfigure]/returns:
Notice: /Stage[main]/Gitlab::Service/Exec[gitlab_reconfigure]/returns: ================================================================================ Notice: /Stage[main]/Gitlab::Service/Exec[gitlab_reconfigure]/returns: Error executing action create on resource 'template[/opt/gitlab/sv/sidekiq/run]' Notice: /Stage[main]/Gitlab::Service/Exec[gitlab_reconfigure]/returns: ================================================================================ Notice: /Stage[main]/Gitlab::Service/Exec[gitlab_reconfigure]/returns:
Notice: /Stage[main]/Gitlab::Service/Exec[gitlab_reconfigure]/returns: Chef::Mixin::Template::TemplateError Notice: /Stage[main]/Gitlab::Service/Exec[gitlab_reconfigure]/returns: ------------------------------------ Notice: /Stage[main]/Gitlab::Service/Exec[gitlab_reconfigure]/returns: undefined method `each' for "['*'] * 4":String

Any additional information you'd like to impart

If I try:

sidekiq                 => {
  queue_groups    => ['[\\'*\\'] * 4'],
},

Gitlab.rb: sidekiq['queue_groups'] = ["['*'] * 4"]

The reconfigure doesn't error, but I also don't get 4 sideqik processes.

https://docs.gitlab.com/ee/administration/sidekiq/extra_sidekiq_processes.html

ccombs-trustamerica avatar Mar 23 '23 22:03 ccombs-trustamerica