ACK for Alexander's two-liner fix to ipa-replica-manage, which I broke with the LDAP refactoring.

Patch was posted via Trac:
https://fedorahosted.org/freeipa/ticket/3490

--
PetrĀ³
From a7a2e3e09eb4a1eb7bb005fe4a43ea0e3ac1c5af Mon Sep 17 00:00:00 2001
From: Alexander Bokovoy <aboko...@redhat.com>
Date: Wed, 6 Mar 2013 13:13:58 +0200
Subject: [PATCH 3/3] ipa-replica-manage: migrate to single_value after
 LDAPEntry updates

---
 install/tools/ipa-replica-manage | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/install/tools/ipa-replica-manage b/install/tools/ipa-replica-manage
index 859809b..82648bd 100755
--- a/install/tools/ipa-replica-manage
+++ b/install/tools/ipa-replica-manage
@@ -632,11 +632,11 @@ def del_master(realm, hostname, options):
         this_services = []
         other_services = []
 
-        for master_cn in [m.getValue('cn') for m in masters]:
+        for master_cn in [m.single_value('cn') for m in masters]:
             master_dn = DN(('cn', master_cn), ('cn', 'masters'), ('cn', 'ipa'), ('cn', 'etc'), ipautil.realm_to_suffix(realm))
             services = delrepl.conn.get_entries(master_dn,
                                                 delrepl.conn.SCOPE_ONELEVEL)
-            services_cns = [s.getValue('cn') for s in services]
+            services_cns = [s.single_value('cn') for s in services]
 
             if master_cn == hostname:
                 this_services = services_cns
-- 
1.8.1.4

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to