summaryrefslogtreecommitdiffstats
path: root/ipalib
diff options
context:
space:
mode:
authorRob Crittenden <rcritten@redhat.com>2011-07-19 09:20:03 -0400
committerRob Crittenden <rcritten@redhat.com>2011-07-18 19:30:43 -0400
commite5a5c781f9c1152ff61cd21d649df99f465722c4 (patch)
treee5c974a78157370c7199174f4753985172e86427 /ipalib
parent07e6353373b4fa2e85942119462d2999eaa0dd39 (diff)
downloadfreeipa-e5a5c781f9c1152ff61cd21d649df99f465722c4.tar.gz
freeipa-e5a5c781f9c1152ff61cd21d649df99f465722c4.tar.xz
freeipa-e5a5c781f9c1152ff61cd21d649df99f465722c4.zip
A removed external host is shown in output when removing external hosts.
This is just a display problem, the host is actually removed from the entry. https://fedorahosted.org/freeipa/ticket/1492
Diffstat (limited to 'ipalib')
-rw-r--r--ipalib/plugins/netgroup.py6
1 files changed, 3 insertions, 3 deletions
diff --git a/ipalib/plugins/netgroup.py b/ipalib/plugins/netgroup.py
index b0c3b652a..97c58a502 100644
--- a/ipalib/plugins/netgroup.py
+++ b/ipalib/plugins/netgroup.py
@@ -236,7 +236,7 @@ class netgroup_add_member(LDAPAddMember):
# hosts that aren't stored in IPA, aka external hosts.
if 'memberhost' in failed and 'host' in failed['memberhost']:
(dn, entry_attrs_) = ldap.get_entry(dn, ['externalhost'])
- members = entry_attrs.get('memberhost', [])
+ members = entry_attrs_.get('memberhost', [])
external_hosts = entry_attrs_.get('externalhost', [])
failed_hosts = []
for host in failed['memberhost']['host']:
@@ -270,8 +270,8 @@ class netgroup_remove_member(LDAPRemoveMember):
# Run through the host failures and gracefully remove any defined as
# as an externalhost.
if 'memberhost' in failed and 'host' in failed['memberhost']:
- (dn, entry_attrs) = ldap.get_entry(dn, ['externalhost'])
- external_hosts = entry_attrs.get('externalhost', [])
+ (dn, entry_attrs_) = ldap.get_entry(dn, ['externalhost'])
+ external_hosts = entry_attrs_.get('externalhost', [])
failed_hosts = []
completed_external = 0
for host in failed['memberhost']['host']: