<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 4, 2014 at 12:31 AM, Steve Martinelli <span dir="ltr"><<a href="mailto:stevemar@ca.ibm.com" target="_blank">stevemar@ca.ibm.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font face="sans-serif">To add to the growing pains of keystone-specs,
one thing I've noticed is, there is inconsistency in the 'REST API Impact'
section.</font>
<br>
<br><font face="sans-serif">To be clear here, I don't mean we shouldn't
include what new APIs will be created, I think that is essential. But rather,
remove the need to specifically spell out the request and response blocks.</font>
<br>
<br><font face="sans-serif">Personally, I find it redundant for
a few reasons:</font></blockquote><div><br></div><div>Agree, we need to eliminate the redundancy...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>


<br><font face="sans-serif">1) We already have identity-api, which
will need to be updated once the spec is completed anyway.</font></blockquote><div><br></div><div>So my thinking is to merge the content of openstack/identity-api into openstack/keystone-specs. We use identity-api just like we use keystone-specs anyway, but only for a subset of our work.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><font face="sans-serif">2) It's easy to get bogged down in the
spec review as it is, I don't want to have to point out mistakes in the
request/response blocks too (as I'll need to do that when reviewing the
identity-api patch anyway).</font></blockquote><div><br></div><div>I personally see value in having them proposed as one patchset - it's all design work, so I think it should be approved as a cohesive piece of design.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><font face="sans-serif">3) Come time to propose the identity-api
patch, there might be differences in what was proposed in the spec.</font></blockquote><div><br></div><div>There *shouldn't* be though... unless you're just talking about typos/etc. It's possible to design an unimplementable or unusable API though, and that can be discovered (at latest) by attempting an implementation... at that point, I think it's fair to go back and revise the spec/API with the solution.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br><font face="sans-serif">Personally I'd be OK with just stating
the HTTP method and the endpoint. Thoughts?</font></blockquote><div><br></div><div>Not all API-impacting changes introduce new endpoint/method combinations, they may just add a new attribute to an existing resource - and this is still a bit redundant with the identity-api repo.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br><font face="sans-serif">Many apologies in advance for my pedantic-ness!<br></font></blockquote><div><br></div><div>Laziness*</div><div><br></div><div>(lazy engineers are just more efficient)</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<font face="sans-serif">
</font>
<br><font size="1" face="Arial">Regards,</font>
<br>
<br><font size="3" color="#8f8f8f" face="Arial"><b>Steve Martinelli</b></font><font size="1" face="Arial"><br>
Software Developer - Openstack<br>
Keystone Core Member</font>
<table width="680" style="border-collapse:collapse">
<tbody><tr height="8">
<td width="680" colspan="2" style="border-style:solid;border-color:#000000;border-width:0px 0px 0px 0px;padding:0px 0px">
<hr>
</td></tr><tr valign="top" height="8">
<td width="420" style="border-style:solid;border-color:#000000;border-width:0px 0px 0px 0px;padding:0px 0px"><font size="1" color="#4181c0" face="Arial"><b>Phone:</b></font><font size="1" color="#5f5f5f" face="Arial">
<a href="tel:1-905-413-2851" value="+19054132851" target="_blank">1-905-413-2851</a></font><font size="1" color="#4181c0" face="Arial"><b><br>
E-mail:</b></font><font size="1" color="#5f5f5f" face="Arial"> </font><a href="mailto:stevemar@ca.ibm.com" target="_blank"><font size="1" color="#5f5f5f" face="Arial"><u>stevemar@ca.ibm.com</u></font></a>
</td><td width="259" style="border-style:solid;border-color:#000000;border-width:0px 0px 0px 0px;padding:0px 0px">
<div align="right"><font size="1" color="#5f5f5f" face="Arial">8200 Warden Ave<br>
Markham, ON L6G 1C7<br>
Canada</font></div></td></tr></tbody></table>
<br>


<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>