Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Add 'Start Maintenance' action to Host #301

Merged
merged 2 commits into from
Apr 15, 2019
Merged

Add 'Start Maintenance' action to Host #301

merged 2 commits into from
Apr 15, 2019

Conversation

jtomasek
Copy link

@jtomasek jtomasek commented Apr 12, 2019

Depends on #274
Depends on kubevirt/web-ui-components#355

@kubevirt-bot kubevirt-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L labels Apr 12, 2019
@jtomasek jtomasek changed the title WIP Add 'Start Maintenance' action to Host Add 'Start Maintenance' action to Host Apr 15, 2019
@kubevirt-bot kubevirt-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 15, 2019
@@ -56,12 +58,11 @@ const HostHeader = props => (
const HostRow = ({ obj: host }) => {
const {
metadata: { name, namespace, uid },

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can also use selectors for these.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

return {
hidden: !canHostStopMaintenance(node),
label: 'Stop Maintenance',
callback: () => makeNodeSchedulable(node),

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't there be a yes/no dialog?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@atiratree atiratree merged commit 86f0732 into kubevirt:master Apr 15, 2019
@jtomasek jtomasek deleted the maintenance_action branch April 15, 2019 15:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants