<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 28, 2014 at 10:32 PM, Steve Gordon <span dir="ltr"><<a href="mailto:sgordon@redhat.com" target="_blank">sgordon@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">----- Original Message -----<br>
> From: "Deepak Shetty" <<a href="mailto:dpkshetty@gmail.com">dpkshetty@gmail.com</a>><br>
> To: "OpenStack Development Mailing List (not for usage questions)" <<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>><br>
><br>
</span><span class="">> But isn't *-specs comes very early in the process where you have an<br>
> idea/proposal of a feature, u don't have it yet implemented. Hence specs<br>
> just end up with Para's on how the feature is supposed to work, but doesn't<br>
> include any real world screen shots as the code is not yet ready at that<br>
> point of time. Along with patch it would make more sense, since the author<br>
> would have tested it so it isn't a big overhead to catch those cli screen<br>
> shots and put it in a .txt or .md file so that patch reviewers can see the<br>
> patch in action and hence can review more effectively<br>
><br>
> thanx,<br>
> deepak<br>
<br>
</span>Sure but in the original email you listed a number of other items, not just CLI screen shots, including:<br>
<span class=""><br>
> >> > 1) What changes are needed in manila.conf to make this work<br>
> >> > 2) How to use the cli with this change incorporated<br>
> >> > 3) Some screen shots of actual usage<br>
</span><span class="">> >> > 4) Any caution/caveats that one has to keep in mind while using this<br>
<br>
</span>Ideally I see 1, 2, and 4 as things that should be added to the spec (retrospectively if necessary) to ensure that it maintains an accurate record of the feature. I can see potential benefits to including listings of real world usage (3) in the client projects, but I don't think all of the items listed belong there.<br></blockquote><div><br></div><div>Agree, IMHO (2) and (3) will be possible only when patch is ready, others can be part of spec.<br><br></div><div>thanx,<br>deepak<br></div></div><br></div></div>