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

add prometheus scrape target tx observer #960

Merged
merged 1 commit into from
Aug 27, 2020

Conversation

nklincoln
Copy link
Contributor

Contributes to #637

Provides a prometheus-tx-observer that enables each worker to act as a scrape target with the following metrics exposed:

  • caliper_tx_submitted (counter)
  • caliper_tx_finished (counter)
  • caliper_tx_e2e_latency (histogram)

User may provide options for:

  • metrics path override
  • exposed port
  • processMetricCollectInterval flag
  • additional labels (applied to all metrics)
  • histogramBuckets override (explicit/linear/exponential)

Follow up PR should modify they PushGateway to use the same metrics

Signed-off-by: nkl199@yahoo.co.uk nkl199@yahoo.co.uk

@nklincoln nklincoln requested a review from aklenik August 26, 2020 12:16
@lgtm-com
Copy link

lgtm-com bot commented Aug 26, 2020

This pull request introduces 1 alert when merging 3db2036 into a561f64 - view on LGTM.com

new alerts:

  • 1 for Information exposure through a stack trace

*/
txFinished(results) {
if (Array.isArray(results)) {
for (let result of results) {
Copy link
Contributor

Choose a reason for hiding this comment

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

let should be const

Signed-off-by: nkl199@yahoo.co.uk <nkl199@yahoo.co.uk>
@nklincoln nklincoln merged commit 4aa36ea into hyperledger:master Aug 27, 2020
@nklincoln nklincoln deleted the prometheus-scrape branch August 27, 2020 15:22
danielporto pushed a commit to danielporto/caliper that referenced this pull request Nov 3, 2020
Signed-off-by: nkl199@yahoo.co.uk <nkl199@yahoo.co.uk>
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