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
Visualize uninstrumented services in the dependency diagrams #5062
base: main
Are you sure you want to change the base?
Conversation
Ref PR: #4853 |
Signed-off-by: nidhey27 <nidhey.indurkar@infracloud.io>
…spans Signed-off-by: nidhey27 <nidhey.indurkar@infracloud.io>
Signed-off-by: nidhey27 <nidhey.indurkar@infracloud.io>
Signed-off-by: nidhey27 <nidhey.indurkar@infracloud.io>
Signed-off-by: nidhey27 <nidhey.indurkar@infracloud.io>
Signed-off-by: nidhey27 <nidhey.indurkar@infracloud.io>
Signed-off-by: nidhey27 <nidhey.indurkar@infracloud.io>
….GetDriver Signed-off-by: nidhey27 <nidhey.indurkar@infracloud.io>
Signed-off-by: nidhey27 <nidhey60@gmail.com>
Signed-off-by: nidhey27 <nidhey60@gmail.com>
+1, we are awaiting this feature. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5062 +/- ##
==========================================
- Coverage 95.20% 93.76% -1.44%
==========================================
Files 343 343
Lines 16813 16846 +33
==========================================
- Hits 16006 15795 -211
- Misses 608 869 +261
+ Partials 199 182 -17
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Which problem is this PR solving?
Description of the changes
GetDependencies
to identify and include client spans without corresponding server spans, ensuring a complete dependency graph.updateServiceDependencyLinks
to streamline the update of service dependencies, reducing code duplication.How was this change tested?
Checklist
jaeger
:make lint test
jaeger-ui
:yarn lint
andyarn test