From 7f6f620ec43b5606413524d25d3cae1003930f55 Mon Sep 17 00:00:00 2001 From: Florence Blanc-Renaud Date: Fri, 6 Jul 2018 12:47:34 +0200 Subject: [PATCH] ipa client uninstall: clean the state store when restoring hostname When ipa client was installed with the --hostname= option, it stores [network] hostname = (current hostname) in /var/lib/ipa-client/sysrestore/sysrestore.state and changes the hostname from (current hostname) to the value provided in --hostname. During uninstall, the previous hostname is restored but the entry does not get removed from sysrestore.state. As the uninstaller checks if all entries from sysrestore.state have been restored, it warns that some state has not been restored. The fix calls statestore.restore_state() instead of statestore.get_state() as this method also clears the entry. https://pagure.io/freeipa/issue/7620 Reviewed-By: Rob Crittenden Reviewed-By: Rob Crittenden --- ipaplatform/redhat/tasks.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ipaplatform/redhat/tasks.py b/ipaplatform/redhat/tasks.py index dcafd735c0161e5cd979ccb113863e7885a14ab3..6a4270defc9f444f76677bdf08d2a680649664bb 100644 --- a/ipaplatform/redhat/tasks.py +++ b/ipaplatform/redhat/tasks.py @@ -380,7 +380,7 @@ class RedHatTaskNamespace(BaseTaskNamespace): statestore.backup_state('network', 'hostname', old_hostname) def restore_hostname(self, fstore, statestore): - old_hostname = statestore.get_state('network', 'hostname') + old_hostname = statestore.restore_state('network', 'hostname') if old_hostname is not None: try: -- 2.17.1