URL: https://github.com/freeipa/freeipa/pull/1103
Author: tomaskrizek
 Title: #1103: ipatests: keep default log level for 389-ds
Action: opened

PR body:
"""
The log level of 8192 for 389-ds is very noisy and conceals useful debug
messages. The default log level 0 should be sufficient for our purposes.

Fixes: https://pagure.io/freeipa/issue/7162
Signed-off-by: Tomas Krizek <tkri...@redhat.com>
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1103/head:pr1103
git checkout pr1103
From bc2d45a7fafe85c8dff3cab5a56efc4e93dc2069 Mon Sep 17 00:00:00 2001
From: Tomas Krizek <tkri...@redhat.com>
Date: Wed, 20 Sep 2017 15:38:14 +0200
Subject: [PATCH] ipatests: keep default log level for 389-ds

The log level of 8192 for 389-ds is very noisy and conceals useful debug
messages. The default log level 0 should be sufficient for our purposes.

Fixes: https://pagure.io/freeipa/issue/7162
Signed-off-by: Tomas Krizek <tkri...@redhat.com>
---
 ipatests/pytest_plugins/integration/tasks.py | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/ipatests/pytest_plugins/integration/tasks.py b/ipatests/pytest_plugins/integration/tasks.py
index 9d3955d993..f53b59922d 100644
--- a/ipatests/pytest_plugins/integration/tasks.py
+++ b/ipatests/pytest_plugins/integration/tasks.py
@@ -236,21 +236,6 @@ def restore_hostname(host):
         host.run_command(['rm', backupname])
 
 
-def enable_replication_debugging(host):
-    logger.info('Enable LDAP replication logging')
-    logging_ldif = textwrap.dedent("""
-        dn: cn=config
-        changetype: modify
-        replace: nsslapd-errorlog-level
-        nsslapd-errorlog-level: 8192
-        """)
-    host.run_command(['ldapmodify', '-x',
-                      '-D', str(host.config.dirman_dn),
-                      '-w', host.config.dirman_password,
-                      '-h', host.hostname],
-                     stdin_text=logging_ldif)
-
-
 def install_master(host, setup_dns=True, setup_kra=False, setup_adtrust=False,
                    extra_args=(), domain_level=None, unattended=True,
                    stdin_text=None, raiseonerr=True):
@@ -286,7 +271,6 @@ def install_master(host, setup_dns=True, setup_kra=False, setup_adtrust=False,
     result = host.run_command(args, raiseonerr=raiseonerr,
                               stdin_text=stdin_text)
     if result.returncode == 0:
-        enable_replication_debugging(host)
         setup_sssd_debugging(host)
         kinit_admin(host)
     return result
@@ -391,7 +375,6 @@ def install_replica(master, replica, setup_ca=True, setup_dns=False,
     result = replica.run_command(args, raiseonerr=raiseonerr,
                                  stdin_text=stdin_text)
     if result.returncode == 0:
-        enable_replication_debugging(replica)
         setup_sssd_debugging(replica)
         kinit_admin(replica)
     return result
_______________________________________________
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org

Reply via email to