Revert "MDS Support for trace analytics (#1752)"

This reverts commit da7d54e.

Author: sumukhswamy

opensearch_dashboards.json

@@ -18,6 +18,6 @@
     "urlForwarding",
     "visualizations"
   ],
-  "optionalPlugins": ["managementOverview", "assistantDashboards", "securityDashboards", "dataSource", "dataSourceManagement"],
+  "optionalPlugins": ["managementOverview", "assistantDashboards", "securityDashboards"],
   "configPath": ["observability"]
 }

public/components/app.tsx

@@ -7,20 +7,19 @@ import { I18nProvider } from '@osd/i18n/react';
 import { QueryManager } from 'common/query_manager';
 import React from 'react';
 import { Provider } from 'react-redux';
-import { CoreStart, MountPoint } from '../../../../src/core/public';
-import { DataSourceManagementPluginSetup } from '../../../../src/plugins/data_source_management/public';
+import { CoreStart } from '../../../../src/core/public';
 import { observabilityID, observabilityTitle } from '../../common/constants/shared';
 import { store } from '../framework/redux/store';
 import { AppPluginStartDependencies } from '../types';
 import { Home as ApplicationAnalyticsHome } from './application_analytics/home';
+import { Home as IntegrationsHome } from './integrations/home';
 import { MetricsListener } from './common/metrics_listener';
 import { Home as CustomPanelsHome } from './custom_panels/home';
-import { Home as DataConnectionsHome } from './datasources/home';
 import { EventAnalytics } from './event_analytics';
-import { Home as IntegrationsHome } from './integrations/home';
 import { Home as MetricsHome } from './metrics/index';
 import { Main as NotebooksHome } from './notebooks/components/main';
 import { Home as TraceAnalyticsHome } from './trace_analytics/home';
