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

Trace quality view & Ddg Decorations #564

Merged
merged 33 commits into from
Apr 27, 2020

Conversation

everett980
Copy link
Collaborator

Which problem is this PR solving?

  • Create OSS Quality Metrics View to display trace quality information for a service
  • Support for visualizing metrics on the DDG

Short description of the changes

  • Add metric ring to ddg nodes to visualize scores
  • Add side panel for selecting which score to visualize
  • Add details within side panel to fetch further information for a single, selected vertex
  • Add trace quality view that shows quality scores and quality metrics

Decorated DDG:

Screen Shot 2020-04-24 at 2 57 46 PM

Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
details panel, render details in details card
TODO: Style table, handle list, handle styled values

Signed-off-by: Everett Ross <reverett@uber.com>
TODO: Style table, handle list, handle styled values

Signed-off-by: Everett Ross <reverett@uber.com>
clickable ddg nodes, fix resizer height css, make top offset css var
TODO: Style table, handle list, handle styled values

Signed-off-by: Everett Ross <reverett@uber.com>
TODO: Style table overflow, handle styled values

Signed-off-by: Everett Ross <reverett@uber.com>
TODO: Handle styled values, loading&err render, modal, beautification

Signed-off-by: Everett Ross <reverett@uber.com>
style card
TODO: Add info modal

Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
decoration header size, destructured variable order,
DeepDependencies/index initial state, stale comments, yarn.lock
@types/node, yarn.lock registry urls, 'value' in paths, existing tests,
op-specific details
Add: linking row cells
TODO: New tests

Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
memoize summary requests
TODO: Test SidePanel/ index, index.track, DetailsPanel

Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
TODO: Render banner text, style components, test

Signed-off-by: Everett Ross <reverett@uber.com>
TODO: Render banner text, render weight, test

Signed-off-by: Everett Ross <reverett@uber.com>
metric documentation tooltip, tweak styles, add BannerText, handle
loading, handle error
TODO: test, cleanup

Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: Everett Ross <reverett@uber.com>
@codecov
Copy link

codecov bot commented Apr 24, 2020

Codecov Report

Merging #564 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #564   +/-   ##
=======================================
  Coverage   89.94%   89.94%           
=======================================
  Files         214      214           
  Lines        5210     5210           
  Branches     1326     1326           
=======================================
  Hits         4686     4686           
  Misses        481      481           
  Partials       43       43           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60eaca9...60eaca9. Read the comment docs.

@yurishkuro yurishkuro merged commit 3e17a7c into jaegertracing:master Apr 27, 2020
vvvprabhakar pushed a commit to vvvprabhakar/jaeger-ui that referenced this pull request Jul 5, 2021
* WIP: Action and types for decorations

Signed-off-by: Everett Ross <reverett@uber.com>

* Add PAD reducer, fix types, fix year

Signed-off-by: Everett Ross <reverett@uber.com>

* Fix and test reducer, fix types, fix another year

Signed-off-by: Everett Ross <reverett@uber.com>

* Add another pad reducer test

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Begin testing action

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Finish action tests TODO: Move stringSupplant

Signed-off-by: Everett Ross <reverett@uber.com>

* Move and test stringSupplant

Signed-off-by: Everett Ross <reverett@uber.com>

* Cleanup

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Decorate nodes, selector/detail side panel

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Style side panel

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Continue styling side panel, fetch details in
details panel, render details in details card
TODO: Style table, handle list, handle styled values

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Improve TS handling of union of arrays
TODO: Style table, handle list, handle styled values

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Limit % circle size, update cursor for
clickable ddg nodes, fix resizer height css, make top offset css var
TODO: Style table, handle list, handle styled values

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Handle list, begin overflow management
TODO: Style table overflow, handle styled values

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Manage overflow, begin handling styled values
TODO: Handle styled values, loading&err render, modal, beautification

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Handle styled values, render loading&err,
style card
TODO: Add info modal

Signed-off-by: Everett Ross <reverett@uber.com>

* Add info modal, begin clean up TODO clean up&test

Signed-off-by: Everett Ross <reverett@uber.com>

* Fix: rowKeys, setViewModifier argument name,
decoration header size, destructured variable order,
DeepDependencies/index initial state, stale comments, yarn.lock
@types/node, yarn.lock registry urls, 'value' in paths, existing tests,
op-specific details
Add: linking row cells
TODO: New tests

Signed-off-by: Everett Ross <reverett@uber.com>

* Handle linked cells, fix cell sort order

Signed-off-by: Everett Ross <reverett@uber.com>

* Test existing files, track decorations viewed,
memoize summary requests
TODO: Test SidePanel/ index, index.track, DetailsPanel

Signed-off-by: Everett Ross <reverett@uber.com>

* Test SidePanel/ index&track WIP test DetailsPanel

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP test DetailsPanel

Signed-off-by: Everett Ross <reverett@uber.com>

* Finish DetailsPanel tests

Signed-off-by: Everett Ross <reverett@uber.com>

* Clean up

Signed-off-by: Everett Ross <reverett@uber.com>

* Add skeleton components and fetch quality metrics

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Render all data and dropdowns except banner
TODO: Render banner text, style components, test

Signed-off-by: Everett Ross <reverett@uber.com>

* WIP: Style components, implement lookback
TODO: Render banner text, render weight, test

Signed-off-by: Everett Ross <reverett@uber.com>

* Debounce InputNumber, limit search url length, add
metric documentation tooltip, tweak styles, add BannerText, handle
loading, handle error
TODO: test, cleanup

Signed-off-by: Everett Ross <reverett@uber.com>

* Cleanup

Signed-off-by: Everett Ross <reverett@uber.com>

* Add support for decoration links

Signed-off-by: Everett Ross <reverett@uber.com>

* Clean up and add quality-metrics top nav link

Signed-off-by: Everett Ross <reverett@uber.com>
Signed-off-by: vvvprabhakar <vvvprabhakar@gmail.com>
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.

None yet

2 participants