[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-11-12 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From 4d7189316ea7318deb4d1b1202264594afcdd89d Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH] Tests: [SSSD-3579]: Skip log check in test
 test_0018_bz1734040.

The logging changed significantly in RHEL-9,
skipping log message check as it no longer works.
The test still tests that sssd does not crash on the flow.
---
 src/tests/multihost/ad/test_adparameters.py | 35 +++--
 1 file changed, 19 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..bda7fe338d 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,38 +692,41 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
 :customerscenario: True
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1734040
 """
+distro = multihost.client[0].distro
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+domain_name = client.get_domain_section_name()
+dom_section = 'domain/%s' % domain_name
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+if "Red Hat Enterprise Linux" in distro and " 9." in distro:
+print("Skipping this part of test as logging changed on RHEL 9.")
+# The test is still valid as sssd was crashing on the
+# "getent passwd 0" part before.
+else:
+assert find.search(log), "Expected log record is missing."
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-11-09 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From a7ea2694e3c39193972dd2f4246fe6717da255b9 Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH] Tests: [SSSD-3579]: Skip log check in test
 test_0018_bz1734040.

The logging changed significantly in RHEL-9,
skipping log message check as it no longer works.
The test still tests that sssd does not crash on the flow.
---
 src/tests/multihost/ad/test_adparameters.py | 35 +++--
 1 file changed, 19 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..bda7fe338d 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,38 +692,41 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
 :customerscenario: True
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1734040
 """
+distro = multihost.client[0].distro
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+domain_name = client.get_domain_section_name()
+dom_section = 'domain/%s' % domain_name
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+if "Red Hat Enterprise Linux" in distro and " 9." in distro:
+print("Skipping this part of test as logging changed on RHEL 9.")
+# The test is still valid as sssd was crashing on the
+# "getent passwd 0" part before.
+else:
+assert find.search(log), "Expected log record is missing."
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-11-09 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From 970080a6d311981441eb3713bbdcb17c21345758 Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH] Tests: [SSSD-3579]: Skip log check in test
 test_0018_bz1734040.

The logging changed significantly in RHEL-9,
skipping log message check as it no longer works.
The test still tests that sssd does not crash on the flow.
---
 src/tests/multihost/ad/test_adparameters.py | 35 +++--
 1 file changed, 19 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..a832d4a7fc 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,38 +692,41 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
 :customerscenario: True
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1734040
 """
+distro = multihost.client[0].distro
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+domain_name = client.get_domain_section_name()
+dom_section = 'domain/%s' % domain_name
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+if "Red Hat Enterprise Linux" in distro and " 9." in distro:
+print("Skipping this part of test as logging changed on RHEL 9.")
+# The test is still valid as sssd was crashing on the
+# "getent passwd 0" part becfore
+else:
+assert find.search(log), "Expected log record is missing."
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-11-09 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From 810f69be5dfdeee8ab754caf33cea83019bb1c8d Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

The test does not work even with fixes for RHEL-9,
skipping for now so it is not blocking RHEL-9 gating,
until proper fix is found.
---
 src/tests/multihost/ad/test_adparameters.py | 32 ++---
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..ef0d91d8fa 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,38 +692,38 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
 :customerscenario: True
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1734040
 """
+distro = multihost.client[0].distro
+if "Red Hat Enterprise Linux" in distro and " 9." in distro:
+pytest.skip("Does not work on RHEL 9.")
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+domain_name = client.get_domain_section_name()
+dom_section = 'domain/%s' % domain_name
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+assert find.search(log), "Expected log record is missing."
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-11-09 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From 2cc7ab2d468ac2942cab30549bd5c09ad928afde Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH 1/5] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

The test does not work even with fixes for RHEL-9,
skipping for now so it is not blocking RHEL-9 gating,
until proper fix is found.
---
 src/tests/multihost/ad/test_adparameters.py | 29 +
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..044ca58df0 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,8 +692,10 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+@pytest.mark.skip(reason="Does not work on RHEL 9, skip until resolved.")
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
@@ -702,28 +704,23 @@ def test_0018_bz1734040(self, multihost, adjoin):
 """
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+dom_section = 'domain/%s' % client.get_domain_section_name()
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+assert find.search(log), "Expected log record is missing."

From d192dd006d631e36aa8346ec46b71e95bbccaf5a Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Tue, 9 Nov 2021 08:47:15 +0100
Subject: [PATCH 2/5] test debug

---
 src/tests/multihost/ad/test_adparameters.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 044ca58df0..b80a6dd110 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -702,6 +702,8 @@ def test_0018_bz1734040(multihost, adjoin):
 :customerscenario: True
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1734040
 """
