Code review comment for lp:~ltrager/maas/power_for_rackcontrollers

Revision history for this message
Lee Trager (ltrager) wrote :

Since all the power commands were originally on the node API endpoint I figured I'd add them back there now that two different node types support power options. This will expand on the concept that the node API is used for general options and be one less thing for people migrating from the 1.0 API to worry about. I disabled them on devices and region controllers as you pointed out since we currently don't support power options on those node types. I could do the mixin thing you are suggestion but then the node API endpoint won't have these options.

If we want to have the power options on the node endpoint we'll have to disable the power commands as I currently do. If we want the power operations only on the machine and rack controller endpoints I can do the mixin as you suggest.

« Back to merge proposal