Skip to content

Conversation

csviri
Copy link
Collaborator

@csviri csviri commented Aug 1, 2025

No description provided.

csviri added 4 commits August 1, 2025 10:43
this was added to fabric8 client meanwhile

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 1, 2025
@csviri csviri changed the title open telemetry feat: open telemetry metrics Aug 1, 2025
@csviri csviri changed the base branch from main to next August 1, 2025 13:00
@csviri csviri linked an issue Aug 1, 2025 that may be closed by this pull request
@csviri
Copy link
Collaborator Author

csviri commented Aug 1, 2025

Started to prototype this, the thing is that there is a OTel adapter for micrometer:
https://docs.micrometer.io/micrometer/reference/implementations/otlp.html

So it should be already possible to integrate with OTel. A native support still would be nice.

However instantly hit this, there is a workaround for sure where we will manage our counts, but it is not nice that counter cannot be closed in sync api.
https://stackoverflow.com/questions/76652117/how-to-close-or-destroy-a-specific-counter-in-opentelemetry

cc @metacosm @xstefank

We might want to discuss it on weekly dev call

@csviri csviri changed the title feat: open telemetry metrics feat: native open telemetry metrics Aug 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate integration possibilities with Opentelemetry
1 participant