Skip to content

Commit 383be7d

Browse files
committed
[ML] Server info service refactor (elastic#50302)
* [ML] Server info service refactor * removing new job defaults * changes based on review * renaming all ml server info getter functions * missed a file
1 parent 86f0c17 commit 383be7d

File tree

18 files changed

+162
-109
lines changed

18 files changed

+162
-109
lines changed

x-pack/legacy/plugins/ml/public/datavisualizer/file_based/file_datavisualizer_directive.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import { getFileDataVisualizerBreadcrumbs } from './breadcrumbs';
1616
import { checkBasicLicense } from '../../license/check_license';
1717
import { checkFindFileStructurePrivilege } from '../../privilege/check_privilege';
1818
import { getMlNodeCount } from '../../ml_nodes_check/check_ml_nodes';
19-
import { loadNewJobDefaults } from '../../jobs/new_job_new/utils/new_job_defaults';
19+
import { loadMlServerInfo } from '../../services/ml_server_info';
2020
import { loadIndexPatterns } from '../../util/index_utils';
2121
import { FileDataVisualizerPage } from './file_datavisualizer';
2222

@@ -36,7 +36,7 @@ uiRoutes
3636
privileges: checkFindFileStructurePrivilege,
3737
indexPatterns: loadIndexPatterns,
3838
mlNodeCount: getMlNodeCount,
39-
loadNewJobDefaults,
39+
loadMlServerInfo,
4040
}
4141
});
4242

x-pack/legacy/plugins/ml/public/jobs/jobs_list/components/edit_job_flyout/edit_utils.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
import { difference } from 'lodash';
99
import chrome from 'ui/chrome';
10-
import { newJobLimits } from 'plugins/ml/jobs/new_job_new/utils/new_job_defaults';
10+
import { getNewJobLimits } from '../../../../services/ml_server_info';
1111
import { mlJobService } from 'plugins/ml/services/job_service';
1212
import { processCreatedBy } from '../../../../../common/util/job_utils';
1313

@@ -157,7 +157,7 @@ function extractGroups(job, newJobData) {
157157
}
158158

159159
function extractMML(job, newJobData) {
160-
const jobLimits = newJobLimits();
160+
const jobLimits = getNewJobLimits();
161161
const mmlData = {};
162162
// if the job's model_memory_limit has changed, add it to the jobData json
163163
if (job.analysis_limits.model_memory_limit !== undefined) {

x-pack/legacy/plugins/ml/public/jobs/jobs_list/components/edit_job_flyout/tabs/datafeed.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ import {
1919
} from '@elastic/eui';
2020

2121
import { calculateDatafeedFrequencyDefaultSeconds } from 'plugins/ml/../common/util/job_utils';
22-
import { newJobDefaults } from 'plugins/ml/jobs/new_job_new/utils/new_job_defaults';
22+
import { getNewJobDefaults } from '../../../../../services/ml_server_info';
2323
import { parseInterval } from 'plugins/ml/../common/util/parse_interval';
2424
import { MLJobEditor } from '../../ml_job_editor';
2525
import { FormattedMessage } from '@kbn/i18n/react';
@@ -47,7 +47,7 @@ export class Datafeed extends Component {
4747
frequency: '',
4848
scrollSize: 0,
4949
},
50-
jobDefaults: newJobDefaults()
50+
jobDefaults: getNewJobDefaults()
5151
};
5252

5353
this.setDatafeed = props.setDatafeed;

x-pack/legacy/plugins/ml/public/jobs/jobs_list/components/node_available_warning/node_available_warning.tsx

Lines changed: 48 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -9,58 +9,58 @@ import React, { Fragment, FC } from 'react';
99
import { EuiCallOut, EuiLink, EuiSpacer } from '@elastic/eui';
1010
import { FormattedMessage } from '@kbn/i18n/react';
1111
import { mlNodesAvailable, permissionToViewMlNodeCount } from '../../../../ml_nodes_check';
12-
import { cloudDeploymentId, isCloud } from '../../../../jobs/new_job_new/utils/new_job_defaults';
12+
import { getCloudDeploymentId, isCloud } from '../../../../services/ml_server_info';
1313

1414
export const NodeAvailableWarning: FC = () => {
1515
if (mlNodesAvailable() === true || permissionToViewMlNodeCount() === false) {
1616
return null;
17-
} else {
18-
const id = cloudDeploymentId();
19-
return (
20-
<Fragment>
21-
<EuiCallOut
22-
title={
23-
<FormattedMessage
24-
id="xpack.ml.jobsList.nodeAvailableWarning.noMLNodesAvailableTitle"
25-
defaultMessage="No ML nodes available"
26-
/>
27-
}
28-
color="warning"
29-
iconType="alert"
30-
>
31-
<p>
32-
<FormattedMessage
33-
id="xpack.ml.jobsList.nodeAvailableWarning.noMLNodesAvailableDescription"
34-
defaultMessage="There are no ML nodes available."
35-
/>
36-
<br />
17+
}
18+
19+
const id = getCloudDeploymentId();
20+
return (
21+
<Fragment>
22+
<EuiCallOut
23+
title={
24+
<FormattedMessage
25+
id="xpack.ml.jobsList.nodeAvailableWarning.noMLNodesAvailableTitle"
26+
defaultMessage="No ML nodes available"
27+
/>
28+
}
29+
color="warning"
30+
iconType="alert"
31+
>
32+
<div>
33+
<FormattedMessage
34+
id="xpack.ml.jobsList.nodeAvailableWarning.noMLNodesAvailableDescription"
35+
defaultMessage="There are no ML nodes available."
36+
/>
37+
</div>
38+
<div>
39+
<FormattedMessage
40+
id="xpack.ml.jobsList.nodeAvailableWarning.unavailableCreateOrRunJobsDescription"
41+
defaultMessage="You will not be able to create or run jobs."
42+
/>
43+
</div>
44+
{isCloud && id !== null && (
45+
<div>
3746
<FormattedMessage
38-
id="xpack.ml.jobsList.nodeAvailableWarning.unavailableCreateOrRunJobsDescription"
39-
defaultMessage="You will not be able to create or run jobs."
47+
id="xpack.ml.jobsList.nodeAvailableWarning.linkToCloudDescription"
48+
defaultMessage="Please edit your {link}. You may enable a free 1GB machine learning node or expand your existing ML configuration."
49+
values={{
50+
link: (
51+
<EuiLink href={`https://cloud.elastic.co/deployments?q=${id}`}>
52+
<FormattedMessage
53+
id="xpack.ml.jobsList.nodeAvailableWarning.linkToCloud.hereLinkText"
54+
defaultMessage="Elastic Cloud deployment"
55+
/>
56+
</EuiLink>
57+
),
58+
}}
4059
/>
41-
{isCloud && id !== null && (
42-
<Fragment>
43-
<br />
44-
<FormattedMessage
45-
id="xpack.ml.jobsList.nodeAvailableWarning.linkToCloudDescription"
46-
defaultMessage="Please edit your {link}. You may enable a free 1GB machine learning node or expand your existing ML configuration."
47-
values={{
48-
link: (
49-
<EuiLink href={`https://cloud.elastic.co/deployments?q=${id}`}>
50-
<FormattedMessage
51-
id="xpack.ml.jobsList.nodeAvailableWarning.linkToCloud.hereLinkText"
52-
defaultMessage="Elastic Cloud deployment"
53-
/>
54-
</EuiLink>
55-
),
56-
}}
57-
/>
58-
</Fragment>
59-
)}
60-
</p>
61-
</EuiCallOut>
62-
<EuiSpacer size="m" />
63-
</Fragment>
64-
);
65-
}
60+
</div>
61+
)}
62+
</EuiCallOut>
63+
<EuiSpacer size="m" />
64+
</Fragment>
65+
);
6666
};

x-pack/legacy/plugins/ml/public/jobs/jobs_list/components/validate_job.js

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

77

8-
import { newJobLimits } from '../../new_job_new/utils/new_job_defaults';
8+
import { getNewJobLimits } from '../../../services/ml_server_info';
99
import { populateValidationMessages } from '../../new_job_new/common/job_validator/util';
1010

1111
import {
@@ -16,7 +16,7 @@ import {
1616
import { isValidLabel, isValidTimeRange } from '../../../util/custom_url_utils';
1717

1818
export function validateModelMemoryLimit(mml) {
19-
const limits = newJobLimits();
19+
const limits = getNewJobLimits();
2020
const tempJob = {
2121
analysis_limits: {
2222
model_memory_limit: mml

x-pack/legacy/plugins/ml/public/jobs/jobs_list/directive.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import { checkFullLicense } from 'plugins/ml/license/check_license';
1616
import { checkGetJobsPrivilege } from 'plugins/ml/privilege/check_privilege';
1717
import { getMlNodeCount } from 'plugins/ml/ml_nodes_check/check_ml_nodes';
1818
import { getJobManagementBreadcrumbs } from 'plugins/ml/jobs/breadcrumbs';
19-
import { loadNewJobDefaults } from 'plugins/ml/jobs/new_job_new/utils/new_job_defaults';
19+
import { loadMlServerInfo } from 'plugins/ml/services/ml_server_info';
2020

2121
import uiRoutes from 'ui/routes';
2222

@@ -31,7 +31,7 @@ uiRoutes
3131
indexPatterns: loadIndexPatterns,
3232
privileges: checkGetJobsPrivilege,
3333
mlNodeCount: getMlNodeCount,
34-
loadNewJobDefaults,
34+
loadMlServerInfo,
3535
}
3636
});
3737

x-pack/legacy/plugins/ml/public/jobs/new_job_new/common/job_validator/job_validator.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
import { ReactElement } from 'react';
88
import { basicJobValidation, basicDatafeedValidation } from '../../../../../common/util/job_utils';
9-
import { newJobLimits } from '../../../new_job_new/utils/new_job_defaults';
9+
import { getNewJobLimits } from '../../../../services/ml_server_info';
1010
import { JobCreatorType } from '../job_creator';
1111
import { populateValidationMessages, checkForExistingJobAndGroupIds } from './util';
1212
import { ExistingJobsAndGroups } from '../../../../services/job_service';
@@ -111,7 +111,7 @@ export class JobValidator {
111111

112112
const jobConfig = this._jobCreator.jobConfig;
113113
const datafeedConfig = this._jobCreator.datafeedConfig;
114-
const limits = newJobLimits();
114+
const limits = getNewJobLimits();
115115

116116
// run standard basic validation
117117
const basicJobResults = basicJobValidation(jobConfig, undefined, limits);

x-pack/legacy/plugins/ml/public/jobs/new_job_new/common/job_validator/util.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { i18n } from '@kbn/i18n';
88
import { BasicValidations } from './job_validator';
99
import { Job, Datafeed } from '../job_creator/configs';
1010
import { ALLOWED_DATA_UNITS, JOB_ID_MAX_LENGTH } from '../../../../../common/constants/validation';
11-
import { newJobLimits } from '../../../new_job_new/utils/new_job_defaults';
11+
import { getNewJobLimits } from '../../../../services/ml_server_info';
1212
import { ValidationResults, ValidationMessage } from '../../../../../common/util/job_utils';
1313
import { ExistingJobsAndGroups } from '../../../../services/job_service';
1414

@@ -18,7 +18,7 @@ export function populateValidationMessages(
1818
jobConfig: Job,
1919
datafeedConfig: Datafeed
2020
) {
21-
const limits = newJobLimits();
21+
const limits = getNewJobLimits();
2222

2323
if (validationResults.contains('job_id_empty')) {
2424
basicValidations.jobId.valid = false;

x-pack/legacy/plugins/ml/public/jobs/new_job_new/pages/components/common/model_memory_limit/model_memory_limit_input.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
import React, { FC, useState, useContext, useEffect } from 'react';
88
import { EuiFieldText } from '@elastic/eui';
9-
import { newJobDefaults } from '../../../../../new_job_new/utils/new_job_defaults';
9+
import { getNewJobDefaults } from '../../../../../../services/ml_server_info';
1010
import { JobCreatorContext } from '../../job_creator_context';
1111
import { Description } from './description';
1212

@@ -23,7 +23,7 @@ export const ModelMemoryLimitInput: FC = () => {
2323
jobCreator.modelMemoryLimit === null ? '' : jobCreator.modelMemoryLimit
2424
);
2525

26-
const { anomaly_detectors: anomalyDetectors } = newJobDefaults();
26+
const { anomaly_detectors: anomalyDetectors } = getNewJobDefaults();
2727
const { model_memory_limit: modelMemoryLimitDefault } = anomalyDetectors;
2828

2929
useEffect(() => {

x-pack/legacy/plugins/ml/public/jobs/new_job_new/pages/components/datafeed_step/components/scroll_size/scroll_size_input.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
import React, { FC, useState, useContext, useEffect } from 'react';
88
import { EuiFieldNumber } from '@elastic/eui';
9-
import { newJobDefaults } from '../../../../../utils/new_job_defaults';
9+
import { getNewJobDefaults } from '../../../../../../../services/ml_server_info';
1010
import { JobCreatorContext } from '../../../job_creator_context';
1111
import { Description } from './description';
1212

@@ -19,7 +19,7 @@ export const ScrollSizeInput: FC = () => {
1919
jobCreator.scrollSize === null ? '' : `${jobCreator.scrollSize}`
2020
);
2121

22-
const { datafeeds } = newJobDefaults();
22+
const { datafeeds } = getNewJobDefaults();
2323
const scrollSizeDefault = datafeeds.scroll_size !== undefined ? `${datafeeds.scroll_size}` : '';
2424

2525
useEffect(() => {

0 commit comments

Comments
 (0)