[openstack-dev] How to improve the specs review process (was Re: [Neutron] Group Based Policy and the way forward)

Ivar Lazzaro ivarlazzaro at gmail.com
Wed Aug 6 20:28:57 UTC 2014


+1 Eugene,

Still, there's a point in Stefano's thread:

There's a time for discussing merging strategies, models, and naming
conventions... And this time is called BP approval :)

Just saying.
Ivar.



On Wed, Aug 6, 2014 at 10:21 PM, Eugene Nikanorov <enikanorov at mirantis.com>
wrote:

>
>
>
> On Wed, Aug 6, 2014 at 11:07 PM, Stefano Maffulli <stefano at openstack.org>
> wrote:
>
>> On 08/06/2014 11:19 AM, Edgar Magana wrote:
>> > That is the beauty of the open source projects, there is always a
>> smartest
>> > reviewer catching out the facts that you don¹t.
>>
>> And yet, the specification clearly talks about 'endpoints' and nobody
>> caught it where it supposed to be caught so I fear that something failed
>> badly here:
>>
>
> I know that there's whole other thread on naming.
> I believe everybody has reviewed this having keystone's "endpoint" in mind
> and understanding that those are different terms where keystone endpoints
> should have been named 'service_endpoints' or something.
> There's no UX or technical reasons to not to reuse terms used in different
> projects and in different domains.
>
> So I don't think it's fair to blame reviewers here.
>
> Thanks,
> Eugene.
>
>>
>> https://review.openstack.org/#/c/89469/10
>>
>> What failed and how we make sure this doesn't happen again? This to me
>> is the most important question to answer.  If I remember correctly we
>> introduced the concept of Specs exactly to discuss on the ideas *before*
>> the implementation starts. We wanted things like architecture, naming
>> conventions and other important decisions to be socialized and agreed
>> upon *before* code was proposed. We wanted to avoid developers to spend
>> time implementing features in ways that are incompatible and likely to
>> be rejected at code review time. And yet, here we are.
>>
>> Something failed and I would ask for all core reviewers to sit down and
>> do an exercise to identify the root cause. If you want we can start from
>> this specific case, do some simple root cause analysis together and take
>> GBP as an example. Thoughts?
>>
>> /stef
>>
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140806/a2491d04/attachment.html>


More information about the OpenStack-dev mailing list