This is an automated email from the git hooks/post-receive script.
spichugi pushed a commit to branch master
in repository lib389.
commit 91699e74c62ed57543922dbc52da4dc5bf35a346
Author: Simon Pichugin <spichugi(a)redhat.com>
Date: Mon Sep 25 08:32:21 2017 +0200
Issue 99 - Fix typo in create_topology
Description: We need to use the roles from the 'for' loop while creating
agreements. If we don't, we will have failures while creating
'asymmetric'
setups like two masters and one consumer.
https://pagure.io/lib389/issue/99
Reviewed by: amsharma, sramling (Thanks!)
---
lib389/topologies.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib389/topologies.py b/lib389/topologies.py
index 1ffc09e..674f1d0 100644
--- a/lib389/topologies.py
+++ b/lib389/topologies.py
@@ -93,11 +93,11 @@ def create_topology(topo_dict):
continue
# Create agreements: master -> masters, consumers
- for inst_num_from in range(1, topo_dict[role]+1):
+ for inst_num_from in range(1, topo_dict[role_from]+1):
roles_to = [ReplicaRole.MASTER, ReplicaRole.CONSUMER]
for role_to in [role for role in topo_dict if role in roles_to]:
- for inst_num_to in range(1, topo_dict[role]+1):
+ for inst_num_to in range(1, topo_dict[role_to]+1):
# Exclude the instance we created it from
if role_from != role_to or inst_num_from != inst_num_to:
inst_from_id = "{}{}".format(role_from.name.lower(),
inst_num_from)
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.