<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 06/06/2016 07:18 PM, Martin Basti
wrote:<br>
</div>
<blockquote
cite="mid:69456999-093e-1d4a-5ff7-6f5cd13e1712@redhat.com"
type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<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>
</blockquote>
<p><font size="-1">Hi Martin,</font></p>
<p><font size="-1">thanks for the review. I am attaching a new patch
with your suggestion. Just for my record, what would be the
command-line options to test the scenario you're referring to?<br>
</font></p>
<p><font size="-1">Flo.</font><br>
</p>
</body>
</html>