<div dir="ltr"><div>+1, we also need some work to clean existing blueprints which are not approved.<br> I think that is responsibility for drafter of those blueprints to follow new process.<br></div><div><div><div><div class="gmail_extra">
<br><div class="gmail_quote">2014-03-07 2:05 GMT+08:00 Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc 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>
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>
 * 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="HOEnZb"><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><br clear="all"><br>-- <br><div dir="ltr"><div>ChangBo Guo(gcb)</div></div>
</div></div></div></div></div>