<div dir="ltr"><div>Thanks. Updated patch attached.<br><br><br></div>Gabe<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 20, 2015 at 10:36 AM, Martin Babinsky <span dir="ltr"><<a href="mailto:mbabinsk@redhat.com" target="_blank">mbabinsk@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 11/20/2015 04:02 PM, Gabe Alford wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
Fix for <a href="https://fedorahosted.org/freeipa/ticket/5470" rel="noreferrer" target="_blank">https://fedorahosted.org/freeipa/ticket/5470</a><br>
<br>
Thanks,<br>
<br>
Gabe<br>
<br>
<br>
</blockquote></div></div>
Hi Gabe,<br>
<br>
patch looks good. IMHO it would be better if you moved the check before API initialization like so:<br>
<br>
"""<br>
@@ -340,6 +340,12 @@ class WinsyncMigrate(admintool.AdminTool):<br>
         the plumbing.<br>
         """<br>
<br>
+        # Check if the IPA server is configured before attempting to migrate<br>
+        try:<br>
+            installutils.check_server_configuration()<br>
+        except RuntimeError as e:<br>
+            sys.exit(e)<br>
+<br>
         # Finalize API<br>
         api.bootstrap(in_server=True, context='server')<br>
         api.finalize()<br>
"""<br>
<br>
There's no point in initializing API if there is no server installed.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Martin^3 Babinsky<br>
</font></span></blockquote></div><br></div>