[ML] Adding permission checks to jobs list (#20487)

* [ML] Adding permission checks to jobs list

* adding comment

* removing horizontal line

* changes based on review
This commit is contained in:
James Gowdy 2018-07-06 08:57:54 +01:00 committed by GitHub
parent aa0c6df8cf
commit c6ba696127
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 114 additions and 93 deletions

View file

@ -4,103 +4,73 @@
* you may not use this file except in compliance with the Elastic License.
*/
import React from 'react';
import {
EuiText,
EuiIcon,
} from '@elastic/eui';
import { checkPermission } from 'plugins/ml/privilege/check_privilege';
import { mlNodesAvailable } from 'plugins/ml/ml_nodes_check/check_ml_nodes';
import {
stopDatafeeds,
cloneJob } from '../utils';
cloneJob,
isStartable,
isStoppable,
} from '../utils';
export function actionsMenuContent(showEditJobFlyout, showDeleteJobModal, showStartDatafeedModal, refreshJobs) {
return [{
render: (item) => {
return (
getStartStop(item, showStartDatafeedModal, refreshJobs)
);
}
}, {
render: (item) => {
return (
<MLText onClick={() => cloneJob(item.id)}>
<EuiIcon type="copy" />
Clone job
</MLText>
);
}
}, {
render: (item) => {
return (
<MLText onClick={() => {
showEditJobFlyout(item);
closeMenu();
}}
>
<EuiIcon type="copy" />
Edit job
</MLText>
);
}
}, {
render: (item) => {
return (
<MLText
color="danger"
onClick={() => {
showDeleteJobModal([item]);
closeMenu();
}}
>
<EuiIcon type="trash" />
Delete job
</MLText>
);
}
}];
}
const canCreateJob = (checkPermission('canCreateJob') && mlNodesAvailable());
const canUpdateJob = checkPermission('canUpdateJob');
const canDeleteJob = checkPermission('canDeleteJob');
const canUpdateDatafeed = checkPermission('canUpdateDatafeed');
const canStartStopDatafeed = (checkPermission('canStartStopDatafeed') && mlNodesAvailable());
function getStartStop(item, showStartDatafeedModal, refreshJobs) {
if (item.datafeedState === 'stopped') {
return (
<MLText onClick={() => {
return [
{
name: 'Start datafeed',
description: 'Start datafeed',
icon: 'play',
enabled: () => (canStartStopDatafeed),
available: (item) => (isStartable([item])),
onClick: (item) => {
showStartDatafeedModal([item]);
closeMenu();
}}
>
<EuiIcon type="play" />
Start datafeed
</MLText>
);
}
if (item.datafeedState === 'started') {
return (
<MLText onClick={() => {
}
}, {
name: 'Stop datafeed',
description: 'Stop datafeed',
icon: 'stop',
enabled: () => (canStartStopDatafeed),
available: (item) => (isStoppable([item])),
onClick: (item) => {
stopDatafeeds([item], refreshJobs);
closeMenu(true);
}}
>
<EuiIcon type="stop" />
Stop datafeed
</MLText>
);
}
}
// EuiText wrapper to stop event propagation so the menu items don't fire twice.
// this appears to be an issue with the EuiBasicTable actions col when specifying
// a custom render function which contains its own onClick
function MLText({ onClick, ...rest }) {
const click = (e) => {
e.stopPropagation();
onClick();
};
return <EuiText onClick={click} {...rest} />;
}
}, {
name: 'Clone job',
description: 'Clone job',
icon: 'copy',
enabled: () => (canCreateJob),
onClick: (item) => {
cloneJob(item.id);
}
}, {
name: 'Edit job',
description: 'Edit job',
icon: 'copy',
enabled: () => (canUpdateJob && canUpdateDatafeed),
onClick: (item) => {
showEditJobFlyout(item);
closeMenu();
}
}, {
name: 'Delete job',
description: 'Delete job',
icon: 'trash',
color: 'danger',
enabled: () => (canDeleteJob),
onClick: (item) => {
showDeleteJobModal([item]);
closeMenu();
}
}
];
}
function closeMenu(now = false) {

View file

@ -66,10 +66,16 @@
opacity: 1;
}
.euiContextMenuPanel .euiContextMenuItem {
white-space: nowrap;
.euiContextMenuPanel {
.euiContextMenuItem {
white-space: nowrap;
}
.euiContextMenuItem:last-child:not(.euiContextMenuItem-isDisabled) {
color: #A30000
}
}
.euiTableCellContent .euiLink {
margin-right: 10px;
}

View file

@ -5,6 +5,8 @@
*/
import { checkPermission } from 'plugins/ml/privilege/check_privilege';
import { mlNodesAvailable } from 'plugins/ml/ml_nodes_check/check_ml_nodes';
import PropTypes from 'prop-types';
import React, {
Component,
@ -29,6 +31,9 @@ export class MultiJobActionsMenu extends Component {
this.state = {
isOpen: false,
};
this.canDeleteJob = checkPermission('canDeleteJob');
this.canStartStopDatafeed = (checkPermission('canStartStopDatafeed') && mlNodesAvailable());
}
onButtonClick = () => {
@ -51,6 +56,7 @@ export class MultiJobActionsMenu extends Component {
iconType="gear"
aria-label="Next"
color="text"
disabled={(this.canDeleteJob === false && this.canStartStopDatafeed === false)}
/>
);
@ -60,6 +66,7 @@ export class MultiJobActionsMenu extends Component {
<EuiContextMenuItem
key="delete"
icon="trash"
disabled={(this.canDeleteJob === false)}
onClick={() => { this.props.showDeleteJobModal(this.props.jobs); this.closePopover(); }}
>
Delete job{s}
@ -72,6 +79,7 @@ export class MultiJobActionsMenu extends Component {
<EuiContextMenuItem
key="stop datafeed"
icon="stop"
disabled={(this.canStartStopDatafeed === false)}
onClick={() => { stopDatafeeds(this.props.jobs, this.props.refreshJobs); this.closePopover(); }}
>
Stop datafeed{s}
@ -84,6 +92,7 @@ export class MultiJobActionsMenu extends Component {
<EuiContextMenuItem
key="start datafeed"
icon="play"
disabled={(this.canStartStopDatafeed === false)}
onClick={() => { this.props.showStartDatafeedModal(this.props.jobs); this.closePopover(); }}
>
Start datafeed{s}

View file

@ -38,7 +38,7 @@
white-space: nowrap;
}
.euiContextMenuItem:last-child {
.euiContextMenuItem:last-child:not(.euiContextMenuItem-isDisabled) {
color: #A30000
}
}

View file

@ -6,6 +6,9 @@
import { checkPermission } from 'plugins/ml/privilege/check_privilege';
import { mlNodesAvailable } from 'plugins/ml/ml_nodes_check/check_ml_nodes';
import React from 'react';
import './styles/main.less';
@ -18,10 +21,12 @@ function newJob() {
}
export function NewJobButton() {
const buttonEnabled = (checkPermission('canCreateJob') && mlNodesAvailable());
return (
<EuiButton
onClick={newJob}
size="s"
disabled={(buttonEnabled === false)}
fill
>
<i className="new-job-button fa fa-plus" />

View file

@ -8,19 +8,49 @@
import './styles/main.less';
import { NewJobButton } from './components/new_job_button';
import { JobsListView } from './components/jobs_list_view';
import { mlNodesAvailable, permissionToViewMlNodeCount } from 'plugins/ml/ml_nodes_check/check_ml_nodes';
import React, {
Component
} from 'react';
import {
EuiHorizontalRule,
EuiCallOut,
EuiLink,
EuiSpacer,
} from '@elastic/eui';
function NodeAvailableWarning() {
const isCloud = false; // placeholder for future specific cloud functionality
if ((mlNodesAvailable() === true) || (permissionToViewMlNodeCount() === false)) {
return (<span />);
} else {
return (
<React.Fragment>
<EuiCallOut
title="No ML nodes available"
color="warning"
iconType="alert"
>
<p>
There are no ML nodes available.<br />
You will not be able to create or run jobs.
{isCloud &&
<span ng-if="isCloud">
&nbsp;This can be configured in Cloud <EuiLink href="#">here</EuiLink>.
</span>
}
</p>
</EuiCallOut>
<EuiSpacer size="m" />
</React.Fragment>
);
}
}
export class JobsPage extends Component {
constructor(props) {
super(props);
this.state = {
};
}
@ -28,6 +58,7 @@ export class JobsPage extends Component {
render() {
return (
<div className="job-management">
<NodeAvailableWarning />
<header>
<div className="new-job-button-container">
<NewJobButton />
@ -36,7 +67,7 @@ export class JobsPage extends Component {
<div className="clear" />
<EuiHorizontalRule margin="m" />
<EuiSpacer size="s" />
<JobsListView />
</div>