<div dir="ltr">The ironic node-list --chassis seems to be easier to understand for me. I've commented on one of the patches that maybe we should deprecate the chassis-node-list if we add this, but then, the deprecation is slow, we have some functional tests already... Having two commands kind of reflects the duplication in our API, where we can do /chassis/<UUID>/nodes and /nodes?chassis_uuid=<UUID>, so maybe having both of them is fine.<div><br></div><div>Vlad</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 29, 2016 at 7:22 PM, Loo, Ruby <span dir="ltr"><<a href="mailto:ruby.loo@intel.com" target="_blank">ruby.loo@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">







<div bgcolor="white" lang="EN-US" link="#0563C1" vlink="#954F72">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt">Hi,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">While working on the openstackclient plugin commands for ironic, I was thinking about the equivalents for 'ironic chassis-node-list' (nodes that are part of specified chassis) and 'ironic-node-port-list' (ports
 that are part of specified node). It didn't make sense to me to have an 'openstack baremetal chassis node list', since a 'chassis' and a 'node' are two different objects in osc lingo and we already have 'openstack baremetal chassis xx' and 'openstack baremetal
 node yy' commands. Furthermore, our REST API supports 'GET /v1/nodes?chassis=c1' and 'GET /v1/ports?node=n1'.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">So I proposed 'openstack baremetal node list --chassis' and 'openstack baremetal port list --node' [1]. To implement this, I need to enhance our corresponding python APIs. The question I have is whether we
 want to only enhance the python API, or also provide 'ironic node-list --chassis' and 'ironic port-list --node' commands. The latter is being proposed [2] and coded at [3]. Doing this would mean two different ironic CLIs to do the same thing, but also provide
 a more obvious 1:1 correspondence between ironic & osc commands, and between ironic CLI and python API.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">Thoughts?<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">It'd be great if we could decide in the next day or so, in order to get the osc-related commands into the client this week for the Newton release.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">--ruby<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">[1] <a href="http://specs.openstack.org/openstack/ironic-specs/specs/approved/ironicclient-osc-plugin.html#openstack-baremetal-node" target="_blank">
http://specs.openstack.org/<wbr>openstack/ironic-specs/specs/<wbr>approved/ironicclient-osc-<wbr>plugin.html#openstack-<wbr>baremetal-node</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">[2] <a href="https://launchpad.net/bugs/1616242" target="_blank">
https://launchpad.net/bugs/<wbr>1616242</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">[3] <a href="https://review.openstack.org/#/c/359520/" target="_blank">https://review.openstack.org/#<wbr>/c/359520/</a><u></u><u></u></span></p>
</div>
</div>

<br>______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
<br></blockquote></div><br></div>