Make check_inst() a standalone function in bindinstance.

When an install instance is created that contains a pointer to a sysrestore
point it loads in the current configuration when instantiated. If an
instance is instantiated but not used then changes may occur to the
system state that it is unaware of. So one needs to take care in the order
that things are done to avoid losing information.

When bind was setup it was overwriting all data in sysrestore.state and
leaving just a [named] section. This caused problems at uninstall.

448173

1 files changed, 2 insertions, 2 deletions

diff --git a/ipa-server/ipa-install/ipa-server-install b/ipa-server/ipa-install/ipa-server-install
index 9b0baa24c..de1add068 100644
--- a/ipa-server/ipa-install/ipa-server-install
+++ b/ipa-server/ipa-install/ipa-server-install
@@ -365,9 +365,8 @@ def main():
     admin_password = ""
 
     # check bind packages are installed
-    bind = ipaserver.bindinstance.BindInstance()
     if options.setup_bind:
-        if not bind.check_inst():
+        if not ipaserver.bindinstance.check_inst():
             print "--setup-bind was specified but bind is not installed on the system"
             print "Please install bind and restart the setup program"
             return 1
@@ -502,6 +501,7 @@ def main():
     webgui = ipaserver.httpinstance.WebGuiInstance()
     webgui.create_instance()
 
+    bind = ipaserver.bindinstance.BindInstance(fstore)
     bind.setup(host_name, ip_address, realm_name, domain_name)
     if options.setup_bind:
         bind.create_instance()