Traces jaeger #150

derek-ho · 2023-01-04T15:47:26Z

Description

Adds support for raw jaeger schema data in trace analytics

Issues Resolved

Check List

New functionality includes testing.
- All tests pass, including unit test, integration test and doctest
New functionality has been documented.
- New functionality has javadoc added
- New functionality has user manual doc added
Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Derek Ho <[email protected]>

…ges to work with this framework Signed-off-by: Derek Ho <[email protected]>

Signed-off-by: Derek Ho <[email protected]>

…e on out Signed-off-by: Derek Ho <[email protected]>

Signed-off-by: Derek Ho <[email protected]>

public/components/trace_analytics/requests/dashboard_request_handler.ts

public/components/trace_analytics/requests/request_handler.ts

public/components/trace_analytics/components/dashboard/dashboard_content.tsx

Signed-off-by: Derek Ho <[email protected]>

public/components/trace_analytics/components/traces/span_detail_table.tsx

public/components/trace_analytics/requests/request_handler.ts

public/components/trace_analytics/requests/queries/services_queries.ts

Signed-off-by: Derek Ho <[email protected]>

Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415)

Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415) Co-authored-by: Derek Ho <[email protected]>

vvvprabhakar · 2023-01-25T16:48:26Z

Pretty Useful feature
Would love to know to more about what are the other roadmap features for trace data with this plugin

derek-ho · 2023-01-25T17:14:08Z

@vvvprabhakar we are still coming up with a roadmap for this feature set - the main one being service map support with spark jobs, but feel free to create issues for features you would like to see/prioritized, thanks!

…search-project#214) Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415) Co-authored-by: Derek Ho <[email protected]> Signed-off-by: Derek Ho <[email protected]>

Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415) Co-authored-by: Derek Ho <[email protected]>

derek-ho added 30 commits January 4, 2023 10:34

get traces page working with queries

97c12d6

Signed-off-by: Derek Ho <[email protected]>

get trace individual page partially working

5fcb2ca

Signed-off-by: Derek Ho <[email protected]>

get service page working initially

71ac297

Signed-off-by: Derek Ho <[email protected]>

get services page mainly working

8bf4747

Signed-off-by: Derek Ho <[email protected]>

get most things working except for dashboards

c361298

Signed-off-by: Derek Ho <[email protected]>

something is broken

d79bc9c

Signed-off-by: Derek Ho <[email protected]>

get time working on traces query

29b8552

Signed-off-by: Derek Ho <[email protected]>

last commit for today

7bf6374

Signed-off-by: Derek Ho <[email protected]>

done for today for real

75fd5f5

Signed-off-by: Derek Ho <[email protected]>

fix date filter

0d6a4be

Signed-off-by: Derek Ho <[email protected]>

trying to fix span table

58d1088

Signed-off-by: Derek Ho <[email protected]>

save work

5abd7d3

Signed-off-by: Derek Ho <[email protected]>

mostly fix trace page

5a99cb4

Signed-off-by: Derek Ho <[email protected]>

services working and traces working

2cb7d9f

Signed-off-by: Derek Ho <[email protected]>

fix some more stuff

555319b

Signed-off-by: Derek Ho <[email protected]>

fix error shown on span detail chart

b848af9

Signed-off-by: Derek Ho <[email protected]>

fix

283fd1d

Signed-off-by: Derek Ho <[email protected]>

make general

041d38f

Signed-off-by: Derek Ho <[email protected]>

make everything more general and get traces and services top level pa…

676f082

…ges to work with this framework Signed-off-by: Derek Ho <[email protected]>

get subpages working and get rid of servicemap when in jaeger mode

58a3a0e

Signed-off-by: Derek Ho <[email protected]>

make most work on both

a3d87ec

Signed-off-by: Derek Ho <[email protected]>

pushing work

fc6bbd7

Signed-off-by: Derek Ho <[email protected]>

fix service map for data prepper

6750c6f

Signed-off-by: Derek Ho <[email protected]>

get more things to work

7237845

Signed-off-by: Derek Ho <[email protected]>

push work

1e7bad3

Signed-off-by: Derek Ho <[email protected]>

get almost everything working

08a0bff

Signed-off-by: Derek Ho <[email protected]>

everything on parity - new features and UX fixes only needed from her…

0742d80

…e on out Signed-off-by: Derek Ho <[email protected]>

fix service flyout pannel

