ced817
From e713c227bb420a841ce3ae146bca55a84a1b0dbf Mon Sep 17 00:00:00 2001
ced817
From: =?UTF-8?q?Fran=C3=A7ois=20Cami?= <fcami@redhat.com>
ced817
Date: Tue, 22 Jun 2021 14:36:51 +0200
ced817
Subject: [PATCH] paths: add IPA_SERVER_CONF
ced817
MIME-Version: 1.0
ced817
Content-Type: text/plain; charset=UTF-8
ced817
Content-Transfer-Encoding: 8bit
ced817
ced817
Related: https://pagure.io/freeipa/issue/8891
ced817
Signed-off-by: François Cami <fcami@redhat.com>
ced817
Reviewed-By: Stanislav Levin <slev@altlinux.org>
ced817
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
ced817
---
ced817
 ipaplatform/base/paths.py | 1 +
ced817
 1 file changed, 1 insertion(+)
ced817
ced817
diff --git a/ipaplatform/base/paths.py b/ipaplatform/base/paths.py
ced817
index 91423b332..de217d9ef 100644
ced817
--- a/ipaplatform/base/paths.py
ced817
+++ b/ipaplatform/base/paths.py
ced817
@@ -71,6 +71,7 @@ class BasePathNamespace:
ced817
     IPA_DEFAULT_CONF = "/etc/ipa/default.conf"
ced817
     IPA_DNSKEYSYNCD_KEYTAB = "/etc/ipa/dnssec/ipa-dnskeysyncd.keytab"
ced817
     IPA_ODS_EXPORTER_KEYTAB = "/etc/ipa/dnssec/ipa-ods-exporter.keytab"
ced817
+    IPA_SERVER_CONF = "/etc/ipa/server.conf"
ced817
     DNSSEC_OPENSSL_CONF = "/etc/ipa/dnssec/openssl.cnf"
ced817
     DNSSEC_SOFTHSM2_CONF = "/etc/ipa/dnssec/softhsm2.conf"
ced817
     DNSSEC_SOFTHSM_PIN_SO = "/etc/ipa/dnssec/softhsm_pin_so"
ced817
-- 
ced817
2.31.1
ced817
ced817
From ee4be290e1583834a573c3896ee1d97b3fbb6c24 Mon Sep 17 00:00:00 2001
ced817
From: =?UTF-8?q?Fran=C3=A7ois=20Cami?= <fcami@redhat.com>
ced817
Date: Tue, 22 Jun 2021 14:45:49 +0200
ced817
Subject: [PATCH] ipatests: smoke test for server debug mode.
ced817
MIME-Version: 1.0
ced817
Content-Type: text/plain; charset=UTF-8
ced817
Content-Transfer-Encoding: 8bit
ced817
ced817
Add a smoke test to make sure the server can be set in debug mode
ced817
without issue.
ced817
ced817
Related: https://pagure.io/freeipa/issue/8891
ced817
Signed-off-by: François Cami <fcami@redhat.com>
ced817
Reviewed-By: Stanislav Levin <slev@altlinux.org>
ced817
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
ced817
---
ced817
 .../test_integration/test_installation.py     | 27 +++++++++++++++++++
ced817
 1 file changed, 27 insertions(+)
ced817
ced817
diff --git a/ipatests/test_integration/test_installation.py b/ipatests/test_integration/test_installation.py
ced817
index 301767b8d..0c96536f0 100644
ced817
--- a/ipatests/test_integration/test_installation.py
ced817
+++ b/ipatests/test_integration/test_installation.py
ced817
@@ -703,6 +703,33 @@ class TestInstallMaster(IntegrationTest):
ced817
     def test_install_master(self):
ced817
         tasks.install_master(self.master, setup_dns=False)
ced817
 
ced817
+    @pytest.mark.skip_if_platform(
ced817
+        "debian", reason="This test hardcodes the httpd service name"
ced817
+    )
ced817
+    def test_smoke_test_for_debug_mode(self):
ced817
+        """Test if an IPA server works in debug mode.
ced817
+        Related: https://pagure.io/freeipa/issue/8891
ced817
+
ced817
+        Note: this test hardcodes the "httpd" service name.
ced817
+        """
ced817
+
ced817
+        target_fname = paths.IPA_SERVER_CONF
ced817
+        assert not self.master.transport.file_exists(target_fname)
ced817
+
ced817
+        # set the IPA server in debug mode
ced817
+        server_conf = "[global]\ndebug=True"
ced817
+        self.master.put_file_contents(target_fname, server_conf)
ced817
+        self.master.run_command(["systemctl", "restart", "httpd"])
ced817
+
ced817
+        # smoke test in debug mode
ced817
+        tasks.kdestroy_all(self.master)
ced817
+        tasks.kinit_admin(self.master)
ced817
+        self.master.run_command(["ipa", "user-show", "admin"])
ced817
+
ced817
+        # rollback
ced817
+        self.master.run_command(["rm", target_fname])
ced817
+        self.master.run_command(["systemctl", "restart", "httpd"])
ced817
+
ced817
     def test_schema_compat_attribute_and_tree_disable(self):
ced817
         """Test if schema-compat-entry-attribute is set
ced817
 
ced817
-- 
ced817
2.31.1
ced817
ced817
From 1539c7383116647ad9c5b125b343f972e9c9653b Mon Sep 17 00:00:00 2001
ced817
From: =?UTF-8?q?Fran=C3=A7ois=20Cami?= <fcami@redhat.com>
ced817
Date: Wed, 23 Jun 2021 06:35:19 +0200
ced817
Subject: [PATCH] rpcserver.py: perf_counter_ns is Python 3.7+
ced817
MIME-Version: 1.0
ced817
Content-Type: text/plain; charset=UTF-8
ced817
Content-Transfer-Encoding: 8bit
ced817
ced817
perf_counter_ns is only available in Python 3.7 and later.
ced817
Define a lambda for 3.6 and lower.
ced817
ced817
Fixes: https://pagure.io/freeipa/issue/8891
ced817
Signed-off-by: François Cami <fcami@redhat.com>
ced817
Reviewed-By: Stanislav Levin <slev@altlinux.org>
ced817
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
ced817
---
ced817
 ipaserver/rpcserver.py | 5 +++++
ced817
 1 file changed, 5 insertions(+)
ced817
ced817
diff --git a/ipaserver/rpcserver.py b/ipaserver/rpcserver.py
ced817
index b121316bf..e612528e0 100644
ced817
--- a/ipaserver/rpcserver.py
ced817
+++ b/ipaserver/rpcserver.py
ced817
@@ -31,6 +31,7 @@ import os
ced817
 import time
ced817
 import traceback
ced817
 from io import BytesIO
ced817
+from sys import version_info
ced817
 from urllib.parse import parse_qs
ced817
 from xmlrpc.client import Fault
ced817
 
ced817
@@ -72,6 +73,10 @@ from requests.auth import AuthBase
ced817
 if six.PY3:
ced817
     unicode = str
ced817
 
ced817
+# time.perf_counter_ns appeared in Python 3.7.
ced817
+if version_info < (3, 7):
ced817
+    time.perf_counter_ns = lambda: int(time.perf_counter() * 10**9)
ced817
+
ced817
 logger = logging.getLogger(__name__)
ced817
 
ced817
 HTTP_STATUS_SUCCESS = '200 Success'
ced817
-- 
ced817
2.31.1
ced817