Skip to content

feat: custom OTEL metrics#1464

Draft
aludwiko wants to merge 1 commit intomainfrom
4664-expose-custom-metrics-in-the-sdk-otel
Draft

feat: custom OTEL metrics#1464
aludwiko wants to merge 1 commit intomainfrom
4664-expose-custom-metrics-in-the-sdk-otel

Conversation

@aludwiko
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot added the documentation documentation related label Mar 27, 2026
@aludwiko
Copy link
Copy Markdown
Contributor Author

ready for review, draft because it must wait for the runtime

Copy link
Copy Markdown
Member

@pvlugter pvlugter left a comment

Choose a reason for hiding this comment

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

LGTM

include::example$key-value-shopping-cart/src/main/java/com/example/application/ShoppingCartMetrics.java[tag=metrics]
----
<1> Injects the OpenTelemetry `Meter` provided by the Akka runtime.
<2> Creates a custom counter metric. Please follow Open Telemetry
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should be "Please follow Open Telemetry semantic conventions"?

Suggested change
<2> Creates a custom counter metric. Please follow Open Telemetry
<2> Creates a custom counter metric. Please follow Open Telemetry semantic conventions.

| `akka.javasdk.Sanitizer`
| Allows for applying sanitization, see xref:sdk:sanitization.adoc[]
| `io.opentelemetry.api.metrics.Meter`
| Allows to create custom Open Telemetry metrics, see xref:sdk:metric.adoc[]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
| Allows to create custom Open Telemetry metrics, see xref:sdk:metric.adoc[]
| Allows creating custom Open Telemetry metrics, see xref:sdk:metric.adoc[]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation documentation related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants