Skip to content

Commit

Permalink
added new metrics for graphs (#2029)
Browse files Browse the repository at this point in the history
* added new metrics for graphs

* added new required and roundtrip into metrics
  • Loading branch information
Adir111 authored Nov 12, 2024
1 parent 4d6683c commit 3f5f821
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 0 deletions.
10 changes: 10 additions & 0 deletions core/pipeline-driver/lib/consts/metricsNames.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ module.exports = {
streaming_edge_processingTimeMs: 'pipeline_driver_streaming_edge_processing_time',
streaming_edge_resRate: 'pipeline_driver_streaming_edge_res_rate',
streaming_edge_reqRate: 'pipeline_driver_streaming_edge_req_rate',
streaming_edge_roundTripTimeMs: 'pipeline_driver_streaming_edge_round_trip',
streaming_edge_required: 'pipeline_driver_streaming_edge_required',

streaming_pods_per_node: 'pipeline_driver_streaming_pods_per_node',
},
Expand Down Expand Up @@ -38,6 +40,14 @@ module.exports = {
propName: 'reqRate',
registerZeroValue: true
},
round_trip: {
propName: 'roundTripTimeMs',
registerZeroValue: false
},
required: {
propName: 'required',
registerZeroValue: true
}
},
streamingGeneralMetricToPropMap: {
pods_per_node: {
Expand Down
10 changes: 10 additions & 0 deletions core/pipeline-driver/lib/metrics/pipeline-metrics.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,16 @@ class PipelineMetrics {
description: 'Pod count per node',
labels: ['pipelineName', 'jobId', 'node'],
});
metrics.addGaugeMeasure({
name: metricsNames.streaming_edge_roundTripTimeMs,
description: 'Round Trip in ms',
labels: ['pipelineName', 'jobId', 'source', 'target']
});
metrics.addGaugeMeasure({
name: metricsNames.streaming_edge_required,
description: 'Required Pods',
labels: ['pipelineName', 'jobId', 'source', 'target']
});
}

startMetrics(options) {
Expand Down

0 comments on commit 3f5f821

Please sign in to comment.