<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class=""><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 class=""><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><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><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;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><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><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;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">The problem, though, is that this only shows the disk properties, nothing else about the driver. The information is different, so that trying to use this command to show the driver information AND the disk property descriptions, doesn't make sense either.</span></p><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"><br></span></p><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">If we thought that we might have some 'openstack baremetal driver raid <something-else>' command in the future, then 'openstack baremetal driver raid logical disk properties show' might make more sense. (Now I'm leaning towards this one.)</span></p></div></div></div></div></blockquote><div><br></div><div>To generalize a bit more, if the 'baremetal driver' has different kinds of properties, or groups of properties that you want to distinguish, I would suggest doing that with a common option name like you have above: '--raid-logical-disk-property' and defining more of those as you need them.  But use the same approach where you set the properties too, etc.</div></div><div><br></div><div>dt</div><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><br>Dean Troyer<br><a href="mailto:dtroyer@gmail.com" target="_blank">dtroyer@gmail.com</a><br></div>
</div></div>