[ML] Adding ML node warning to overview and analytics pages (#50766)

elastic · Nov 17, 2019 · 429c28d · 429c28d
1 parent 4c33f2a
commit 429c28d
Show file tree

Hide file tree

Showing 10 changed files with 33 additions and 16 deletions.
diff --git a/...omponents/node_available_warning/index.ts → ...omponents/node_available_warning/index.ts b/...omponents/node_available_warning/index.ts → ...omponents/node_available_warning/index.ts
diff --git a/...ilable_warning/node_available_warning.tsx → ...ilable_warning/node_available_warning.tsx b/...ilable_warning/node_available_warning.tsx → ...ilable_warning/node_available_warning.tsx
@@ -8,8 +8,8 @@ import React, { Fragment, FC } from 'react';
 
 import { EuiCallOut, EuiLink, EuiSpacer } from '@elastic/eui';
 import { FormattedMessage } from '@kbn/i18n/react';
-import { mlNodesAvailable, permissionToViewMlNodeCount } from '../../../../ml_nodes_check';
-import { getCloudDeploymentId, isCloud } from '../../../../services/ml_server_info';
+import { mlNodesAvailable, permissionToViewMlNodeCount } from '../../ml_nodes_check';
+import { getCloudDeploymentId, isCloud } from '../../services/ml_server_info';
 
 export const NodeAvailableWarning: FC = () => {
   if (mlNodesAvailable() === true || permissionToViewMlNodeCount() === false) {

diff --git a/...ins/ml/public/components/upgrade/index.js → ...ins/ml/public/components/upgrade/index.ts b/...ins/ml/public/components/upgrade/index.js → ...ins/ml/public/components/upgrade/index.ts
@@ -4,5 +4,4 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-
 export { UpgradeWarning } from './upgrade_warning';
diff --git a/...lic/components/upgrade/upgrade_warning.js → ...ic/components/upgrade/upgrade_warning.tsx b/...lic/components/upgrade/upgrade_warning.js → ...ic/components/upgrade/upgrade_warning.tsx
@@ -4,26 +4,24 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
+import React, { FC } from 'react';
 
-import React from 'react';
-
-import {
-  EuiCallOut,
-  EuiSpacer,
-} from '@elastic/eui';
+import { EuiCallOut, EuiSpacer } from '@elastic/eui';
 import { FormattedMessage } from '@kbn/i18n/react';
 
 import { isUpgradeInProgress } from '../../services/upgrade_service';
 
-export function UpgradeWarning() {
+export const UpgradeWarning: FC = () => {
   if (isUpgradeInProgress() === true) {
     return (
       <React.Fragment>
         <EuiCallOut
-          title={(<FormattedMessage
-            id="xpack.ml.upgrade.upgradeWarning.upgradeInProgressWarningTitle"
-            defaultMessage="Index migration in progress"
-          />)}
+          title={
+            <FormattedMessage
+              id="xpack.ml.upgrade.upgradeWarning.upgradeInProgressWarningTitle"
+              defaultMessage="Index migration in progress"
+            />
+          }
           color="warning"
           iconType="alert"
         >
@@ -45,4 +43,4 @@ export function UpgradeWarning() {
   }
 
   return null;
-}
+};
diff --git a/.../data_frame_analytics/pages/analytics_management/hooks/use_create_analytics_form/state.ts b/.../data_frame_analytics/pages/analytics_management/hooks/use_create_analytics_form/state.ts
@@ -6,6 +6,7 @@
 
 import { DeepPartial } from '../../../../../../common/types/common';
 import { checkPermission } from '../../../../../privilege/check_privilege';
+import { mlNodesAvailable } from '../../../../../ml_nodes_check/check_ml_nodes';
 
 import { DataFrameAnalyticsId, DataFrameAnalyticsConfig } from '../../../../common';
 
@@ -102,6 +103,7 @@ export const getInitialState = (): State => ({
   },
   jobConfig: {},
   disabled:
+    !mlNodesAvailable() ||
     !checkPermission('canCreateDataFrameAnalytics') ||
     !checkPermission('canStartStopDataFrameAnalytics'),
   indexNames: [],

diff --git a/x-pack/legacy/plugins/ml/public/data_frame_analytics/pages/analytics_management/page.tsx b/x-pack/legacy/plugins/ml/public/data_frame_analytics/pages/analytics_management/page.tsx
@@ -22,6 +22,8 @@ import { NavigationMenu } from '../../../components/navigation_menu';
 import { DataFrameAnalyticsList } from './components/analytics_list';
 import { useRefreshInterval } from './components/analytics_list/use_refresh_interval';
 import { useCreateAnalyticsForm } from './hooks/use_create_analytics_form';
+import { NodeAvailableWarning } from '../../../components/node_available_warning';
+import { UpgradeWarning } from '../../../components/upgrade';
 
 export const Page: FC = () => {
   const [blockRefresh, setBlockRefresh] = useState(false);
@@ -62,6 +64,10 @@ export const Page: FC = () => {
               </EuiTitle>
             </EuiPageHeaderSection>
           </EuiPageHeader>
+
+          <NodeAvailableWarning />
+          <UpgradeWarning />
+
           <DataFrameAnalyticsList
             blockRefresh={blockRefresh}
             createAnalyticsForm={createAnalyticsForm}

diff --git a/x-pack/legacy/plugins/ml/public/data_frame_analytics/pages/analytics_management/route.ts b/x-pack/legacy/plugins/ml/public/data_frame_analytics/pages/analytics_management/route.ts
@@ -9,6 +9,8 @@ import uiRoutes from 'ui/routes';
 // @ts-ignore
 import { checkFullLicense } from '../../../license/check_license';
 import { checkGetJobsPrivilege } from '../../../privilege/check_privilege';
+import { loadMlServerInfo } from '../../../services/ml_server_info';
+import { getMlNodeCount } from '../../../ml_nodes_check/check_ml_nodes';
 import {
   loadCurrentIndexPattern,
   loadCurrentSavedSearch,
@@ -26,5 +28,7 @@ uiRoutes.when('/data_frame_analytics/?', {
     privileges: checkGetJobsPrivilege,
     indexPattern: loadCurrentIndexPattern,
     savedSearch: loadCurrentSavedSearch,
+    mlNodeCount: getMlNodeCount,
+    loadMlServerInfo,
   },
 });
diff --git a/x-pack/legacy/plugins/ml/public/jobs/jobs_list/components/jobs_list_view/jobs_list_view.js b/x-pack/legacy/plugins/ml/public/jobs/jobs_list/components/jobs_list_view/jobs_list_view.js
@@ -21,7 +21,7 @@ import { CreateWatchFlyout } from '../create_watch_flyout';
 import { MultiJobActions } from '../multi_job_actions';
 import { NewJobButton } from '../new_job_button';
 import { JobStatsBar } from '../jobs_stats_bar';
-import { NodeAvailableWarning } from '../node_available_warning';
+import { NodeAvailableWarning } from '../../../../components/node_available_warning';
 import { UpgradeWarning } from '../../../../components/upgrade';
 import { RefreshJobsListButton } from '../refresh_jobs_list_button';
 import { isEqual } from 'lodash';

diff --git a/x-pack/legacy/plugins/ml/public/overview/overview_page.tsx b/x-pack/legacy/plugins/ml/public/overview/overview_page.tsx
@@ -11,17 +11,23 @@ import { mlNodesAvailable } from '../ml_nodes_check/check_ml_nodes';
 import { NavigationMenu } from '../components/navigation_menu';
 import { OverviewSideBar } from './components/sidebar';
 import { OverviewContent } from './components/content';
+import { NodeAvailableWarning } from '../components/node_available_warning';
+import { UpgradeWarning } from '../components/upgrade';
 
 export const OverviewPage: FC = () => {
   const disableCreateAnomalyDetectionJob = !checkPermission('canCreateJob') || !mlNodesAvailable();
   const disableCreateAnalyticsButton =
+    !mlNodesAvailable() ||
     !checkPermission('canCreateDataFrameAnalytics') ||
     !checkPermission('canStartStopDataFrameAnalytics');
   return (
     <Fragment>
       <NavigationMenu tabId="overview" />
       <EuiPage data-test-subj="mlPageOverview">
         <EuiPageBody>
+          <NodeAvailableWarning />
+          <UpgradeWarning />
+
           <EuiFlexGroup>
             <OverviewSideBar createAnomalyDetectionJobDisabled={disableCreateAnomalyDetectionJob} />
             <OverviewContent

diff --git a/x-pack/legacy/plugins/ml/public/overview/route.ts b/x-pack/legacy/plugins/ml/public/overview/route.ts
@@ -9,6 +9,7 @@ import { getMlNodeCount } from '../ml_nodes_check/check_ml_nodes';
 // @ts-ignore no declaration module
 import { checkFullLicense } from '../license/check_license';
 import { checkGetJobsPrivilege } from '../privilege/check_privilege';
+import { loadMlServerInfo } from '../services/ml_server_info';
 import { getOverviewBreadcrumbs } from './breadcrumbs';
 import './directive';
 
@@ -21,5 +22,6 @@ uiRoutes.when('/overview/?', {
     CheckLicense: checkFullLicense,
     privileges: checkGetJobsPrivilege,
     mlNodeCount: getMlNodeCount,
+    loadMlServerInfo,
   },
 });