support multi nodes operations
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-
node-get-console
node-get-
node-set-
node-set-
node-set-
node-set-
node-set-
node-show-states
node-update
node-validate
node-vendor-
An example of node-set-
Without this bp, current behavior:
ironic node-set-
With this bp:
ironic node-set-
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
- Completed by
Whiteboard
Add bulk operations with multi-node-ids as input for node-set-
https:/
Add bulk operations with multi-node-ids as input for node-get-
Add bulk operations with multi-node-ids as input for node-get-
Add bulk operations with multi-node-ids as input for node-set-
=======
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-
node-get-console
node-get-
node-set-
node-set-
node-set-
node-set-
node-set-
node-show-states
node-update
node-validate
node-vendor-
An example of node-set-
Without this bp, current behavior:
ironic node-set-
With this bp:
ironic node-set-
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:/
Thanks
Haomeng 2015-11-04
=======
Thanks Jim
Haomeng 2015-11-18
