Skip to content

Fix sidekiq-cluster name selector with colons

Sean McGivern requested to merge fix-sidekiq-cluster-name-matching-colons into master

What does this MR do?

A queue name can have colons (to separate it from its namespace). This wasn't part of the term regex so trying to select a queue by name, when the name contained a colon, would fail.

This also allows every other predicate to accept colons. Although none of them strictly need to, it doesn't harm them: feature categories and resource boundaries can't contain colons anyway, and colons weren't part of the general syntax.

Merge request reports

Loading