<html><head><style type='text/css'>p { margin: 0; }</style></head><body><div style='font-family: Times New Roman; font-size: 12pt; color: #000000'><span>You're welcome.  <br><br>As for the "</span>Signed-off-by" line, sure.  I disabled that in my .gitconfig because another GIT repo I use doesn't like it.  I'll have to investigate enabling/disabling it per repo.<br><br>Cheers,<br>Keith<br><span id="aac1f842-56a9-4bee-8bd8-ba034af2c5c4"><br><span name="x"></span><img src="http://www.redhat.com/g/chrome/logo_rh_home.png"><br>Keith Robertson</span>Signed-off-by<br><span id="aac1f842-56a9-4bee-8bd8-ba034af2c5c4">1-919-754-4004<br><span name="x"></span><br></span><br><hr id="zwchr"><b>From: </b>"Bryn M. Reeves" <bmr@redhat.com><br><b>To: </b>"Keith Robertson" <kroberts@redhat.com><br><b>Cc: </b>sos-devel@redhat.com<br><b>Sent: </b>Tuesday, May 10, 2011 10:08:11 AM<br><b>Subject: </b>Re: [sos-devel] [PATCH 2/2] iscsi enhancements 1. Added some useful commands to get more info about iscsi devices<br><br>On 05/10/2011 02:59 PM, Keith Robertson wrote:<br>> ---<br>>  sos/plugins/iscsi.py |    2 ++<br>>  1 files changed, 2 insertions(+), 0 deletions(-)<br><br>Ack.<br><br>Thanks again!<br><br>May I assume a Signed-off-by line for both patches? We're trying to get all<br>contributions to include a DCO now (again, something I need to document!).<br><br>Have a couple of comments on the other patch (nothing major, just cmdline opts<br>etc.) but I have a call in a moment so will have to get back to you a little later.<br><br>Regards,<br>Bryn.<br><br></div></body></html>