summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexander Bokovoy <abokovoy@redhat.com>2011-07-01 12:15:12 +0300
committerAlexander Bokovoy <abokovoy@redhat.com>2011-07-01 12:19:23 +0300
commitd29143ce5f6364dfc93dd0228dc58199f956b0a6 (patch)
treeef922b914704860b99ebb0723916d004971b88fe
parentc746abd63ac544273a2ddec44a3845e4eb880e62 (diff)
downloadfreeipa-d29143ce5f6364dfc93dd0228dc58199f956b0a6.zip
freeipa-d29143ce5f6364dfc93dd0228dc58199f956b0a6.tar.gz
freeipa-d29143ce5f6364dfc93dd0228dc58199f956b0a6.tar.xz
Make error reporting more 'local' for various configurations of nss_ldap packages
https://fedorahosted.org/freeipa/ticket/1369 When nss_ldap-based configuration does not work, report proper package name instead of always assuming nss_ldap. At least, in RHEL6 and Fedora appropriate package is called nss-pam-ldapd while in older releases and other distributions it might be called differently. The change makes less confusing error reporting. It also introduces common utility function package_installed_name() which provides an interface to query package manager for existence of mutually exclusive packages which is helpful to distinguish between different configuration paths.
-rwxr-xr-xipa-client/ipa-install/ipa-client-install3
-rw-r--r--ipapython/ipautil.py18
2 files changed, 20 insertions, 1 deletions
diff --git a/ipa-client/ipa-install/ipa-client-install b/ipa-client/ipa-install/ipa-client-install
index 884dd21..d3218d3 100755
--- a/ipa-client/ipa-install/ipa-client-install
+++ b/ipa-client/ipa-install/ipa-client-install
@@ -974,7 +974,8 @@ def main():
n = n + 1
if not found:
- print "nss_ldap is not able to use DNS discovery!"
+ package = package_installed_name(["nss-pam-ldapd","nss_ldap"])
+ print "Unable to use DNS discovery! Recognized configuration: %s" % (package)
print "Changing configuration to use hardcoded server name: " +cli_server
try:
diff --git a/ipapython/ipautil.py b/ipapython/ipautil.py
index 91d19e9..1573e0a 100644
--- a/ipapython/ipautil.py
+++ b/ipapython/ipautil.py
@@ -1169,3 +1169,21 @@ def bind_port_responder(port, socket_stream=True, socket_timeout=None, responder
s.sendto(responder_data, addr)
finally:
s.close()
+
+def package_installed_name(packages):
+ """
+ Find out which of mutually exclusive packages is installed
+
+ packages is a list of package names to check
+
+ Returns package name or None
+ """
+
+ args = ["/bin/rpm","-q","--queryformat","%{NAME}"]
+ for package in packages:
+ try:
+ (package_name, error, retcode) = run(args+[package])
+ return package_name
+ except CalledProcessError:
+ continue
+ return None