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

milanisko k vetrisko at gmail.com
Wed May 17 12:13:44 UTC 2017


Ășt 16. 5. 2017 v 18:03 odesĂ­latel Dmitry Tantsur <dtantsur at redhat.com>
napsal:

> 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 :)
>
>  +1 :D

> >
> > 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.
>

 +1 I'm sometimes wondering about inspector ;)


> >
> > 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
> >
>
>
> __________________________________________________________________________
> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20170517/1ef5ed36/attachment.html>


More information about the OpenStack-dev mailing list