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
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
this was added to fabric8 client meanwhile

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@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
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 16, 2025
@openshift-merge-robot
Copy link

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

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. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate integration possibilities with Opentelemetry

2 participants