[openstack-dev] [Fuel] Code review process in Fuel and related issues

Igor Kalnitsky ikalnitsky at mirantis.com
Tue Sep 1 10:43:49 UTC 2015


Hi folks,

So basically..

* core reviewers won't be feature leads anymore
* core reviewers won't be assigned to features (or at least not full-time)
* core reviewers will spend time doing review and participate design meetings
* core reviewers will spend time triaging bugs

Is that correct?

Thanks,
Igor

On Sun, Aug 30, 2015 at 2:29 AM, Tomasz Napierala
<tnapierala at mirantis.com> wrote:
>> On 27 Aug 2015, at 07:58, Evgeniy L <eli at mirantis.com> wrote:
>>
>> Hi Mike,
>>
>> I have several comments.
>>
>> >> SLA should be the driver of doing timely reviews, however we can’t allow to fast-track code into master suffering quality of review ...
>>
>> As for me the idea of SLA contradicts to qualitative reviews.
>
> We expect cores to be less loaded after this change, so you guys should have more time to spend on right reviews, and not minor stuff. We hope this will also help keeping SLAs.
>
>> Another thing is I got a bit confused by the difference between Core Reviewer and Component Lead,
>> aren't those the same persons? Shouldn't every Core Reviewer know the architecture, best practises
>> and participate in design architecture sessions?
>
> Not really. You can have  many core reviewers, but there should be one component lead. Currently, while Fuel is monolithic, we cannot implement it in technical way. But if we succeed splitting Fuel into smaller projects, component lead will be responsible for (most likely) one repo.
>
> Regards,
> --
> Tomasz 'Zen' Napierala
> Product Engineering - Poland
>
>
>
>
>
>
>
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev



More information about the OpenStack-dev mailing list