-
Notifications
You must be signed in to change notification settings - Fork 57
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
First take #1
First take #1
Conversation
"endpoints": [{"region": "ORD", | ||
"tenantId": "851153", | ||
"publicURL": "http://localhost:8080/v2/851153/loadbalancers"}], | ||
"type": "rax:load-balancer"}]}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you need to return autoscale endpoint too? AFAIK, otter and bobby don't need it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm, I was thinking not cause the the same reason.
Can the |
@manishtomar agree. Also wondering if the auth and nova mocks should be on separate services, or maybe at least under a different subpath? I don't think nova and auth have any overlapping endpoints, but possibly load balancers may overlap with one or the other? This can be done in a different PR though. |
updated folder names. Will change auth, nova to be separate services in the next PR and might need your help :) |
+1 LGTM |
@lekhajee happy to! |
This reverts commit 37e410e.
No description provided.