Bug getting queues with regex using RabbitMQ Scaler

[JorTurFer]

Report

I have to apologize because the RabbitMQ Scaler is not working with regex :(

Idk why I totally forgot 2 important things which should be part of the query string, the page (with pagination=false is still needed the parameter page=1 ) and the regex in url format.
I'm really sorry about it and I will fix it ASAP (I start with this now) and I will also update the integration test to ensure that it works

And sorry again :(

Expected Behavior

Use regex and it works

Actual Behavior

Now all the queues are returned

KEDA Version

2.4.0

Scaler Details

RabbitMQ

[zroubalik]

OK, no worrries.

wrt testing, the best thing would be to extend current RabbitMQ e2e test and add regex scenario there: https://github.com/kedacore/keda/tree/main/tests

[JorTurFer]

The problem is that I did the test, but it is wrong... I will fix it also :)

[crisp2u]

Guys, any idea when we can expect a release with this fix ? I'm trying to use RabbitMQ scaler with regexe but it's not working. I've also tried running main image but the helm chart is broken.

[JorTurFer]

Hi,
next release will be in the first half of November, AFAIR. Probably you are getting errors with helm and chart because an argument has changed after the kubebuilder updates.
Before the release, you need to update this line after release the chart, or maybe you can use the chart from your local

[tomkerkhove]

Guys, any idea when we can expect a release with this fix ? I'm trying to use RabbitMQ scaler with regexe but it's not working. I've also tried running main image but the helm chart is broken.

Can you elaborate on the chart issue please?

[JorTurFer]

Guys, any idea when we can expect a release with this fix ? I'm trying to use RabbitMQ scaler with regexe but it's not working. I've also tried running main image but the helm chart is broken.

Can you elaborate on the chart issue please?

I think that it's not an issue by itself, @zroubalik updated kubebuilder version and that changed the election leader flag, the chart is updated, but it's not possible to deploy KEDA using the chart v2.4 with the image tag main because the flag in v2.4 is different.
Am I right @zroubalik ?

[crisp2u]

@JorTurFer yeah, that's the issue, i did not want to go into details of that. I've also tried to escape metricName and queueName and add page=1 outside but still no luck

[JorTurFer]

Maybe, could you try using local chart (or modifying the deployment manually) until the next release? Otherwise, you should wait until it 😞

[crisp2u]

Thanks for the suggestions. I'm running a local chart now and all seems good. I'm gonna keep this in development for a while so I can wait for the next release.

[JorTurFer]

Nice! :)