Update configuration.py

The code that enables/disables the instance in configuration.py
has been moved into instance_layout.py.

Author: edewata

base/server/python/pki/server/__init__.py

@@ -1252,6 +1252,10 @@ def undeploy_webapp(
             'localhost',
             webapp_id + '.xml')
 
+        #if not os.path.exists(context_xml):
+        #    logger.info('%s web application is not deployed', webapp_id)
+        #    return
+
         logger.info('Removing %s', context_xml)
         pki.util.remove(context_xml, force=force)
 

base/server/python/pki/server/deployment/__init__.py

@@ -5839,11 +5839,6 @@ def destroy(self):
         scriptlet.instance = self.instance
         scriptlet.destroy(self)
 
-        scriptlet = pki.server.deployment.scriptlets.finalization.PkiScriptlet()
-        scriptlet.deployer = self
-        scriptlet.instance = self.instance
-        scriptlet.destroy(self)
-
         scriptlet = pki.server.deployment.scriptlets.fapolicy_setup.PkiScriptlet()
         scriptlet.deployer = self
         scriptlet.instance = self.instance

base/server/python/pki/server/deployment/scriptlets/configuration.py

@@ -226,13 +226,6 @@ def spawn(self, deployer):
 
         if config.str2bool(deployer.mdict['pki_systemd_service_create']):
 
-            # Optionally, programmatically 'enable' the configured PKI instance
-            # to be started upon system boot (default is True)
-            if not config.str2bool(deployer.mdict['pki_enable_on_system_boot']):
-                instance.disable()
-            else:
-                instance.enable()
-
             if (len(instance.get_subsystems()) == 1 or
                     config.str2bool(deployer.mdict['pki_hsm_enable'])):
                 logger.info('Starting PKI server')

base/server/python/pki/server/deployment/scriptlets/finalization.py

@@ -79,11 +79,3 @@ def spawn(self, deployer):
         logger.info(log.PKISPAWN_END_MESSAGE_2,
                     deployer.subsystem_type,
                     instance.name)
-
-    def destroy(self, deployer):
-
-        instance = self.instance
-
-        logger.info(log.PKIDESTROY_END_MESSAGE_2,
-                    deployer.subsystem_type,
-                    instance.name)

base/server/python/pki/server/deployment/scriptlets/instance_layout.py

@@ -224,17 +224,15 @@ def spawn(self, deployer):
             deployer.write_systemd_overrides()
             deployer.systemd.daemon_reload()
 
-            # Link /etc/systemd/system/pki-tomcatd.target.wants/pki-tomcatd@<instance>.service
-            # to /lib/systemd/system/pki-tomcatd@.service
+            # Optionally, programmatically 'enable' the configured PKI instance
+            # to be started upon system boot (default is True)
 
-            systemd_service_link = os.path.join(
-                pki.server.instance.PKIInstance.TARGET_WANTS,
-                instance.service_name + '.service')
-
-            instance.symlink(
-                pki.server.instance.PKIInstance.UNIT_FILE,
-                systemd_service_link,
-                exist_ok=True)
+            if config.str2bool(deployer.mdict['pki_enable_on_system_boot']):
+                '''
+                this command creates /etc/systemd/system/pki-tomcatd.target.want/pki-tomcatd@<instance>.service
+                or /etc/systemd/system/pki-tomcatd-nuxwdog.target.wants/pki-tomcatd-nuxwdog@<instance>.service
+                '''  # noqa: E501
+                instance.enable()
 
     def destroy(self, deployer):
 
@@ -255,11 +253,14 @@ def destroy(self, deployer):
                 timeout=deployer.request_timeout)
             return
 
-        logger.info('Removing %s instance', instance.name)
+        logger.info('Disabling PKI server')
+        '''
+        this command removes /etc/systemd/system/pki-tomcatd.target.want/pki-tomcatd@<instance>.service
+        or /etc/systemd/system/pki-tomcatd-nuxwdog.target.wants/pki-tomcatd-nuxwdog@<instance>.service
+        '''  # noqa: E501
+        instance.disable()
 
-        logger.info('Removing %s', deployer.systemd.systemd_link)
-        pki.util.unlink(link=deployer.systemd.systemd_link,
-                        force=deployer.force)
+        logger.info('Removing %s instance', instance.name)
 
         if os.path.exists(deployer.systemd.base_override_dir):
             logger.info('Removing %s', deployer.systemd.base_override_dir)
@@ -309,6 +310,3 @@ def destroy(self, deployer):
             # Remove /var/lib/pki/<instance> if empty
             logger.info('Removing %s', instance.base_dir)
             pki.util.rmtree(path=instance.base_dir, force=deployer.force)
-
-        logger.info('Disabling PKI server')
-        instance.disable()