diff options
author | Tomas Babej <tomasbabej@gmail.com> | 2013-11-21 15:25:27 +0100 |
---|---|---|
committer | Petr Viktorin <pviktori@redhat.com> | 2013-11-26 15:23:42 +0100 |
commit | d361e12ae55f391a13b613a7220c164f503954cc (patch) | |
tree | f4cc3f1882030c05df02fa3245b723bc85cf5362 /ipaserver | |
parent | 76c7f24919d30fdd53e4a1cd32880b55c2437ace (diff) | |
download | freeipa-d361e12ae55f391a13b613a7220c164f503954cc.tar.gz freeipa-d361e12ae55f391a13b613a7220c164f503954cc.tar.xz freeipa-d361e12ae55f391a13b613a7220c164f503954cc.zip |
trusts: Always stop and disable smb service on uninstall
https://fedorahosted.org/freeipa/ticket/4042
Diffstat (limited to 'ipaserver')
-rw-r--r-- | ipaserver/install/adtrustinstance.py | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/ipaserver/install/adtrustinstance.py b/ipaserver/install/adtrustinstance.py index 5e3d0acbb..2f1c99949 100644 --- a/ipaserver/install/adtrustinstance.py +++ b/ipaserver/install/adtrustinstance.py @@ -881,11 +881,16 @@ class ADTRUSTInstance(service.Service): if self.is_configured(): self.print_msg("Unconfiguring %s" % self.service_name) - running = self.restore_state("running") - enabled = self.restore_state("enabled") + # Call restore_state so that we do not leave mess in the statestore + # Otherwise this does nothing + self.restore_state("running") + self.restore_state("enabled") + # Always try to stop and disable smb service, since we do not leave + # working configuration after uninstall try: self.stop() + self.disable() except: pass @@ -917,9 +922,3 @@ class ADTRUSTInstance(service.Service): # Remove our keys from samba's keytab self.clean_samba_keytab() - - if not enabled is None and not enabled: - self.disable() - - if not running is None and running: - self.start() |