81d3abf

Signed-off-by: Derek Ho <[email protected]>

fix app analytics

9ec8533

Signed-off-by: Derek Ho <[email protected]>

default to data prepper for app analytics

5dba245

Signed-off-by: Derek Ho <[email protected]>

derek-ho added 2 commits January 10, 2023 13:59

add timeout

57cd03f

Signed-off-by: Derek Ho <[email protected]>

revert version

ab65109

Signed-off-by: Derek Ho <[email protected]>

derek-ho commented Jan 10, 2023

View reviewed changes

public/components/trace_analytics/requests/dashboard_request_handler.ts Show resolved Hide resolved

joshuali925 reviewed Jan 10, 2023

View reviewed changes

public/components/trace_analytics/requests/request_handler.ts Outdated Show resolved Hide resolved

derek-ho commented Jan 10, 2023

View reviewed changes

public/components/trace_analytics/components/dashboard/dashboard_content.tsx Outdated Show resolved Hide resolved

derek-ho added 2 commits January 10, 2023 15:09

fix timeout

5f7dbe2

Signed-off-by: Derek Ho <[email protected]>

test fixes and message fix

4a06925

Signed-off-by: Derek Ho <[email protected]>

derek-ho marked this pull request as ready for review January 10, 2023 21:36

derek-ho requested a review from a team as a code owner January 10, 2023 21:36

fix toast into a single one

3dd96c5

Signed-off-by: Derek Ho <[email protected]>

pjfitzgibbons reviewed Jan 10, 2023

View reviewed changes

public/components/trace_analytics/components/traces/span_detail_table.tsx Show resolved Hide resolved

joshuali925 reviewed Jan 10, 2023

View reviewed changes

public/components/trace_analytics/requests/request_handler.ts Outdated Show resolved Hide resolved

public/components/trace_analytics/requests/request_handler.ts Outdated Show resolved Hide resolved

ps48 reviewed Jan 10, 2023

View reviewed changes

public/components/trace_analytics/requests/request_handler.ts Outdated Show resolved Hide resolved

ps48 reviewed Jan 10, 2023

View reviewed changes

public/components/trace_analytics/requests/queries/services_queries.ts Outdated Show resolved Hide resolved

address pr comments

45cdc1e

Signed-off-by: Derek Ho <[email protected]>

joshuali925 approved these changes Jan 11, 2023

View reviewed changes

ps48 approved these changes Jan 11, 2023

View reviewed changes

joshuali925 merged commit f700415 into opensearch-project:2.5 Jan 11, 2023

derek-ho added backport 2.x backport main labels Jan 20, 2023

opensearch-trigger-bot bot mentioned this pull request Jan 20, 2023

[Backport 2.x] Traces jaeger #214

Merged

opensearch-trigger-bot bot pushed a commit that referenced this pull request Jan 20, 2023

Support Jaeger data in trace-analytics (#150)

7a9f711

Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415)

opensearch-trigger-bot bot pushed a commit that referenced this pull request Jan 20, 2023

Support Jaeger data in trace-analytics (#150)

2613008

Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415)

opensearch-trigger-bot bot mentioned this pull request Jan 20, 2023

[Backport main] Traces jaeger #215

Merged

derek-ho added a commit that referenced this pull request Jan 24, 2023

Support Jaeger data in trace-analytics (#150) (#214)

90b4d12

Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415) Co-authored-by: Derek Ho <[email protected]>

derek-ho added a commit that referenced this pull request Jan 24, 2023

Support Jaeger data in trace-analytics (#150) (#215)

6b956a6

Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415) Co-authored-by: Derek Ho <[email protected]>

pjfitzgibbons pushed a commit that referenced this pull request Mar 28, 2023

Support Jaeger data in trace-analytics (#150) (#214)

4fd4e08

Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit f700415) Co-authored-by: Derek Ho <[email protected]>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Traces jaeger #150

Traces jaeger #150

derek-ho commented Jan 4, 2023 •

edited

Loading

vvvprabhakar commented Jan 25, 2023

derek-ho commented Jan 25, 2023

Traces jaeger #150

Traces jaeger #150

Conversation

derek-ho commented Jan 4, 2023 • edited Loading

Description

Issues Resolved

Check List

vvvprabhakar commented Jan 25, 2023

derek-ho commented Jan 25, 2023

derek-ho commented Jan 4, 2023 •

edited

Loading