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

Move graph service to service tracer. #6695

Merged
merged 2 commits into from
Jul 17, 2023
Merged

Move graph service to service tracer. #6695

merged 2 commits into from
Jul 17, 2023

Conversation

ainmosni
Copy link
Contributor

@ainmosni ainmosni commented Jul 3, 2023

This changes the graph service away from using global tracers,
which makes debugging tracing issues easier going forward.

@update-docs
Copy link

update-docs bot commented Jul 3, 2023

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@ainmosni ainmosni requested a review from butonic July 3, 2023 10:39
@ownclouders
Copy link
Contributor

💥 Acceptance test localApiTests-apiSpaces-ocis failed. Further test are cancelled...

@ainmosni ainmosni force-pushed the ainmosni/tracing/graph branch 3 times, most recently from 598eb03 to c6d75d7 Compare July 7, 2023 09:35
ainmosni added 2 commits July 12, 2023 12:56
This changes the graph service away from using global tracers,
which makes debugging tracing issues easier going forward.
@ainmosni ainmosni force-pushed the ainmosni/tracing/graph branch from c6d75d7 to 2d1e504 Compare July 12, 2023 10:56
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

27.3% 27.3% Coverage
0.0% 0.0% Duplication

@micbar micbar merged commit 2fb2a1c into master Jul 17, 2023
@delete-merged-branch delete-merged-branch bot deleted the ainmosni/tracing/graph branch July 17, 2023 11:00
ownclouders pushed a commit that referenced this pull request Jul 17, 2023
Move graph service to service tracer.
@micbar micbar mentioned this pull request Jul 24, 2023
68 tasks
@butonic butonic mentioned this pull request Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants