<html><body>
<h3>Yedidyah Bar David created OVIRT-1895:</h3>
<pre>   Summary: CI votes not forwarded on commit message change
       Key: OVIRT-1895
       URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1895
   Project: oVirt - virtualization made easy
Issue Type: By-EMAIL
  Reporter: Yedidyah Bar David
  Assignee: infra</pre>
<p>Hi all,</p>
<p>If a change goes through this flow:</p>
<ol><li><p>Push an update N</p></li>
<li><p>CI triggers check-patch for N</p></li>
<li><p>Push another update N+1 that changes only commit message</p></li>
<li><p>CI marks +1 N</p></li></ol>
<p>Then we have logic to not trigger CI for N+1 (with the assumption that a change to commit message should not affect the result), but also do not mark +1 N+1, leaving the patch unmarked.</p>
<p>We should either forward the +1, when giving it, to all consecutive future patchsets that only changed the commit message, or give up on the optimization and always trigger CI.</p>
<p>As an example, see:</p>
<p><a href="https://gerrit.ovirt.org/87515">https://gerrit.ovirt.org/87515</a></p>
<p>Best regards, &mdash; Didi</p>
<p>&mdash; This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100079)</p>

<img src="https://u4043402.ct.sendgrid.net/wf/open?upn=i5TMWGV99amJbNxJpSp2-2BJ33BSM3tuiUfRTk64K-2BOjHk0t8rl8MpMdxMsjDtQ-2B6XOX7pNdoswlBAGXkxIW25gp1cBdrqzGA1YyFPKxhR1tde8I-2Fd-2F6hAZASgSEDnTN-2BZhF71Pw5-2BlaTduml3toFyueyTBXC8wz9tCMysHoyhTf1KX-2FmOTQN1ncO7U5qung-2FXhk4qovCffitmeJ1CbPHJCRP42OFfxYJQfGsnB9WY00RngSd8AQuseAa94K5AINOnhX3881VTIKAXzhy2iCnH8piPgi-2FxkIc8CsyxbPtnr4KRZdEKiTneYvteb74r7Kpyp5lMJsWVvjsHD9OAYj5HNnFgeyYTIBgwCEmRnTPf7NpTaNL0DFLCEdJcOKQk1Ftazn9XvqvwZfh6JaPb1OYdg4Wca2VEFzmPkF6bZl2PFUcmboCKNvuwKYGm7ac1UUXl" alt="" width="1" height="1" border="0" style="height:1px !important;width:1px !important;border-width:0 !important;margin-top:0 !important;margin-bottom:0 !important;margin-right:0 !important;margin-left:0 !important;padding-top:0 !important;padding-bottom:0 !important;padding-right:0 !important;padding-left:0 !important;"/>
</body></html>