<div dir="ltr">@Aleksandr: it seems you are right, after my first broken commit message I was careful not to mess them up again and I didn't even notice that this check was turned off. Just curious: why was it turned off?</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 29, 2015 at 11:24 AM, Igor Kalnitsky <span dir="ltr"><<a href="mailto:ikalnitsky@mirantis.com" target="_blank">ikalnitsky@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Guys,<br>
<br>
How do you suppose to know that only commit message was changed? Do<br>
you want to implement manual comparison between patch sets?!<br>
<br>
Currently Gerrit checks whether patchset was changed or not by<br>
tracking Git commit SHA1 sum, and, btw, chaning commit message will<br>
lead to changing commit sha1 sum.<br>
<br>
So it looks pretty tricky to implement, and it may lead to false<br>
positive results.<br>
<br>
Thanks,<br>
Igor<br>
<span class=""><br>
On Wed, Jul 29, 2015 at 12:06 PM, Sergii Golovatiuk<br>
<<a href="mailto:sgolovatiuk@mirantis.com">sgolovatiuk@mirantis.com</a>> wrote:<br>
> I agree with Maciej<br>
><br>
> 1. Simple change in CommitMessage shouldn't spin up CI check<br>
> 2. There should be simple task where we check "Implements: blueprint" or<br>
> "Closes-Bug:" or Related-Bug:" to set -1 automatically<br>
> 3. There can be additional checks like Short Summary should be 50 or less<br>
> symbols. Long summary should be wrapped to 80 symbols<br>
><br>
><br>
> --<br>
> Best regards,<br>
> Sergii Golovatiuk,<br>
> Skype #golserge<br>
> IRC #holser<br>
><br>
</span>> On Wed, Jul 29, 2015 at 10:58 AM, Aleksandr Didenko <<a href="mailto:adidenko@mirantis.com">adidenko@mirantis.com</a>><br>
<span class="">> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> > I think that checking commit message compliance to commit message<br>
>> > guidelines (for example ending the first line with dot) is part of CI jobs,<br>
>> > and they will vote -1 if message is wrongly structured.<br>
>><br>
>><br>
>> Maciej, we don't have such checks at the moment. You can craft any commit<br>
>> message you want and it will not cause any problems with CI. I think,<br>
>> reviewers could do the job on commit message verification and they already<br>
>> do this :)<br>
>><br>
>> Regards,<br>
>> Alex<br>
>><br>
>><br>
>> On Wed, Jul 29, 2015 at 11:23 AM, Sergey Vasilenko<br>
</span><span class="">>> <<a href="mailto:svasilenko@mirantis.com">svasilenko@mirantis.com</a>> wrote:<br>
>>><br>
>>> -1 to Maciej<br>
>>> +1 to Sergii<br>
>>><br>
>>><br>
>>><br>
>>> /sv<br>
>>><br>
>>><br>
>>><br>
>>> __________________________________________________________________________<br>
>>> OpenStack Development Mailing List (not for usage questions)<br>
>>> Unsubscribe:<br>
</span>>>> <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<span class="">>>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
>>><br>
>><br>
>><br>
>> __________________________________________________________________________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
</span>>> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<span class="">>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
>><br>
><br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
</span>> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<span class="">> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
</span>Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div>