Skip to content

feat: native open telemetry metrics#2882

Draft
csviri wants to merge 4 commits intomainfrom
open-telemetry
Draft

feat: native open telemetry metrics#2882
csviri wants to merge 4 commits intomainfrom
open-telemetry

Conversation

@csviri
Copy link
Collaborator

@csviri csviri commented Aug 1, 2025 •
edited
Loading

No description provided.

csviri added 4 commits August 1, 2025 10:43
@csviri
feat: field selectors support for InformerEventSource (#2835)
3e09911 
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@csviri
improve: remove owner refernce check (#2838)
c711d59 
this was added to fabric8 client meanwhile

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@csviri
feat: open telemetry metrics
42ae61e 
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@csviri
wip
28f5b3a 
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
Investigate integration possibilities with Opentelemetry #2135
Open
@csviri
Copy link
Collaborator Author

csviri commented Aug 1, 2025 •
edited
Loading

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

openshift-merge-robot commented Oct 16, 2025

PR needs rebase.

Details

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.

@csviri csviri force-pushed the next branch 2 times, most recently from 3075f11 to 4972764 Compare October 23, 2025 13:17
@csviri csviri force-pushed the next branch 2 times, most recently from 09f7950 to 26de0d2 Compare November 25, 2025 12:15
Base automatically changed from next to main November 25, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Investigate integration possibilities with Opentelemetry

2 participants

@csviri @openshift-merge-robot