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.

Sign up for GitHub

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

[Backport 2.x] Remove redundant traces call for related services #2316

Merged
merged 1 commit into from
Jan 24, 2025
Merged

[Backport 2.x] Remove redundant traces call for related services #2316

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion common/constants/trace_analytics.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ export const TRACE_ANALYTICS_PLOTS_DATE_FORMAT = 'MMM D, YYYY HH:mm:ss.SSS';
export const SERVICE_MAP_MAX_NODES = 500;
// size limit when requesting edge related queries, not necessarily the number of edges
export const SERVICE_MAP_MAX_EDGES = 1000;
export const TRACES_MAX_NUM = 3000;
export const TRACES_MAX_NUM = 500;
export const TRACE_ANALYTICS_DOCUMENTATION_LINK =
'https://opensearch.org/docs/latest/observability-plugin/trace/index/';

Expand Down
4 changes: 4 additions & 0 deletions public/components/application_analytics/__tests__/__snapshots__/log_config.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,10 +90,12 @@ exports[`Log Config component renders empty log config 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -760,10 +762,12 @@ exports[`Log Config component renders with query 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down
4 changes: 4 additions & 0 deletions public/components/application_analytics/__tests__/__snapshots__/service_config.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,10 +90,12 @@ exports[`Service Config component renders empty service config 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -1454,10 +1456,12 @@ exports[`Service Config component renders with one service selected 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down
4 changes: 4 additions & 0 deletions public/components/application_analytics/__tests__/__snapshots__/trace_config.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,10 +89,12 @@ exports[`Trace Config component renders empty trace config 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -1015,10 +1017,12 @@ exports[`Trace Config component renders with one trace selected 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down
8 changes: 8 additions & 0 deletions public/components/custom_panels/__tests__/__snapshots__/custom_panel_view.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1237,10 +1237,12 @@ exports[`Panels View Component renders panel view container with visualizations
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -3144,10 +3146,12 @@ exports[`Panels View Component renders panel view container with visualizations
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -3605,10 +3609,12 @@ exports[`Panels View Component renders panel view container without visualizatio
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -5396,10 +5402,12 @@ exports[`Panels View Component renders panel view container without visualizatio
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down
2 changes: 2 additions & 0 deletions ...s/custom_panels/panel_modules/panel_grid/__tests__/__snapshots__/panel_grid.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -247,10 +247,12 @@ exports[`Panel Grid Component renders panel grid component with empty visualizat
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down
4 changes: 4 additions & 0 deletions public/components/metrics/view/__tests__/__snapshots__/metrics_grid.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,10 +100,12 @@ exports[`Metrics Grid Component renders Metrics Grid Component 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -277,10 +279,12 @@ exports[`Metrics Grid Component renders Metrics Grid Component 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down
12 changes: 12 additions & 0 deletions ...ents/trace_analytics/components/dashboard/__tests__/__snapshots__/dashboard.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,10 +103,12 @@ exports[`Dashboard component renders dashboard 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -496,10 +498,12 @@ exports[`Dashboard component renders dashboard 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -1361,10 +1365,12 @@ exports[`Dashboard component renders empty dashboard 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -1753,10 +1759,12 @@ exports[`Dashboard component renders empty dashboard 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -2618,10 +2626,12 @@ exports[`Dashboard component renders empty jaeger dashboard 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -3012,10 +3022,12 @@ exports[`Dashboard component renders empty jaeger dashboard 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down
12 changes: 12 additions & 0 deletions ...onents/trace_analytics/components/services/__tests__/__snapshots__/services.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,10 +103,12 @@ exports[`Services component renders empty services page 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -334,10 +336,12 @@ exports[`Services component renders empty services page 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -2543,10 +2547,12 @@ exports[`Services component renders jaeger services page 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -2774,10 +2780,12 @@ exports[`Services component renders jaeger services page 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -4245,10 +4253,12 @@ exports[`Services component renders services page 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down Expand Up @@ -4475,10 +4485,12 @@ exports[`Services component renders services page 1`] = `
"getCenter$": [MockFunction],
"getLeft$": [MockFunction],
"getLeftBottom$": [MockFunction],
"getPrimaryHeaderRight$": [MockFunction],
"getRight$": [MockFunction],
"registerCenter": [MockFunction],
"registerLeft": [MockFunction],
"registerLeftBottom": [MockFunction],
"registerPrimaryHeaderRight": [MockFunction],
"registerRight": [MockFunction],
},
"navGroup": Object {
Expand Down
Loading
Loading