+distro = multihost.client[0].distro
+print(distro)
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
 client.backup_sssd_conf()

From 96ee4a3f951b9ecda35883d7c1bb2cd330ef5f8b Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Tue, 9 Nov 2021 09:56:45 +0100
Subject: [PATCH 3/5] test debug

---
 src/tests/multihost/ad/test_adparameters.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index b80a6dd110..80aee4927f 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -704,6 +704,8 @@ def test_0018_bz1734040(multihost, adjoin):
 """
 distro = multihost.client[0].distro
 print(distro)
+multihost.client[0].run_command(f'echo "Distro: {distro}"',
+raiseonerr=True)

[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-11-09 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From 2cc7ab2d468ac2942cab30549bd5c09ad928afde Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH 1/4] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

The test does not work even with fixes for RHEL-9,
skipping for now so it is not blocking RHEL-9 gating,
until proper fix is found.
---
 src/tests/multihost/ad/test_adparameters.py | 29 +
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..044ca58df0 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,8 +692,10 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+@pytest.mark.skip(reason="Does not work on RHEL 9, skip until resolved.")
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
@@ -702,28 +704,23 @@ def test_0018_bz1734040(self, multihost, adjoin):
 """
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+dom_section = 'domain/%s' % client.get_domain_section_name()
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+assert find.search(log), "Expected log record is missing."

From d192dd006d631e36aa8346ec46b71e95bbccaf5a Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Tue, 9 Nov 2021 08:47:15 +0100
Subject: [PATCH 2/4] test debug

---
 src/tests/multihost/ad/test_adparameters.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 044ca58df0..b80a6dd110 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -702,6 +702,8 @@ def test_0018_bz1734040(multihost, adjoin):
 :customerscenario: True
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1734040
 """
+distro = multihost.client[0].distro
+print(distro)
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
 client.backup_sssd_conf()

From 96ee4a3f951b9ecda35883d7c1bb2cd330ef5f8b Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Tue, 9 Nov 2021 09:56:45 +0100
Subject: [PATCH 3/4] test debug

---
 src/tests/multihost/ad/test_adparameters.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index b80a6dd110..80aee4927f 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -704,6 +704,8 @@ def test_0018_bz1734040(multihost, adjoin):
 """
 distro = multihost.client[0].distro
 print(distro)
+multihost.client[0].run_command(f'echo "Distro: {distro}"',
+raiseonerr=True)

[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-11-09 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From 2cc7ab2d468ac2942cab30549bd5c09ad928afde Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH 1/3] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

The test does not work even with fixes for RHEL-9,
skipping for now so it is not blocking RHEL-9 gating,
until proper fix is found.
---
 src/tests/multihost/ad/test_adparameters.py | 29 +
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..044ca58df0 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,8 +692,10 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+@pytest.mark.skip(reason="Does not work on RHEL 9, skip until resolved.")
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
@@ -702,28 +704,23 @@ def test_0018_bz1734040(self, multihost, adjoin):
 """
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+dom_section = 'domain/%s' % client.get_domain_section_name()
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+assert find.search(log), "Expected log record is missing."

From d192dd006d631e36aa8346ec46b71e95bbccaf5a Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Tue, 9 Nov 2021 08:47:15 +0100
Subject: [PATCH 2/3] test debug

---
 src/tests/multihost/ad/test_adparameters.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 044ca58df0..b80a6dd110 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -702,6 +702,8 @@ def test_0018_bz1734040(multihost, adjoin):
 :customerscenario: True
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1734040
 """
+distro = multihost.client[0].distro
+print(distro)
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
 client.backup_sssd_conf()

From 96ee4a3f951b9ecda35883d7c1bb2cd330ef5f8b Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Tue, 9 Nov 2021 09:56:45 +0100
Subject: [PATCH 3/3] test debug

---
 src/tests/multihost/ad/test_adparameters.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index b80a6dd110..80aee4927f 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -704,6 +704,8 @@ def test_0018_bz1734040(multihost, adjoin):
 """
 distro = multihost.client[0].distro
 print(distro)
+multihost.client[0].run_command(f'echo "Distro: {distro}"',
+raiseonerr=True)

[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-11-08 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From 2cc7ab2d468ac2942cab30549bd5c09ad928afde Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH 1/2] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

The test does not work even with fixes for RHEL-9,
skipping for now so it is not blocking RHEL-9 gating,
until proper fix is found.
---
 src/tests/multihost/ad/test_adparameters.py | 29 +
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..044ca58df0 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,8 +692,10 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+@pytest.mark.skip(reason="Does not work on RHEL 9, skip until resolved.")
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
@@ -702,28 +704,23 @@ def test_0018_bz1734040(self, multihost, adjoin):
 """
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+dom_section = 'domain/%s' % client.get_domain_section_name()
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+assert find.search(log), "Expected log record is missing."

From d192dd006d631e36aa8346ec46b71e95bbccaf5a Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Tue, 9 Nov 2021 08:47:15 +0100
Subject: [PATCH 2/2] test debug

---
 src/tests/multihost/ad/test_adparameters.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 044ca58df0..b80a6dd110 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -702,6 +702,8 @@ def test_0018_bz1734040(multihost, adjoin):
 :customerscenario: True
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1734040
 """
+distro = multihost.client[0].distro
+print(distro)
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
 client.backup_sssd_conf()
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

[SSSD] [sssd PR#5786][synchronized] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

2021-09-20 Thread jakub-vavra-cz 
   URL: https://github.com/SSSD/sssd/pull/5786
Author: jakub-vavra-cz
 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5786/head:pr5786
git checkout pr5786
From 2cc7ab2d468ac2942cab30549bd5c09ad928afde Mon Sep 17 00:00:00 2001
From: Jakub Vavra 
Date: Wed, 15 Sep 2021 12:05:28 +0200
Subject: [PATCH] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.

The test does not work even with fixes for RHEL-9,
skipping for now so it is not blocking RHEL-9 gating,
until proper fix is found.
---
 src/tests/multihost/ad/test_adparameters.py | 29 +
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/src/tests/multihost/ad/test_adparameters.py b/src/tests/multihost/ad/test_adparameters.py
index 2967709229..044ca58df0 100644
--- a/src/tests/multihost/ad/test_adparameters.py
+++ b/src/tests/multihost/ad/test_adparameters.py
@@ -692,8 +692,10 @@ def test_0017_gssspnego_adjoin(self, multihost):
 multihost.client[0].run_command(remove_pcap)
 assert status == 'PASS'
 
+@staticmethod
 @pytest.mark.tier1
-def test_0018_bz1734040(self, multihost, adjoin):
+@pytest.mark.skip(reason="Does not work on RHEL 9, skip until resolved.")
+def test_0018_bz1734040(multihost, adjoin):
 """
 :title: ad_parameters: sssd crash in ad_get_account_domain_search
 :id: dcca509e-b316-4010-a173-20f541dafd52
@@ -702,28 +704,23 @@ def test_0018_bz1734040(self, multihost, adjoin):
 """
 adjoin(membersw='adcli')
 client = sssdTools(multihost.client[0])
-domain_name = client.get_domain_section_name()
 client.backup_sssd_conf()
 client.remove_sss_cache('/var/log/sssd')
-sssdcfg = multihost.client[0].get_file_contents(SSSD_DEFAULT_CONF)
-sssdcfg = re.sub(b'ad_domain = %s' % domain_name.encode('utf-8'),
- b'ad_domain = example.com \ndebug_level = 9', sssdcfg)
-multihost.client[0].put_file_contents(SSSD_DEFAULT_CONF, sssdcfg)
+dom_section = 'domain/%s' % client.get_domain_section_name()
+sssd_params = {'ad_domain': 'example.com', 'debug_level': '9'}
+client.sssd_conf(dom_section, sssd_params)
+sssd_params = {'enable_files_domain': 'True',
+   'debug_level': '9'}
+client.sssd_conf('sssd', sssd_params)
 client.clear_sssd_cache()
 cmd = multihost.client[0].run_command('getent passwd 0',
   raiseonerr=True)
-if cmd.returncode != 0:
-status = 'FAIL'
-else:
-status = 'PASS'
 time.sleep(10)
 domain_log = '/var/log/sssd/sssd_%s.log' % domain_name
 log = multihost.client[0].get_file_contents(domain_log).decode('utf-8')
-msg = 'Flags\s.0x0001.'
+msg = r'AccountDomain.*Flags\s.0x0001.'
 find = re.compile(r'%s' % msg)
-if not find.search(log):
-status = 'FAIL'
-else:
-status = 'PASS'
+
 client.restore_sssd_conf()
-assert status == 'PASS'
+assert cmd.returncode == 0, "'getent passwd 0' failed!"
+assert find.search(log), "Expected log record is missing."
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure