Skip to content

Commit 582faa0

Browse files
[SearchProfiler] Move out of legacy (#55331) (#55543)
* Initial move of searchprofiler into new platform directory, lots of things need testing * Whitespace, clean up types and remove unused files * First iteration of end-to-end plugin working - Updated license check to only check for presence of basic license (not search profiler as a feature - Updated the payload: removed types from validation - Also added README in public regarding the location of styles * Added extractProfilerErrorMessage function to interface with new error reporting from profiler endpoint * Fix paths to test_utils * Update I18n for search profiler * Fix react hooks ordering bug with license status updates and fix test (wait for first license object before rendering) * Added index.ts file to common in searchprofiler route Marked types and values as internal Removed unnecessary "async" from function Update import to not use "src" alias Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
1 parent 0c6ed2a commit 582faa0

File tree

104 files changed

+527
-509
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

104 files changed

+527
-509
lines changed

.github/CODEOWNERS

Lines changed: 156 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,156 @@
1+
# GitHub CODEOWNERS definition
2+
# Identify which groups will be pinged by changes to different parts of the codebase.
3+
# For more info, see https://help.github.com/articles/about-codeowners/
4+
5+
# App
6+
/x-pack/legacy/plugins/lens/ @elastic/kibana-app
7+
/x-pack/legacy/plugins/graph/ @elastic/kibana-app
8+
/src/plugins/share/ @elastic/kibana-app
9+
/src/legacy/server/url_shortening/ @elastic/kibana-app
10+
/src/legacy/server/sample_data/ @elastic/kibana-app
11+
/src/legacy/core_plugins/kibana/public/dashboard/ @elastic/kibana-app
12+
/src/legacy/core_plugins/kibana/public/discover/ @elastic/kibana-app
13+
/src/legacy/core_plugins/kibana/public/visualize/ @elastic/kibana-app
14+
/src/legacy/core_plugins/kibana/public/local_application_service/ @elastic/kibana-app
15+
/src/legacy/core_plugins/kibana/public/home/ @elastic/kibana-app
16+
/src/legacy/core_plugins/kibana/public/dev_tools/ @elastic/kibana-app
17+
/src/legacy/core_plugins/metrics/ @elastic/kibana-app
18+
/src/legacy/core_plugins/vis_type_vislib/ @elastic/kibana-app
19+
/src/plugins/home/ @elastic/kibana-app
20+
/src/plugins/kibana_legacy/ @elastic/kibana-app
21+
/src/plugins/timelion/ @elastic/kibana-app
22+
/src/plugins/dev_tools/ @elastic/kibana-app
23+
24+
# App Architecture
25+
/src/plugins/data/ @elastic/kibana-app-arch
26+
/src/plugins/embeddable/ @elastic/kibana-app-arch
27+
/src/plugins/expressions/ @elastic/kibana-app-arch
28+
/src/plugins/kibana_react/ @elastic/kibana-app-arch
29+
/src/plugins/kibana_utils/ @elastic/kibana-app-arch
30+
/src/plugins/navigation/ @elastic/kibana-app-arch
31+
/src/plugins/ui_actions/ @elastic/kibana-app-arch
32+
/src/plugins/visualizations/ @elastic/kibana-app-arch
33+
/x-pack/plugins/advanced_ui_actions/ @elastic/kibana-app-arch
34+
/src/legacy/core_plugins/data/ @elastic/kibana-app-arch
35+
/src/legacy/core_plugins/elasticsearch/lib/create_proxy.js @elastic/kibana-app-arch
36+
/src/legacy/core_plugins/embeddable_api/ @elastic/kibana-app-arch
37+
/src/legacy/core_plugins/interpreter/ @elastic/kibana-app-arch
38+
/src/legacy/core_plugins/kibana_react/ @elastic/kibana-app-arch
39+
/src/legacy/core_plugins/kibana/public/management/ @elastic/kibana-app-arch
40+
/src/legacy/core_plugins/kibana/server/field_formats/ @elastic/kibana-app-arch
41+
/src/legacy/core_plugins/kibana/server/routes/api/management/ @elastic/kibana-app-arch
42+
/src/legacy/core_plugins/kibana/server/routes/api/suggestions/ @elastic/kibana-app-arch
43+
/src/legacy/core_plugins/visualizations/ @elastic/kibana-app-arch
44+
/src/legacy/server/index_patterns/ @elastic/kibana-app-arch
45+
46+
# APM
47+
/x-pack/legacy/plugins/apm/ @elastic/apm-ui
48+
/x-pack/test/functional/apps/apm/ @elastic/apm-ui
49+
/src/legacy/core_plugins/apm_oss/ @elastic/apm-ui
50+
51+
# Beats
52+
/x-pack/legacy/plugins/beats_management/ @elastic/beats
53+
54+
# Canvas
55+
/x-pack/legacy/plugins/canvas/ @elastic/kibana-canvas
56+
57+
# Logs & Metrics UI
58+
/x-pack/legacy/plugins/infra/ @elastic/logs-metrics-ui
59+
/x-pack/legacy/plugins/integrations_manager/ @elastic/epm
60+
61+
# Machine Learning
62+
/x-pack/legacy/plugins/ml/ @elastic/ml-ui
63+
/x-pack/test/functional/apps/machine_learning/ @elastic/ml-ui
64+
/x-pack/test/functional/services/machine_learning/ @elastic/ml-ui
65+
/x-pack/test/functional/services/ml.ts @elastic/ml-ui
66+
# ML team owns the transform plugin, ES team added here for visibility
67+
# because the plugin lives in Kibana's Elasticsearch management section.
68+
/x-pack/legacy/plugins/transform/ @elastic/ml-ui @elastic/es-ui
69+
/x-pack/test/functional/apps/transform/ @elastic/ml-ui
70+
/x-pack/test/functional/services/transform_ui/ @elastic/ml-ui
71+
/x-pack/test/functional/services/transform.ts @elastic/ml-ui
72+
73+
# Maps
74+
/x-pack/legacy/plugins/maps/ @elastic/kibana-gis
75+
/x-pack/test/api_integration/apis/maps/ @elastic/kibana-gis
76+
/x-pack/test/functional/apps/maps/ @elastic/kibana-gis
77+
/x-pack/test/functional/es_archives/maps/ @elastic/kibana-gis
78+
/x-pack/test/visual_regression/tests/maps/index.js @elastic/kibana-gis
79+
80+
# Operations
81+
/src/dev/ @elastic/kibana-operations
82+
/src/setup_node_env/ @elastic/kibana-operations
83+
/src/optimize/ @elastic/kibana-operations
84+
/packages/*eslint*/ @elastic/kibana-operations
85+
/packages/*babel*/ @elastic/kibana-operations
86+
/packages/kbn-dev-utils*/ @elastic/kibana-operations
87+
/packages/kbn-es/ @elastic/kibana-operations
88+
/packages/kbn-pm/ @elastic/kibana-operations
89+
/packages/kbn-test/ @elastic/kibana-operations
90+
/packages/kbn-ui-shared-deps/ @elastic/kibana-operations
91+
/src/legacy/server/keystore/ @elastic/kibana-operations
92+
/src/legacy/server/pid/ @elastic/kibana-operations
93+
/src/legacy/server/sass/ @elastic/kibana-operations
94+
/src/legacy/server/utils/ @elastic/kibana-operations
95+
/src/legacy/server/warnings/ @elastic/kibana-operations
96+
97+
# Platform
98+
/src/core/ @elastic/kibana-platform
99+
/config/kibana.yml @elastic/kibana-platform
100+
/x-pack/plugins/features/ @elastic/kibana-platform
101+
/x-pack/plugins/licensing/ @elastic/kibana-platform
102+
/packages/kbn-config-schema/ @elastic/kibana-platform
103+
/src/legacy/server/config/ @elastic/kibana-platform
104+
/src/legacy/server/http/ @elastic/kibana-platform
105+
/src/legacy/server/i18n/ @elastic/kibana-platform
106+
/src/legacy/server/logging/ @elastic/kibana-platform
107+
/src/legacy/server/saved_objects/ @elastic/kibana-platform
108+
/src/legacy/server/status/ @elastic/kibana-platform
109+
110+
# Security
111+
/src/core/server/csp/ @elastic/kibana-security @elastic/kibana-platform
112+
/x-pack/legacy/plugins/security/ @elastic/kibana-security
113+
/x-pack/legacy/plugins/spaces/ @elastic/kibana-security
114+
/x-pack/plugins/spaces/ @elastic/kibana-security
115+
/x-pack/legacy/plugins/encrypted_saved_objects/ @elastic/kibana-security
116+
/x-pack/plugins/encrypted_saved_objects/ @elastic/kibana-security
117+
/x-pack/plugins/security/ @elastic/kibana-security
118+
/x-pack/test/api_integration/apis/security/ @elastic/kibana-security
119+
120+
# Kibana Localization
121+
/src/dev/i18n/ @elastic/kibana-localization
122+
123+
# Pulse
124+
/packages/kbn-analytics/ @elastic/pulse
125+
/src/legacy/core_plugins/ui_metric/ @elastic/pulse
126+
/src/plugins/usage_collection/ @elastic/pulse
127+
/x-pack/legacy/plugins/telemetry/ @elastic/pulse
128+
129+
# Kibana Alerting Services
130+
/x-pack/legacy/plugins/alerting/ @elastic/kibana-alerting-services
131+
/x-pack/legacy/plugins/actions/ @elastic/kibana-alerting-services
132+
/x-pack/plugins/event_log/ @elastic/kibana-alerting-services
133+
/x-pack/plugins/task_manager/ @elastic/kibana-alerting-services
134+
/x-pack/test/alerting_api_integration/ @elastic/kibana-alerting-services
135+
/x-pack/test/plugin_api_integration/plugins/task_manager/ @elastic/kibana-alerting-services
136+
/x-pack/test/plugin_api_integration/test_suites/task_manager/ @elastic/kibana-alerting-services
137+
/x-pack/legacy/plugins/triggers_actions_ui/ @elastic/kibana-alerting-services
138+
/x-pack/test/functional_with_es_ssl/apps/triggers_actions_ui/ @elastic/kibana-alerting-services
139+
/x-pack/test/functional_with_es_ssl/fixtures/plugins/alerts/ @elastic/kibana-alerting-services
140+
141+
# Design
142+
**/*.scss @elastic/kibana-design
143+
144+
# Elasticsearch UI
145+
/src/legacy/core_plugins/console/ @elastic/es-ui
146+
/src/plugins/es_ui_shared/ @elastic/es-ui
147+
/x-pack/legacy/plugins/console_extensions/ @elastic/es-ui
148+
/x-pack/legacy/plugins/cross_cluster_replication/ @elastic/es-ui
149+
/x-pack/legacy/plugins/index_lifecycle_management/ @elastic/es-ui
150+
/x-pack/legacy/plugins/index_management/ @elastic/es-ui
151+
/x-pack/legacy/plugins/license_management/ @elastic/es-ui
152+
/x-pack/legacy/plugins/remote_clusters/ @elastic/es-ui
153+
/x-pack/legacy/plugins/rollup/ @elastic/es-ui
154+
/x-pack/plugins/searchprofiler/ @elastic/es-ui
155+
/x-pack/legacy/plugins/snapshot_restore/ @elastic/es-ui
156+
/x-pack/legacy/plugins/watcher/ @elastic/es-ui

x-pack/.i18nrc.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@
2727
"xpack.main": "legacy/plugins/xpack_main",
2828
"xpack.monitoring": "legacy/plugins/monitoring",
2929
"xpack.remoteClusters": "legacy/plugins/remote_clusters",
30-
"xpack.reporting": [ "plugins/reporting", "legacy/plugins/reporting" ],
30+
"xpack.reporting": ["plugins/reporting", "legacy/plugins/reporting"],
3131
"xpack.rollupJobs": "legacy/plugins/rollup",
32-
"xpack.searchProfiler": "legacy/plugins/searchprofiler",
32+
"xpack.searchProfiler": "plugins/searchprofiler",
3333
"xpack.siem": "legacy/plugins/siem",
3434
"xpack.security": ["legacy/plugins/security", "plugins/security"],
3535
"xpack.server": "legacy/server",

x-pack/legacy/plugins/searchprofiler/README.md

Lines changed: 0 additions & 26 deletions
This file was deleted.

x-pack/legacy/plugins/searchprofiler/index.ts

Lines changed: 5 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,10 @@
55
*/
66

77
import { resolve } from 'path';
8-
import Boom from 'boom';
98

10-
import { CoreSetup } from 'src/core/server';
11-
import { Server } from 'src/legacy/server/kbn_server';
12-
import { LegacySetup } from './server/np_ready/types';
13-
import { plugin } from './server/np_ready';
9+
// TODO:
10+
// Until we can process SCSS in new platform, this part of Searchprofiler
11+
// legacy must remain here.
1412

1513
export const searchprofiler = (kibana: any) => {
1614
const publicSrc = resolve(__dirname, 'public');
@@ -22,43 +20,8 @@ export const searchprofiler = (kibana: any) => {
2220
publicDir: publicSrc,
2321

2422
uiExports: {
25-
// NP Ready
26-
devTools: [`${publicSrc}/legacy`],
27-
styleSheetPaths: `${publicSrc}/np_ready/application/index.scss`,
28-
// Legacy
29-
home: ['plugins/searchprofiler/register_feature'],
30-
},
31-
init(server: Server) {
32-
const serverPlugin = plugin();
33-
const thisPlugin = this;
34-
35-
const commonRouteConfig = {
36-
pre: [
37-
function forbidApiAccess() {
38-
const licenseCheckResults = server.plugins.xpack_main.info
39-
.feature(thisPlugin.id)
40-
.getLicenseCheckResults();
41-
if (licenseCheckResults.showAppLink && licenseCheckResults.enableAppLink) {
42-
return null;
43-
} else {
44-
throw Boom.forbidden(licenseCheckResults.message);
45-
}
46-
},
47-
],
48-
};
49-
50-
const legacySetup: LegacySetup = {
51-
route: (args: Parameters<typeof server.route>[0]) => server.route(args),
52-
plugins: {
53-
__LEGACY: {
54-
thisPlugin,
55-
xpackMain: server.plugins.xpack_main,
56-
elasticsearch: server.plugins.elasticsearch,
57-
commonRouteConfig,
58-
},
59-
},
60-
};
61-
serverPlugin.setup({} as CoreSetup, legacySetup);
23+
styleSheetPaths: `${publicSrc}/index.scss`,
6224
},
25+
init() {},
6326
});
6427
};

x-pack/legacy/plugins/searchprofiler/public/legacy.ts

Lines changed: 0 additions & 28 deletions
This file was deleted.

x-pack/legacy/plugins/searchprofiler/public/np_ready/plugin.ts

Lines changed: 0 additions & 62 deletions
This file was deleted.

x-pack/legacy/plugins/searchprofiler/public/register_feature.js

Lines changed: 0 additions & 28 deletions
This file was deleted.

0 commit comments

Comments
 (0)