Open Stack

Hi,

> I am working on extending the current set of patches that implement
> the OSC plugin for Ironic. I would like some discussion/guidance about
> a couple of command structures.
>
> Currently provisioning state is set via 'openstack baremetal set
> --provision-state [active|deleted|rebuild|inspect|provide|manage]
> $NODE'
>
> dtantsur suggests it be top-level a command (which I concur)
> 'openstack baremetal [active|delete|rebuild|inspect|provide|manage] $NODE'
>
> Question there is does that make sense?
>

I'm good with both ways.

> Secondly, maintenance mode makes sense to be part of 'openstack baremetal
> set' command, but implementation would be easier and less error-prone
> if an antonym for maintenance were used as a flag.
>
> * openstack baremetal set --maintenance --reason $REASON $NODE
> * openstack baremetal set --maintenance-antonym $NODE
>

Maybe:

* openstack baremetal set --maintenance --reason $REASON $NODE
* openstack baremetal unset --maintenance $NODE

("unset" instead of "set" for removing a node from maintenance)

Cheers,
Lucas