On 12/01/2015 12:28 PM, Martin Babinsky wrote:
On 12/01/2015 12:21 PM, Martin Babinsky wrote:
This patch fixes a regression caused by recently pushed
topologysuffix-related patches.
self-NACK.
This patch should actually work.
--
Martin^3 Babinsky
From 1357e9ab7a01875afd29c5282b38ee65bb92d966 Mon Sep 17 00:00:00 2001
From: Martin Babinsky
Date: Tue, 1 Dec 2015 12:14:07 +0100
Subject: [PATCH] fix 'iparepltopomanagedsuffix' attribute consumers
Commit 46ae52569a179f73b1445922f7bac993d598c953 reimplemented reporting of
managed topology suffixes in server-find/show commands using membership
attributes. This patch fixes consumers of this attribute in ipa-replica-manage
command and webui to reflect this change.
---
install/tools/ipa-replica-manage | 8 ++--
install/ui/src/freeipa/topology.js | 4 ++--
2 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/install/tools/ipa-replica-manage b/install/tools/ipa-replica-manage
index 085aa29b7904958ced5281a2946a2d318ac371f6..7bbef3593ef291b4af209d7f59c21e23d3d22944 100755
--- a/install/tools/ipa-replica-manage
+++ b/install/tools/ipa-replica-manage
@@ -573,14 +573,10 @@ def check_last_link(delrepl, realm, dirman_passwd, force):
def map_masters_to_suffixes(masters, suffixes):
masters_to_suffix = {}
-suffix_name_to_root = {
-s['iparepltopoconfroot'][0]: s['cn'][0] for s in suffixes
-}
for master in masters:
-managed_suffixes = master['iparepltopomanagedsuffix']
-for suffix in managed_suffixes:
-suffix_name = suffix_name_to_root[suffix]
+managed_suffixes = master['iparepltopomanagedsuffix_topologysuffix']
+for suffix_name in managed_suffixes:
try:
masters_to_suffix[suffix_name].append(master)
except KeyError:
diff --git a/install/ui/src/freeipa/topology.js b/install/ui/src/freeipa/topology.js
index 4c77f1b1b2660cdb78ffe6c5299852d7b1d316fa..8e74918bf237dba335ce2f63441423191b2f89e8 100644
--- a/install/ui/src/freeipa/topology.js
+++ b/install/ui/src/freeipa/topology.js
@@ -206,7 +206,7 @@ return {
'cn',
'ipamindomainlevel',
'ipamaxdomainlevel',
-'iparepltopomanagedsuffix'
+'iparepltopomanagedsuffix_topologysuffix'
]
},
{
@@ -220,7 +220,7 @@ return {
{ name: 'cn', read_only: true },
{ name: 'ipamindomainlevel', read_only: true },
{ name: 'ipamaxdomainlevel', read_only: true },
-{ $type: 'multivalued', name: 'iparepltopomanagedsuffix', read_only: true }
+{ $type: 'multivalued', name: 'iparepltopomanagedsuffix_topologysuffix', read_only: true }
]
}
]
--
2.5.0
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code