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

Allow custom registry for initialization of collector metrics #885

Merged
merged 4 commits into from
Jun 23, 2023

Conversation

marctc
Copy link
Contributor

@marctc marctc commented May 31, 2023

This PR changes the initialization of SNMP internal metrics allowing to specify a custom registry. This is aims to solve the issue described here but in a generic way. I took a similar approach to this PR to solve the issue.

@marctc marctc force-pushed the explicit_metrics_registration branch from 74e06aa to d21c801 Compare May 31, 2023 10:40
@marctc
Copy link
Contributor Author

marctc commented May 31, 2023

cc @SuperQ

@marctc marctc force-pushed the explicit_metrics_registration branch from cbd7b0c to e9f2286 Compare May 31, 2023 10:45
@marctc marctc changed the title Explicit registration of internal exporter metrics Allow custom registry for initialization of internal exporter metrics May 31, 2023
marctc added 3 commits May 31, 2023 14:16
Signed-off-by: Marc Tuduri <[email protected]>
Signed-off-by: Marc Tuduri <[email protected]>
@marctc marctc force-pushed the explicit_metrics_registration branch from e9f2286 to 595a6a3 Compare May 31, 2023 12:16
@SuperQ SuperQ changed the title Allow custom registry for initialization of internal exporter metrics Allow custom registry for initialization of collector metrics Jun 23, 2023
@SuperQ SuperQ merged commit 4a3227b into prometheus:main Jun 23, 2023
SuperQ pushed a commit that referenced this pull request Jun 23, 2023
* Explicit registration of collector metrics

---------

Signed-off-by: Marc Tuduri <[email protected]>
stephan-windischmann-sky pushed a commit to stephan-windischmann-sky/snmp_exporter that referenced this pull request Oct 27, 2023
…heus#885)

* Explicit registration of collector metrics

---------

Signed-off-by: Marc Tuduri <[email protected]>
Signed-off-by: Stephan Windischmann <[email protected]>
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.

2 participants