<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue., Sep. 14, 2021, 15:57 Lee Yarwood, <<a href="mailto:lyarwood@redhat.com">lyarwood@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 14-09-21 13:55:14, Artom Lifshitz wrote:<br>
> Hey Elod and Vlad (and the ML, cc'ed)<br>
> <br>
> This is continuing the conversation Elod and I had on IRC. The<br>
> immediate context is [1], which is a test refactor backport that in<br>
> turn enables conflict-free backporting of at least two fixes.<br>
> <br>
> From Red Hat's point of view, we will have to care about stable/train<br>
> for *very* long time, so we'd like to do everything we can to make our<br>
> lives easier, including backportings things are aren't strictly<br>
> necessary (like test refactors) in order to greatly facilitate future<br>
> backports, which tend to be of the "func reproducer test + actual fix"<br>
> variety.<br>
> <br>
> Elod made the excellent point that while this approach does make<br>
> backports to train much easier to both write and review, it also<br>
> "postpones" all of the manual conflict resolution work onto anyone<br>
> doing stable/stein backports.<br>
<br>
I can't say I agree with the objection here but regardless I think the<br>
ship has already sailed on us backporting many of these test refactors to<br>
stable/train.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Maybe, but at least in some cases it's still possible, like my [1] link in my previous message.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We have already backported so by manually resolving conflicts and that<br>
would now conflict again with any attempt to backport the refactors. At<br>
least that has been my experience the last few times I tried to do this<br>
and I've now given up and manually resolved conflicts, for example:<br>
<br>
<a href="https://review.opendev.org/c/openstack/nova/+/785847" rel="noreferrer noreferrer" target="_blank">https://review.opendev.org/c/openstack/nova/+/785847</a><br>
<br>
Going forward however and with my dusty old Red Hat on I want to try to<br>
do this as much as possible back to stable/wallaby to allow fixes to<br>
flow through our stable branches as seamlessly as possible.<br>
<br>
I don't see why such contributions should be blocked, the work required<br>
to resolve these conflicts would eventually have to be done by someone, <br>
pushing that to contributors who care about a given older release only<br>
seems fair IMHO.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Fair, hard to argue with "if you care about something, put in the work."</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> I went and gathered data. I looked at the last 6 months of backports<br>
> in Gerrit. At the time of this writing, stable/train 72 [2] had and<br>
> stable/stein had 24 [3]. Out of the latter, Vlad Gusev was the main<br>
> non-Red Hat contributor.<br>
<br>
Slight tangent but only 20 landing for train and 10 landing for stein.<br>
<br>
If anyone is interested in becoming a stable core on Nova please feel<br>
free to reach out as we could really use more reviewers.<br>
<br>
> So my proposal is simple: I'm willing to help resolve conflicts in<br>
> stable/stein backport for folks like Vlad, if it means making<br>
> backports to stable/train easier. I'm confident that it would still<br>
> mean an overall reduction in man-hours for backports, even if I become<br>
> personally on the hook to deal with the fallout of backporting test<br>
> refactors to stable/train.<br>
<br>
Well that's good of you Artom but I don't think that should be required<br>
to land such backports on a given stable branch.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Just trying to be a good community citizen, and show that I'm open to listen to others' concerns :) (And, full disclosure, also making the bet that it'll happen so rarely that it won't affect me.)</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
<br>
Lee<br>
<br>
> [1] <a href="https://review.opendev.org/c/openstack/nova/+/791481" rel="noreferrer noreferrer" target="_blank">https://review.opendev.org/c/openstack/nova/+/791481</a><br>
> [2] <a href="https://review.opendev.org/q/project:openstack/nova+branch:stable/train+-age:6month" rel="noreferrer noreferrer" target="_blank">https://review.opendev.org/q/project:openstack/nova+branch:stable/train+-age:6month</a><br>
> [3] <a href="https://review.opendev.org/q/project:openstack/nova+branch:stable/stein+-age:6month" rel="noreferrer noreferrer" target="_blank">https://review.opendev.org/q/project:openstack/nova+branch:stable/stein+-age:6month</a><br>
<br>
-- <br>
Lee Yarwood                 A5D1 9385 88CB 7E5F BE64  6618 BCA6 6E33 F672 2D76<br>
</blockquote></div></div></div>