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.

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

[Tracing Instrumentation] Add instrumentation at Rest Layer #8552

Closed
Tracked by #8557
Gaganjuneja opened this issue Jul 9, 2023 · 2 comments · Fixed by #9415
Closed
Tracked by #8557

[Tracing Instrumentation] Add instrumentation at Rest Layer #8552

Gaganjuneja opened this issue Jul 9, 2023 · 2 comments · Fixed by #9415
Labels
distributed framework enhancement Enhancement or improvement to existing feature or request

Comments

@Gaganjuneja
Copy link
Contributor

Gaganjuneja commented Jul 9, 2023

Is your feature request related to a problem? Please describe.

Now the distributed tracing framework #7543 is available. We need to add the tracing instrumentation at several generic components like Rest Layer, Transport Actions and Task Manager levels. Opening up this issue specifically to address the instrumentation in the Rest layer component for all the incoming requests. If the tracing information is passed from the client as part of the header then we should also be able to use it as a root/parent span.

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@Gaganjuneja Gaganjuneja added enhancement Enhancement or improvement to existing feature or request untriaged labels Jul 9, 2023
@dblock
Copy link
Member

dblock commented Jul 12, 2023

@Gaganjuneja can you please put some details in the issue?

@Gaganjuneja
Copy link
Contributor Author

@Gaganjuneja can you please put some details in the issue?

Thanks @dblock for calling it out. I had just created initially these tasks as place holders and now have updated the description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
distributed framework enhancement Enhancement or improvement to existing feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants