-
-
Notifications
You must be signed in to change notification settings - Fork 242
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
Support prometheus >= 2.0 #48
Conversation
Hi @sathieu, thanks for this PR. Are you able to provide tests for this? |
@bastelfreak No. Sorry. |
Todo for me later: We can maybe add a simple acceptance test to prove this is working |
Following voxpupuli#83, and preparing tests for voxpupuli#48.
PR updated with tests. |
Thanks for the unit tests @sathieu. We should probably add basic acceptance tests for this before we merge it. I will have a look later. |
I will test them in #86 |
@sathieu can you rebase please? Acceptance tests are now available in our master branch. |
9a0fabc
to
4d84542
Compare
The acceptance tests are sometimes failing, but I think it's not due to this PR... |
Damn, I've no idea where the issue is. I thought I got it fixed while implementing the tests. |
can you restart those failing checks and see if it works? |
Restarted the failed one |
Thanks for this @sathieu! |
Following voxpupuli#83, and preparing tests for voxpupuli#48.
Following voxpupuli#83, and preparing tests for voxpupuli#48.
Support prometheus >= 2.0
Following voxpupuli#83, and preparing tests for voxpupuli#48.
Support prometheus >= 2.0
No description provided.