<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p><br>
</p>
<br>
<div class="moz-cite-prefix">On 02.06.2016 14:58, Florence
Blanc-Renaud wrote:<br>
</div>
<blockquote
cite="mid:03e2c4d9-54b3-c3ac-537a-e6c9069af409@redhat.com"
type="cite">
<meta http-equiv="content-type" content="text/html;
charset=windows-1252">
<p>Hi,</p>
<p>this patch modifies ipa-replica-conncheck when it performs the
SSH connection to the master, so that the username is always
fully qualified.<br>
</p>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://fedorahosted.org/freeipa/ticket/5812">https://fedorahosted.org/freeipa/ticket/5812</a><br>
<pre class="moz-signature" cols="72">--
Florence Blanc-Renaud
Identity Management Team, Red Hat</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
<br>
LGTM, but because current issues with replica install in master
branch, I couldn't test it and I would like to be sure that
ipa-replica-install using NTP will work too<br>
<br>
<br>
Just little nitpick, for better readibility, 'command' should be on
new line<br>
- '%s@%s' % (self.user, self.addr), command<br>
+ '-o User=%s' % self.user,<br>
+ '%s' % self.addr, command<br>
<br>
Martin^2<br>
</body>
</html>