[SSSD] [sssd PR#5303][comment] tests: run TIER-0 multihost tests in PRCI

2020-09-18 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5303 Title: #5303: tests: run TIER-0 multihost tests in PRCI mrniranjan commented: """ I tested with the updated spec file in this PR and all the kcm tests passed. """ See the full comment at https://github.com/SSSD/sssd/pu

[SSSD] [sssd PR#5303][comment] tests: run TIER-0 multihost tests in PRCI

2020-09-17 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5303 Title: #5303: tests: run TIER-0 multihost tests in PRCI mrniranjan commented: """ @pbrezina the kcm tests uses a fixture which enables kcm before running the tests. """ See the full comment at https://github.com/SS

[SSSD] [sssd PR#5303][comment] tests: run TIER-0 multihost tests in PRCI

2020-09-17 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5303 Title: #5303: tests: run TIER-0 multihost tests in PRCI mrniranjan commented: """ Since current sssd-kcm package in fedora-32 already has /etc/krb5.conf.d/kcm_default_ccache , would it be possible to have it same even in th

[SSSD] [sssd PR#5303][comment] tests: run TIER-0 multihost tests in PRCI

2020-09-17 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5303 Title: #5303: tests: run TIER-0 multihost tests in PRCI mrniranjan commented: """ The reason some kcm tests are failing because the test automation assumes that the default cache is kcm as it assumes that the there is /

[SSSD] [sssd PR#5303][comment] tests: run TIER-0 multihost tests in PRCI

2020-09-16 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5303 Title: #5303: tests: run TIER-0 multihost tests in PRCI mrniranjan commented: """ @pbrezina i am looking in to this issue. will update as soon as possible. """ See the full comment at https://github.com/SSSD/sssd/pu

[SSSD] [sssd PR#5303][comment] tests: run TIER-0 multihost tests in PRCI

2020-09-07 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5303 Title: #5303: tests: run TIER-0 multihost tests in PRCI mrniranjan commented: """ @pbrezina is it possible to get rpms so that i can install and run failing tests individually to debug the issue. """ See the full

[SSSD] [sssd PR#5305][opened] pytest/testlib: Execute pk12util command to create ca.p12

2020-09-03 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5305 Author: mrniranjan Title: #5305: pytest/testlib: Execute pk12util command to create ca.p12 Action: opened PR body: """ execute the pk12util command to create ca.p12 file Minor fix: return variable should be out of the for loop scope

[SSSD] [sssd PR#5253][comment] libdirsrv should be modified to be compatible with new DS

2020-08-31 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5253 Title: #5253: libdirsrv should be modified to be compatible with new DS mrniranjan commented: """ @aborah-sudo can you move the -t to the next line ie. ` set_trust_cmd = 'certutil -M -d %s -n %s ' \ ' -t "

[SSSD] [sssd PR#5253][comment] libdirsrv should be modified to be compatible with new DS

2020-08-31 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5253 Title: #5253: libdirsrv should be modified to be compatible with new DS mrniranjan commented: """ @aborah-sudo Can you modify the line in the patch with below changes: ` set_trust_cmd = 'certut

[SSSD] [sssd PR#5253][comment] libdirsrv should be modified to be compatible with new DS

2020-08-31 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/5253 Title: #5253: libdirsrv should be modified to be compatible with new DS mrniranjan commented: """ @aborah-sudo Can you modify the line in the patch with below changes: ` set_trust_cmd = 'certut

[SSSD] [sssd PR#895][synchronized] pytest/multihost: Update instructions to execute multihost tests

2019-10-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Author: mrniranjan Title: #895: pytest/multihost: Update instructions to execute multihost tests Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/895/head:pr895 git checkout

[SSSD] [sssd PR#895][synchronized] pytest/multihost: Update instructions to execute multihost tests

2019-10-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Author: mrniranjan Title: #895: pytest/multihost: Update instructions to execute multihost tests Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/895/head:pr895 git checkout

[SSSD] [sssd PR#895][synchronized] pytest/multihost: Update instructions to execute multihost tests

2019-10-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Author: mrniranjan Title: #895: pytest/multihost: Update instructions to execute multihost tests Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/895/head:pr895 git checkout

[SSSD] [sssd PR#895][synchronized] pytest/multihost: Update instructions to execute multihost tests

2019-10-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Author: mrniranjan Title: #895: pytest/multihost: Update instructions to execute multihost tests Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/895/head:pr895 git checkout

[SSSD] [sssd PR#895][comment] pytest/multihost: Update instructions to execute multihost tests

2019-10-03 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Title: #895: pytest/multihost: Update instructions to execute multihost tests mrniranjan commented: """ @pbrezina please have a look at the updated instructions. """ See the full comment at https://github.com/S

[SSSD] [sssd PR#895][synchronized] pytest/multihost: Update instructions to execute multihost tests

2019-10-03 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Author: mrniranjan Title: #895: pytest/multihost: Update instructions to execute multihost tests Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/895/head:pr895 git checkout

[SSSD] [sssd PR#895][edited] pytest/multihost: Update instructions to execute multihost tests

2019-10-01 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Author: mrniranjan Title: #895: pytest/multihost: Update instructions to execute multihost tests Action: edited Changed field: body Original value: """ 1. Update Instructions to execute multihost tests 2. Add python-ldap as depe

[SSSD] [sssd PR#895][synchronized] pytest/multihost: Update instructions to execute multihost tests

2019-10-01 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Author: mrniranjan Title: #895: pytest/multihost: Update instructions to execute multihost tests Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/895/head:pr895 git checkout

[SSSD] [sssd PR#895][comment] pytest/multihost: Update instructions to execute multihost tests

2019-10-01 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Title: #895: pytest/multihost: Update instructions to execute multihost tests mrniranjan commented: """ @pbrezina can i updated the patch to modify Makefile.am to use README.md ? """ See the full comment at https:/

[SSSD] [sssd PR#895][comment] pytest/multihost: Update instructions to execute multihost tests

2019-10-01 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Title: #895: pytest/multihost: Update instructions to execute multihost tests mrniranjan commented: """ @pbrezina can i use README.md instead of README, the instructions are in markdown syntax. """ See the full comme

[SSSD] [sssd PR#895][comment] pytest/multihost: Update instructions to execute multihost tests

2019-09-30 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Title: #895: pytest/multihost: Update instructions to execute multihost tests mrniranjan commented: """ I am unable to see the CI results to find out why the tests failed. Any idea how to check what failed . """

[SSSD] [sssd PR#895][opened] pytest/multihost: Update instructions to execute multihost tests

2019-09-30 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/895 Author: mrniranjan Title: #895: pytest/multihost: Update instructions to execute multihost tests Action: opened PR body: """ 1. Update Instructions to execute multihost tests 2. Add python-ldap as dependency for sssd multihost test li

[SSSD] [sssd PR#873][comment] pytest: Use idm:DL1 module to install 389-ds

2019-09-12 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/873 Title: #873: pytest: Use idm:DL1 module to install 389-ds mrniranjan commented: """ @pbrezina RHEL8 yum is symlinked to /usr/bin/dnf-3 . I have been told to use yum only on RHEL8 """ See the full comment at https:/

[SSSD] [sssd PR#873][comment] pytest: Use idm:DL1 module to install 389-ds

2019-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/873 Title: #873: pytest: Use idm:DL1 module to install 389-ds mrniranjan commented: """ @pbrezina updated the code please check. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#873][synchronized] pytest: Use idm:DL1 module to install 389-ds

2019-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/873 Author: mrniranjan Title: #873: pytest: Use idm:DL1 module to install 389-ds Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/873/head:pr873 git checkout pr873 From

[SSSD] [sssd PR#873][comment] pytest: Use idm:DL1 module to install 389-ds

2019-08-22 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/873 Title: #873: pytest: Use idm:DL1 module to install 389-ds mrniranjan commented: """ @jhrozek @mzidek-rh please review. This change was already done in downstream and passed in osci pipeline. """ See the full comment at

[SSSD] [sssd PR#749][comment] TEST: Add a multihost test for not returning / for an empty home dir

2019-06-18 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/749 Title: #749: TEST: Add a multihost test for not returning / for an empty home dir mrniranjan commented: """ @jhrozek ACK """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#749][+Accepted] TEST: Add a multihost test for not returning / for an empty home dir

2019-06-18 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/749 Title: #749: TEST: Add a multihost test for not returning / for an empty home dir Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-l

[SSSD] [sssd PR#782][comment] TESTS: Test changing LDAP password with extended operation and modification

2019-06-18 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/782 Title: #782: TESTS: Test changing LDAP password with extended operation and modification mrniranjan commented: """ @jhrozek Looks good to me. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#749][comment] TEST: Add a multihost test for not returning / for an empty home dir

2019-06-18 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/749 Title: #749: TEST: Add a multihost test for not returning / for an empty home dir mrniranjan commented: """ @jhrozek Looks good to me. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#766][comment] TESTS: Test kvno correctly displays vesion numbers of principals

2019-06-17 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals mrniranjan commented: """ @jhrozek @sumit-bose can you review the latest changes. """ See the full comment at https://github.com/S

[SSSD] [sssd PR#766][comment] TESTS: Test kvno correctly displays vesion numbers of principals

2019-06-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals mrniranjan commented: """ Ran the kcm tests on (unpatched version) sssd-1.16.0-4.el8+5.s390x . ``` test_kcm.py::TestSanityKCM::test_kinit_kcm test_kcm.py

[SSSD] [sssd PR#766][comment] TESTS: Test kvno correctly displays vesion numbers of principals

2019-06-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals mrniranjan commented: """ Ran the kcm tests on (unpatched version) sssd-1.16.0-4.el8+5.s390x . ``` test_kcm.py::TestSanityKCM::test_kinit_kcm test_kcm.py

[SSSD] [sssd PR#766][comment] TESTS: Test kvno correctly displays vesion numbers of principals

2019-06-02 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals mrniranjan commented: """ @sumit-bose the latest patch addresses the version number being checked against integer instead of hardcoded number . "&quo

[SSSD] [sssd PR#766][synchronized] TESTS: Test kvno correctly displays vesion numbers of principals

2019-06-02 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Author: mrniranjan Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/766/head:pr766 git checkout

[SSSD] [sssd PR#766][comment] TESTS: Test kvno correctly displays vesion numbers of principals

2019-04-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals mrniranjan commented: """ @jhrozek this patch is to address bz-[1585313 ](https://bugzilla.redhat.com/show_bug.cgi?id=1585313) """

[SSSD] [sssd PR#766][comment] TESTS: Test kvno correctly displays vesion numbers of principals

2019-04-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals mrniranjan commented: """ @jhrozek i could run the tests against a build which doesn't contain the fix. I have only checked this against latest build

[SSSD] [sssd PR#756][comment] TESTS: Add test case title with @title in docstrings

2019-04-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Title: #756: TESTS: Add test case title with @title in docstrings mrniranjan commented: """ @jhrozek Updated the PR for adding title to ifp test case also. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#756][synchronized] TESTS: Add test case title with @title in docstrings

2019-04-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Author: mrniranjan Title: #756: TESTS: Add test case title with @title in docstrings Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/756/head:pr756 git checkout pr756 From

[SSSD] [sssd PR#756][comment] TESTS: Add test case title with @title in docstrings

2019-04-02 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Title: #756: TESTS: Add test case title with @title in docstrings mrniranjan commented: """ @jhrozek can you have a look at this PR, i have made changes to docstrings as requested. """ See the full comment at https:/

[SSSD] [sssd PR#756][comment] TESTS: Add test case title with @title in docstrings

2019-03-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Title: #756: TESTS: Add test case title with @title in docstrings mrniranjan commented: """ I see failures, but from the details its not clear if it's pep8 related failure. In the ci-pep8.log i see only the below message ``` Sta

[SSSD] [sssd PR#756][comment] TESTS: Add test case title with @title in docstrings

2019-03-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Title: #756: TESTS: Add test case title with @title in docstrings mrniranjan commented: """ I see failures, but from the details its not clear if it's pep8 related failure. In the ci-pep8.log i see only the below error. ``` Start:

[SSSD] [sssd PR#756][comment] TESTS: Add test case title with @title in docstrings

2019-03-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Title: #756: TESTS: Add test case title with @title in docstrings mrniranjan commented: """ @jhrozek I have made changes to test case names as we discussed internally. """ See the full comment at https://github.com/S

[SSSD] [sssd PR#756][synchronized] TESTS: Add test case title with @title in docstrings

2019-03-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Author: mrniranjan Title: #756: TESTS: Add test case title with @title in docstrings Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/756/head:pr756 git checkout pr756 From

[SSSD] [sssd PR#766][comment] TESTS: Test kvno correctly displays vesion numbers of principals

2019-03-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals mrniranjan commented: """ Ran Below tests on s390x . (Linux ibm-z-130.rhts.eng.bos.redhat.com 4.18.0-74.el8.s390x #1 SMP Wed Feb 27 17:18:28 UTC 2019 s390x s

[SSSD] [sssd PR#766][opened] TESTS: Test kvno correctly displays vesion numbers of principals

2019-03-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/766 Author: mrniranjan Title: #766: TESTS: Test kvno correctly displays vesion numbers of principals Action: opened PR body: """ Multihost tests for: https://pagure.io/SSSD/sssd/issue/3757. Signed-off-by: Niranjan M.R """

[SSSD] [sssd PR#756][comment] TESTS: Add test case title with @title in docstrings

2019-02-20 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Title: #756: TESTS: Add test case title with @title in docstrings mrniranjan commented: """ > I have two questions: > > 1. Is the IDM-SSSD-TC string significant in any way for downstream > testing? What happens if we ad

[SSSD] [sssd PR#756][comment] TESTS: Add test case title with @title in docstrings

2019-02-17 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Title: #756: TESTS: Add test case title with @title in docstrings mrniranjan commented: """ @jhrozek @pbrezina can't figure out what's failing , pep8 seems to fine. """ See the full comment at https://gi

[SSSD] [sssd PR#756][synchronized] TESTS: Add test case title with @title in docstrings

2019-02-16 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Author: mrniranjan Title: #756: TESTS: Add test case title with @title in docstrings Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/756/head:pr756 git checkout pr756 From

[SSSD] [sssd PR#756][synchronized] TESTS: Add test case title with @title in docstrings

2019-02-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Author: mrniranjan Title: #756: TESTS: Add test case title with @title in docstrings Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/756/head:pr756 git checkout pr756 From

[SSSD] [sssd PR#756][synchronized] TESTS: Add test case title with @title in docstrings

2019-02-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Author: mrniranjan Title: #756: TESTS: Add test case title with @title in docstrings Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/756/head:pr756 git checkout pr756 From

[SSSD] [sssd PR#756][synchronized] TESTS: Add test case title with @title in docstrings

2019-02-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Author: mrniranjan Title: #756: TESTS: Add test case title with @title in docstrings Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/756/head:pr756 git checkout pr756 From

[SSSD] [sssd PR#756][opened] TESTS: Add test case title with @title in docstrings

2019-02-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/756 Author: mrniranjan Title: #756: TESTS: Add test case title with @title in docstrings Action: opened PR body: """ Adding test case titles starting with @title. When pytest is run with --junit-xml, the resulting junit file contains

[SSSD] [sssd PR#739][comment] TESTS: Add a test for extra attributes in the output of sssctl user-checks

2019-02-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/739 Title: #739: TESTS: Add a test for extra attributes in the output of sssctl user-checks mrniranjan commented: """ @jhrozek Looks good to me """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#726][comment] TESTS: Add a simple integration test for retrieving the extraAttributes property

2019-02-14 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/726 Title: #726: TESTS: Add a simple integration test for retrieving the extraAttributes property mrniranjan commented: """ @jhrozek Looks good, apologise for the delay """ See the full comment at https://github.com/S

[SSSD] [sssd PR#725][comment] MULTIHOST: Do not use the deprecated namespace

2019-02-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/725 Title: #725: MULTIHOST: Do not use the deprecated namespace mrniranjan commented: """ @jhrozek looks good to me """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#725][comment] MULTIHOST: Do not use the deprecated namespace

2019-02-04 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/725 Title: #725: MULTIHOST: Do not use the deprecated namespace mrniranjan commented: """ @mrniranjan looks good to me. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#680][comment] pytest: Add test case for Expired sudo rule

2018-11-14 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/680 Title: #680: pytest: Add test case for Expired sudo rule mrniranjan commented: """ @jhrozek Can you review this ? """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#680][comment] pytest: Add test case for Expired sudo rule

2018-10-21 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/680 Title: #680: pytest: Add test case for Expired sudo rule mrniranjan commented: """ I have fixed the issues raised in the review comments, but for some reason github is not updating the PR with latest commits. """

[SSSD] [sssd PR#680][comment] pytest: Add test case for Expired sudo rule

2018-10-21 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/680 Title: #680: pytest: Add test case for Expired sudo rule mrniranjan commented: """ I have fixed the issues raised in the review comments, but for some reason github is not updating the PR with latest commits. """

[SSSD] [sssd PR#680][synchronized] pytest: Add test case for Expired sudo rule

2018-10-21 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/680 Author: mrniranjan Title: #680: pytest: Add test case for Expired sudo rule Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/680/head:pr680 git checkout pr680 From

[SSSD] [sssd PR#680][synchronized] pytest: Add test case for Expired sudo rule

2018-10-21 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/680 Author: mrniranjan Title: #680: pytest: Add test case for Expired sudo rule Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/680/head:pr680 git checkout pr680 From

[SSSD] [sssd PR#680][comment] pytest: Add test case for Expired sudo rule

2018-10-19 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/680 Title: #680: pytest: Add test case for Expired sudo rule mrniranjan commented: """ On the buggy version, i could see the sssd_sudo crashes. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#680][synchronized] pytest: Add test case for Expired sudo rule

2018-10-17 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/680 Author: mrniranjan Title: #680: pytest: Add test case for Expired sudo rule Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/680/head:pr680 git checkout pr680 From

[SSSD] [sssd PR#680][opened] pytest: Add test case for Expired sudo rule

2018-10-16 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/680 Author: mrniranjan Title: #680: pytest: Add test case for Expired sudo rule Action: opened PR body: """ """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/680

[SSSD] [sssd PR#672][synchronized] testlib/utils: use SSSDException and decode str to bytes.

2018-10-16 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/672 Author: mrniranjan Title: #672: testlib/utils: use SSSDException and decode str to bytes. Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/672/head:pr672 git checkout pr672

[SSSD] [sssd PR#672][comment] testlib/utils: use SSSDException and decode str to bytes.

2018-10-16 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/672 Title: #672: testlib/utils: use SSSDException and decode str to bytes. mrniranjan commented: """ Q: can you explain in the commit message why is the new exception better than the old one? Exception is too general and broad . mostly tr

[SSSD] [sssd PR#672][synchronized] testlib/utils: use SSSDException and decode str to bytes.

2018-10-16 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/672 Author: mrniranjan Title: #672: testlib/utils: use SSSDException and decode str to bytes. Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/672/head:pr672 git checkout pr672

[SSSD] [sssd PR#672][opened] testlib/utils: use SSSDException and decode str to bytes.

2018-10-09 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/672 Author: mrniranjan Title: #672: testlib/utils: use SSSDException and decode str to bytes. Action: opened PR body: """ Signed-off-by: Niranjan M.R """ To pull the PR as Git branch: git remote add ghsssd https://github.

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-10-01 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-10-01 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-10-01 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][comment] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-28 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains mrniranjan commented: """ @jhrozek can you review this """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-26 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#659][opened] pytest: Remove installing idm module

2018-09-26 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/659 Author: mrniranjan Title: #659: pytest: Remove installing idm module Action: opened PR body: """ Signed-off-by: Niranjan M.R """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fe

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-24 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-24 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-24 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-24 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-24 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][synchronized] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-24 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/658

[SSSD] [sssd PR#658][opened] pytest: Test case for sudo: search with lower cased name for case insensitive domains

2018-09-20 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/658 Author: mrniranjan Title: #658: pytest: Test case for sudo: search with lower cased name for case insensitive domains Action: opened PR body: """ """ To pull the PR as Git branch: git remote add ghsssd https://github.

[SSSD] [sssd PR#652][synchronized] Multihost-SanityTests: New test case for ssh login with KCM as default

2018-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/652 Author: mrniranjan Title: #652: Multihost-SanityTests: New test case for ssh login with KCM as default Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/652/head:pr652 git

[SSSD] [sssd PR#652][opened] Multihost-SanityTests: New test case for ssh login with KCM as default

2018-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/652 Author: mrniranjan Title: #652: Multihost-SanityTests: New test case for ssh login with KCM as default Action: opened PR body: """ Signed-off-by: Niranjan M.R """ To pull the PR as Git branch: git remote add ghsssd ht

[SSSD] [sssd PR#651][synchronized] test-library: fixes related to KCM, TLS on Directory server

2018-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/651 Author: mrniranjan Title: #651: test-library: fixes related to KCM, TLS on Directory server Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/651/head:pr651 git checkout pr651

[SSSD] [sssd PR#651][synchronized] test-library: fixes related to KCM, TLS on Directory server

2018-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/651 Author: mrniranjan Title: #651: test-library: fixes related to KCM, TLS on Directory server Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/651/head:pr651 git checkout pr651

[SSSD] [sssd PR#651][opened] test-library: fixes related to KCM, TLS on Directory server

2018-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/651 Author: mrniranjan Title: #651: test-library: fixes related to KCM, TLS on Directory server Action: opened PR body: """ Fixes related to enabling of ssl in Directory Server Minor fixes on KCM, adding new exceptions Signed-off-b

[SSSD] [sssd PR#642][comment] TESTS: Fixes to test library and add new test case for kcm.

2018-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/642 Title: #642: TESTS: Fixes to test library and add new test case for kcm. mrniranjan commented: """ Latest patch addresses it """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#642][synchronized] TESTS: Fixes to test library and add new test case for kcm.

2018-09-11 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/642 Author: mrniranjan Title: #642: TESTS: Fixes to test library and add new test case for kcm. Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/642/head:pr642 git checkout pr642

[SSSD] [sssd PR#642][synchronized] TESTS: Fixes to test library and add new test case for kcm.

2018-09-03 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/642 Author: mrniranjan Title: #642: TESTS: Fixes to test library and add new test case for kcm. Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/642/head:pr642 git checkout pr642

[SSSD] [sssd PR#642][synchronized] TESTS: Fixes to test library and add new test case for kcm.

2018-08-30 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/642 Author: mrniranjan Title: #642: TESTS: Fixes to test library and add new test case for kcm. Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/642/head:pr642 git checkout pr642

[SSSD] [sssd PR#642][comment] TESTS: Fixes to test library and add new test case for kcm.

2018-08-29 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/642 Title: #642: TESTS: Fixes to test library and add new test case for kcm. mrniranjan commented: """ Regarding the failures from the jenkins job it says there are some pep8 failures, When i ran pep8 on my laptop for all the python files

[SSSD] [sssd PR#642][opened] TESTS: Fixes to test library and add new test case for kcm.

2018-08-28 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/642 Author: mrniranjan Title: #642: TESTS: Fixes to test library and add new test case for kcm. Action: opened PR body: """ Fixes related to enabling of ssl in Directory Server Minor fixes on KCM, adding new exceptions Adding a new test c

[SSSD] [sssd PR#641][opened] Minor fixes related to converting of ldap attributes to bytes

2018-08-17 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/641 Author: mrniranjan Title: #641: Minor fixes related to converting of ldap attributes to bytes Action: opened PR body: """ Minor fixes related to converting attribute values to bytes in function related to adding posix group Signed-o

[SSSD] [sssd PR#640][opened] Python3 changes to multihost tests

2018-08-16 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/640 Author: mrniranjan Title: #640: Python3 changes to multihost tests Action: opened PR body: """ 1. Python3 related changes 2. converting ldap attributes to use bytes Signed-off-by: Niranjan M.R """ To pull the PR

[SSSD] [sssd PR#139][comment] Initial revision of sssd pytest framework

2017-11-16 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/139 Title: #139: Initial revision of sssd pytest framework mrniranjan commented: """ +1 from me for team """ See the full comment at https://github.com/SSSD/sssd/pull/139#issuecomment-344963921

[SSSD] [sssd PR#139][comment] Initial revision of sssd pytest framework

2017-11-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/139 Title: #139: Initial revision of sssd pytest framework mrniranjan commented: """ Till we have some idea on the review process, the directory structure of where tests should be pushed and how to run those tests , i am okay with not havi

[SSSD] [sssd PR#139][comment] Initial revision of sssd pytest framework

2017-11-15 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/139 Title: #139: Initial revision of sssd pytest framework mrniranjan commented: """ Thanks will apply the changes suggested and resend the patch """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#139][comment] Initial revision of sssd pytest framework

2017-11-13 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/139 Title: #139: Initial revision of sssd pytest framework mrniranjan commented: """ I am unable to access the link given """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#139][comment] Initial revision of sssd pytest framework

2017-11-13 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/139 Title: #139: Initial revision of sssd pytest framework mrniranjan commented: """ I have fixed most of the errors suggested, i would like if you we don't enforce 80chars limit, the readeability really hampers, Can we have 90 or

  1   2   >