Re: Changing Submit Type on review.gluster.org

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 07, 2017 at 11:50:21AM +0530, Nigel Babu wrote:
> Hello folks,
> 
> A few times, we've merged dependent patches out of order because the Submit
> type[1] did not block us from doing so. The last few times we've talked about
> this, we didn't actually take a strong decision either way. In yesterday's
> maintainers meeting, we agreed to change the Submit type to
> Rebase-If-Necessary. This change will happen on 18th September 2017.
> 
> What this means:
> * No more metadata flags added by Gerrit. There will only be a Change-Id,
>   Signed-off-by, and BUG (if you've added it). Gerrit itself will not add any
>   metadata.
> * If you push a patch on top of another patch, the Submit button will either be
>   grayed out because the dependent patches cannot be merged or they will be
>   submited in the correct order in one go.
> 
> Some of the concerns that have been raised:
> Q: With the Reviewed-on flag gone, how do we keep track of changesets
>    (especially backports)?
> A: The Change-Id will get you all the data directly on Gerrit. As long you
>    retain the Change-Id, Gerrit will get you the matching changesets.
> 
> Q: Will who-wrote-what continue to work?
> A: As far as I can see, it continues to work. I ran the script against
>    build-jobs repo and it works correctly. Additionally, we'll be setting up an
>    instance of Gerrit Stats[2] to provide more detailed stats.
> 
> Q: Can we have some of the metadata if not all?
> Q: Why can't we have the metadata if we change the submit type?
> A: There's no good answer to this other than, this is how Gerrit works and
>    I can neither change it nor control it.

Q: Can patches of a series be merged before all patches in the series
have a +2? Initial changes that prepare things, or add new (unused) core
functionalities should be mergable so that follow-up patches can be
posted against the HEAD of the branch.

A: Nigel?

Thanks,
Niels


> 
> [1]: https://review.gluster.org/Documentation/intro-project-owner.html#submit-type
> [2]: http://gerritstats-demo.firebaseapp.com/
> 
> --
> nigelb
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel@xxxxxxxxxxx
> http://lists.gluster.org/mailman/listinfo/gluster-devel
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://lists.gluster.org/mailman/listinfo/gluster-devel



[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux