-
Notifications
You must be signed in to change notification settings - Fork 282
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
feat(quorum): add prometheus exporter #602
feat(quorum): add prometheus exporter #602
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there is almost no problem. However, could you please reply to or revise your comments on the minor point?
68b4cd6
to
9e0d621
Compare
8be6aab
to
121db02
Compare
121db02
to
fc5f9e3
Compare
fc5f9e3
to
596c2b9
Compare
858df34
596c2b9
to
858df34
Compare
@jagpreetsinghsasan I recommend rebasing this to the PR that improves the GHA flakiness (and then also making sure to mark this as dependent on that) |
@jagpreetsinghsasan After rebasing to main the CI will hopefully pass and then this is ready to go |
Primary Change -------------- 1. The quorum ledger connector plugin now includes the prometheus metrics exporter integration 2. OpenAPI spec now has api endpoint for the getting the prometheus metrics Refactorings that were also necessary to accomodate 1) and 2) ------------------------------------------------------------ 3. GetPrometheusMetricsV1 class is created to handle the corresponding api endpoint 4. IPluginLedgerConnectorQuorumOptions interface in PluginLedgerConnectorQuorum class now has a prometheusExporter optional field 5. The PluginLedgerConnectorQuorum class has relevant functions and codes to incorporate prometheus exporter 6. Added Readme.md on the prometheus exporter usage Fixes hyperledger-cacti#534 Signed-off-by: Jagpreet Singh Sasan <jagpreet.singh.sasan@accenture.com>
Commit to be reviewed
feat(quorum): prometheus exporter metrics integration
Signed-off-by: Jagpreet Singh Sasan jagpreet.singh.sasan@accenture.com