<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Nov 12, 2015 at 4:45 PM, Nir Soffer <span dir="ltr"><<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi all,<br>
<br>
Our QE (and sometimes our users) are working hard testing and<br>
reporting bugs, but<br>
their effort is never mentioned in our code.<br>
<br>
Looking at kernel git history, I found that they are using the<br>
Reported-By header for<br>
giving credit to the person reporting a bug. I suggest we adapt this header.<br>
<br>
Here are some examples how we can use it:<br>
<br>
- <a href="https://gerrit.ovirt.org/#/c/48483/3//COMMIT_MSG" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/48483/3//COMMIT_MSG</a><br>
- <a href="https://github.com/oVirt/vdsm/commit/fb4c72af5e4c200409c74834111d44d92959ebbd" rel="noreferrer" target="_blank">https://github.com/oVirt/vdsm/commit/fb4c72af5e4c200409c74834111d44d92959ebbd</a><br>
- <a href="https://github.com/oVirt/vdsm/commit/f8127d88add881a4775e7030dde2433125c7b598" rel="noreferrer" target="_blank">https://github.com/oVirt/vdsm/commit/f8127d88add881a4775e7030dde2433125c7b598</a><br>
<br>
Thanks,<br>
Nir<br>
_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/devel</a><br>
</blockquote></div><br></div><div class="gmail_extra">+1</div><div class="gmail_extra"><br></div><div class="gmail_extra">In case they help with the verification of the change, I suggest adding Tested-By which is just as important.</div><div class="gmail_extra">From <a href="https://www.kernel.org/doc/Documentation/SubmittingPatches">https://www.kernel.org/doc/Documentation/SubmittingPatches</a> :</div><div class="gmail_extra"><br></div><div class="gmail_extra"><pre style="color:rgb(0,0,0);word-wrap:break-word;white-space:pre-wrap">A Tested-by: tag indicates that the patch has been successfully tested (in
some environment) by the person named. This tag informs maintainers that
some testing has been performed, provides a means to locate testers for
future patches, and ensures credit for the testers.</pre><pre style="color:rgb(0,0,0);word-wrap:break-word;white-space:pre-wrap"><br></pre><pre style="color:rgb(0,0,0);word-wrap:break-word;white-space:pre-wrap">Y.</pre></div></div>