|
 |
3280a9 |
From 00ccec335792e3fa44712427463c64eb1ff9c5be Mon Sep 17 00:00:00 2001
|
|
|
3280a9 |
From: progier389 <progier@redhat.com>
|
|
|
3280a9 |
Date: Tue, 12 Jan 2021 17:45:41 +0100
|
|
|
3280a9 |
Subject: [PATCH] Issue 4504 - insure that repl_monitor_test use ldapi (for
|
|
|
3280a9 |
RHEL) - fix merge issue (#4533)
|
|
|
3280a9 |
|
|
|
3280a9 |
(cherry picked from commit a880fddc192414d6283ea6832491b7349e5471dc)
|
|
|
3280a9 |
---
|
|
|
3280a9 |
.../tests/suites/clu/repl_monitor_test.py | 47 ++++++++++++++-----
|
|
|
3280a9 |
1 file changed, 36 insertions(+), 11 deletions(-)
|
|
|
3280a9 |
|
|
|
3280a9 |
diff --git a/dirsrvtests/tests/suites/clu/repl_monitor_test.py b/dirsrvtests/tests/suites/clu/repl_monitor_test.py
|
|
|
3280a9 |
index b2cb840b3..caf6a9099 100644
|
|
|
3280a9 |
--- a/dirsrvtests/tests/suites/clu/repl_monitor_test.py
|
|
|
3280a9 |
+++ b/dirsrvtests/tests/suites/clu/repl_monitor_test.py
|
|
|
3280a9 |
@@ -9,6 +9,7 @@
|
|
|
3280a9 |
import time
|
|
|
3280a9 |
import subprocess
|
|
|
3280a9 |
import pytest
|
|
|
3280a9 |
+import re
|
|
|
3280a9 |
|
|
|
3280a9 |
from lib389.cli_conf.replication import get_repl_monitor_info
|
|
|
3280a9 |
from lib389.tasks import *
|
|
|
3280a9 |
@@ -69,6 +70,25 @@ def check_value_in_log_and_reset(content_list, second_list=None, single_value=No
|
|
|
3280a9 |
log.info('Reset log file')
|
|
|
3280a9 |
f.truncate(0)
|
|
|
3280a9 |
|
|
|
3280a9 |
+def get_hostnames_from_log(port1, port2):
|
|
|
3280a9 |
+ # Get the supplier host names as displayed in replication monitor output
|
|
|
3280a9 |
+ with open(LOG_FILE, 'r') as logfile:
|
|
|
3280a9 |
+ logtext = logfile.read()
|
|
|
3280a9 |
+ # search for Supplier :hostname:port
|
|
|
3280a9 |
+ # and use \D to insure there is no more number is after
|
|
|
3280a9 |
+ # the matched port (i.e that 10 is not matching 101)
|
|
|
3280a9 |
+ regexp = '(Supplier: )([^:]*)(:' + str(port1) + '\D)'
|
|
|
3280a9 |
+ match=re.search(regexp, logtext)
|
|
|
3280a9 |
+ host_m1 = 'localhost.localdomain'
|
|
|
3280a9 |
+ if (match is not None):
|
|
|
3280a9 |
+ host_m1 = match.group(2)
|
|
|
3280a9 |
+ # Same for master 2
|
|
|
3280a9 |
+ regexp = '(Supplier: )([^:]*)(:' + str(port2) + '\D)'
|
|
|
3280a9 |
+ match=re.search(regexp, logtext)
|
|
|
3280a9 |
+ host_m2 = 'localhost.localdomain'
|
|
|
3280a9 |
+ if (match is not None):
|
|
|
3280a9 |
+ host_m2 = match.group(2)
|
|
|
3280a9 |
+ return (host_m1, host_m2)
|
|
|
3280a9 |
|
|
|
3280a9 |
@pytest.mark.ds50545
|
|
|
3280a9 |
@pytest.mark.bz1739718
|
|
|
3280a9 |
@@ -177,20 +197,9 @@ def test_dsconf_replication_monitor(topology_m2, set_log_file):
|
|
|
3280a9 |
'001',
|
|
|
3280a9 |
m1.host + ':' + str(m1.port)]
|
|
|
3280a9 |
|
|
|
3280a9 |
- dsrc_content = '[repl-monitor-connections]\n' \
|
|
|
3280a9 |
- 'connection1 = ' + m1.host + ':' + str(m1.port) + ':' + DN_DM + ':' + PW_DM + '\n' \
|
|
|
3280a9 |
- 'connection2 = ' + m2.host + ':' + str(m2.port) + ':' + DN_DM + ':' + PW_DM + '\n' \
|
|
|
3280a9 |
- '\n' \
|
|
|
3280a9 |
- '[repl-monitor-aliases]\n' \
|
|
|
3280a9 |
- 'M1 = ' + m1.host + ':' + str(m1.port) + '\n' \
|
|
|
3280a9 |
- 'M2 = ' + m2.host + ':' + str(m2.port)
|
|
|
3280a9 |
-
|
|
|
3280a9 |
connections = [m1.host + ':' + str(m1.port) + ':' + DN_DM + ':' + PW_DM,
|
|
|
3280a9 |
m2.host + ':' + str(m2.port) + ':' + DN_DM + ':' + PW_DM]
|
|
|
3280a9 |
|
|
|
3280a9 |
- aliases = ['M1=' + m1.host + ':' + str(m1.port),
|
|
|
3280a9 |
- 'M2=' + m2.host + ':' + str(m2.port)]
|
|
|
3280a9 |
-
|
|
|
3280a9 |
args = FakeArgs()
|
|
|
3280a9 |
args.connections = connections
|
|
|
3280a9 |
args.aliases = None
|
|
|
3280a9 |
@@ -198,8 +207,24 @@ def test_dsconf_replication_monitor(topology_m2, set_log_file):
|
|
|
3280a9 |
|
|
|
3280a9 |
log.info('Run replication monitor with connections option')
|
|
|
3280a9 |
get_repl_monitor_info(m1, DEFAULT_SUFFIX, log, args)
|
|
|
3280a9 |
+ (host_m1, host_m2) = get_hostnames_from_log(m1.port, m2.port)
|
|
|
3280a9 |
check_value_in_log_and_reset(content_list, connection_content, error_list=error_list)
|
|
|
3280a9 |
|
|
|
3280a9 |
+ # Prepare the data for next tests
|
|
|
3280a9 |
+ aliases = ['M1=' + host_m1 + ':' + str(m1.port),
|
|
|
3280a9 |
+ 'M2=' + host_m2 + ':' + str(m2.port)]
|
|
|
3280a9 |
+
|
|
|
3280a9 |
+ alias_content = ['Supplier: M1 (' + host_m1 + ':' + str(m1.port) + ')',
|
|
|
3280a9 |
+ 'Supplier: M2 (' + host_m2 + ':' + str(m2.port) + ')']
|
|
|
3280a9 |
+
|
|
|
3280a9 |
+ dsrc_content = '[repl-monitor-connections]\n' \
|
|
|
3280a9 |
+ 'connection1 = ' + m1.host + ':' + str(m1.port) + ':' + DN_DM + ':' + PW_DM + '\n' \
|
|
|
3280a9 |
+ 'connection2 = ' + m2.host + ':' + str(m2.port) + ':' + DN_DM + ':' + PW_DM + '\n' \
|
|
|
3280a9 |
+ '\n' \
|
|
|
3280a9 |
+ '[repl-monitor-aliases]\n' \
|
|
|
3280a9 |
+ 'M1 = ' + host_m1 + ':' + str(m1.port) + '\n' \
|
|
|
3280a9 |
+ 'M2 = ' + host_m2 + ':' + str(m2.port)
|
|
|
3280a9 |
+
|
|
|
3280a9 |
log.info('Run replication monitor with aliases option')
|
|
|
3280a9 |
args.aliases = aliases
|
|
|
3280a9 |
get_repl_monitor_info(m1, DEFAULT_SUFFIX, log, args)
|
|
|
3280a9 |
--
|
|
|
3280a9 |
2.26.2
|
|
|
3280a9 |
|