<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Mar 6, 2014 at 12:05 PM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</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">One of the issues that the Nova team has definitely hit is Blueprint<br>


overload. At some point there were over 150 blueprints. Many of them<br>
were a single sentence.<br>
<br>
The results of this have been that design review today is typically not<br>
happening on Blueprint approval, but is instead happening once the code<br>
shows up in the code review. So -1s and -2s on code review are a mix of<br>
design and code review. A big part of which is that design was never in<br>
any way sufficiently reviewed before the code started.<br>
<br>
In today's Nova meeting a new thought occurred. We already have Gerrit<br>
which is good for reviewing things. It gives you detailed commenting<br>
abilities, voting, and history. Instead of attempting (and usually<br>
failing) on doing blueprint review in launchpad (or launchpad + an<br>
etherpad, or launchpad + a wiki page) we could do something like follows:<br>
<br>
1. create bad blueprint<br></blockquote><div><br></div><div>or create a good one with a great template! </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">


2. create gerrit review with detailed proposal on the blueprint<br>
3. iterate in gerrit working towards blueprint approval<br>
4. once approved copy back the approved text into the blueprint (which<br>
should now be sufficiently detailed)<br>
<br>
Basically blueprints would get design review, and we'd be pretty sure we<br>
liked the approach before the blueprint is approved. This would<br>
hopefully reduce the late design review in the code reviews that's<br>
happening a lot now.<br>
<br>
There are plenty of niggly details that would be need to be worked out<br>
<br>
 * what's the basic text / template format of the design to be reviewed<br>
(probably want a base template for folks to just keep things consistent).<br>
 * is this happening in the nova tree (somewhere in docs/ - NEP (Nova<br>
Enhancement Proposals), or is it happening in a separate gerrit tree.<br></blockquote><div><br></div><div>I think this is really worthwhile to try -- and it might offer an interesting, readable history of decisions made. Plus how funny it was also brought up at the Ops Summit. Convergence, cool. </div>

<div><br></div><div>It also goes along with our hope to move API design docs into the repo. </div><div><br></div><div>Other projects up to try it? The only possible addition is that we might need to work out is cross-project blueprints and which repo should those live in? We're doing well on integration, be careful about siloing.<br>

</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">
 * are there timelines for blueprint approval in a cycle? after which<br>
point, we don't review any new items.<br>
<br>
Anyway, plenty of details to be sorted. However we should figure out if<br>
the big idea has support before we sort out the details on this one.<br>
<br>
Launchpad blueprints will still be used for tracking once things are<br>
approved, but this will give us a standard way to iterate on that<br>
content and get to agreement on approach.<br>
<span class=""><font color="#888888"><br>
        -Sean<br>
<br>
--<br>
Sean Dague<br>
Samsung Research America<br>
<a href="mailto:sean@dague.net">sean@dague.net</a> / <a href="mailto:sean.dague@samsung.com">sean.dague@samsung.com</a><br>
<a href="http://dague.net" target="_blank">http://dague.net</a><br>
<br>
</font></span><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>