[openstack-dev] [ironic] Ironic-UI review requirements - single core reviews
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>
> 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.
> 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
> 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
> > :
> > 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
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the OpenStack-dev