Skip to content
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

ResourceManager.search doesn't take the endpoint path into account #140

Open
thomasgoirand opened this issue May 8, 2024 · 0 comments
Open

Comments

@thomasgoirand
Copy link

In here:
https://github.com/gnocchixyz/python-gnocchiclient/blob/master/gnocchiclient/v1/resource.py#L174

gnocchiclient does:
url = "v1/search/resource/%s?%%s" % resource_type

Unfortunately, when the endpoint is set to something like this:
http://api.example.com/metric

gnocchiclient then fails to add /metric to the above url = variable, and therefore fails.

The solution would be to check the endpoint, and find the path, to add it before the content of url=

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants