Open Stack

On Mon, Sep 12, 2016 at 12:38 PM, Dean Troyer <dtroyer at gmail.com> wrote:

> On Mon, Sep 12, 2016 at 10:51 AM, Ruby Loo <opensrloo at gmail.com> wrote:
>
>> My preference is:
>>
>>     * openstack baremetal driver show --raid-logical-disk-properties
>>
>
> Agreed here.
>
> The process we encourage is always to first identify the resource you are
> operating on.  I would add that a 'property' is not a resource, it
> describes a resource, there may be many of them per resource, etc.  Here,
> the resource is 'baremetal driver'.
>
> because we already have 'openstack baremetal driver show' as a command,
>> and other than wanting to see a description of the logical disk properties,
>> the only other command related to the disk properties is 'openstack
>> baremetal node set --target-raid-config'.
>>
>
> Possible confusion on my part... the properties you want to display as
> part of 'baremetal driver show' are set with 'baremetal node set'?  That
> assymetry may be part of the reason this was ambiguous to start with.
>

A baremetal node has an associated driver; it is the driver that handles
the node (powering, booting, deploying, RAID configuration, ...). The RAID
logical disk properties describe the information that the driver needs, for
doing RAID configuration. e.g. [1]. Maybe it should be
--raid-logical-disk-property-descriptions?  But the actual configuration is
specific to a particular node, and that is set via the ..node set
--target-raid-config (which can specify one or more logical disks). e.g.
[2].

Maybe we could make it easier on ourselves by not providing an API to get
descriptions of things, but "just" document them. :)

--ruby

[1]
http://developer.openstack.org/api-ref/baremetal/index.html?expanded=show-driver-logical-disk-properties-detail
[2]
http://developer.openstack.org/api-ref/baremetal/index.html?expanded=set-raid-config-detail
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20160915/6827b97b/attachment.html>