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

Return Prometheus Warnings #699

Merged
merged 4 commits into from Dec 13, 2019

Conversation

joe-elliott
Copy link
Contributor

This PR replaces #648.

  • Moved warnings from the api package to api/prometheus/v1
  • Split the api.Client interface. Previously it was being used in both api and v1. Now a separate interface (v1.apiClient) better defines this internal http client.
  • Moved DoGetFallback into method into v1
  • Moved tests as appropriate.
  • Correctly returned warnings from Do and DoGetFallback.

Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
@joe-elliott joe-elliott changed the title Return Prometheus Warnings Correctly Return Prometheus Warnings Dec 11, 2019
@beorn7
Copy link
Member

beorn7 commented Dec 11, 2019

Many thanks. I'll have a look ASAP.

@krasi-georgiev in case you find time: You might have some context here. If you see anything we haven't seen, please let us know.

@krasi-georgiev
Copy link
Contributor

sorry I wouldn't be able to have a look anytime soon.

Copy link
Member

@beorn7 beorn7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To my best (limited) knowledge, this looks good.
I'll merge, and we'll see if it works as expected.

I checked the godoc examples. They should all work unchanged.

@beorn7 beorn7 merged commit e7776d2 into prometheus:master Dec 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants