<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 6, 2015 at 10:03 AM, Steven Kaufer <span dir="ltr"><<a href="mailto:kaufer@us.ibm.com" target="_blank">kaufer@us.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>
<p><font face="sans-serif">This is a follow up thread to [1]</font><br>
<br>
<font face="sans-serif">In order to have consistency across clients, I am proposing that the client side sorting has the following syntax: --sort <key>[:<direction>]</font><br>
<br>
<font face="sans-serif">Where the --sort parameter is comma-separated and is used to specify one or more sort keys and directions. The direction defaults to 'desc' for each sort key and the user can supply 'asc' to override.</font><br>
<br>
<font face="sans-serif">For example:</font><br>
<br>
<font face="sans-serif">  nova list --sort display_name</font><br>
<font face="sans-serif">  nova list --sort display_name,vm_state</font><br>
<font face="sans-serif">  nova list --sort display_name:asc,vm_state:asc</font><br>
<br>
<font face="sans-serif">If approved, then the following changes are needed for glance and cinder (note that nova already uses this syntax):</font><br>
<br>
<font face="sans-serif">  Cinder: Deprecate --sort_key and --sort_dir and add support for --sort</font><br>
<font face="sans-serif">  Glance: Modify [2] to this new syntax</font><br>
<br>
<font face="sans-serif">I have not verified how all other projects handle sorting, there may be other projects that would also need to be changed.</font><br>
<br>
<font face="sans-serif">Thoughts?  Objections?</font><br>
<br>
<font face="sans-serif">Also, if there is a more formal way to propose/review this standard please let me know.</font></p></div></blockquote><div>I'm all for consistency. Sounds like a great case for the API Working Group to document. You can propose a patch describing the way we want sorting to work. </div><div><br></div><div>See <a href="https://review.openstack.org/#/q/project:openstack/api-wg,n,z">https://review.openstack.org/#/q/project:openstack/api-wg,n,z</a></div><div><br></div><div>Anne</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><p><br>
<br>
<font face="sans-serif">[1] </font><a href="http://www.mail-archive.com/openstack-dev@lists.openstack.org/msg42854.html" target="_blank"><font face="sans-serif">http://www.mail-archive.com/openstack-dev@lists.openstack.org/msg42854.html</font></a><br>
<font face="sans-serif">[2] </font><a href="https://review.openstack.org/#/c/120777/" target="_blank"><font face="sans-serif">https://review.openstack.org/#/c/120777/</font></a><br>
<br>
<font face="sans-serif">Thanks,</font><br>
<font face="sans-serif">Steven Kaufer</font></p></div><br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div></div>