<div dir="ltr"><div dir="ltr">Dear Matt,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 12, 2019 at 1:25 PM Matt Caswell <<a href="mailto:matt@openssl.org">matt@openssl.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
On 12/12/2019 09:29, Dmitry Belyavsky wrote:<br>
> - the contributor agreed to sign the CLA and<br>
> - there was a mark that CLA is signed and<br>
> - all the necessary approves were present<br>
> I decided that there is no problem to merge.<br>
<br>
The only thing I could see in that PR was a message from the author<br>
saying that they would submit a CLA. There doesn't seem to be a message<br>
saying that the CLA had actually been processed.<br></blockquote><div><br></div><div>Well, then it's my misinterpretation. I saw a author's words and green checkbox related to CLA.</div><div>My fault, sorry.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It's not sufficient for the author to *say* they have submitted a CLA.<br>
It must actually have been submitted, and accepted and be on file.<br>
Sometimes there are problems with submitted CLAs (e.g. missing fields,<br>
or a user sends an ICLA when they really also need a CCLA, etc).<br>
Normally the git hooks would not allow you to push a commit where the<br>
author does not have a CLA. However those checks are suppressed where<br>
"CLA: trivial" appears in the commit description. So we need to be extra<br>
careful in that case, i.e. perhaps we should insist that commits<br>
containing "CLA: trivial" have the line removed if we don't think the<br>
commit really is trivial. This ensures that the git hooks do their job<br>
and we can be absolutely sure that the author has a registered CLA.<br>
<br>
Matt<br>
<br>
> <br>
> Regards,<br>
> <br>
> Paul Yang<br>
> <br>
>> On Dec 12, 2019, at 5:29 PM, Dmitry Belyavsky <<a href="mailto:beldmit@gmail.com" target="_blank">beldmit@gmail.com</a><br>
>> <mailto:<a href="mailto:beldmit@gmail.com" target="_blank">beldmit@gmail.com</a>>> wrote:<br>
>><br>
>> Dear Matt,<br>
>><br>
>> As <br>
>> - the contributor agreed to sign the CLA and <br>
>> - there was a mark that CLA is signed and<br>
>> - all the necessary approves were present<br>
>> I decided that there is no problem to merge.<br>
>><br>
>> BTW, I am not sure the PR was trivial enough.<br>
>><br>
>> Anyway, the responsibility was mine, not the git one :)<br>
>><br>
>><br>
>> On Thu, Dec 12, 2019 at 12:20 PM Matt Caswell <<a href="mailto:matt@openssl.org" target="_blank">matt@openssl.org</a><br>
>> <mailto:<a href="mailto:matt@openssl.org" target="_blank">matt@openssl.org</a>>> wrote:<br>
>><br>
>>     I notice that PR 10594 (Add support for otherName:NAIRealm in output)<br>
>>     got merged yesterday:<br>
>>     <a href="https://github.com/openssl/openssl/pull/10594" rel="noreferrer" target="_blank">https://github.com/openssl/openssl/pull/10594</a><br>
>><br>
>>     The commit description contained "CLA: trivial" and so the "hold: cla<br>
>>     required" label was not automatically applied to the PR. But the<br>
>>     discussion in the PR suggested a CLA should be submitted. But it got<br>
>>     merged anyway! Fortunately the CLA had already been processed -<br>
>>     just not<br>
>>     noted in the PR. So, in this case, it makes no difference.<br>
>><br>
>>     I think this points to a possible flaw in our workflow for dealing<br>
>>     with<br>
>>     trivial changes. Because the "CLA: trivial" header suppresses the<br>
>>     "hold:<br>
>>     cla required" label and the git hooks don't complain when commits get<br>
>>     pushed with the "CLA: trivial" header and no CLA on file, it seems<br>
>>     possible to me that we could push commit all the way through the<br>
>>     process<br>
>>     without the reviewers even realising that the author is claiming<br>
>>     triviality on the commit.<br>
>><br>
>>     Not sure what the solution to that is.<br>
>><br>
>>     Matt<br>
>><br>
>><br>
>><br>
>> -- <br>
>> SY, Dmitry Belyavsky<br>
> <br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">SY, Dmitry Belyavsky</div></div>