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

Support sending native histograms with remote_write #429

Merged
merged 3 commits into from
Mar 22, 2024

Conversation

aptomaKetil
Copy link
Contributor

Expose this setting under externalServices.prometheus, for backends that support it.

@petewall
Copy link
Collaborator

Thanks for creating this!
Make sure to run make regenerate-example-outputs and add the changes from that. That should get the failing tests to pass.

@petewall
Copy link
Collaborator

Hmm, I can see that your deployment is adding the scrape annotation to Node exporter as well.
Can you tell me what platform you're running? Windows/Mac/Linux? What version of Helm?

@aptomaKetil
Copy link
Contributor Author

MacOS, was using Helm v3.9. Updated to v3.14 now and regenerated, and that got rid of the extra annotations.

@petewall petewall merged commit a886c82 into grafana:main Mar 22, 2024
5 checks passed
@petewall
Copy link
Collaborator

That really helps! Thanks. You're the second person who had issues with this.

@petewall
Copy link
Collaborator

Your PR has been merged. Thanks for the contribution!!

@aptomaKetil aptomaKetil deleted the feat/rw-native-histograms branch March 23, 2024 07:36
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

2 participants