Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

How to modify view with HttpClientMetrics #6819

Closed
yingziisme opened this issue Oct 6, 2022 · 3 comments · Fixed by #9440
Closed

How to modify view with HttpClientMetrics #6819

yingziisme opened this issue Oct 6, 2022 · 3 comments · Fixed by #9440
Labels
enhancement New feature or request

Comments

@yingziisme
Copy link
Contributor

How to modify view with HttpClientMetrics or HttpServerMetrics? Only by modifying TemporaryMetricsView?
@yingziisme yingziisme added the bug Something isn't working label Oct 6, 2022
@trask
Copy link
Member

trask commented Oct 6, 2022

hi @yingziisme! there is no good way currently, though this is needed. can you describe your use case? is it to capture additional attributes which are already on the span? or to remove some attributes? or to populate new attributes that are not already on the span?

@mateuszrzeszutek
Copy link
Member

Linking a related issue #6209

@anuragagarwal561994 anuragagarwal561994 mentioned this issue Dec 27, 2022
Closed
6 tasks
@mateuszrzeszutek mateuszrzeszutek mentioned this issue Dec 28, 2022
Closed
@trask trask added enhancement New feature or request and removed bug Something isn't working labels Aug 15, 2023
@trask
Copy link
Member

trask commented Sep 4, 2023

getting closer to being able to support this with open-telemetry/opentelemetry-java#5677 and open-telemetry/opentelemetry-java#5722

@mateuszrzeszutek mateuszrzeszutek mentioned this issue Sep 12, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
3 participants
@trask @yingziisme @mateuszrzeszutek