Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Monitoring Improvements: Prometheus client to export receiver up stat… #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dharani-oqc
Copy link

This PR addresses key points discussed previously:

Reuse Classes Available for Testing:
Reused the classes available in metrics.py to improve consistency and reduce redundancy in our test suite.

Related Issues
This PR is based on discussions in PR #4 "Monitoring: Prometheus client to export receiver up statuses and other metrics".

…ues and other metrics

Signed-off-by: dharani-oqc <dadusumalli@oqc.tech>
Copy link

github-actions bot commented Aug 7, 2024

CLA Assistant Lite bot All Contributors have signed the CLA.

@dharani-oqc
Copy link
Author

I have read the Contributor License Agreement and I hereby accept the Terms.

github-actions bot added a commit that referenced this pull request Aug 7, 2024
Signed-off-by: dharani-oqc <dadusumalli@oqc.tech>
Copy link
Contributor

@keriksson-rosenqvist keriksson-rosenqvist left a comment

Choose a reason for hiding this comment

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

LGTM

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants