support multi nodes operations

Registered by Haomeng,Wang

so far, we just support multi node operations with 'ironic node-delete' command, in this blueprint, will cover more action to support multi node operations which should be easy for user to handle multi nodes with single commands.

And this is not BIG change, so think the spec is not required.

The list of commands which will be address in this bp:

node-get-boot-device
node-get-console
node-get-supported-boot-devices
node-set-boot-device
node-set-console-mode
node-set-maintenance
node-set-power-state
node-set-provision-state
node-show-states
node-update
node-validate
node-vendor-passthru

An example of node-set-power-state command changes:

Without this bp, current behavior:
ironic node-set-power-state <node> <power-state>

With this bp:
ironic node-set-power-state <node> [<node> ...] <power-state>

And also, the related osc-plugin code which is handing above commands for ironic client will be synced with these new option changes.

Blueprint information

Status:
Started
Approver:
Jim Rollenhagen
Priority:
Low
Drafter:
Haomeng,Wang
Direction:
Approved
Assignee:
Haomeng,Wang
Definition:
Approved
Series goal:
None
Implementation:
Started
Milestone target:
None
Started by
Haomeng,Wang

Related branches

Sprints

Whiteboard

Add bulk operations with multi-node-ids as input for node-set-power-state command
https://review.openstack.org/#/c/228301/

Add bulk operations with multi-node-ids as input for node-get-boot-device command
Add bulk operations with multi-node-ids as input for node-get-supported-boot-devices command
Add bulk operations with multi-node-ids as input for node-set-console-mode command

======================================================

Haomeng, I do agree a spec is not required. But, could you add more info here? I'd like to see:
* A list of commands this will be changed for
* An example for each command

======================================================

Yes, Jim, agree with you, added these info as below:

The list of commands which will be address in this bp:

node-get-boot-device
node-get-console
node-get-supported-boot-devices
node-set-boot-device
node-set-console-mode
node-set-maintenance
node-set-power-state
node-set-provision-state
node-show-states
node-update
node-validate
node-vendor-passthru

An example of node-set-power-state command changes:

Without this bp, current behavior:
ironic node-set-power-state <node> <power-state>

With this bp:
ironic node-set-power-state <node> [<node> ...] <power-state>

So let me know if you have more concern, thanks.

==========================================================

Haomeng, why not also commands like node-update?
// jroll 2015-10-16

Jim, good catch, I just consider for these 'actions' operations, yes, for node-update, we can support multi-nodeid as input, but they will changed to the SAME property value for all these input nodes, that should be useful for some cases which update all to same value.
So I added it into the command list. Thanks for your good comments.

// haomeng 2015-10-17

Jim, updated the command list, have added all commands which has node_id as input which all can be covered by this bp to support multi-node-id input, so can you help to review again and approval, and I will go ahead to commit the patches.

// haomeng 2015-10-21

==========================================================

Hi Lucas / Dmitry,

Based on IRC chat with Jim, he suggest to get your comments, so add you as subscribers, if you have time, would you please help to take this simple bp a look and welcome your comments?

[19:24:22] <jroll> Haomeng: let me check with other folks on that blueprint, but it looks fine to me
[19:24:40] <jroll> dtantsur|afk: lucas-afk: when you're around, y'all good with approving https://blueprints.launchpad.net/ironic/+spec/multi-nodes-operations ?

Thanks
Haomeng 2015-11-04

===========================================================

Thanks Jim
Haomeng 2015-11-18

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.