<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Sep 12, 2016 at 12:38 PM, Dean Troyer <span dir="ltr"><<a href="mailto:dtroyer@gmail.com" target="_blank">dtroyer@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="gmail-">On Mon, Sep 12, 2016 at 10:51 AM, Ruby Loo <span dir="ltr"><<a href="mailto:opensrloo@gmail.com" target="_blank">opensrloo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><span><div class="gmail_extra"><p style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><span style="color:rgb(0,0,0);font-family:arial;font-size:14.6667px;line-height:1.38;background-color:transparent">My preference is:</span><br></p></div></span><div class="gmail_extra"><div class="gmail_extra"><span><p style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><span style="font-size:14.6667px;font-family:arial;color:rgb(0,0,0);vertical-align:baseline;background-color:transparent">    </span><span style="font-size:14.6667px;font-family:arial;color:rgb(0,0,0);vertical-align:baseline;background-color:transparent">* openstack baremetal driver show --raid-logical-disk-properties</span></p></span></div></div></div></blockquote><div><br></div></span><div>Agreed here.</div><div><br></div><div>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'.</div><span class="gmail-"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div><div class="gmail_extra"><p style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><span style="font-size:14.6667px;font-family:arial;color:rgb(0,0,0);vertical-align:baseline;background-color:transparent">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'.</span></p></div></div></div></div></blockquote><div><br></div></span><div>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.</div></div></div></div></blockquote><div><br></div><div>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].</div><div><br></div><div>Maybe we could make it easier on ourselves by not providing an API to get descriptions of things, but "just" document them. :)</div><div><br></div><div>--ruby</div><div><br></div><div>[1] <a href="http://developer.openstack.org/api-ref/baremetal/index.html?expanded=show-driver-logical-disk-properties-detail">http://developer.openstack.org/api-ref/baremetal/index.html?expanded=show-driver-logical-disk-properties-detail</a></div><div>[2] <a href="http://developer.openstack.org/api-ref/baremetal/index.html?expanded=set-raid-config-detail">http://developer.openstack.org/api-ref/baremetal/index.html?expanded=set-raid-config-detail</a></div></div></div></div>