summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTomas Babej <tbabej@redhat.com>2013-12-12 15:01:14 +0100
committerPetr Viktorin <pviktori@redhat.com>2013-12-20 16:04:22 +0100
commit2a2f5ac4e68ed9a819ddc3de1d35366b2e2b58c7 (patch)
tree3c5426df0872eb0663c2e2984700970be204505a
parent1357eade4c5086e6c837a49f3008616317f88e5f (diff)
downloadfreeipa-2a2f5ac4e68ed9a819ddc3de1d35366b2e2b58c7.tar.gz
freeipa-2a2f5ac4e68ed9a819ddc3de1d35366b2e2b58c7.tar.xz
freeipa-2a2f5ac4e68ed9a819ddc3de1d35366b2e2b58c7.zip
Fix incorrect path in error message on sysrestore failure
On sysrestore failure, user is prompted out to remove the sysrestore file. However, the path to the sysrestore file mentioned in the sentence is not correct. https://fedorahosted.org/freeipa/ticket/4080
-rwxr-xr-xinstall/tools/ipa-server-install15
1 files changed, 10 insertions, 5 deletions
diff --git a/install/tools/ipa-server-install b/install/tools/ipa-server-install
index 458ebba5..9c9b205a 100755
--- a/install/tools/ipa-server-install
+++ b/install/tools/ipa-server-install
@@ -89,6 +89,8 @@ VALID_SUBJECT_ATTRS = ['st', 'o', 'ou', 'dnqualifier', 'c',
'incorporationlocality', 'incorporationstate',
'incorporationcountry', 'businesscategory']
+SYSRESTORE_DIR_PATH = '/var/lib/ipa/sysrestore'
+
def subject_callback(option, opt_str, value, parser):
"""
Make sure the certificate subject base is a valid DN
@@ -525,16 +527,19 @@ def uninstall():
sysupgrade.remove_upgrade_file()
if fstore.has_files():
- root_logger.error('Some files have not been restored, see /var/lib/ipa/sysrestore/sysrestore.index')
+ root_logger.error('Some files have not been restored, see %s/sysrestore.index' % SYSRESTORE_DIR_PATH)
has_state = False
for module in IPA_MODULES: # from installutils
if sstore.has_state(module):
- root_logger.error('Some installation state for %s has not been restored, see /var/lib/ipa/sysrestore/sysrestore.state' % module)
+ root_logger.error('Some installation state for %s has not been restored, see %s/sysrestore.state' % (module, SYSRESTORE_DIR_PATH))
has_state = True
rv = 1
if has_state:
- root_logger.error('Some installation state has not been restored.\nThis may cause re-installation to fail.\nIt should be safe to remove /var/lib/ipa/sysrestore.state but it may\nmean your system hasn\'t be restored to its pre-installation state.')
+ root_logger.error('Some installation state has not been restored.\n'
+ 'This may cause re-installation to fail.\n'
+ 'It should be safe to remove %s/sysrestore.state but it may\n'
+ 'mean your system hasn\'t be restored to its pre-installation state.' % SYSRESTORE_DIR_PATH)
# Note that this name will be wrong after the first uninstall.
dirname = dsinstance.config_dirname(dsinstance.realm_to_serverid(api.env.realm))
@@ -603,9 +608,9 @@ def main():
root_logger.debug("missing options might be asked for interactively later\n")
global fstore
- fstore = sysrestore.FileStore('/var/lib/ipa/sysrestore')
+ fstore = sysrestore.FileStore(SYSRESTORE_DIR_PATH)
global sstore
- sstore = sysrestore.StateFile('/var/lib/ipa/sysrestore')
+ sstore = sysrestore.StateFile(SYSRESTORE_DIR_PATH)
# Configuration for ipalib, we will bootstrap and finalize later, after
# we are sure we have the configuration file ready.