[telemetry] Procedure for merging patches

Rong Zhu aaronzhu1121 at gmail.com
Thu Apr 11 07:46:53 UTC 2019


Hi,

As usual, There are have +2s, then the next core can +2/A, why we need PTL
final +A?

Thanks,
Rong Zhu

Trinh Nguyen <dangtrinhnt at gmail.com>于2019年4月11日 周四15:39写道:

> Hi team,
>
> One of the thing that has been brought up in the last meeting is what is
> the process of merging a patch. So, let's try this first:
>
>    - For small patches (e.g., change strings, fix typos, update docs):
>    any core can approve for merging.
>    - For a patch that implements a new feature, removes a feature, patch
>    that has huge changes (in the number of changed files and loc, etc.) or any
>    patch that is in doubt: we need at least 2 cores to give a +2 and then the
>    PTL will merge it.
>
> Of course, we can change the procedure when we see issues. For now, this
> should be a guideline for the team.
>
> Please feel free to comment.
>
> Yours,
>
> --
> *Trinh Nguyen*
> *www.edlab.xyz <https://www.edlab.xyz>*
>
> --
Thanks,
Rong Zhu
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-discuss/attachments/20190411/86e2aefe/attachment-0001.html>


More information about the openstack-discuss mailing list