Vishnu Sreekumar has posted comments on this change. Change subject: Passed pep8 and pyflakes. Use a global variable for the ssh command + common options. Updated all the email addresses as given in Code-review-1 Use argparse for command line options. Print return code, error and output on failures. Signed-off-by: Vishnu S ......................................................................
Patch Set 3: Please ignore patch set 2. I ammended the commit without pushing the changes. :) -- To view, visit http://gerrit.ovirt.org/37426 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic911411a61e097c734d0199cc04057c393b974f9 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Vishnu Sreekumar <vishnu.sr...@gmail.com> Gerrit-Reviewer: Barak Korren <bkor...@redhat.com> Gerrit-Reviewer: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: Eyal Edri <ee...@redhat.com> Gerrit-Reviewer: Vishnu Sreekumar <vishnu.sr...@gmail.com> Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches