[SSSD] [sssd PR#5946][+Changes requested] Tests: Fix yum repoquery --recommends sssd-tools test

2022-01-11 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5946
Title: #5946: Tests: Fix yum repoquery --recommends sssd-tools test

Label: +Changes requested
___
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#5946][+Tests] Tests: Fix yum repoquery --recommends sssd-tools test

2022-01-11 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5946
Title: #5946: Tests: Fix yum repoquery --recommends sssd-tools test

Label: +Tests
___
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#5908][synchronized] Check default debug level of sssd and corresponding logs

2022-01-11 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5908
Author: sgoveas
 Title: #5908: Check default debug level of sssd and corresponding logs
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5908/head:pr5908
git checkout pr5908
From f465958208f569bc683a0eee77ff7daef02f7592 Mon Sep 17 00:00:00 2001
From: Madhuri Upadhye 
Date: Fri, 22 Jan 2021 15:35:17 +0530
Subject: [PATCH] Check default debug level of sssd and corresponding logs

It consists of five test cases:
  1. Check default debug level when sssd start
successfully
  2. Check default debug level by
successful authentication of the user
  3. Check default level as 0 and 1
  4. Check default level as 2
  5. Check SBUS code should not trigger failure
message during modules startup

Verifies:
Issues: #5422
#5425

Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1893159
  https://bugzilla.redhat.com/show_bug.cgi?id=1915319
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../alltests/test_default_debug_level.py  | 150 ++
 2 files changed, 151 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_default_debug_level.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..e23f23d21d 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+defaultdebuglevel: Test default debug level sssd
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_default_debug_level.py b/src/tests/multihost/alltests/test_default_debug_level.py
new file mode 100644
index 00..bcadea0dc5
--- /dev/null
+++ b/src/tests/multihost/alltests/test_default_debug_level.py
@@ -0,0 +1,150 @@
+"""Automation for default debug level
+
+:requirement: SSSD - Default debug level
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.defaultdebuglevel
+@pytest.mark.tier1_2
+class TestDefaultDebugLevel(object):
+""" Check sssd default debug level """
+def test_0001_check_default_debug_level(self, multihost, backupsssdconf):
+"""
+:title: default debug logs: Check default debug level when sssd start
+ successfully
+:id: 1f38b560-27dc-4144-895d-e667420b0467
+"""
+section = f"domain/{ds_instance_name}"
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+# stop sssd, delete sssd logs and cache, start sssd
+tools.clear_sssd_cache()
+time.sleep(5)
+log_list = ['sssd', f'sssd_{ds_instance_name}',
+'sssd_nss', 'sssd_pam']
+for log_filename in log_list:
+log = f'/var/log/sssd/{log_filename}.log'
+log_str = multihost.client[0].get_file_contents(log).decode(
+'utf-8')
+pattern1 = re.compile(r'Starting with debug level = 0x0070')
+default_debug = pattern1.search(log_str)
+assert default_debug is not None
+log_split = log_str.split("\n")
+for index in range(len(log_split)-1):
+log_single_line = log_split[index]
+pattern2 = re.compile(r'(0x\w+)')
+pattern3 = re.compile(r'(0x\d+)')
+debug_str1 = pattern2.search(log_single_line)
+debug_str2 = pattern3.search(log_single_line)
+assert debug_str1.group() == '0x1f7c0' or \
+debug_str2.group() <= 0x0040
+
+def test_0002_check_default_level_with_auth(self, multihost,
+backupsssdconf):
+"""
+:title: default debug logs: Check successful login with default
+ log level doesn't generate any logs
+:id: f40a7c66-6b5f-4f3c-8fcb-6aa12f415473
+"""
+section = f"domain/{ds_instance_name}"
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+domain_params = {'fallback_homed

[SSSD] [sssd PR#5908][comment] Check default debug level of sssd and corresponding logs

2022-01-10 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5908
Title: #5908: Check default debug level of sssd and corresponding logs

sgoveas commented:
"""
> @sgoveas Could you implement f-strings instead of currently used string 
> formatting.

Done
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5908#issuecomment-1009643589
___
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#5908][synchronized] Check default debug level of sssd and corresponding logs

2022-01-10 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5908
Author: sgoveas
 Title: #5908: Check default debug level of sssd and corresponding logs
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5908/head:pr5908
git checkout pr5908
From 710a7127e0e622c91c1c33428b05b6407f0d9531 Mon Sep 17 00:00:00 2001
From: Madhuri Upadhye 
Date: Fri, 22 Jan 2021 15:35:17 +0530
Subject: [PATCH] Check default debug level of sssd and corresponding logs

It consists of five test cases:
  1. Check default debug level when sssd start
successfully
  2. Check default debug level by
successful authentication of the user
  3. Check default level as 0 and 1
  4. Check default level as 2
  5. Check SBUS code should not trigger failure
message during modules startup

Verifies:
Issues: #5422
#5425

Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1893159
  https://bugzilla.redhat.com/show_bug.cgi?id=1915319
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../alltests/test_default_debug_level.py  | 153 ++
 2 files changed, 154 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_default_debug_level.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..e23f23d21d 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+defaultdebuglevel: Test default debug level sssd
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_default_debug_level.py b/src/tests/multihost/alltests/test_default_debug_level.py
new file mode 100644
index 00..3728c86117
--- /dev/null
+++ b/src/tests/multihost/alltests/test_default_debug_level.py
@@ -0,0 +1,153 @@
+"""Automation for default debug level
+
+:requirement: SSSD - Default debug level
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.defaultdebuglevel
+@pytest.mark.tier1_2
+class TestDefaultDebugLevel(object):
+""" Check sssd default debug level """
+def test_0001_check_default_debug_level(self, multihost, backupsssdconf):
+"""
+:title: default debug logs: Check default debug level when sssd start
+ successfully
+:id: 1f38b560-27dc-4144-895d-e667420b0467
+"""
+section = f"domain/{ds_instance_name}"
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+# stop sssd, delete sssd logs and cache, start sssd
+tools.clear_sssd_cache()
+time.sleep(5)
+log_list = ['sssd', f'sssd_{ds_instance_name}',
+'sssd_nss', 'sssd_pam']
+for log_filename in log_list:
+log = f'/var/log/sssd/{log_filename}.log'
+log_str = multihost.client[0].get_file_contents(log).decode(
+'utf-8')
+pattern1 = re.compile(r'Starting with debug level = 0x0070')
+default_debug = pattern1.search(log_str)
+assert default_debug is not None
+log_split = log_str.split("\n")
+for index in range(len(log_split)-1):
+log_single_line = log_split[index]
+pattern2 = re.compile(r'(0x\w+)')
+pattern3 = re.compile(r'(0x\d+)')
+debug_str1 = pattern2.search(log_single_line)
+debug_str2 = pattern3.search(log_single_line)
+assert debug_str1.group() == '0x1f7c0' or \
+debug_str2.group() <= 0x0040
+
+def test_0002_check_default_level_with_auth(self, multihost,
+backupsssdconf):
+"""
+:title: default debug logs: Check successful login with default
+ log level doesn't generate any logs
+:id: f40a7c66-6b5f-4f3c-8fcb-6aa12f415473
+"""
+section = f"domain/{ds_instance_name}"
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+domain_params = {'fallback_homed

[SSSD] [sssd PR#5944][+Accepted] Tests: Fix python-alltests-tier1-2 Add local users

2022-01-07 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5944
Title: #5944: Tests: Fix python-alltests-tier1-2 Add local users

Label: +Accepted
___
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#5944][+Tests] Tests: Fix python-alltests-tier1-2 Add local users

2022-01-07 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5944
Title: #5944: Tests: Fix python-alltests-tier1-2 Add local users

Label: +Tests
___
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#5929][synchronized] Tests: Add tests for poor man's backtrace

2022-01-06 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5929
Author: sgoveas
 Title: #5929: Tests: Add tests for poor man's backtrace
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5929/head:pr5929
git checkout pr5929
From 2ab89e3399021b54bf84358e63993e49a8fe2080 Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Sun, 5 Dec 2021 19:27:51 +0530
Subject: [PATCH] Tests: Add tests for poor man's backtrace

Tests are added to check that no duplicate backtrace is generated and it
is generated when
* no debug level is defined / default debug level
* debug level is set to 0 or 1
* no backtrace when debug level is >= 9
* backtrace can be disabled

Verifies: #5585
  https://bugzilla.redhat.com/show_bug.cgi?id=1949149
  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../multihost/alltests/test_backtrace.py  | 167 ++
 2 files changed, 168 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_backtrace.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..8754deabef 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+backtrace: test poor man's backtrace in logs for debug level < 9
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_backtrace.py b/src/tests/multihost/alltests/test_backtrace.py
new file mode 100644
index 00..9c890c4a0f
--- /dev/null
+++ b/src/tests/multihost/alltests/test_backtrace.py
@@ -0,0 +1,167 @@
+"""Automation poor man's backtrace
+
+:requirement: Poor Man's Backtrace
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+:bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1949149
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.backtrace
+@pytest.mark.tier1_2
+class TestPoorManBacktrace(object):
+""" Check sssd backtrace feature """
+def test_0001_bz2021196(self, multihost, backupsssdconf):
+"""
+:title: avoid duplicate backtraces
+:id: d4d8a0a0-ab90-4c8f-8087-95dc7ad3f3ae
+:customerscenario: true
+:bugzilla:
+  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
+  https://bugzilla.redhat.com/show_bug.cgi?id=2021499
+"""
+hostname = multihost.master[0].sys_hostname
+bad_ldap_uri = f"ldaps://typo.{hostname}"
+tools = sssdTools(multihost.client[0])
+domain_params = {'ldap_uri': bad_ldap_uri}
+tools.sssd_conf(f'domain/{ds_instance_name}', domain_params)
+tools.clear_sssd_cache()
+logfile = '/var/log/sssd/sssd_nss.log'
+cmd = f'getent passwd fakeuser@{ds_instance_name}'
+multihost.client[0].run_command(cmd, raiseonerr=False)
+msg = 'BACKTRACE DUMP ENDS HERE'
+msg2 = '... skipping repetitive backtrace ...'
+pattern = re.compile(fr'{msg}')
+pattern2 = re.compile(fr'{msg2}')
+log_str1 = multihost.client[0].get_file_contents(logfile). \
+decode('utf-8')
+multihost.client[0].run_command(f'> {logfile}')
+multihost.client[0].run_command(cmd, raiseonerr=False)
+time.sleep(2)
+log_str2 = multihost.client[0].get_file_contents(logfile). \
+decode('utf-8')
+# Check the backtrace is dumped first time and no backtrace is skipped
+assert pattern.search(log_str1) and not pattern2.search(log_str1)
+# Check there is no new backtrace with the same issue and repeative
+# backtrace is skipped
+assert pattern2.search(log_str2) and not pattern.search(log_str2)
+
+def test_0001_bz1949149(self, multihost, backupsssdconf):
+"""
+:title: backtrace is disabled if debug level >= 9
+:id: 50f2d501-3296-4229-86a0-b81844381637
+"""
+section = f"domain/{ds_instance_name}"
+param = {'debug_level': '9'}
+serv_list = ['sssd', section, 'nss', 'pam']
+tools = sssdTools(multihost.client[0])
+for serv in serv_list:
+tools.sssd_conf(serv, param)
+tools.clear_sssd_cache()
+cmd_kill = 'kil

[SSSD] [sssd PR#5942][+Accepted] Tests: Fix python-alltests-tier1-2

2022-01-05 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5942
Title: #5942: Tests: Fix python-alltests-tier1-2

Label: +Accepted
___
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#5892][+Accepted] Tests: Add a test for BZ2004406

2022-01-05 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5892
Title: #5892: Tests: Add a test for BZ2004406

Label: +Accepted
___
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#5892][-Waiting for review] Tests: Add a test for BZ2004406

2022-01-05 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5892
Title: #5892: Tests: Add a test for BZ2004406

Label: -Waiting for review
___
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#5933][+Tests] Fix for test bz1294670

2021-12-23 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5933
Title: #5933: Fix for test bz1294670

Label: +Tests
___
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#5933][+Accepted] Fix for test bz1294670

2021-12-23 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5933
Title: #5933: Fix for test bz1294670

Label: +Accepted
___
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#5908][synchronized] Check default debug level of sssd and corresponding logs

2021-12-21 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5908
Author: sgoveas
 Title: #5908: Check default debug level of sssd and corresponding logs
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5908/head:pr5908
git checkout pr5908
From be0a00f91b4095842cc2d2ec859f5dcb8b17aaea Mon Sep 17 00:00:00 2001
From: Madhuri Upadhye 
Date: Fri, 22 Jan 2021 15:35:17 +0530
Subject: [PATCH] Check default debug level of sssd and corresponding logs

It consists of five test cases:
  1. Check default debug level when sssd start
successfully
  2. Check default debug level by
successful authentication of the user
  3. Check default level as 0 and 1
  4. Check default level as 2
  5. Check SBUS code should not trigger failure
message during modules startup

Verifies:
Issues: #5422
#5425

Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1893159
  https://bugzilla.redhat.com/show_bug.cgi?id=1915319
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../alltests/test_default_debug_level.py  | 153 ++
 2 files changed, 154 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_default_debug_level.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..e23f23d21d 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+defaultdebuglevel: Test default debug level sssd
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_default_debug_level.py b/src/tests/multihost/alltests/test_default_debug_level.py
new file mode 100644
index 00..d9c2d759a9
--- /dev/null
+++ b/src/tests/multihost/alltests/test_default_debug_level.py
@@ -0,0 +1,153 @@
+"""Automation for default debug level
+
+:requirement: SSSD - Default debug level
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.defaultdebuglevel
+@pytest.mark.tier1_2
+class TestDefaultDebugLevel(object):
+""" Check sssd default debug level """
+def test_0001_check_default_debug_level(self, multihost, backupsssdconf):
+"""
+:title: default debug logs: Check default debug level when sssd start
+ successfully
+:id: 1f38b560-27dc-4144-895d-e667420b0467
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+# stop sssd, delete sssd logs and cache, start sssd
+tools.clear_sssd_cache()
+time.sleep(5)
+log_list = ['sssd', f'sssd_{ds_instance_name}',
+'sssd_nss', 'sssd_pam']
+for log_filename in log_list:
+log = '/var/log/sssd/%s.log' % log_filename
+log_str = multihost.client[0].get_file_contents(log).decode(
+'utf-8')
+pattern1 = re.compile(r'Starting with debug level = 0x0070')
+default_debug = pattern1.search(log_str)
+assert default_debug is not None
+log_split = log_str.split("\n")
+for index in range(len(log_split)-1):
+log_single_line = log_split[index]
+pattern2 = re.compile(r'(0x\w+)')
+pattern3 = re.compile(r'(0x\d+)')
+debug_str1 = pattern2.search(log_single_line)
+debug_str2 = pattern3.search(log_single_line)
+assert debug_str1.group() == '0x1f7c0' or \
+debug_str2.group() <= 0x0040
+
+def test_0002_check_default_level_with_auth(self, multihost,
+backupsssdconf):
+"""
+:title: default debug logs: Check successful login with default
+ log level doesn't generate any logs
+:id: f40a7c66-6b5f-4f3c-8fcb-6aa12f415473
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+

[SSSD] [sssd PR#5908][comment] Check default debug level of sssd and corresponding logs

2021-12-21 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5908
Title: #5908: Check default debug level of sssd and corresponding logs

sgoveas commented:
"""
> PEP check is failing:
> 
> * https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5908/3/fedora34/ci-pep8.log
> * https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5908/3/rhel8/ci-pep8.log

Fixed the errors from this PR. Will send a separate commit for the other errors 
if they still exist
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5908#issuecomment-998567450
___
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#5908][synchronized] Check default debug level of sssd and corresponding logs

2021-12-21 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5908
Author: sgoveas
 Title: #5908: Check default debug level of sssd and corresponding logs
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5908/head:pr5908
git checkout pr5908
From 052829b3d1445c03aa015983511f2cd6398179ee Mon Sep 17 00:00:00 2001
From: Madhuri Upadhye 
Date: Fri, 22 Jan 2021 15:35:17 +0530
Subject: [PATCH] Check default debug level of sssd and corresponding logs

It consists of five test cases:
  1. Check default debug level when sssd start
successfully
  2. Check default debug level by
successful authentication of the user
  3. Check default level as 0 and 1
  4. Check default level as 2
  5. Check SBUS code should not trigger failure
message during modules startup

Verifies:
Issues: #5422
#5425

Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1893159
  https://bugzilla.redhat.com/show_bug.cgi?id=1915319
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../alltests/test_default_debug_level.py  | 153 ++
 2 files changed, 154 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_default_debug_level.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..e23f23d21d 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+defaultdebuglevel: Test default debug level sssd
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_default_debug_level.py b/src/tests/multihost/alltests/test_default_debug_level.py
new file mode 100644
index 00..b96498a1e6
--- /dev/null
+++ b/src/tests/multihost/alltests/test_default_debug_level.py
@@ -0,0 +1,153 @@
+"""Automation for default debug level
+
+:requirement: SSSD - Default debug level
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.defaultdebuglevel
+@pytest.mark.tier1_2
+class TestDefaultDebugLevel(object):
+""" Check sssd default debug level """
+def test_0001_check_default_debug_level(self, multihost, backupsssdconf):
+"""
+:title: default debug logs: Check default debug level when sssd start
+ successfully
+:id: 1f38b560-27dc-4144-895d-e667420b0467
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+# stop sssd, delete sssd logs and cache, start sssd
+tools.clear_sssd_cache()
+time.sleep(5)
+log_list = ['sssd', f'sssd_{ds_instance_name}',
+'sssd_nss', 'sssd_pam']
+for log_filename in log_list:
+log = '/var/log/sssd/%s.log' % log_filename
+log_str = multihost.client[0].get_file_contents(log).decode(
+'utf-8')
+pattern1 = re.compile(r'Starting with debug level = 0x0070')
+default_debug = pattern1.search(log_str)
+assert default_debug is not None
+log_split = log_str.split("\n")
+for index in range(len(log_split)-1):
+log_single_line = log_split[index]
+pattern2 = re.compile(r'(0x\w+)')
+pattern3 = re.compile(r'(0x\d+)')
+debug_str1 = pattern2.search(log_single_line)
+debug_str2 = pattern3.search(log_single_line)
+assert debug_str1.group() == '0x1f7c0' or \
+   debug_str2.group() <= 0x0040
+
+def test_0002_check_default_level_with_auth(self, multihost,
+backupsssdconf):
+"""
+:title: default debug logs: Check successful login with default
+ log level doesn't generate any logs
+:id: f40a7c66-6b5f-4f3c-8fcb-6aa12f415473
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+

[SSSD] [sssd PR#5925][-Waiting for review] TEST: Current value of ssh_hash_known_hosts causes error in the default configuration in FIPS mode

2021-12-20 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5925
Title: #5925: TEST: Current value of ssh_hash_known_hosts causes error in the 
default configuration in FIPS mode

Label: -Waiting for review
___
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#5925][+Accepted] TEST: Current value of ssh_hash_known_hosts causes error in the default configuration in FIPS mode

2021-12-20 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5925
Title: #5925: TEST: Current value of ssh_hash_known_hosts causes error in the 
default configuration in FIPS mode

Label: +Accepted
___
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#5925][+Waiting for review] TEST: Current value of ssh_hash_known_hosts causes error in the default configuration in FIPS mode

2021-12-20 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5925
Title: #5925: TEST: Current value of ssh_hash_known_hosts causes error in the 
default configuration in FIPS mode

Label: +Waiting for review
___
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#5892][comment] Tests: Add a test for BZ2004406

2021-12-20 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5892
Title: #5892: Tests: Add a test for BZ2004406

sgoveas commented:
"""
Please add short explanation/detail of the test in the commit message and a 
relevant title
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5892#issuecomment-997778639
___
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#5908][+Waiting for review] Check default debug level of sssd and corresponding logs

2021-12-20 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5908
Title: #5908: Check default debug level of sssd and corresponding logs

Label: +Waiting for review
___
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#5892][+Changes requested] Tests: Add a test for BZ2004406

2021-12-20 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5892
Title: #5892: Tests: Add a test for BZ2004406

Label: +Changes requested
___
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#5892][-Waiting for review] Tests: Add a test for BZ2004406

2021-12-20 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5892
Title: #5892: Tests: Add a test for BZ2004406

Label: -Waiting for review
___
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#5929][synchronized] Tests: Add tests for poor man's backtrace

2021-12-18 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5929
Author: sgoveas
 Title: #5929: Tests: Add tests for poor man's backtrace
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5929/head:pr5929
git checkout pr5929
From 4c5b146029387780a65f09fa9a89526fac39c34d Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Sun, 5 Dec 2021 19:27:51 +0530
Subject: [PATCH] Tests: Add tests for poor man's backtrace

Tests are added to check that no duplicate backtrace is generated and it
is generated when
* no debug level is defined / default debug level
* debug level is set to 0 or 1
* no backtrace when debug level is >= 9
* backtrace can be disabled

Verifies: #5585
  https://bugzilla.redhat.com/show_bug.cgi?id=1949149
  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../multihost/alltests/test_backtrace.py  | 167 ++
 2 files changed, 168 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_backtrace.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..8754deabef 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+backtrace: test poor man's backtrace in logs for debug level < 9
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_backtrace.py b/src/tests/multihost/alltests/test_backtrace.py
new file mode 100644
index 00..15bd45bd29
--- /dev/null
+++ b/src/tests/multihost/alltests/test_backtrace.py
@@ -0,0 +1,167 @@
+"""Automation poor man's backtrace
+
+:requirement: Poor Man's Backtrace
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+:bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1949149
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.backtrace
+@pytest.mark.tier1_2
+class TestPoorManBacktrace(object):
+""" Check sssd backtrace feature """
+def test_0001_bz2021196(self, multihost, backupsssdconf):
+"""
+:title: avoid duplicate backtraces
+:id: d4d8a0a0-ab90-4c8f-8087-95dc7ad3f3ae
+:customerscenario: true
+:bugzilla:
+  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
+  https://bugzilla.redhat.com/show_bug.cgi?id=2021499
+"""
+hostname = multihost.master[0].sys_hostname
+bad_ldap_uri = "ldaps://typo.%s" % hostname
+tools = sssdTools(multihost.client[0])
+domain_params = {'ldap_uri': bad_ldap_uri}
+tools.sssd_conf('domain/%s' % ds_instance_name, domain_params)
+tools.clear_sssd_cache()
+logfile = '/var/log/sssd/sssd_nss.log'
+cmd = f'getent passwd fakeuser@{ds_instance_name}'
+multihost.client[0].run_command(cmd, raiseonerr=False)
+msg = 'BACKTRACE DUMP ENDS HERE'
+msg2 = '... skipping repetitive backtrace ...'
+pattern = re.compile(fr'{msg}')
+pattern2 = re.compile(fr'{msg2}')
+log_str1 = multihost.client[0].get_file_contents(logfile). \
+decode('utf-8')
+multihost.client[0].run_command(f'> {logfile}')
+multihost.client[0].run_command(cmd, raiseonerr=False)
+time.sleep(2)
+log_str2 = multihost.client[0].get_file_contents(logfile). \
+decode('utf-8')
+# Check the backtrace is dumped first time and no backtrace is skipped
+assert pattern.search(log_str1) and not pattern2.search(log_str1)
+# Check there is no new backtrace with the same issue and repeative
+# backtrace is skipped
+assert pattern2.search(log_str2) and not pattern.search(log_str2)
+
+def test_0001_bz1949149(self, multihost, backupsssdconf):
+"""
+:title: backtrace is disabled if debug level >= 9
+:id: 50f2d501-3296-4229-86a0-b81844381637
+"""
+section = "domain/%s" % ds_instance_name
+param = {'debug_level': '9'}
+serv_list = ['sssd', section, 'nss', 'pam']
+tools = sssdTools(multihost.client[0])
+for serv in serv_list:
+tools.sssd_conf(serv, param)
+tools.clear_sssd_cache()
+cmd_kill = 'kil

[SSSD] [sssd PR#5929][synchronized] Tests: Add tests for poor man's backtrace

2021-12-18 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5929
Author: sgoveas
 Title: #5929: Tests: Add tests for poor man's backtrace
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5929/head:pr5929
git checkout pr5929
From b4d2a6b1d2f21db6e326e9f7786f577e329e61b7 Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Sun, 5 Dec 2021 19:27:51 +0530
Subject: [PATCH] Tests: Add tests for poor man's backtrace

Tests are added to check that no duplicate backtrace is generated and it
is generated when
* no debug level is defined / default debug level
* debug level is set to 0 or 1
* no backtrace when debug level is >= 9
* backtrace can be disabled

Verifies: #5585
  https://bugzilla.redhat.com/show_bug.cgi?id=1949149
  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../multihost/alltests/test_backtrace.py  | 167 ++
 2 files changed, 168 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_backtrace.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..8754deabef 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+backtrace: test poor man's backtrace in logs for debug level < 9
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_backtrace.py b/src/tests/multihost/alltests/test_backtrace.py
new file mode 100644
index 00..05c52fad51
--- /dev/null
+++ b/src/tests/multihost/alltests/test_backtrace.py
@@ -0,0 +1,167 @@
+"""Automation poor man's backtrace
+
+:requirement: Poor Man's Backtrace
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+:bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1949149
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.backtrace
+@pytest.mark.tier1_2
+class TestPoorManBacktrace(object):
+""" Check sssd backtrace feature """
+def test_0001_bz2021196(self, multihost, backupsssdconf):
+"""
+:title: avoid duplicate backtraces
+:id: d4d8a0a0-ab90-4c8f-8087-95dc7ad3f3ae
+:customerscenario: true
+:bugzilla:
+  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
+  https://bugzilla.redhat.com/show_bug.cgi?id=2021499
+"""
+hostname = multihost.master[0].sys_hostname
+bad_ldap_uri = "ldaps://typo.%s" % hostname
+tools = sssdTools(multihost.client[0])
+domain_params = {'ldap_uri': bad_ldap_uri}
+tools.sssd_conf('domain/%s' % ds_instance_name, domain_params)
+tools.clear_sssd_cache()
+logfile = '/var/log/sssd/sssd_nss.log'
+cmd = f'getent passwd fakeuser@{ds_instance_name}'
+multihost.client[0].run_command(cmd, raiseonerr=False)
+msg = 'BACKTRACE DUMP ENDS HERE'
+msg2 = '... skipping repetitive backtrace ...'
+pattern = re.compile(fr'{msg}')
+pattern2 = re.compile(fr'{msg2}')
+log_str1 = multihost.client[0].get_file_contents(logfile). \
+decode('utf-8')
+multihost.client[0].run_command(f'> {logfile}')
+multihost.client[0].run_command(cmd, raiseonerr=False)
+time.sleep(2)
+log_str2 = multihost.client[0].get_file_contents(logfile). \
+decode('utf-8')
+# Check the backtrace is dumped first time and no backtrace is skipped
+assert pattern.search(log_str1) and not pattern2.search(log_str1)
+# Check there is no new backtrace with the same issue and repeative
+# backtrace is skipped
+assert pattern2.search(log_str2) and not pattern.search(log_str2)
+
+def test_0001_bz1949149(self, multihost, backupsssdconf):
+"""
+:title: backtrace is disabled if debug level >= 9
+:id: 50f2d501-3296-4229-86a0-b81844381637
+"""
+section = "domain/%s" % ds_instance_name
+param = {'debug_level': '9'}
+serv_list = ['sssd', section, 'nss', 'pam']
+tools = sssdTools(multihost.client[0])
+for serv in serv_list:
+tools.sssd_conf(serv, param)
+tools.clear_sssd_cache()
+cmd_kill = 'kil

[SSSD] [sssd PR#5929][+Waiting for review] Tests: Add tests for poor man's backtrace

2021-12-18 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5929
Title: #5929: Tests: Add tests for poor man's backtrace

Label: +Waiting for review
___
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#5929][+Tests] Tests: Add tests for poor man's backtrace

2021-12-18 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5929
Title: #5929: Tests: Add tests for poor man's backtrace

Label: +Tests
___
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#5929][opened] Tests: Add tests for poor man's backtrace

2021-12-18 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5929
Author: sgoveas
 Title: #5929: Tests: Add tests for poor man's backtrace
Action: opened

PR body:
"""
Tests are added to check that no duplicate backtrace is generated and it
is generated when
* no debug level is defined / default debug level
* debug level is set to 0 or 1
* no backtrace when debug level is >= 9
* backtrace can be disabled

Verifies: #5585
  https://bugzilla.redhat.com/show_bug.cgi?id=1949149
  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5929/head:pr5929
git checkout pr5929
From 1521cad7a289236d74a98ead1bd0ddf75c3fa06e Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Sun, 5 Dec 2021 19:27:51 +0530
Subject: [PATCH] Tests: Add tests for poor man's backtrace

Tests are added to check that no duplicate backtrace is generated and it
is generated when
* no debug level is defined / default debug level
* debug level is set to 0 or 1
* no backtrace when debug level is >= 9
* backtrace can be disabled

Verifies: #5585
  https://bugzilla.redhat.com/show_bug.cgi?id=1949149
  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../multihost/alltests/test_automount.py  |   2 +-
 .../multihost/alltests/test_backtrace.py  | 167 ++
 3 files changed, 169 insertions(+), 1 deletion(-)
 create mode 100644 src/tests/multihost/alltests/test_backtrace.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..8754deabef 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+backtrace: test poor man's backtrace in logs for debug level < 9
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_automount.py b/src/tests/multihost/alltests/test_automount.py
index 0efd18c723..a39fd5d7fe 100644
--- a/src/tests/multihost/alltests/test_automount.py
+++ b/src/tests/multihost/alltests/test_automount.py
@@ -448,7 +448,7 @@ def test_008_wildcardsearch(self, multihost, indirect_nismaps,
 for i in range(1, 10):
 key = 'foo%d' % i
 ldap_filter = '\(\&\(cn=%s\)\(objectclass=nisObject\)\)' % key
-log_1 = re.compile(r'%s' % (ldap_filter))
+log_1 = re.compile(r'%s' % ldap_filter)
 assert log_1.search(tcpdump_ascii)
 
 # delete the pcap file
diff --git a/src/tests/multihost/alltests/test_backtrace.py b/src/tests/multihost/alltests/test_backtrace.py
new file mode 100644
index 00..05c52fad51
--- /dev/null
+++ b/src/tests/multihost/alltests/test_backtrace.py
@@ -0,0 +1,167 @@
+"""Automation poor man's backtrace
+
+:requirement: Poor Man's Backtrace
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+:bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1949149
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.backtrace
+@pytest.mark.tier1_2
+class TestPoorManBacktrace(object):
+""" Check sssd backtrace feature """
+def test_0001_bz2021196(self, multihost, backupsssdconf):
+"""
+:title: avoid duplicate backtraces
+:id: d4d8a0a0-ab90-4c8f-8087-95dc7ad3f3ae
+:customerscenario: true
+:bugzilla:
+  https://bugzilla.redhat.com/show_bug.cgi?id=2021196
+  https://bugzilla.redhat.com/show_bug.cgi?id=2021499
+"""
+hostname = multihost.master[0].sys_hostname
+bad_ldap_uri = "ldaps://typo.%s" % hostname
+tools = sssdTools(multihost.client[0])
+domain_params = {'ldap_uri': bad_ldap_uri}
+tools.sssd_conf('domain/%s' % ds_instance_name, domain_params)
+tools.clear_sssd_cache()
+logfile = '/var/log/sssd/sssd_nss.log'
+cmd = f'getent passwd fakeuser@{ds_instance_name}'
+multihost.client[0].run_command(cmd, raiseonerr=False)
+msg = 'BACKTRACE DUMP ENDS HERE'
+msg2 = '... skipping repetitive backtrace ...'
+pattern = re.compile(fr'{msg}')
+pattern2 = re.compile(fr'{msg2}')
+log_str1 = multihost.client[0].get_file_contents(logfile). \
+

[SSSD] [sssd PR#5908][comment] Check default debug level of sssd and corresponding logs

2021-12-16 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5908
Title: #5908: Check default debug level of sssd and corresponding logs

sgoveas commented:
"""
> ```
> Check default level as 0 and 1
> Check default level as 2
> ```
> 
> Default level is 
> `(SSSDBG_FATAL_FAILURE|SSSDBG_CRIT_FAILURE|SSSDBG_OP_FAILURE)`, this 
> corresponds to `2`.
> 
> How is it - "default level as 0 and 1"?
> 
> UPD: having read tests code, I think I understand now. But I think 
> description is unclear. Something like "Check that messages with levels 0 and 
> 1 are printed with default log level" would be better, imo.

Updated the title
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5908#issuecomment-995764217
___
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#5908][synchronized] Check default debug level of sssd and corresponding logs

2021-12-16 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5908
Author: sgoveas
 Title: #5908: Check default debug level of sssd and corresponding logs
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5908/head:pr5908
git checkout pr5908
From 9cccace78c4cb60966b215ba624810704c3a3ec8 Mon Sep 17 00:00:00 2001
From: Madhuri Upadhye 
Date: Fri, 22 Jan 2021 15:35:17 +0530
Subject: [PATCH] Check default debug level of sssd and corresponding logs

It consists of five test cases:
  1. Check default debug level when sssd start
successfully
  2. Check default debug level by
successful authentication of the user
  3. Check default level as 0 and 1
  4. Check default level as 2
  5. Check SBUS code should not trigger failure
message during modules startup

Verifies:
Issues: #5422
#5425

Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1893159
  https://bugzilla.redhat.com/show_bug.cgi?id=1915319
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../alltests/test_default_debug_level.py  | 150 ++
 2 files changed, 151 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_default_debug_level.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..e23f23d21d 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+defaultdebuglevel: Test default debug level sssd
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_default_debug_level.py b/src/tests/multihost/alltests/test_default_debug_level.py
new file mode 100644
index 00..731bebbcb3
--- /dev/null
+++ b/src/tests/multihost/alltests/test_default_debug_level.py
@@ -0,0 +1,150 @@
+"""Automation for default debug level
+
+:requirement: SSSD - Default debug level
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.defaultdebuglevel
+@pytest.mark.tier1_2
+class TestDefaultDebugLevel(object):
+""" Check sssd default debug level """
+def test_0001_check_default_debug_level(self, multihost, backupsssdconf):
+"""
+:title: default debug logs: Check default debug level when sssd start
+ successfully
+:id: 1f38b560-27dc-4144-895d-e667420b0467
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+# stop sssd, delete sssd logs and cache, start sssd
+tools.clear_sssd_cache()
+log_list = ['sssd', f'sssd_{ds_instance_name}',
+   'sssd_nss', 'sssd_pam']
+for log_filename in log_list:
+log = '/var/log/sssd/%s.log' % log_filename
+log_str = multihost.client[0].get_file_contents(log).decode(
+'utf-8')
+pattern1 = re.compile(r'Starting with debug level = 0x0070')
+default_debug = pattern1.search(log_str)
+log_split = log_str.split("\n")
+for index in range(len(log_split)-1):
+log_single_line = log_split[index]
+pattern2 = re.compile(r'(0x\w+)')
+debug_str = pattern2.search(log_single_line)
+if debug_str.group() != '0x1f7c0':
+assert False
+assert default_debug is not None
+
+def test_0002_check_default_level_with_auth(self, multihost,
+backupsssdconf):
+"""
+:title: default debug logs: Check successful login with default
+ log level doesn't generate any logs
+:id: f40a7c66-6b5f-4f3c-8fcb-6aa12f415473
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+domain_params = {'fallback_homedir': '/home/%u'}
+tools.sssd_conf(section, domain_params)
+# stop sssd, delete logs and cache, start sssd
+tools.clear_sssd_cache

[SSSD] [sssd PR#5908][synchronized] Check default debug level of sssd and corresponding logs

2021-12-16 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5908
Author: sgoveas
 Title: #5908: Check default debug level of sssd and corresponding logs
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5908/head:pr5908
git checkout pr5908
From a7342fd5907e703f8682935ba8bc67417c4d6dce Mon Sep 17 00:00:00 2001
From: Madhuri Upadhye 
Date: Fri, 22 Jan 2021 15:35:17 +0530
Subject: [PATCH] Check default debug level of sssd and corresponding logs

It consists of five test cases:
  1. Check default debug level when sssd start
successfully
  2. Check default debug level by
successful authentication of the user
  3. Check default level as 0 and 1
  4. Check default level as 2
  5. Check SBUS code should not trigger failure
message during modules startup

Verifies:
Issues: #5422
#5425

Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1893159
  https://bugzilla.redhat.com/show_bug.cgi?id=1915319
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../alltests/test_default_debug_level.py  | 150 ++
 2 files changed, 151 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_default_debug_level.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..e23f23d21d 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+defaultdebuglevel: Test default debug level sssd
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_default_debug_level.py b/src/tests/multihost/alltests/test_default_debug_level.py
new file mode 100644
index 00..53c6bfb469
--- /dev/null
+++ b/src/tests/multihost/alltests/test_default_debug_level.py
@@ -0,0 +1,150 @@
+"""Automation for default debug level
+
+:requirement: SSSD - Default debug logs
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.defaultdebuglevel
+@pytest.mark.tier1_2
+class TestDefaultDebugLevel(object):
+""" Check sssd default debug level """
+def test_0001_check_default_debug_level(self, multihost, backupsssdconf):
+"""
+:title: default debug logs: Check default debug level when sssd start
+ successfully
+:id: 1f38b560-27dc-4144-895d-e667420b0467
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+# stop sssd, delete sssd logs and cache, start sssd
+tools.clear_sssd_cache()
+log_list = ['sssd', f'sssd_{ds_instance_name}',
+   'sssd_nss', 'sssd_pam']
+for log_filename in log_list:
+log = '/var/log/sssd/%s.log' % log_filename
+log_str = multihost.client[0].get_file_contents(log).decode(
+'utf-8')
+pattern1 = re.compile(r'Starting with debug level = 0x0070')
+default_debug = pattern1.search(log_str)
+log_split = log_str.split("\n")
+for index in range(len(log_split)-1):
+log_single_line = log_split[index]
+pattern2 = re.compile(r'(0x\w+)')
+debug_str = pattern2.search(log_single_line)
+if debug_str.group() != '0x1f7c0':
+assert False
+assert default_debug is not None
+
+def test_0002_check_default_level_with_auth(self, multihost,
+backupsssdconf):
+"""
+:title: default debug logs: Check successful login with default
+ log level doesn't generate any logs
+:id: f40a7c66-6b5f-4f3c-8fcb-6aa12f415473
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+domain_params = {'fallback_homedir': '/home/%u'}
+tools.sssd_conf(section, domain_params)
+# stop sssd, delete logs and cache, start sssd
+tools.clear_sssd_cache

[SSSD] [sssd PR#5912][+Accepted] Tests: Fix pytest-alltests-tier1

2021-12-09 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5912
Title: #5912: Tests: Fix pytest-alltests-tier1

Label: +Accepted
___
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#5912][+Tests] Tests: Fix pytest-alltests-tier1

2021-12-09 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5912
Title: #5912: Tests: Fix pytest-alltests-tier1

Label: +Tests
___
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#5908][opened] Check default debug level of sssd and corresponding logs

2021-12-05 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5908
Author: sgoveas
 Title: #5908: Check default debug level of sssd and corresponding logs
Action: opened

PR body:
"""
It consists of five test cases:
  1. Check default debug level when sssd start
successfully
  2. Check default debug level by
successful authentication of the user
  3. Check default level as 0 and 1
  4. Check default level as 2
  5. Check SBUS code should not trigger failure
message during modules startup

Verifies:
Issues: #5422
#5425

Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1893159
  https://bugzilla.redhat.com/show_bug.cgi?id=1915319
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5908/head:pr5908
git checkout pr5908
From 1b502e02abd896cae7c0f62cb44383ae2f04f6e5 Mon Sep 17 00:00:00 2001
From: Madhuri Upadhye 
Date: Fri, 22 Jan 2021 15:35:17 +0530
Subject: [PATCH] Check default debug level of sssd and corresponding logs

It consists of five test cases:
  1. Check default debug level when sssd start
successfully
  2. Check default debug level by
successful authentication of the user
  3. Check default level as 0 and 1
  4. Check default level as 2
  5. Check SBUS code should not trigger failure
message during modules startup

Verifies:
Issues: #5422
#5425

Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1893159
  https://bugzilla.redhat.com/show_bug.cgi?id=1915319
---
 src/tests/multihost/alltests/pytest.ini   |   1 +
 .../alltests/test_default_debug_level.py  | 144 ++
 2 files changed, 145 insertions(+)
 create mode 100644 src/tests/multihost/alltests/test_default_debug_level.py

diff --git a/src/tests/multihost/alltests/pytest.ini b/src/tests/multihost/alltests/pytest.ini
index ac127c07f3..e23f23d21d 100644
--- a/src/tests/multihost/alltests/pytest.ini
+++ b/src/tests/multihost/alltests/pytest.ini
@@ -25,6 +25,7 @@ markers =
 ssh: Tests related to ssh responder
 ldaplibdebuglevel: Test ldap_library_debug_level option
 no_tier: test cases are not executed on any tier
+defaultdebuglevel: Test default debug level sssd
 tier1: tier1 test cases with run time of aproximately 60 minutes
 tier1_2: tier1 test cases split to keep runtime upto 60 minutes
 tier2: tier2 test cases
diff --git a/src/tests/multihost/alltests/test_default_debug_level.py b/src/tests/multihost/alltests/test_default_debug_level.py
new file mode 100644
index 00..bbdfd4dacf
--- /dev/null
+++ b/src/tests/multihost/alltests/test_default_debug_level.py
@@ -0,0 +1,144 @@
+"""Automation for default debug level
+
+:requirement: IDM-SSSD-REQ : LDAP Provider
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+"""
+
+from __future__ import print_function
+import re
+import time
+import pytest
+from sssd.testlib.common.utils import sssdTools
+from sssd.testlib.common.expect import pexpect_ssh
+from constants import ds_instance_name
+
+
+@pytest.mark.usefixtures('setup_sssd', 'create_posix_usersgroups')
+@pytest.mark.defaultdebuglevel
+@pytest.mark.tier1_2
+class TestDefaultDebugLevel(object):
+""" Check sssd default debug level """
+def test_0001_check_default_debug_level(self, multihost, backupsssdconf):
+"""
+:title: default debug logs: Check default debug level when sssd start
+ successfully
+:id: 1f38b560-27dc-4144-895d-e667420b0467
+"""
+section = "domain/%s" % ds_instance_name
+domain_params = {'debug_level': ''}
+tools = sssdTools(multihost.client[0])
+tools.sssd_conf(section, domain_params, action='delete')
+# stop sssd, delete sssd logs and cache, start sssd
+tools.clear_sssd_cache()
+log_list = ['sssd', 'sssd_example1', 'sssd_nss',
+'sssd_pam', 'sssd_implicit_files']
+for log_filename in log_list:
+log = '/var/log/sssd/%s.log' % log_filename
+log_str = multihost.client[0].get_file_contents(log).decode(
+'utf-8')
+log_split = log_str.split("\n")
+for index in range(len(log_split)-1):
+log_single_line = log_split[index]
+pattern = re.compile(r'0x\d*')
+debug_str = pattern.search(log_single_line)
+if debug_str.group() > '0x0040':
+assert False
+
+def test_0002_check_default_level_with_auth(self, multihost,
+backupsssdconf):
+"""
+:title: default debug logs: Check successful login with default
+ log level doesn't generate any logs
+:id: f40a7c66-6b5f-4f3c-8fcb-6aa12f415473
+"""
+section = "domain/%s" % ds_insta

[SSSD] [sssd PR#5881][comment] SDAP: Do not fail ASQ search when parsing a referenced entry fails

2021-11-30 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5881
Title: #5881: SDAP: Do not fail ASQ search when parsing a referenced entry fails

sgoveas commented:
"""
> @sgoveas , @sidecontrol , do we have any tests that could be easily extended 
> to cover those cases?

We have old tests for 'ldap_deref_threshold' downstream. It will be better to 
add new tests in pytest upstream than extending it.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5881#issuecomment-982612670
___
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#5885][comment] LDAP: expire accounts when today >= shadowExpire

2021-11-30 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5885
Title: #5885: LDAP: expire accounts when today >= shadowExpire

sgoveas commented:
"""
> @sgoveas , @sidecontrol ,
> 
> any chance this will affect any existing tests?

This will impact some of the old downstream tests, but we can port them to 
pytest and add it upstream
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5885#issuecomment-982608345
___
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#5887][+Accepted] Tests: proxy provider, user's removed secondary group remains in sssd cache

2021-11-25 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5887
Title: #5887: Tests: proxy provider, user's removed secondary group remains in 
sssd cache

Label: +Accepted
___
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#5889][+Tests] TEST: Add missing polarion requirements to tests

2021-11-25 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5889
Title: #5889: TEST: Add missing polarion requirements to tests

Label: +Tests
___
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#5889][+Waiting for review] TEST: Add missing polarion requirements to tests

2021-11-25 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5889
Title: #5889: TEST: Add missing polarion requirements to tests

Label: +Waiting for review
___
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#5889][opened] TEST: Add missing polarion requirements to tests

2021-11-25 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5889
Author: sgoveas
 Title: #5889: TEST: Add missing polarion requirements to tests
Action: opened

PR body:
"""
Some tests were not linked to polarion requirements
The subid tests added recently is linked to
"IDM-IPA-REQ: ipa subid range" in this PR
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5889/head:pr5889
git checkout pr5889
From d2b92fd673c2ad8826cc069807b1b97fd995c1a9 Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Fri, 26 Nov 2021 11:30:10 +0530
Subject: [PATCH] TEST: Add missing polarion requirements to tests

Some tests were not linked to polarion requirements
The subid tests added recently is linked to
"IDM-IPA-REQ: ipa subid range" in this PR
---
 src/tests/multihost/alltests/test_krb5.py| 1 +
 src/tests/multihost/ipa/test_subid_ranges.py | 8 +++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/tests/multihost/alltests/test_krb5.py b/src/tests/multihost/alltests/test_krb5.py
index 849437707f..5800a1c9ee 100644
--- a/src/tests/multihost/alltests/test_krb5.py
+++ b/src/tests/multihost/alltests/test_krb5.py
@@ -74,6 +74,7 @@ def test_0002_generating_lot_of(self, multihost, backupsssdconf):
  queries in a very large environment
 :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1772513
 :id: 74a60320-e48b-11eb-ba19-845cf3eff344
+:requirement: IDM-SSSD-REQ : LDAP Provider
 :steps:
   1. Start SSSD with any configuration
   2. Call 'getent passwd username@domain'
diff --git a/src/tests/multihost/ipa/test_subid_ranges.py b/src/tests/multihost/ipa/test_subid_ranges.py
index bbbd19d42e..b3df9b3d60 100644
--- a/src/tests/multihost/ipa/test_subid_ranges.py
+++ b/src/tests/multihost/ipa/test_subid_ranges.py
@@ -1,4 +1,10 @@
-""" Automation of IPA bugs """
+""" Automation of IPA subid feature bugs
+
+:requirement: IDM-IPA-REQ: ipa subid range
+:casecomponent: sssd
+:subsystemteam: sst_idm_sssd
+:upstream: yes
+"""
 
 import pytest
 import subprocess
___
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#5887][+Changes requested] Tests: proxy provider, user's removed secondary group remains in sssd cache

2021-11-25 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5887
Title: #5887: Tests: proxy provider, user's removed secondary group remains in 
sssd cache

Label: +Changes requested
___
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#5879][synchronized] TEST: Remove check for rhel 9 to enable CRB repo

2021-11-24 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5879
Author: sgoveas
 Title: #5879: TEST: Remove check for rhel 9 to enable CRB repo
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5879/head:pr5879
git checkout pr5879
From 49910a763d469c646d06a2d7674096866125705a Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Wed, 17 Nov 2021 12:59:23 +0530
Subject: [PATCH] TEST: Remove check for rhel 9 to enable CRB repo

Tests will run for 8.6 and rhel 9 and both need CRB to be enabled.
Removing the check for rhel 9, to make it work for 8.6 as well
---
 src/tests/multihost/ipa/conftest.py | 14 +++---
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/src/tests/multihost/ipa/conftest.py b/src/tests/multihost/ipa/conftest.py
index c039488433..4bf9061c58 100644
--- a/src/tests/multihost/ipa/conftest.py
+++ b/src/tests/multihost/ipa/conftest.py
@@ -171,18 +171,10 @@ def environment_setup(session_multihost, request):
 """
 Install necessary packages
 """
-pre_version = "sed -rn 's/.*([0-9])\.[0-9].*/\\1/p' /etc/redhat-release"
 client = session_multihost.client[0]
-if "Red Hat" in client.run_command("cat "
-   "/etc/redhat-release").stdout_text:
-version = [int(i) for i in
-   client.run_command(pre_version).stdout_text.split()
-   if i.isdigit()][0]
-if version >= 9:
-client.run_command("yum "
-   "--enablerepo=rhel-CRB install"
-   " -y shadow-utils*")
-client.run_command("yum install -y shadow-utils*")
+client.run_command("yum "
+   "--enablerepo=rhel-CRB install"
+   " -y shadow-utils*")
 client.run_command("yum install -y gcc")
 with pytest.raises(subprocess.CalledProcessError):
 client.run_command(f"grep subid /etc/nsswitch.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#5880][+Accepted] TEST: Lookup with fully-qualified name with 'cache_first = True'

2021-11-18 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5880
Title: #5880: TEST: Lookup with fully-qualified name with 'cache_first = True'

Label: +Accepted
___
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#5880][-Changes requested] TEST: Lookup with fully-qualified name with 'cache_first = True'

2021-11-18 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5880
Title: #5880: TEST: Lookup with fully-qualified name with 'cache_first = True'

Label: -Changes requested
___
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#5880][edited] TEST: Lookup with fully-qualified name with 'cache_first = True'

2021-11-18 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5880
Author: dparmar18
 Title: #5880: TEST: Lookup with fully-qualified name with 'cache_first = True'
Action: edited

 Changed field: body
Original value:
"""
TEST: Lookup with fully-qualified name with 'cache_first = True'

Verifies
  Issue: SSSD#5744
  Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2013294
"""

 Changed field: title
Original value:
"""
Automation code for bug 2013294/1992973/2013379
"""

___
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#5880][edited] Automation code for bug 2013294/1992973/2013379

2021-11-18 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5880
Author: dparmar18
 Title: #5880: Automation code for bug 2013294/1992973/2013379
Action: edited

 Changed field: body
Original value:
"""
Bugzilla links:

https://bugzilla.redhat.com/show_bug.cgi?id=2013294
https://bugzilla.redhat.com/show_bug.cgi?id=1992973
https://bugzilla.redhat.com/show_bug.cgi?id=2013379
"""

___
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#5880][+Tests] Automation code for bug 2013294/1992973/2013379

2021-11-17 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5880
Title: #5880: Automation code for bug 2013294/1992973/2013379

Label: +Tests
___
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#5880][+Changes requested] Automation code for bug 2013294/1992973/2013379

2021-11-17 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5880
Title: #5880: Automation code for bug 2013294/1992973/2013379

Label: +Changes requested
___
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#5879][+Waiting for review] TEST: Remove check for rhel 9 to enable CRB repo

2021-11-16 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5879
Title: #5879: TEST: Remove check for rhel 9 to enable CRB repo

Label: +Waiting for review
___
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#5879][+Tests] TEST: Remove check for rhel 9 to enable CRB repo

2021-11-16 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5879
Title: #5879: TEST: Remove check for rhel 9 to enable CRB repo

Label: +Tests
___
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#5879][synchronized] TEST: Remove check for rhel 9 to enable CRB repo

2021-11-16 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5879
Author: sgoveas
 Title: #5879: TEST: Remove check for rhel 9 to enable CRB repo
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5879/head:pr5879
git checkout pr5879
From c5700793a9711de38221a197c5127be854815f4c Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Wed, 17 Nov 2021 12:59:23 +0530
Subject: [PATCH] TEST: Remove check for rhel 9 to enable CRB repo

Tests will run for 8.6 and rhel 9 and both need CRB to be enabled.
Removing the check for rhel 9, to make it work for 8.6 as well
---
 src/tests/multihost/ipa/conftest.py | 14 +++---
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/src/tests/multihost/ipa/conftest.py b/src/tests/multihost/ipa/conftest.py
index c039488433..4bf9061c58 100644
--- a/src/tests/multihost/ipa/conftest.py
+++ b/src/tests/multihost/ipa/conftest.py
@@ -171,18 +171,10 @@ def environment_setup(session_multihost, request):
 """
 Install necessary packages
 """
-pre_version = "sed -rn 's/.*([0-9])\.[0-9].*/\\1/p' /etc/redhat-release"
 client = session_multihost.client[0]
-if "Red Hat" in client.run_command("cat "
-   "/etc/redhat-release").stdout_text:
-version = [int(i) for i in
-   client.run_command(pre_version).stdout_text.split()
-   if i.isdigit()][0]
-if version >= 9:
-client.run_command("yum "
-   "--enablerepo=rhel-CRB install"
-   " -y shadow-utils*")
-client.run_command("yum install -y shadow-utils*")
+client.run_command("yum "
+   "--enablerepo=rhel-CRB install"
+   " -y shadow-utils*")
 client.run_command("yum install -y gcc")
 with pytest.raises(subprocess.CalledProcessError):
 client.run_command(f"grep subid /etc/nsswitch.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#5879][opened] TEST: Remove check for rhel 9 to enable CRB repo

2021-11-16 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5879
Author: sgoveas
 Title: #5879: TEST: Remove check for rhel 9 to enable CRB repo
Action: opened

PR body:
"""
Tests will run for 8.6 and rhel 9 and both need CRB to be enabled.
Removing the check for rhel 9, to make it work for 8.6 as well
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5879/head:pr5879
git checkout pr5879
From 376660a557e67bb69a7b2d74a538fcfb3da0780a Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Wed, 17 Nov 2021 12:59:23 +0530
Subject: [PATCH] TEST: Remove check for rhel 9 to enable CRB repo

Tests will run for 8.6 and rhel 9 and both need CRB to be enabled.
Removing the check for rhel 9, to make it work for 8.6 as well
---
 src/tests/multihost/ipa/conftest.py | 15 +++
 1 file changed, 3 insertions(+), 12 deletions(-)

diff --git a/src/tests/multihost/ipa/conftest.py b/src/tests/multihost/ipa/conftest.py
index c039488433..a9186ae74a 100644
--- a/src/tests/multihost/ipa/conftest.py
+++ b/src/tests/multihost/ipa/conftest.py
@@ -171,18 +171,9 @@ def environment_setup(session_multihost, request):
 """
 Install necessary packages
 """
-pre_version = "sed -rn 's/.*([0-9])\.[0-9].*/\\1/p' /etc/redhat-release"
-client = session_multihost.client[0]
-if "Red Hat" in client.run_command("cat "
-   "/etc/redhat-release").stdout_text:
-version = [int(i) for i in
-   client.run_command(pre_version).stdout_text.split()
-   if i.isdigit()][0]
-if version >= 9:
-client.run_command("yum "
-   "--enablerepo=rhel-CRB install"
-   " -y shadow-utils*")
-client.run_command("yum install -y shadow-utils*")
+client.run_command("yum "
+   "--enablerepo=rhel-CRB install"
+   " -y shadow-utils*")
 client.run_command("yum install -y gcc")
 with pytest.raises(subprocess.CalledProcessError):
 client.run_command(f"grep subid /etc/nsswitch.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#5870][+Tests] TESTS: Add tier2 marker for ipa tests

2021-11-11 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5870
Title: #5870: TESTS: Add tier2 marker for ipa tests

Label: +Tests
___
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#5870][+Waiting for review] TESTS: Add tier2 marker for ipa tests

2021-11-11 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5870
Title: #5870: TESTS: Add tier2 marker for ipa tests

Label: +Waiting for review
___
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#5870][synchronized] TESTS: Add tier2 marker for ipa tests

2021-11-11 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5870
Author: sgoveas
 Title: #5870: TESTS: Add tier2 marker for ipa tests
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5870/head:pr5870
git checkout pr5870
From 04d64db729a8304a6e6c0332b6786e722e73daef Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Thu, 11 Nov 2021 17:28:16 +0530
Subject: [PATCH] TESTS: Add tier2 marker for ipa tests

Some of the ipa tests would be executed as tier1 tests. Added markers
for the ones that were not marked and would run as tier2 tests
---
 src/tests/multihost/ipa/pytest.ini  | 1 +
 src/tests/multihost/ipa/test_adhbac.py  | 1 +
 src/tests/multihost/ipa/test_adtrust.py | 1 +
 src/tests/multihost/ipa/test_hbac.py| 1 +
 4 files changed, 4 insertions(+)

diff --git a/src/tests/multihost/ipa/pytest.ini b/src/tests/multihost/ipa/pytest.ini
index 6be7556333..98efaa70a1 100644
--- a/src/tests/multihost/ipa/pytest.ini
+++ b/src/tests/multihost/ipa/pytest.ini
@@ -1,6 +1,7 @@
 [pytest]
 markers =
 tier1: tier1 test cases
+tier2: tier2 test cases
 hbac: Tests hbac in ipa provider environment
 trust: Tests related to IPA AD Trust
 adhbac: Test related to HBAC in IPA-AD Trust environment
diff --git a/src/tests/multihost/ipa/test_adhbac.py b/src/tests/multihost/ipa/test_adhbac.py
index 5d960bf636..971a14729c 100644
--- a/src/tests/multihost/ipa/test_adhbac.py
+++ b/src/tests/multihost/ipa/test_adhbac.py
@@ -16,6 +16,7 @@
 @pytest.mark.usefixtures('disable_allow_all_hbac',
  'create_ad_users',
  'create_ad_groups')
+@pytest.mark.tier2
 @pytest.mark.adhbac
 class TestADTrustHbac(object):
 """ AD Trust HBAC Test cases """
diff --git a/src/tests/multihost/ipa/test_adtrust.py b/src/tests/multihost/ipa/test_adtrust.py
index 64ff28aa3c..3f2fc66c1b 100644
--- a/src/tests/multihost/ipa/test_adtrust.py
+++ b/src/tests/multihost/ipa/test_adtrust.py
@@ -15,6 +15,7 @@
 
 
 @pytest.mark.usefixtures('setup_ipa_client')
+@pytest.mark.tier2
 @pytest.mark.trust
 class TestADTrust(object):
 """ IPA AD Trust tests """
diff --git a/src/tests/multihost/ipa/test_hbac.py b/src/tests/multihost/ipa/test_hbac.py
index da62ef7c1c..510be0cec0 100644
--- a/src/tests/multihost/ipa/test_hbac.py
+++ b/src/tests/multihost/ipa/test_hbac.py
@@ -18,6 +18,7 @@
 @pytest.mark.usefixtures('default_ipa_users',
  'disable_allow_all_hbac',
  'reset_password')
+@pytest.mark.tier2
 @pytest.mark.hbac
 class Testipahbac(object):
 
___
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#5870][edited] TESTS: Add tier2 marker for ipa tests

2021-11-11 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5870
Author: sgoveas
 Title: #5870: TESTS: Add tier2 marker for ipa tests
Action: edited

 Changed field: body
Original value:
"""
Some of the ipa tests would be executed as tier1 tests. Added markers
for the ones that were not marked and would run as tier2 tests
"""

 Changed field: title
Original value:
"""
Mark tier2
"""

___
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#5870][edited] Mark tier2

2021-11-11 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5870
Author: sgoveas
 Title: #5870: Mark tier2
Action: edited

___
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#5870][synchronized] Mark tier2

2021-11-11 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5870
Author: sgoveas
 Title: #5870: Mark tier2
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5870/head:pr5870
git checkout pr5870
From 2258710b1206fa6a016c72430477bf26724adaa7 Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Thu, 11 Nov 2021 17:28:16 +0530
Subject: [PATCH] TESTS: Add tier2 marker for ipa tests

Some of the ipa tests would be executed as tier1 tests. Added markers
for the ones that were not marked and would run as tier2 tests
---
 src/tests/multihost/ipa/pytest.ini  | 1 +
 src/tests/multihost/ipa/test_adhbac.py  | 1 +
 src/tests/multihost/ipa/test_adtrust.py | 1 +
 src/tests/multihost/ipa/test_hbac.py| 1 +
 4 files changed, 4 insertions(+)

diff --git a/src/tests/multihost/ipa/pytest.ini b/src/tests/multihost/ipa/pytest.ini
index 6be7556333..d6b535fd3f 100644
--- a/src/tests/multihost/ipa/pytest.ini
+++ b/src/tests/multihost/ipa/pytest.ini
@@ -1,6 +1,7 @@
 [pytest]
 markers =
 tier1: tier1 test cases
+tier1: tier2 test cases
 hbac: Tests hbac in ipa provider environment
 trust: Tests related to IPA AD Trust
 adhbac: Test related to HBAC in IPA-AD Trust environment
diff --git a/src/tests/multihost/ipa/test_adhbac.py b/src/tests/multihost/ipa/test_adhbac.py
index 5d960bf636..971a14729c 100644
--- a/src/tests/multihost/ipa/test_adhbac.py
+++ b/src/tests/multihost/ipa/test_adhbac.py
@@ -16,6 +16,7 @@
 @pytest.mark.usefixtures('disable_allow_all_hbac',
  'create_ad_users',
  'create_ad_groups')
+@pytest.mark.tier2
 @pytest.mark.adhbac
 class TestADTrustHbac(object):
 """ AD Trust HBAC Test cases """
diff --git a/src/tests/multihost/ipa/test_adtrust.py b/src/tests/multihost/ipa/test_adtrust.py
index 64ff28aa3c..3f2fc66c1b 100644
--- a/src/tests/multihost/ipa/test_adtrust.py
+++ b/src/tests/multihost/ipa/test_adtrust.py
@@ -15,6 +15,7 @@
 
 
 @pytest.mark.usefixtures('setup_ipa_client')
+@pytest.mark.tier2
 @pytest.mark.trust
 class TestADTrust(object):
 """ IPA AD Trust tests """
diff --git a/src/tests/multihost/ipa/test_hbac.py b/src/tests/multihost/ipa/test_hbac.py
index da62ef7c1c..510be0cec0 100644
--- a/src/tests/multihost/ipa/test_hbac.py
+++ b/src/tests/multihost/ipa/test_hbac.py
@@ -18,6 +18,7 @@
 @pytest.mark.usefixtures('default_ipa_users',
  'disable_allow_all_hbac',
  'reset_password')
+@pytest.mark.tier2
 @pytest.mark.hbac
 class Testipahbac(object):
 
___
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#5812][synchronized] Tests: Modify ported AD tests marker to tier1_2 and fix a restore error

2021-11-11 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5812
Author: sgoveas
 Title: #5812: Tests: Modify ported AD tests marker to tier1_2 and fix a 
restore error
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5812/head:pr5812
git checkout pr5812
From b76d014633e90b563e2e6ba9a1998fb8b0a5f5b2 Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Wed, 6 Oct 2021 13:20:26 +0530
Subject: [PATCH 1/2] Test: Update marker to tier1_2 for some ad tier1 tests

To reduce test runtime to around 1 hour
---
 src/tests/multihost/ad/pytest.ini |  3 +-
 .../multihost/ad/test_adparameters_ported.py  | 43 +--
 2 files changed, 3 insertions(+), 43 deletions(-)

diff --git a/src/tests/multihost/ad/pytest.ini b/src/tests/multihost/ad/pytest.ini
index bd4f102151..043f6c7f24 100644
--- a/src/tests/multihost/ad/pytest.ini
+++ b/src/tests/multihost/ad/pytest.ini
@@ -8,6 +8,7 @@ markers =
smbsecretrotation: Samba Secret database rotation
sudo: Test cases related to sudo using AD backend.
cifs: Cifs test cases
-   tier1: All tier1 test cases
+   tier1: tier1 test cases with run time of aproximately 60 minutes
+   tier1_2: tier1 test cases split to keep runtime upto 60 minutes
tier2: All tier2 test cases
tier3: All tier3 test cases
diff --git a/src/tests/multihost/ad/test_adparameters_ported.py b/src/tests/multihost/ad/test_adparameters_ported.py
index 21a4739738..b84aa645cb 100644
--- a/src/tests/multihost/ad/test_adparameters_ported.py
+++ b/src/tests/multihost/ad/test_adparameters_ported.py
@@ -180,13 +180,13 @@ def set_ssh_key_ldap(session_multihost, user, pubkey, operation="replace"):
 return cmd.returncode == 0
 
 
+@pytest.mark.tier1_2
 @pytest.mark.adparameters
 @pytest.mark.usefixtures("change_client_hostname")
 class TestADParamsPorted:
 """ BZ Automated Test Cases for AD Parameters ported from bash"""
 
 @staticmethod
-@pytest.mark.tier1
 def test_0001_ad_parameters_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -260,7 +260,6 @@ def test_0001_ad_parameters_domain(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0002_ad_parameters_junk_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -333,7 +332,6 @@ def test_0002_ad_parameters_junk_domain(
 assert "segfault" not in log_msg_str, "Segfault present in the log!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0003_ad_parameters_junk_domain_invalid_keytab(
 multihost, adjoin, create_aduser_group):
 """
@@ -450,7 +448,6 @@ def test_0003_ad_parameters_junk_domain_invalid_keytab(
 assert "Option krb5_realm set to JUNK" in log_str
 
 @staticmethod
-@pytest.mark.tier1
 def test_0004_ad_parameters_valid_domain_shorthost(
 multihost, adjoin, create_aduser_group):
 """
@@ -527,7 +524,6 @@ def test_0004_ad_parameters_valid_domain_shorthost(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0005_ad_parameters_blank_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -607,7 +603,6 @@ def test_0005_ad_parameters_blank_domain(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0006_ad_parameters_homedir_override_nss(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -657,7 +652,6 @@ def test_0006_ad_parameters_homedir_override_nss(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 def test_0007_ad_parameters_homedir_override_domain(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -706,7 +700,6 @@ def test_0007_ad_parameters_homedir_override_domain(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 def test_0008_ad_parameters_homedir_override_both(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -756,7 +749,6 @@ def test_0008_ad_parameters_homedir_override_both(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 @pytest.mark.skip(reason="The ldap sasl authid just does not work.")
 def test_0009_ad_parameters_ldap_sasl_full(
 multihost, adjoin, create_aduser_group, fetch_ca_cert):
@@ -859,7 +851,6 @@ def test_0009_ad_parameters_ldap_sasl_full(
 assert su_result, f"Su for user {aduser} failed!"
 
 @staticmethod
-@pytest.mark.tier1
 @pytest.mark.skip(reason="The ldap sasl authid just does not work.")
 def test_0010_ad

[SSSD] [sssd PR#5870][opened] Mark tier2

2021-11-11 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5870
Author: sgoveas
 Title: #5870: Mark tier2
Action: opened

PR body:
"""
None
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5870/head:pr5870
git checkout pr5870
From fac012048e57331587f91ba4bb0a1a1e4b3d202f Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Wed, 6 Oct 2021 13:20:26 +0530
Subject: [PATCH 1/2] TEST: Update marker to tier1_2 for some ad tier1 tests

To reduce test runtime to around 1 hour
---
 src/tests/multihost/ad/pytest.ini |  3 +-
 .../multihost/ad/test_adparameters_ported.py  | 43 +--
 2 files changed, 3 insertions(+), 43 deletions(-)

diff --git a/src/tests/multihost/ad/pytest.ini b/src/tests/multihost/ad/pytest.ini
index bd4f102151..043f6c7f24 100644
--- a/src/tests/multihost/ad/pytest.ini
+++ b/src/tests/multihost/ad/pytest.ini
@@ -8,6 +8,7 @@ markers =
smbsecretrotation: Samba Secret database rotation
sudo: Test cases related to sudo using AD backend.
cifs: Cifs test cases
-   tier1: All tier1 test cases
+   tier1: tier1 test cases with run time of aproximately 60 minutes
+   tier1_2: tier1 test cases split to keep runtime upto 60 minutes
tier2: All tier2 test cases
tier3: All tier3 test cases
diff --git a/src/tests/multihost/ad/test_adparameters_ported.py b/src/tests/multihost/ad/test_adparameters_ported.py
index 21a4739738..b84aa645cb 100644
--- a/src/tests/multihost/ad/test_adparameters_ported.py
+++ b/src/tests/multihost/ad/test_adparameters_ported.py
@@ -180,13 +180,13 @@ def set_ssh_key_ldap(session_multihost, user, pubkey, operation="replace"):
 return cmd.returncode == 0
 
 
+@pytest.mark.tier1_2
 @pytest.mark.adparameters
 @pytest.mark.usefixtures("change_client_hostname")
 class TestADParamsPorted:
 """ BZ Automated Test Cases for AD Parameters ported from bash"""
 
 @staticmethod
-@pytest.mark.tier1
 def test_0001_ad_parameters_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -260,7 +260,6 @@ def test_0001_ad_parameters_domain(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0002_ad_parameters_junk_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -333,7 +332,6 @@ def test_0002_ad_parameters_junk_domain(
 assert "segfault" not in log_msg_str, "Segfault present in the log!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0003_ad_parameters_junk_domain_invalid_keytab(
 multihost, adjoin, create_aduser_group):
 """
@@ -450,7 +448,6 @@ def test_0003_ad_parameters_junk_domain_invalid_keytab(
 assert "Option krb5_realm set to JUNK" in log_str
 
 @staticmethod
-@pytest.mark.tier1
 def test_0004_ad_parameters_valid_domain_shorthost(
 multihost, adjoin, create_aduser_group):
 """
@@ -527,7 +524,6 @@ def test_0004_ad_parameters_valid_domain_shorthost(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0005_ad_parameters_blank_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -607,7 +603,6 @@ def test_0005_ad_parameters_blank_domain(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0006_ad_parameters_homedir_override_nss(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -657,7 +652,6 @@ def test_0006_ad_parameters_homedir_override_nss(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 def test_0007_ad_parameters_homedir_override_domain(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -706,7 +700,6 @@ def test_0007_ad_parameters_homedir_override_domain(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 def test_0008_ad_parameters_homedir_override_both(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -756,7 +749,6 @@ def test_0008_ad_parameters_homedir_override_both(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 @pytest.mark.skip(reason="The ldap sasl authid just does not work.")
 def test_0009_ad_parameters_ldap_sasl_full(
 multihost, adjoin, create_aduser_group, fetch_ca_cert):
@@ -859,7 +851,6 @@ def test_0009_ad_parameters_ldap_sasl_full(
 assert su_result, f"Su for user {aduser} failed!"
 
 @staticmethod
-@pytest.mark.tier1
 @pytest.mark.skip(reason="The ldap sasl authid just does not work.")
 def test_0010_ad_parameters_ldap_sasl_short(
   

[SSSD] [sssd PR#5836][+Accepted] Tests: Regression 8.5 - sssd-ipa

2021-10-21 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5836
Title: #5836: Tests: Regression 8.5 - sssd-ipa

Label: +Accepted
___
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#5814][-Changes requested] Tests: sss_override does not take precedence over override_homedir di…

2021-10-18 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5814
Title: #5814: Tests: sss_override does not take precedence over 
override_homedir di…

Label: -Changes requested
___
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#5814][+Accepted] Tests: sss_override does not take precedence over override_homedir di…

2021-10-18 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5814
Title: #5814: Tests: sss_override does not take precedence over 
override_homedir di…

Label: +Accepted
___
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#5814][comment] Tests: sss_override does not take precedence over override_homedir di…

2021-10-13 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5814
Title: #5814: Tests: sss_override does not take precedence over 
override_homedir di…

sgoveas commented:
"""
Please move test to a separate file like test_local_overrides.py with marker 
'localoverrides'. Add requirement "local_overrides"
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5814#issuecomment-941970297
___
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#5814][+Changes requested] Tests: sss_override does not take precedence over override_homedir di…

2021-10-13 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5814
Title: #5814: Tests: sss_override does not take precedence over 
override_homedir di…

Label: +Changes requested
___
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#5755][+Accepted] Tests: support subid ranges managed by FreeIPA

2021-10-11 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5755
Title: #5755: Tests: support subid ranges managed by FreeIPA

Label: +Accepted
___
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#5755][-Waiting for review] Tests: support subid ranges managed by FreeIPA

2021-10-11 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5755
Title: #5755: Tests: support subid ranges managed by FreeIPA

Label: -Waiting for review
___
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#5812][+Tests] Tests: Modify ported AD tests marker to tier1_2 and fix a restore error

2021-10-06 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5812
Title: #5812: Tests: Modify ported AD tests marker to tier1_2 and fix a restore 
error

Label: +Tests
___
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#5812][+Waiting for review] Tests: Modify ported AD tests marker to tier1_2 and fix a restore error

2021-10-06 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5812
Title: #5812: Tests: Modify ported AD tests marker to tier1_2 and fix a restore 
error

Label: +Waiting for review
___
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#5812][opened] Tests: Modify ported AD tests marker to tier1_2 and fix a restore error

2021-10-06 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5812
Author: sgoveas
 Title: #5812: Tests: Modify ported AD tests marker to tier1_2 and fix a 
restore error
Action: opened

PR body:
"""
Use tier1_2 marker for newly ported ad parameters tests to reduce runtime to 1 
hour and fix and ldap.conf backup restore
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5812/head:pr5812
git checkout pr5812
From bfbea33ccc8bc373662e13660bc732361dc69b3a Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Wed, 6 Oct 2021 13:20:26 +0530
Subject: [PATCH 1/2] Test: Update marker to tier1_2 for some ad tier1 tests

To reduce test runtime to around 1 hour
---
 src/tests/multihost/ad/pytest.ini |  3 +-
 .../multihost/ad/test_adparameters_ported.py  | 43 +--
 2 files changed, 3 insertions(+), 43 deletions(-)

diff --git a/src/tests/multihost/ad/pytest.ini b/src/tests/multihost/ad/pytest.ini
index bd4f102151..043f6c7f24 100644
--- a/src/tests/multihost/ad/pytest.ini
+++ b/src/tests/multihost/ad/pytest.ini
@@ -8,6 +8,7 @@ markers =
smbsecretrotation: Samba Secret database rotation
sudo: Test cases related to sudo using AD backend.
cifs: Cifs test cases
-   tier1: All tier1 test cases
+   tier1: tier1 test cases with run time of aproximately 60 minutes
+   tier1_2: tier1 test cases split to keep runtime upto 60 minutes
tier2: All tier2 test cases
tier3: All tier3 test cases
diff --git a/src/tests/multihost/ad/test_adparameters_ported.py b/src/tests/multihost/ad/test_adparameters_ported.py
index 21a4739738..b84aa645cb 100644
--- a/src/tests/multihost/ad/test_adparameters_ported.py
+++ b/src/tests/multihost/ad/test_adparameters_ported.py
@@ -180,13 +180,13 @@ def set_ssh_key_ldap(session_multihost, user, pubkey, operation="replace"):
 return cmd.returncode == 0
 
 
+@pytest.mark.tier1_2
 @pytest.mark.adparameters
 @pytest.mark.usefixtures("change_client_hostname")
 class TestADParamsPorted:
 """ BZ Automated Test Cases for AD Parameters ported from bash"""
 
 @staticmethod
-@pytest.mark.tier1
 def test_0001_ad_parameters_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -260,7 +260,6 @@ def test_0001_ad_parameters_domain(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0002_ad_parameters_junk_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -333,7 +332,6 @@ def test_0002_ad_parameters_junk_domain(
 assert "segfault" not in log_msg_str, "Segfault present in the log!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0003_ad_parameters_junk_domain_invalid_keytab(
 multihost, adjoin, create_aduser_group):
 """
@@ -450,7 +448,6 @@ def test_0003_ad_parameters_junk_domain_invalid_keytab(
 assert "Option krb5_realm set to JUNK" in log_str
 
 @staticmethod
-@pytest.mark.tier1
 def test_0004_ad_parameters_valid_domain_shorthost(
 multihost, adjoin, create_aduser_group):
 """
@@ -527,7 +524,6 @@ def test_0004_ad_parameters_valid_domain_shorthost(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0005_ad_parameters_blank_domain(
 multihost, adjoin, create_aduser_group):
 """
@@ -607,7 +603,6 @@ def test_0005_ad_parameters_blank_domain(
 assert su_result, "The su command failed!"
 
 @staticmethod
-@pytest.mark.tier1
 def test_0006_ad_parameters_homedir_override_nss(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -657,7 +652,6 @@ def test_0006_ad_parameters_homedir_override_nss(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 def test_0007_ad_parameters_homedir_override_domain(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -706,7 +700,6 @@ def test_0007_ad_parameters_homedir_override_domain(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 def test_0008_ad_parameters_homedir_override_both(
 multihost, adjoin, create_plain_aduser_group):
 """
@@ -756,7 +749,6 @@ def test_0008_ad_parameters_homedir_override_both(
usr_cmd.stdout_text
 
 @staticmethod
-@pytest.mark.tier1
 @pytest.mark.skip(reason="The ldap sasl authid just does not work.")
 def test_0009_ad_parameters_ldap_sasl_full(
 multihost, adjoin, create_aduser_group, fetch_ca_cert):
@@ -859,7 +851,6 @@ def test_0009_ad_parameters_ldap_sasl_full(
 assert su_result, f"Su for user {aduser} failed!"
 

[SSSD] [sssd PR#5804][+Accepted] Tests: Fix Failure of sssctl_local test

2021-10-04 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5804
Title: #5804: Tests: Fix Failure of sssctl_local test

Label: +Accepted
___
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#5792][-Waiting for review] Tests: improve sssd refresh timers for sudo queries

2021-10-01 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5792
Title: #5792: Tests: improve sssd refresh timers for sudo queries

Label: -Waiting for review
___
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#5792][+Accepted] Tests: improve sssd refresh timers for sudo queries

2021-10-01 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5792
Title: #5792: Tests: improve sssd refresh timers for sudo queries

Label: +Accepted
___
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#5799][+Tests] Tests: Fix RHEL8.5 failures for IDM-CI

2021-09-28 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5799
Title: #5799: Tests: Fix RHEL8.5 failures for IDM-CI

Label: +Tests
___
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#5799][+Accepted] Tests: Fix RHEL8.5 failures for IDM-CI

2021-09-28 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5799
Title: #5799: Tests: Fix RHEL8.5 failures for IDM-CI

Label: +Accepted
___
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#5797][closed] Tests: Install iptabls as its not installed by default

2021-09-28 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5797
Author: aborah-sudo
 Title: #5797: Tests:  Install iptabls as its not installed by default
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5797/head:pr5797
git checkout pr5797
___
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#5797][comment] Tests: Install iptabls as its not installed by default

2021-09-28 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5797
Title: #5797: Tests:  Install iptabls as its not installed by default

sgoveas commented:
"""
#5798 covers this requirement. Closing PR.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5797#issuecomment-929089440
___
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#5798][synchronized] Tests: Add firewalld package install on clients

2021-09-28 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5798
Author: sgoveas
 Title: #5798: Tests: Add firewalld package install on clients
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5798/head:pr5798
git checkout pr5798
From 7d75191475797b171f7f815437e6b372f1b98922 Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Tue, 28 Sep 2021 16:02:05 +0530
Subject: [PATCH] Tests: Add firewalld package install on clients

---
 src/tests/multihost/sssd/testlib/common/utils.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tests/multihost/sssd/testlib/common/utils.py b/src/tests/multihost/sssd/testlib/common/utils.py
index 0ca16bd97e..3b12a8b429 100644
--- a/src/tests/multihost/sssd/testlib/common/utils.py
+++ b/src/tests/multihost/sssd/testlib/common/utils.py
@@ -62,7 +62,7 @@ def client_install_pkgs(self):
 pkgs = 'adcli realmd samba samba-common-tools krb5-workstation '\
'oddjob oddjob-mkhomedir ldb-tools samba-winbind '\
'samba-winbind-clients autofs nfs-utils authconfig '\
-   'authselect cifs-utils openldap-clients '\
+   'authselect cifs-utils openldap-clients firewalld '\
'tcpdump wireshark-cli expect rsyslog gcc gcc-c++ pam-devel '\
'tdb-tools libkcapi-hmaccalc'
 sssd_pkgs = 'sssd sssd-tools sssd-proxy sssd-winbind-idmap '\
___
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#5798][edited] Tests: Add firewalld package install on clients

2021-09-28 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5798
Author: sgoveas
 Title: #5798: Tests: Add firewalld package install on clients
Action: edited

___
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#5798][+Waiting for review] Tests: Add firewalld package install on clients

2021-09-28 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5798
Title: #5798: Tests: Add firewalld package install on clients

Label: +Waiting for review
___
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#5798][+Tests] Tests: Add firewalld package install on clients

2021-09-28 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5798
Title: #5798: Tests: Add firewalld package install on clients

Label: +Tests
___
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#5798][edited] Tests: Add firewalld package install on clients

2021-09-28 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5798
Author: sgoveas
 Title: #5798: Tests: Add firewalld package install on clients
Action: edited

 Changed field: title
Original value:
"""
Add firewalld package install on clients
"""

___
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#5798][opened] Add firewalld package install on clients

2021-09-28 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5798
Author: sgoveas
 Title: #5798: Add firewalld package install on clients
Action: opened

PR body:
"""
None
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5798/head:pr5798
git checkout pr5798
From bbea27a64020c66e60a8628507e53a9886a19b3c Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Tue, 28 Sep 2021 16:02:05 +0530
Subject: [PATCH] Add firewalld package install on clients

---
 src/tests/multihost/sssd/testlib/common/utils.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tests/multihost/sssd/testlib/common/utils.py b/src/tests/multihost/sssd/testlib/common/utils.py
index 0ca16bd97e..3b12a8b429 100644
--- a/src/tests/multihost/sssd/testlib/common/utils.py
+++ b/src/tests/multihost/sssd/testlib/common/utils.py
@@ -62,7 +62,7 @@ def client_install_pkgs(self):
 pkgs = 'adcli realmd samba samba-common-tools krb5-workstation '\
'oddjob oddjob-mkhomedir ldb-tools samba-winbind '\
'samba-winbind-clients autofs nfs-utils authconfig '\
-   'authselect cifs-utils openldap-clients '\
+   'authselect cifs-utils openldap-clients firewalld '\
'tcpdump wireshark-cli expect rsyslog gcc gcc-c++ pam-devel '\
'tdb-tools libkcapi-hmaccalc'
 sssd_pkgs = 'sssd sssd-tools sssd-proxy sssd-winbind-idmap '\
___
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#5796][+Accepted] Tests: sss_cache prints spurious error messages

2021-09-27 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5796
Title: #5796: Tests: sss_cache prints spurious error messages

Label: +Accepted
___
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#5796][+Tests] Tests: sss_cache prints spurious error messages

2021-09-27 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5796
Title: #5796: Tests: sss_cache prints spurious error messages

Label: +Tests
___
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#5730][+Accepted] Tests: SSSD logs improvements: clarify which config option applies to…

2021-09-19 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5730
Title: #5730: Tests: SSSD logs improvements: clarify which config option 
applies to…

Label: +Accepted
___
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#5764][+Accepted] Tests: Suppress log message

2021-09-15 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5764
Title: #5764: Tests: Suppress log message

Label: +Accepted
___
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#5742][+Accepted] Tests: RHEL9.0 Regression - alltests-tier1_2

2021-09-14 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5742
Title: #5742: Tests: RHEL9.0 Regression - alltests-tier1_2

Label: +Accepted
___
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#5773][+Waiting for review] TEST: usermod -d needs absolute path

2021-09-06 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5773
Title: #5773: TEST: usermod -d needs absolute path

Label: +Waiting for review
___
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#5773][+Tests] TEST: usermod -d needs absolute path

2021-09-06 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5773
Title: #5773: TEST: usermod -d needs absolute path

Label: +Tests
___
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#5773][opened] TEST: usermod -d needs absolute path

2021-09-06 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5773
Author: sgoveas
 Title: #5773: TEST: usermod -d needs absolute path
Action: opened

PR body:
"""
usermod -d failed when it was given an empty string to update. This was
noticed in #5754. Updating test to check for modified home dir
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5773/head:pr5773
git checkout pr5773
From 6f7a52671d01bd1f5259b9d97ecfa548e96d79d5 Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Mon, 6 Sep 2021 11:44:04 +0530
Subject: [PATCH] TEST: usermod -d needs absolute path

usermod -d failed when it was given an empty string to update. This was
noticed in #5754. Updating test to check for modified home dir
---
 src/tests/multihost/basic/conftest.py   | 2 +-
 src/tests/multihost/basic/test_files.py | 8 
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/tests/multihost/basic/conftest.py b/src/tests/multihost/basic/conftest.py
index 39c3ebdae9..3d0debb3aa 100644
--- a/src/tests/multihost/basic/conftest.py
+++ b/src/tests/multihost/basic/conftest.py
@@ -303,7 +303,7 @@ def files_domain_users_class(request, session_multihost):
 for user in no_home_users:
 useradd_cmd = "useradd --no-create-home %s" % (user)
 session_multihost.master[0].run_command(useradd_cmd)
-usermod_cmd = "usermod -d '' %s" % (user)
+usermod_cmd = "usermod -d /tmp %s" % (user)
 session_multihost.master[0].run_command(usermod_cmd)
 
 def teardown_files_domain_users():
diff --git a/src/tests/multihost/basic/test_files.py b/src/tests/multihost/basic/test_files.py
index 18c9bb9379..10b8b9f307 100644
--- a/src/tests/multihost/basic/test_files.py
+++ b/src/tests/multihost/basic/test_files.py
@@ -53,11 +53,11 @@ def test_files_enumeration(self, multihost):
 cmd = multihost.master[0].run_command('getent passwd -s sss')
 assert len(cmd.stdout_text) == 0
 
-def test_no_homedir_no_slash(self, multihost):
+def test_updated_homedir(self, multihost):
 """
-:title: files: Test that sssd returns an empty value with no homedir
-:id: 58010941-f1d6-453f-86f7-ade11dc81bb5
+:title: files: Test that homedir is updated
+:id: a9a0a911-1818-40d1-b897-0397ef107fd4
 """
 exit_status, output = get_sss_user(multihost, 'no_home_user')
 assert exit_status == 0
-assert ":/:" not in output
+assert ":/tmp:" in output
___
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#5740][-Changes requested] Test: Fix RHEL9.0 Regression - alltests-tier1

2021-08-25 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5740
Title: #5740: Test: Fix RHEL9.0 Regression - alltests-tier1

Label: -Changes requested
___
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#5740][+Accepted] Test: Fix RHEL9.0 Regression - alltests-tier1

2021-08-25 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5740
Title: #5740: Test: Fix RHEL9.0 Regression - alltests-tier1

Label: +Accepted
___
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#5740][+Accepted] Test: Fix RHEL9.0 Regression - alltests-tier1

2021-08-24 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5740
Title: #5740: Test: Fix RHEL9.0 Regression - alltests-tier1

Label: +Accepted
___
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#5728][opened] TEST: Add id and fix indentation in docstrings

2021-07-28 Thread sgoveas
   URL: https://github.com/SSSD/sssd/pull/5728
Author: sgoveas
 Title: #5728: TEST: Add id and fix indentation in docstrings
Action: opened

PR body:
"""

"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5728/head:pr5728
git checkout pr5728
From c5820e4664bae00e61aaf1f88fc3424b31986a70 Mon Sep 17 00:00:00 2001
From: Steeve Goveas 
Date: Thu, 29 Jul 2021 11:14:47 +0530
Subject: [PATCH] TEST: Add id and fix indentation in docstrings

---
 src/tests/multihost/alltests/test_krb5.py | 40 +++
 src/tests/multihost/ipa/test_misc.py  |  6 ++--
 2 files changed, 24 insertions(+), 22 deletions(-)

diff --git a/src/tests/multihost/alltests/test_krb5.py b/src/tests/multihost/alltests/test_krb5.py
index e94b7be763..849437707f 100644
--- a/src/tests/multihost/alltests/test_krb5.py
+++ b/src/tests/multihost/alltests/test_krb5.py
@@ -70,26 +70,26 @@ def test_0001_krb5_not_working_based_on_k5login(self,
 @pytest.mark.tier1_2
 def test_0002_generating_lot_of(self, multihost, backupsssdconf):
 """
-:title: SSSD is generating lot of LDAP
-queries in a very large environment
-:bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1772513
-:id: 74a60320-e48b-11eb-ba19-845cf3eff344
-:steps:
-1. Start SSSD with any configuration
-2. Call 'getent passwd username@domain'
-3. Check the entry is present in data and timestamp cache
-4. Now stop SSSD and remove the timestamp cache
-5. Start SSSD and call 'sss_cache -E'
-6. Call 'getent passwd username@domain'
-7. Do the ldbsearch checks again
-:expectedresults:
-1. Should succeed
-2. Should succeed
-3. Should succeed
-4. Should succeed
-5. Should succeed
-6. Should succeed
-7. Should succeed
+:title: SSSD is generating lot of LDAP
+ queries in a very large environment
+:bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1772513
+:id: 74a60320-e48b-11eb-ba19-845cf3eff344
+:steps:
+  1. Start SSSD with any configuration
+  2. Call 'getent passwd username@domain'
+  3. Check the entry is present in data and timestamp cache
+  4. Now stop SSSD and remove the timestamp cache
+  5. Start SSSD and call 'sss_cache -E'
+  6. Call 'getent passwd username@domain'
+  7. Do the ldbsearch checks again
+:expectedresults:
+  1. Should succeed
+  2. Should succeed
+  3. Should succeed
+  4. Should succeed
+  5. Should succeed
+  6. Should succeed
+  7. Should succeed
 """
 multihost.client[0].service_sssd('restart')
 cmd = multihost.client[0].run_command('getent passwd '
diff --git a/src/tests/multihost/ipa/test_misc.py b/src/tests/multihost/ipa/test_misc.py
index 05341d766b..2e3c4254cb 100644
--- a/src/tests/multihost/ipa/test_misc.py
+++ b/src/tests/multihost/ipa/test_misc.py
@@ -138,8 +138,10 @@ def test_filter_groups(self, multihost, default_ipa_groups,
 def test_asymmetric_auth_for_nsupdate(self, multihost,
   create_reverse_zone):
 """
-@Title: Support asymmetric auth for nsupdate
-@Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1884301
+:title: Support asymmetric auth for nsupdate
+:id: 2bc5c4c7-7296-434b-8f38-2b7297b32b9b
+:requirement: dyndns
+:bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1884301
 """
 client = sssdTools(multihost.client[0])
 client_hostname = multihost.client[0].sys_hostname
___
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#5719][+Tests] Tests: fix sss_cache to also reset cached timestamp

2021-07-28 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5719
Title: #5719: Tests: fix sss_cache to also reset cached timestamp

Label: +Tests
___
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#5719][+Accepted] Tests: fix sss_cache to also reset cached timestamp

2021-07-28 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5719
Title: #5719: Tests: fix sss_cache to also reset cached timestamp

Label: +Accepted
___
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#5725][+Accepted] Tests: Add support to verify authentication indicators in pam_sss_gss

2021-07-26 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5725
Title: #5725: Tests: Add support to verify authentication indicators in 
pam_sss_gss

Label: +Accepted
___
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#5630][-Waiting for review] Tests: Add support to verify authentication indicators in pam_sss_gss

2021-07-23 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5630
Title: #5630: Tests: Add support to verify authentication indicators in 
pam_sss_gss

Label: -Waiting for review
___
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#5630][+Accepted] Tests: Add support to verify authentication indicators in pam_sss_gss

2021-07-23 Thread sgoveas
  URL: https://github.com/SSSD/sssd/pull/5630
Title: #5630: Tests: Add support to verify authentication indicators in 
pam_sss_gss

Label: +Accepted
___
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


  1   2   >