DogtagInstance.enable() overrides Service.enable() and does nothing usefulll. Also removing it solves bug discovered recently in uninstall procedure.
--
David Kupka
From dbf020b1703a9cf2f4d3614f4c9caa83f340f571 Mon Sep 17 00:00:00 2001
From: David Kupka <dku...@redhat.com>
Date: Tue, 12 May 2015 12:43:36 +0200
Subject: [PATCH] Remove unused enable() method from DogtagInstance.

This method only overrides the enable() method from Service class
and causes uninstallation to fail.
---
 ipaserver/install/dogtaginstance.py | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/ipaserver/install/dogtaginstance.py b/ipaserver/install/dogtaginstance.py
index 98929b8640fc91f1c520ddc4c21630464779f2df..33f39f7930b4151200f2880d02a0bc2c152c0025 100644
--- a/ipaserver/install/dogtaginstance.py
+++ b/ipaserver/install/dogtaginstance.py
@@ -178,14 +178,6 @@ class DogtagInstance(service.Service):
         except ipautil.CalledProcessError, e:
             self.handle_setup_error(e)
 
-    def enable(self):
-        self.backup_state("enabled", self.is_enabled())
-        # We do not let the system start IPA components on its own,
-        # Instead we reply on the IPA init script to start only enabled
-        # components as found in our LDAP configuration tree
-        # We need to install DS before we can actually ldap_enable a service.
-        # so actual enablement is delayed.
-
     def restart_instance(self):
         try:
             self.restart(self.dogtag_constants.PKI_INSTANCE_NAME)
-- 
2.4.0

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to