see
https://www.redhat.com/archives/freeipa-devel/2015-November/msg00485.html
--
Petr Vobornik
From ae546045ff286b7787e90e6eed945c70463404a8 Mon Sep 17 00:00:00 2001
From: Petr Vobornik <pvobo...@redhat.com>
Date: Fri, 27 Nov 2015 17:00:23 +0100
Subject: [PATCH] rename topology suffixes to "domain" and "ca"
https://www.redhat.com/archives/freeipa-devel/2015-November/msg00485.html
---
install/share/ca-topology.uldif | 4 ++--
install/share/topology-entries.ldif | 4 ++--
install/updates/20-replication.update | 4 ++--
ipalib/constants.py | 3 +++
ipatests/test_integration/tasks.py | 5 +++--
ipatests/test_integration/test_topology.py | 15 ++++++++-------
6 files changed, 20 insertions(+), 15 deletions(-)
diff --git a/install/share/ca-topology.uldif b/install/share/ca-topology.uldif
index 8c9998cbcd151664a3bee881df346e6636b5167b..7ce3cb18bab94a2d5f6e7bef09f0964c2971759f 100644
--- a/install/share/ca-topology.uldif
+++ b/install/share/ca-topology.uldif
@@ -4,11 +4,11 @@ add: objectclass: ipaReplTopoManagedServer
add: ipaReplTopoManagedSuffix: o=ipaca
# add IPA CA topology configuration area
-dn: cn=ipaca,cn=topology,cn=ipa,cn=etc,$SUFFIX
+dn: cn=ca,cn=topology,cn=ipa,cn=etc,$SUFFIX
default: objectclass: top
default: objectclass: iparepltopoconf
default: ipaReplTopoConfRoot: o=ipaca
-default: cn: ipaca
+default: cn: ca
# Update CA replication settings
dn: cn=o\3Dipaca,cn=mapping tree,cn=config
diff --git a/install/share/topology-entries.ldif b/install/share/topology-entries.ldif
index 0f92a882601336bd7037dd9f10b12a0f41610795..d6c952b94c735cd9a9aa35342b197e57b2f60d11 100644
--- a/install/share/topology-entries.ldif
+++ b/install/share/topology-entries.ldif
@@ -5,7 +5,7 @@ objectclass: nsContainer
cn: topology
# default topology configuration area
-dn: cn=realm,cn=topology,cn=ipa,cn=etc,$SUFFIX
+dn: cn=domain,cn=topology,cn=ipa,cn=etc,$SUFFIX
changetype: add
objectclass: top
objectclass: iparepltopoconf
@@ -13,4 +13,4 @@ ipaReplTopoConfRoot: $SUFFIX
nsDS5ReplicatedAttributeList: $EXCLUDES
nsDS5ReplicatedAttributeListTotal: $TOTAL_EXCLUDES
nsds5ReplicaStripAttrs: $STRIP_ATTRS
-cn: realm
+cn: domain
diff --git a/install/updates/20-replication.update b/install/updates/20-replication.update
index 34b47e4ac5e0bbd74a917ca60d0e339c6f1c6650..a471742532cf5954be1b76dbe4a6d908e4cefa2c 100644
--- a/install/updates/20-replication.update
+++ b/install/updates/20-replication.update
@@ -22,11 +22,11 @@ default: objectclass: nsContainer
default: cn: topology
# Default topology configuration area
-dn: cn=realm,cn=topology,cn=ipa,cn=etc,$SUFFIX
+dn: cn=domain,cn=topology,cn=ipa,cn=etc,$SUFFIX
default: objectclass: top
default: objectclass: iparepltopoconf
default: ipaReplTopoConfRoot: $SUFFIX
-default: cn: realm
+default: cn: domain
add: nsDS5ReplicatedAttributeList: $EXCLUDES
add: nsDS5ReplicatedAttributeListTotal: $TOTAL_EXCLUDES
add: nsds5ReplicaStripAttrs: $STRIP_ATTRS
diff --git a/ipalib/constants.py b/ipalib/constants.py
index fc0560ba4fe746f11e8ff3175508ace2e50c3187..68c7eb172da138b49270af5c49fc8441db2c1653 100644
--- a/ipalib/constants.py
+++ b/ipalib/constants.py
@@ -258,3 +258,6 @@ REPL_AGMT_STRIP_ATTRS = ('modifiersName',
'modifyTimestamp',
'internalModifiersName',
'internalModifyTimestamp')
+
+DOMAIN_SUFFIX = 'domain'
+CA_SUFFIX = 'ca'
diff --git a/ipatests/test_integration/tasks.py b/ipatests/test_integration/tasks.py
index 0ee50a8e289b0648103ed86d5739d0cce08aa070..8392c87467529f51730169880af50667e91bcdab 100644
--- a/ipatests/test_integration/tasks.py
+++ b/ipatests/test_integration/tasks.py
@@ -38,6 +38,7 @@ from ipatests.test_integration import util
from ipatests.test_integration.env_config import env_to_script
from ipatests.test_integration.host import Host
from ipalib.util import get_reverse_zone_default
+from ipalib.constants import DOMAIN_SUFFIX
log = log_mgr.get_logger(__name__)
@@ -659,7 +660,7 @@ def create_segment(master, leftnode, rightnode):
lefthost = leftnode.hostname
righthost = rightnode.hostname
segment_name = "%s-to-%s" % (lefthost, righthost)
- result = master.run_command(["ipa", "topologysegment-add", "realm",
+ result = master.run_command(["ipa", "topologysegment-add", DOMAIN_SUFFIX,
segment_name,
"--leftnode=%s" % lefthost,
"--rightnode=%s" % righthost], raiseonerr=False)
@@ -681,7 +682,7 @@ def destroy_segment(master, segment_name):
kinit_admin(master)
command = ["ipa",
"topologysegment-del",
- "realm",
+ DOMAIN_SUFFIX,
segment_name]
result = master.run_command(command, raiseonerr=False)
return result.returncode, result.stderr_text
diff --git a/ipatests/test_integration/test_topology.py b/ipatests/test_integration/test_topology.py
index 2bd134a84b160536ad5804efd8b4325de8bd9252..d005551cedea81e2816ad42e99da248303fa5573 100644
--- a/ipatests/test_integration/test_topology.py
+++ b/ipatests/test_integration/test_topology.py
@@ -9,6 +9,7 @@ import pytest
from ipatests.test_integration.base import IntegrationTest
from ipatests.test_integration import tasks
from env_config import get_global_config
+from ipalib.constants import DOMAIN_SUFFIX
config = get_global_config()
reasoning = "Topology plugin disabled due to domain level 0"
@@ -59,7 +60,7 @@ class TestTopologyOptions(IntegrationTest):
"""
tasks.kinit_admin(self.master)
result1 = self.master.run_command(['ipa', 'topologysegment-find',
- 'realm'])
+ DOMAIN_SUFFIX])
first_segment_name = "%s-to-%s" % (self.master.hostname,
self.replicas[0].hostname)
output1 = result1.stdout_text
@@ -73,11 +74,11 @@ class TestTopologyOptions(IntegrationTest):
# We need to make sure topology information is consistent across all
# replicas
result2 = self.master.run_command(['ipa', 'topologysegment-find',
- 'realm'])
+ DOMAIN_SUFFIX])
result3 = self.replicas[0].run_command(['ipa', 'topologysegment-find',
- 'realm'])
+ DOMAIN_SUFFIX])
result4 = self.replicas[1].run_command(['ipa', 'topologysegment-find',
- 'realm'])
+ DOMAIN_SUFFIX])
segments = self.tokenize_topologies(result2.stdout_text)
assert(len(segments) == 2)
assert(result2.stdout_text == result3.stdout_text)
@@ -87,7 +88,7 @@ class TestTopologyOptions(IntegrationTest):
tasks.uninstall_master(self.replicas[1])
tasks.clean_replication_agreement(self.master, self.replicas[1])
result5 = self.master.run_command(['ipa', 'topologysegment-find',
- 'realm'])
+ DOMAIN_SUFFIX])
assert(self.noentries_re.search(result5.stdout_text).group(1) == "1")
def test_add_remove_segment(self):
@@ -108,7 +109,7 @@ class TestTopologyOptions(IntegrationTest):
assert err == "", err
# Make sure the new segment is shown by `ipa topologysegment-find`
result1 = self.master.run_command(['ipa', 'topologysegment-find',
- 'realm'])
+ DOMAIN_SUFFIX])
assert(result1.stdout_text.find(segment['name']) > 0)
# Remove master <-> replica2 segment and make sure that the changes get
# there through replica1
@@ -118,7 +119,7 @@ class TestTopologyOptions(IntegrationTest):
assert returncode == 0, error
# make sure replica1 does not have segment that was deleted on master
result3 = self.replicas[0].run_command(['ipa', 'topologysegment-find',
- 'realm'])
+ DOMAIN_SUFFIX])
assert(result3.stdout_text.find(deleteme) < 0)
# Create test data on master and make sure it gets all the way down to
# replica2 through replica1
--
2.4.3
--
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
