[openstack-dev] [ironic] Ironic-UI review requirements - single core reviews

Dmitry Tantsur dtantsur at redhat.com
Tue May 16 15:58:40 UTC 2017


On 05/15/2017 09:10 PM, Julia Kreger wrote:
> All,
> 
> In our new reality, in order to maximize velocity, I propose that we
> loosen the review requirements for ironic-ui to allow faster
> iteration. To this end, I suggest we move ironic-ui to using a single
> core reviewer for code approval, along the same lines as Horizon[0].

Ok, Horizon example makes me feel a bit better about this :)

> 
> Our new reality is a fairly grim one, but there is always hope. We
> have several distinct active core reviewers. The problem is available
> time to review, and then getting any two reviewers to be on the same,
> at the same time, with the same patch set. Reducing the requirements
> will help us iterate faster and reduce the time a revision waits for
> approval to land, which should ultimately help everyone contributing.
> 
> If there are no objections from my fellow ironic folk, then I propose
> we move to this for ironic-ui immediately.

I'm fine with that. As I mentioned to you, it's clearly more important to be 
able to move forward than to make sure we never miss a sub-perfect patch. 
Especially for leaf projects like UI.

> 
> Thanks,
> 
> -Julia
> 
> [0]: http://lists.openstack.org/pipermail/openstack-dev/2017-February/113029.html
> 
> __________________________________________________________________________
> 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