diff options
| author | Rob Crittenden <rcritten@redhat.com> | 2011-07-19 09:20:03 -0400 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2011-07-18 19:30:43 -0400 |
| commit | e5a5c781f9c1152ff61cd21d649df99f465722c4 (patch) | |
| tree | e5c974a78157370c7199174f4753985172e86427 /ipalib | |
| parent | 07e6353373b4fa2e85942119462d2999eaa0dd39 (diff) | |
| download | freeipa-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.py | 6 |
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']: |