Build health metrics dimensions

[mrh666]

What feature do you want to see added?

@cyrille-leclerc on that page https://plugins.jenkins.io/opentelemetry/ there is a screenshot of kibana https://raw.githubusercontent.com/jenkinsci/opentelemetry-plugin/master/docs/images/kibana_jenkins_overview_dashboard.png with all graphs I need, e.g. job duration, failed steps, long steps, etc. How can we get those metrics exported to Dynatrace?

Upstream changes

No response

Are you interested in contributing this feature?

No response

[cyrille-leclerc]

Report per pipeline

A solution we are looking at could be to produce an histogram metric per pipeline and per result. It would be inspired by the standardized http.server.request.duration metric.

I was waiting for the OTel CI/CD SIG to standardize such a metric, work is in progress with:

• CI/CD conventions for metrics open-telemetry/semantic-conventions#1111

⚠️ I'm worried about the cardinality of such a metric as we can potentially produce 5 x count(pipeline) histograms which is a lot.

@christophe-kamphaus-jemmic did you put thoughts on such metrics?

The metric could look like

ci.pipeline.run.duration: histogram {
   // Pipeline full name
   // See org.jenkinsci.plugins.workflow.job.WorkflowJob#getFullName()
   ci.pipeline.id="/my-team/my-war/master",
   // see hudson.model.Run#getResult() 
   // SUCCESS, UNSTABLE, FAILURE, NOT_BUILT, ABORTED
   ci.pipeline.run.result="SUCCESS"
}

Report per pipeline step

High cardinality problems look even more a risk here. I'm wondering if we should not stick to solve this doing metrics queries on the traces similar to what TraceQL metrics queries offer.

Controlling cardinality

I'm thinking of helping Jenkins admins control cardinality of such metrics enabling allow & deny lists of pipeline names as we have seen Jenkins instances with thousands of pipelines.

@mrh666 Is it he kind of ideas you had in mind?

[mrh666]

@cyrille-leclerc that's exactly what I have in mind!

I'm worried about the cardinality of such a metric as we can potentially produce 5 x count(pipeline) histograms which is a lot.

You have reasonable worries about cardinality. In influx world it's easily can kill all the DB performance. But just make it optional. Something like otel.exporter.otlp.metrics.build_health.enabled

if we should not stick to solve this doing metrics queries on the traces similar to

In Dynatrace world it's impossible or close to impossible. I've digging into such a functionality and not achieved any results.

[cyrille-leclerc]

Thanks @mrh666 . Can you please share with us:

• Total count of pipelines
• Count of pipelines for which you want performance metrics
• Could it be possible with allo/deny lists based on regex to just collect metrics on the pipelines that matter to you?

Same question for build steps

[mrh666]

In the current project:
We have 24 pipelines running at the moment
12 of those required pipeline metrics

Could it be possible with allo/deny lists based on regex to just collect metrics on the pipelines that matter to you?

This one is really important!

[cyrille-leclerc]

Here is a proposal:

• Allow and Deny lists using regex to specify the job names for which we create a time series to control cardinality
• Histogram metric

ci.pipeline.run.duration: unit=second {
   ci.pipeline.id: if (in-allow-lit && ! in-deny-list) ?
      hudson.model.Job.getParent().getFullName() :
      "#other#"
   ci.pipeline.run.result: hudson.model.Result
   ci.pipeline.run.completed: hudson.model.Result.isCompleted()

}

See:

• Add ci.pipeline.run.duration metric #959

Feedback welcome cc @mrh666

[christophe-kamphaus-jemmic]

I was waiting for the OTel CI/CD SIG to standardize such a metric

Indeed work in the OTel CI/CD SIG related to metrics is in progress.
We are currently standardizing metrics related to VCS and we plan to follow that up with metrics related to pipelines, queues and agents.
ci.pipeline.run.duration: histogram sounds like a good metric. I will propose it in the SIG.

One issue I can see with using a histogram is that the chosen buckets might not give enough insight to take any action. Some jobs might be of very short duration, while others could take hours or even days to complete.
So how would you define the buckets?

I had very good success in using metrics queries on traces/spans for job duration as well as stage duration using the steps I introduced in #827 (example use in a pipeline here: #811 (comment)). This allowed me to have very detailed statistics (eg. average duration per day or job) and is filterable per job.

I'm thinking of helping Jenkins admins control cardinality of such metrics enabling allow & deny lists of pipeline names as we have seen Jenkins instances with thousands of pipelines.

For sure cardinality is an issue when the number of time series scales with a dynamic value like the number of jobs managed by Jenkins. It's not as bad as when we would have a separate time series per build, but still it needs to be managed. (prometheus-plugin has per-build metrics guarded by a checkbox config option)

Controlling which jobs generate this metric on Jenkins-side I think is a very good option.

Alternatively it's also possible to filter later:

• using metric relabelling rules in Prometheus or ServiceMonitor
• with opentelemetry-collector a filterprocessor could be used

  processors:
    filter/ci:
      error_mode: ignore
      metrics:
        metric:
            - 'name == "ci.pipeline.run.duration" and not(IsMatch(attributes["ci.pipeline.id"], "my-otel-pipelines.*"))'
  

[cyrille-leclerc]

Cc @miraccan00

[cyrille-leclerc]

Please use the ci.pipeline.run.duration{ci.pipeline.id="<<pipeline full name>>", ci.pipeline.result="<<SUCCESS, UNSTABLE, FAILURE, NOT_BUILT, ABORTED>>"} histogram metric we have just released.
ℹ Use the otel.instrumentation.jenkins.run.metric.duration.allow_list and otel.instrumentation.jenkins.run.metric.duration.deny_list to specify the pipelines for which you want to capture the run duration, other pipelines will be aggregated in the ci.pipeline.id="#other#" time series.

See documentation https://github.com/jenkinsci/opentelemetry-plugin/blob/main/docs/monitoring-metrics.md#build-duration

I'm marking your enhancement request as solved. Please open new enhancement requests if needed.

[mrh666]

@cyrille-leclerc Thank you! Now I've been trying to use it with Jenkins. Here is config line from jenkins:

Nov 28, 2024 2:40:44 PM FINE io.jenkins.plugins.opentelemetry.jenkins.OpenTelemetryConfigurerComputerListener$OpenTelemetryConfigurerMasterToSlaveCallable

Configure OpenTelemetry SDK with properties: {otel.exporter.otlp.timeout=30000, otel.exporter.otlp.endpoint=https://dynatrace-otel-collector.xxxx.com, otel.exporter.otlp.protocol=http/protobuf, jenkins.url=https://xxxx/, jenkins.version=2.462.1, otel.metrics.exporter=otlp, otel.instrumentation.jenkins.run.metric.duration.allow_list=".*/.*", otel.traces.exporter=otlp, service.instance.id=xxxx, otel.instrumentation.jenkins.remote.span.enabled=true, otel.instrumentation.jenkins.agent.enabled=true, otel.imr.export.interval=30000, otel.instrumentation.jenkins.remoting.enabled=true, otel.exporter.otlp.{signal}.protocol=http/protobuf, otel.exporter.otlp.metrics.temporality.preference=DELTA, otel.java.disabled.resource.providers=io.opentelemetry.instrumentation.resources.ProcessResourceProvider}, resource:{jenkins.opentelemetry.plugin.version=3.1423.v0d1a_2fcd2429, service.name=jenkins, jenkins.computer.name=xxxx, service.namespace=jenkins}

I changed those params a lot of times. But I can't trigger the metric. Like:

otel.instrumentation.jenkins.run.metric.duration.allow_list=".*/.*"
otel.instrumentation.jenkins.run.metric.duration.allow_list=".*"
otel.instrumentation.jenkins.run.metric.duration.allow_list=.*
otel.instrumentation.jenkins.run.metric.duration.allow_list=/.*/

Am I doing wrong with regex?

And another point - please fix the documentation https://github.com/jenkinsci/opentelemetry-plugin/blob/main/docs/monitoring-metrics.md#build-duration:

You wrote:

Configuration parameters to control the cardinality of the ci.pipeline.id attribute:

    otel.instrumentation.jenkins.run.metric.duration.allow_list: Java regex, default value: $^ (ie match nothing). Example jenkins_folder_a/.*|jenkins_folder_b/.*
    otel.instrumentation.jenkins.run.metric.duration.deny_list: Java regex, default value: $^ (ie match nothing). Example .*test.*

Should be I believe:

Configuration parameters to control the cardinality of the ci.pipeline.id attribute:

    otel.instrumentation.jenkins.run.metric.duration.allow_list: Java regex, default value: ^$ (ie match nothing). Example jenkins_folder_a/.*|jenkins_folder_b/.*
    otel.instrumentation.jenkins.run.metric.duration.deny_list: Java regex, default value: ^$ (ie match nothing). Example .*test.*