+import { Home as DataConnectionsHome } from './datasources/home';
 
 interface ObservabilityAppDeps {
   CoreStartProp: CoreStart;
@@ -31,10 +30,6 @@ interface ObservabilityAppDeps {
   timestampUtils: any;
   queryManager: QueryManager;
   startPage: string;
-  dataSourceEnabled: boolean;
-  dataSourceManagement: DataSourceManagementPluginSetup;
-  setActionMenu: (menuMount: MountPoint | undefined) => void;
-  savedObjectsMDSClient: CoreStart['savedObjects'];
 }
 
 // for cypress to test redux store
@@ -63,10 +58,6 @@ export const App = ({
   queryManager,
   startPage,
   dataSourcePluggables,
-  dataSourceManagement,
-  setActionMenu,
-  dataSourceEnabled,
-  savedObjectsMDSClient,
 }: ObservabilityAppDeps) => {
   const { chrome, http, notifications, savedObjects: coreSavedObjects } = CoreStartProp;
   const parentBreadcrumb = {
@@ -98,10 +89,6 @@ export const App = ({
             parentBreadcrumbs={[parentBreadcrumb]}
             setBreadcrumbs={chrome.setBreadcrumbs}
             dataSourcePluggables={dataSourcePluggables}
-            dataSourceManagement={dataSourceManagement}
-            dataSourceEnabled={dataSourceEnabled}
-            setActionMenu={setActionMenu}
-            savedObjectsMDSClient={savedObjectsMDSClient}
           />
         </MetricsListener>
       </I18nProvider>

public/components/application_analytics/components/config_components/service_config.tsx

@@ -15,14 +15,15 @@ import {
   EuiText,
 } from '@elastic/eui';
 import DSLService from 'public/services/requests/dsl';
-import React, { useEffect, useState } from 'react';
-import { OptionType } from '../../../../../common/types/application_analytics';
+import React, { useState } from 'react';
+import { useEffect } from 'react';
 import { FilterType } from '../../../trace_analytics/components/common/filters/filters';
 import { ServiceObject } from '../../../trace_analytics/components/common/plots/service_map';
 import { ServiceMap } from '../../../trace_analytics/components/services';
 import { handleServiceMapRequest } from '../../../trace_analytics/requests/services_request_handler';
-import { getClearModal } from '../../helpers/modal_containers';
 import { AppAnalyticsComponentDeps } from '../../home';
+import { OptionType } from '../../../../../common/types/application_analytics';
+import { getClearModal } from '../../helpers/modal_containers';
 
 interface ServiceConfigProps extends AppAnalyticsComponentDeps {
   dslService: DSLService;
@@ -49,7 +50,7 @@ export const ServiceConfig = (props: ServiceConfigProps) => {
   const [modalLayout, setModalLayout] = useState(<EuiOverlayMask />);
 
   useEffect(() => {
-    handleServiceMapRequest(http, dslService, mode, '', setServiceMap);
+    handleServiceMapRequest(http, dslService, mode, setServiceMap);
   }, []);
 
   useEffect(() => {

public/components/application_analytics/components/flyout_components/service_detail_flyout.tsx

@@ -4,6 +4,7 @@
  */
 /* eslint-disable react-hooks/exhaustive-deps */
 
+import _ from 'lodash';
 import {
   EuiFlyout,
   EuiFlyoutBody,
@@ -13,20 +14,19 @@ import {
   EuiText,
   EuiTitle,
 } from '@elastic/eui';
-import _ from 'lodash';
 import React, { useEffect, useMemo, useState } from 'react';
+import {
+  handleServiceMapRequest,
+  handleServiceViewRequest,
+} from '../../../../../public/components/trace_analytics/requests/services_request_handler';
 import {
   filtersToDsl,
   processTimeStamp,
 } from '../../../../../public/components/trace_analytics/components/common/helper_functions';
-import { ServiceObject } from '../../../../../public/components/trace_analytics/components/common/plots/service_map';
 import { ServiceMap } from '../../../../../public/components/trace_analytics/components/services';
+import { ServiceObject } from '../../../../../public/components/trace_analytics/components/common/plots/service_map';
 import { SpanDetailTable } from '../../../../../public/components/trace_analytics/components/traces/span_detail_table';
 import { TraceAnalyticsComponentDeps } from '../../../../../public/components/trace_analytics/home';
-import {
-  handleServiceMapRequest,
-  handleServiceViewRequest,
-} from '../../../../../public/components/trace_analytics/requests/services_request_handler';
 import { getListItem } from '../../helpers/utils';
 
 interface ServiceFlyoutProps extends TraceAnalyticsComponentDeps {
@@ -131,7 +131,7 @@ export function ServiceDetailFlyout(props: ServiceFlyoutProps) {
       appConfigs
     );
     handleServiceViewRequest(serviceName, http, serviceDSL, setFields, mode);
-    handleServiceMapRequest(http, serviceDSL, mode, '', setServiceMap, serviceName);
+    handleServiceMapRequest(http, serviceDSL, mode, setServiceMap, serviceName);
     const spanDSL = filtersToDsl(mode, filters, query, startTime, endTime, 'app', appConfigs);
     spanDSL.query.bool.must.push({
       term: {

public/components/index.tsx

@@ -3,12 +3,11 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { QueryManager } from 'common/query_manager';
 import React from 'react';
 import ReactDOM from 'react-dom';
+import { QueryManager } from 'common/query_manager';
 import { AppMountParameters, CoreStart } from '../../../../src/core/public';
-import { DataSourceManagementPluginSetup } from '../../../../src/plugins/data_source_management/public';
-import { AppPluginStartDependencies } from '../types';
+import { AppPluginStartDependencies, SetupDependencies } from '../types';
 import { App } from './app';
 
 export const Observability = (
@@ -21,12 +20,8 @@ export const Observability = (
   timestampUtils: any,
   queryManager: QueryManager,
   startPage: string,
-  dataSourcePluggables,
-  dataSourceManagement: DataSourceManagementPluginSetup,
-  savedObjectsMDSClient: CoreStart['savedObjects']
+  dataSourcePluggables
 ) => {
-  const { setHeaderActionMenu } = AppMountParametersProp;
-  const { dataSource } = DepsStart;
   ReactDOM.render(
     <App
       CoreStartProp={CoreStartProp}
@@ -38,10 +33,6 @@ export const Observability = (
       queryManager={queryManager}
       startPage={startPage}
       dataSourcePluggables={dataSourcePluggables}
-      dataSourceManagement={dataSourceManagement}
-      setActionMenu={setHeaderActionMenu}
-      dataSourceEnabled={!!dataSource}
-      savedObjectsMDSClient={savedObjectsMDSClient}
     />,
     AppMountParametersProp.element
   );

public/components/trace_analytics/components/dashboard/__tests__/__snapshots__/dashboard.test.tsx.snap

@@ -149,14 +149,6 @@ exports[`Dashboard component renders dashboard 1`] = `
     }
   }
   dataPrepperIndicesExist={true}
-  dataSourceMDSId={
-    Array [
-      Object {
-        "id": "",
-        "label": "",
-      },
-    ]
-  }
   endTime="now"
   filters={Array []}
   http={[MockFunction]}
@@ -487,14 +479,6 @@ exports[`Dashboard component renders dashboard 1`] = `
       }
     }
     dataPrepperIndicesExist={true}
-    dataSourceMDSId={
-      Array [
-        Object {
-          "id": "",
-          "label": "",
-        },
-      ]
-    }
     endTime="now"
     filters={Array []}
     http={[MockFunction]}
@@ -1277,14 +1261,6 @@ exports[`Dashboard component renders empty dashboard 1`] = `
     }
   }
   dataPrepperIndicesExist={true}
-  dataSourceMDSId={
-    Array [
-      Object {
-        "id": "",
-        "label": "",
-      },
-    ]
-  }
   endTime="now"
   filters={Array []}
   http={[MockFunction]}
@@ -1614,14 +1590,6 @@ exports[`Dashboard component renders empty dashboard 1`] = `
       }
     }
     dataPrepperIndicesExist={true}
-    dataSourceMDSId={
-      Array [
-        Object {
-          "id": "",
-          "label": "",
-        },
-      ]
-    }
     endTime="now"
     filters={Array []}
     http={[MockFunction]}
@@ -2404,14 +2372,6 @@ exports[`Dashboard component renders empty jaeger dashboard 1`] = `
     }
   }
   dataPrepperIndicesExist={false}
-  dataSourceMDSId={
-    Array [
-      Object {
-        "id": "",
-        "label": "",
-      },
-    ]
-  }
   endTime="now"
   filters={Array []}
   http={[MockFunction]}
@@ -2743,14 +2703,6 @@ exports[`Dashboard component renders empty jaeger dashboard 1`] = `
       }
     }
     dataPrepperIndicesExist={false}
-    dataSourceMDSId={
-      Array [
-        Object {
-          "id": "",
-          "label": "",
-        },
-      ]
-    }
     endTime="now"
     filters={Array []}
     http={[MockFunction]}

public/components/trace_analytics/components/dashboard/__tests__/dashboard.test.tsx

@@ -7,6 +7,7 @@ import { configure, mount } from 'enzyme';
 import Adapter from 'enzyme-adapter-react-16';
 import React from 'react';
 import { Dashboard } from '..';
+import { CoreStart } from '../../../../../../../../src/core/public';
 import { coreStartMock } from '../../../../../../test/__mocks__/coreMocks';
 
 describe('Dashboard component', () => {
@@ -55,7 +56,6 @@ describe('Dashboard component', () => {
         mode="data_prepper"
         dataPrepperIndicesExist={true}
         modes={modes}
-        dataSourceMDSId={[{ id: '', label: '' }]}
       />
     );
 
@@ -102,7 +102,6 @@ describe('Dashboard component', () => {
         dataPrepperIndicesExist={false}
         jaegerIndicesExist={true}
         modes={modes}
-        dataSourceMDSId={[{ id: '', label: '' }]}
       />
     );
 
@@ -148,7 +147,6 @@ describe('Dashboard component', () => {
         mode="data_prepper"
         dataPrepperIndicesExist={true}
         modes={modes}
-        dataSourceMDSId={[{ id: '', label: '' }]}
       />
     );
 

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

@@ -6,8 +6,7 @@
 import { EuiBreadcrumb, EuiTitle } from '@elastic/eui';
 import { Toast } from '@elastic/eui/src/components/toast/global_toast_list';
 import React from 'react';
-import { DataSourceOption } from '../../../../../../../src/plugins/data_source_management/public/components/data_source_menu/types';
-import { TraceAnalyticsComponentDeps } from '../../home';
+import { TraceAnalyticsComponentDeps, TraceAnalyticsMode } from '../../home';
 import { DashboardContent } from './dashboard_content';
 import { DataSourcePicker } from './mode_picker';
 
@@ -21,7 +20,6 @@ export interface DashboardProps extends TraceAnalyticsComponentDeps {
     text?: React.ReactChild | undefined,
     side?: string | undefined
   ) => void;
-  dataSourceMDSId: DataSourceOption[];
 }
 
 export function Dashboard(props: DashboardProps) {