/var/log/SSSD is platform specific, thus should be added to ipaplatform module

Patch attached.
From 5873f755428903b9300f260221b5417a3697f279 Mon Sep 17 00:00:00 2001
From: Martin Basti <mba...@redhat.com>
Date: Tue, 8 Mar 2016 16:26:23 +0100
Subject: [PATCH] Use platform path constant for SSSD log dir

The path to SSSD log directory is platform specific and should be in
ipaplatform module.
---
 ipaplatform/base/paths.py          | 1 +
 ipatests/test_integration/tasks.py | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/ipaplatform/base/paths.py b/ipaplatform/base/paths.py
index d7a9219a098896b88f42d3a3680b4e39c389aabc..e4b8bd76d5f23c226269f1b3880a9aa3e2ebf63d 100644
--- a/ipaplatform/base/paths.py
+++ b/ipaplatform/base/paths.py
@@ -321,6 +321,7 @@ class BasePathNamespace(object):
     TOMCAT_KRA_ARCHIVE_DIR = "/var/log/pki/pki-tomcat/kra/archive"
     TOMCAT_KRA_SIGNEDAUDIT_DIR = "/var/log/pki/pki-tomcat/kra/signedAudit"
     LOG_SECURE = "/var/log/secure"
+    VAR_LOG_SSSD_DIR = "/var/log/sssd"
     NAMED_RUN = "/var/named/data/named.run"
     VAR_OPENDNSSEC_DIR = "/var/opendnssec"
     OPENDNSSEC_KASP_DB = "/var/opendnssec/kasp.db"
diff --git a/ipatests/test_integration/tasks.py b/ipatests/test_integration/tasks.py
index ead038c8891e23bc8ce98a4447404ca27fef64a4..b6699d6911c7b97a4f3691262f1b3d9a1635bcd9 100644
--- a/ipatests/test_integration/tasks.py
+++ b/ipatests/test_integration/tasks.py
@@ -547,7 +547,7 @@ def setup_sssd_debugging(host):
                       paths.SSSD_CONF],
                      raiseonerr=False)
 
-    host.collect_log('/var/log/sssd/*')
+    host.collect_log(os.path.join(paths.VAR_LOG_SSSD_DIR, '*'))
 
     # Clear the cache and restart SSSD
     clear_sssd_cache(host)
-- 
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

Reply via email to