From e5a5c781f9c1152ff61cd21d649df99f465722c4 Mon Sep 17 00:00:00 2001 From: Rob Crittenden Date: Tue, 19 Jul 2011 09:20:03 -0400 Subject: 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 --- ipalib/plugins/netgroup.py | 6 +++--- 1 file 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']: -- cgit