arturictus / sidekiq_alive

Liveness probe for Sidekiq in Kubernetes deployments
MIT License
188 stars 57 forks source link

Fails to attach instance-specific queue (by hostname) with Sidekiq7 #87

Closed fwolfst closed 1 year ago

fwolfst commented 1 year ago

After updating sidekiq_alive (2.2.0) and sidekiq (7.1.0), inidvidual hosts don't get an own queue anymore (only the sidekiq_alive queue, but not prefix_hostname. The Workers are scheduled and set for the correct queues though.

image

Before it looked like

image

When debugging, in https://github.com/arturictus/sidekiq_alive/blob/66f2f3f435795da36745eb8ac95651e036b280e2/lib/sidekiq_alive.rb#L21 the queue array is correct, so it seems to be reset somewhere...?

fwolfst commented 1 year ago

Seems that the queue does not get shown because it has no weight.

fwolfst commented 1 year ago

See PR to set a default weight (of 1), which fixes it in my case.