<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 19, 2015 at 2:23 AM, David Ostrovsky <span dir="ltr"><<a href="mailto:d.ostrovsky@gmx.de" target="_blank">d.ostrovsky@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Nov 18, 2015 at 19:20:03 PST, Ashod Nakashian <ashnakash at<br>
<a href="http://gmail.com" rel="noreferrer" target="_blank">gmail.com</a>> wrote:<br>
<br>
>Sorry, I didn't explain well.<br>
><br>
>The patches are related. They are updates based on feedback, partial<br>
>failure or improvement.<br>
<br>
[...]<br>
<br>
>Why do people send multiple patches per push? That's the right question<br>
>to ask.<br>
><br>
>And the answer is: to improve the previous patch.<br>
><br>
>Hence my question. If a patch has partially failed, or I got feedback<br>
to<br>
>improve it, or (insert reason here), and I want to push an update, why<br>
>should the previous patch still build when it's not necessary?<br>
><br>
>Hope this makes sense.<br>
<br>
Yes, indeed. LibreOffice is using Jenkins CI and Gerrit trigger plugin<br>
for integration between Jenkins and Gerrit. This plugin provides the<br>
feature you are asking for: job cancellation when patch set became<br>
obsolete because of uploading new patch during the old patch set is<br>
still building. Moreover, when patch_set_42 is building, and you've<br>
uploaded another 10 patch sets simultaneously, so that for every patch<br>
set i, 1 <= i <= 10, the build for patch_set_42 + i wouldn't be even<br>
started, but only the build for patch_set_53. Unfortunately for some<br>
reasons this feature wasn't activated yet. See also the announcement<br>
on the trigger site: [1].<br>
<br>
"<br>
EXPERIMENTAL: Cancel old running jobs when a new Patch Set is uploaded<br>
on the same change.<br>
This feature is set as experimental due to problems we've found during<br>
testing.<br>
You can enable the feature from the Manage Jenkins/Gerrit Trigger page.<br>
All help in debugging the problems we've found is appreciated, that's<br>
why it is in this release.<br>
"<br>
<br>
So yes, I agree that this feature should be activated on our infra. To<br>
do that, we would need to investigate, track down and fix the problems.<br>
To be able to do so, we need staging Jenkins and Gerrit instances, where<br>
we could reproduce and troubleshoot this and other issues, like simulate<br>
Gerrit upgrade to newer version, etc.<br></blockquote><div><br></div><div>Thanks David. This is exactly the answer I was looking for.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Unfortunately, it cannot be done, because our infra team doesn't have<br>
free resources to set up new VM and install Gerrit and Jenkins on them.<br>
Last time i asked why they don't have resources and the answer was: this<br>
project is not OpenStack that is spending substantial resources for<br>
their infrastructure. No wonder that my request to set up staging Gerrit<br>
instance is pending for years now: [2]. So I wouldn't expect that<br>
staging Jenkins instance would be available in 5-10 years (if at all),<br>
if you would request it today on infra issue tracker.<br>
<br>
IOW, just accept that all patch sets are get built. Well, at least you<br>
have asked and now you know the answer why it is how it is.<br>
<br>
[1] <a href="https://wiki.jenkins-ci.org/pages/viewpage.action?pageId=63931391" rel="noreferrer" target="_blank">https://wiki.jenkins-ci.org/pages/viewpage.action?pageId=63931391</a><br>
[2] <a href="https://redmine.documentfoundation.org/issues/1147" rel="noreferrer" target="_blank">https://redmine.documentfoundation.org/issues/1147</a><br>
<br></blockquote><div><br></div><div>Not great. I'm hoping this could still be tackled, especially when the benefits are more than the case at hand (as you mentioned staging is a major advantage).</div><div> </div><div>I'm still hopeful this is doable. Perhaps it could be suggested for consideration in next year's budget?</div></div></div></div>