<font size=2 face="sans-serif">IBM uses the "signed off by"
feature to enforce/track code reviews before any code gets contributed
to ANL. The intent is that the commit author != the commit signer when
the code goes out. Often a developer at ANL will also review the code before
it gets integrated into the master branch (IBM developers don't have push
authority) and sign the commit again.</font>
<br>
<br><font size=2 face="sans-serif">We use the text "Signed-off-by:xxxx
<yyy@zzz>" simply because this is what is supported by the git
tools. You can use commands like `git cherry-pick -s`, `git commit --amend
-s`, and `git am -s` to ease the signing effort. </font>
<br><font size=2 face="sans-serif"><br>
Michael Blocksome<br>
Parallel Environment MPI Middleware Team Lead, TCEM<br>
POWER, x86, and Blue Gene HPC Messaging <br>
blocksom@us.ibm.com<br>
</font>
<br>
<br>
<br>
<br><font size=1 color=#5f5f5f face="sans-serif">From:      
 </font><font size=1 face="sans-serif">Pavan Balaji <balaji@mcs.anl.gov></font>
<br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">discuss@mpich.org,
</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">01/04/2014 10:00 PM</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">Re: [mpich-discuss]
Use of Signed-off-by in MPICH</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Sent by:    
   </font><font size=1 face="sans-serif">discuss-bounces@mpich.org</font>
<br>
<hr noshade>
<br>
<br>
<br><tt><font size=2><br>
Please use devel@mpich.org for developer oriented discussion.<br>
<br>
We use signoff as a way to get a second confirmation on the correctness
of the patch.  In some cases, multiple confirmations.  There’s
also a "silent signoff” done by the jenkins testing before things
are merged into mpich/master.<br>
<br>
  — Pavan<br>
<br>
On Jan 4, 2014, at 7:55 PM, Jed Brown <jedbrown@mcs.anl.gov> wrote:<br>
<br>
> I'm confused by the current use of Signed-off-by tags in MPICH.  Is
this<br>
> intended to be a developer's certificate of origin (DCO) following
the<br>
> model of the Linux kernel [1]?  I can't tell if IBM is using
it that<br>
> way, where Michael Blocksome is the one authorized by IBM to assert
on<br>
> IBM's behalf that the work has clean IP and may be distributed under
the<br>
> MPICH license.  But what does this mean?<br>
> <br>
> commit 87b20136ac20a0fbcffd8e29edcc500fcebbfc22<br>
> Author:     Pavan Balaji <balaji@mcs.anl.gov><br>
> AuthorDate: Wed Jan 1 19:33:36 2014 -0600<br>
> Commit:     Pavan Balaji <balaji@mcs.anl.gov><br>
> CommitDate: Thu Jan 2 15:39:07 2014 -0600<br>
> <br>
>    Move default eager threshold setting to ch3.<br>
> <br>
>    We were setting the default eager threshold inside nemesis,
instead of<br>
>    in ch3.  This was causing other ch3 channels, notably
sock, to get a<br>
>    garbage default value for the eager threshold.<br>
> <br>
>    Signed-off-by: Wesley Bland <wbland@mcs.anl.gov><br>
> <br>
> <br>
> If this were a DCO, we would interpret it as meaning that Wesley is
the<br>
> one certifying the IP, but Pavan is the author.<br>
> <br>
> Or is Signed-off-by being used where Reviewed-by would be more<br>
> appropriate?<br>
> <br>
> <br>
> [1] </font></tt><a href=https://www.kernel.org/doc/Documentation/SubmittingPatches><tt><font size=2>https://www.kernel.org/doc/Documentation/SubmittingPatches</font></tt></a><tt><font size=2><br>
> _______________________________________________<br>
> discuss mailing list     discuss@mpich.org<br>
> To manage subscription options or unsubscribe:<br>
> </font></tt><a href=https://lists.mpich.org/mailman/listinfo/discuss><tt><font size=2>https://lists.mpich.org/mailman/listinfo/discuss</font></tt></a><tt><font size=2><br>
<br>
--<br>
Pavan Balaji<br>
</font></tt><a href=http://www.mcs.anl.gov/~balaji><tt><font size=2>http://www.mcs.anl.gov/~balaji</font></tt></a><tt><font size=2><br>
<br>
_______________________________________________<br>
discuss mailing list     discuss@mpich.org<br>
To manage subscription options or unsubscribe:<br>
</font></tt><a href=https://lists.mpich.org/mailman/listinfo/discuss><tt><font size=2>https://lists.mpich.org/mailman/listinfo/discuss</font></tt></a><tt><font size=2><br>
<br>
</font></tt>
<br>