On Mon, Aug 29, 2016 at 10:55 AM, Poornima Gurusiddaiah <[email protected]
> wrote:

> Hi,
>
> Regarding the enforcement of the dependencies while merging, i see that
> the dependent patches on any patch is mentioned in the "Related Changes"
> column [1]. It still doesn't enforce, in the cherry-pick way of
> submitting changes, by default it ignores the lineage [2]. But there are
> ways to enforce this. Will let the gluster infra maintainers to comment
> on the same.
>
> [1] https://gerrit-review.googlesource.com/Documentation/user-review-ui.
> html#related-changes
> [2] https://gerrit-review.googlesource.com/Documentation/project-
> configuration.html#project_options
>
> Regards,
> Poornima
>

Thank you Poornima for pointing this out. You're right, it's worth changing
our submission type to either "Rebase If Necessary" or "Merge If Necessary"
to enforce this. I'm not sure what the implications are, so I'll report
back after I've setup a test on staging so we can experiment and see what
works.

-- 
nigelb
_______________________________________________
Gluster-devel mailing list
[email protected]
http://www.gluster.org/mailman/listinfo/gluster-devel

Reply via email to