-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Blank vhost for Rabbitmq Scaler apiHost is erroring out when making api calls #790
Comments
So you're saying that the fix for #768 is not working out? |
@tbickford @holyketzer seems like RabitMQ scaler needs more love 😄 |
@mpduarte which value you provide in
They all should work. One case I'm not sure is |
Yes case without trailing slash wasn't covered. @mpduarte you can just add a trailing slash to your |
I had a trailing slash, the issue is I had is I specified /api/ at the end, for some reason I thought it was required, once I removed it, everything started working great. appreciate the quick responses! |
Blank vhost for Rabbitmq Scaler apiHost is erroring out when making api calls to the queue.
Expected Behavior
With no vhost specified the api call to RabbitMQ should look like this:
http://guest:guest@rabbitmq:15672/api/queues/%2F/queuename
Actual Behavior
With no vhost specified the api call to RabbitMQ looks like this:
http://guest:guest@rabbitmq:15672/api/queues/api//queuename
Debug log from Controller:
{"level":"debug","ts":"2020-04-29T00:15:39.676Z","logger":"scalehandler","msg":"Error getting scale decision","ScaledObject.Namespace":"cust0-herald-test","ScaledObject.Name":"celery-scaledobject","ScaledObject.ScaleType":"deployment","Error":"error inspecting rabbitMQ: error requesting rabbitMQ API status: 404 Not Found, response: {"error":"Object Not Found","reason":"Not Found"}, from: http://guest:guest@rabbitmq.cust0-herald-test.svc.cluster.local:15672/api/queues/api//celery"}
Steps to Reproduce the Problem
Specifications
The text was updated successfully, but these errors were encountered: