[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread martinpitt
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error martinpitt commented: """ @ikerexxe : I'm happy to test this on Fedora 35, rhel 8.5, 8.6 or 9.0, or centos stream 8. Thanks! """ See the full comment at

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error ikerexxe commented: """ > @ikerexxe, is this still "Changes requested"? Yes, I'd like to run the CI and see that everything works correctly before changing its status. But apparently this is not going to

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread stanislavlevin
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements stanislavlevin commented: """ ``` [root@master1 /]# sssctl analyze request list -v NOTE: Tevent chain ID support missing, request analysis will be limited. It is recommended to use

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread stanislavlevin
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements stanislavlevin commented: """ Are there corresponding tests? Can't find them. """ See the full comment at https://github.com/SSSD/sssd/pull/5863#issuecomment-996523962

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread stanislavlevin
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements stanislavlevin commented: """ cherry-pick is OK. """ See the full comment at https://github.com/SSSD/sssd/pull/5863#issuecomment-996521088

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error ikerexxe commented: """ > I looked at the diff, and the new exception looks very reasonable/useful for > me, thanks @ikerexxe ! I didn't test it yet -- is there some packit-like rpm > being produced in CI

[SSSD] [sssd PR#5914][comment] Tests: add docstring in intg/test_infopipe.py

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5914 Title: #5914: Tests: add docstring in intg/test_infopipe.py ikerexxe commented: """ > @ikerexxe I rebased but, how do I force it to rerun the pipeline tests? The rerun is triggered each time that the branch is pushed. """ See the full comment at

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread martinpitt
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error martinpitt commented: """ I looked at the diff, and the new exception looks very reasonable/useful for me, thanks @ikerexxe ! I didn't test it yet -- is there some packit-like rpm being produced in CI which I

[SSSD] [sssd PR#5927][+branch: sssd-1-16] ad: add required 'cn' attribute to subdomain object

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5927 Title: #5927: ad: add required 'cn' attribute to subdomain object Label: +branch: sssd-1-16 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#5927][+Bugzilla] ad: add required 'cn' attribute to subdomain object

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5927 Title: #5927: ad: add required 'cn' attribute to subdomain object Label: +Bugzilla ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread stanislavlevin
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements stanislavlevin commented: """ > ``` > [root@master1 /]# sssctl analyze request list -v > NOTE: Tevent chain ID support missing, request analysis will be limited. > It is

[SSSD] [sssd PR#5927][opened] ad: add required 'cn' attribute to subdomain object

2021-12-17 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/5927 Author: sumit-bose Title: #5927: ad: add required 'cn' attribute to subdomain object Action: opened PR body: """ If the forest root is not part of the return trusted domain objects from the local domain controller we generate an object for further

[SSSD] [sssd PR#5914][comment] Tests: add docstring in intg/test_infopipe.py

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5914 Title: #5914: Tests: add docstring in intg/test_infopipe.py alexey-tikhonov commented: """ > > @ikerexxe I rebased but, how do I force it to rerun the pipeline tests? > > The rerun is triggered each time that the branch is pushed. This or it's

[SSSD] [sssd PR#5920][-Waiting for review] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread thalman
URL: https://github.com/SSSD/sssd/pull/5920 Title: #5920: krb5: write kdcinfo.* file with port configuration Label: -Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#5920][+Accepted] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread thalman
URL: https://github.com/SSSD/sssd/pull/5920 Title: #5920: krb5: write kdcinfo.* file with port configuration Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements alexey-tikhonov commented: """ From my point of view, it's ok to change "will be limited" to "unsupported" and just quit at this point. There is really very little we can do

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements alexey-tikhonov commented: """ From my point of view, it's ok to change "will be limited" to "unsupported" and just quit at this point. There is really very little we can do

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error ikerexxe commented: """ > @ikerexxe : I'm happy to test this on Fedora 35, rhel 8.5, 8.6 or 9.0, or > centos stream 8. Thanks! Fedora 35 can be tested using the following

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error alexey-tikhonov commented: """ Thanks, ACK. """ See the full comment at https://github.com/SSSD/sssd/pull/5918#issuecomment-996824912 ___ sssd-devel mailing list --

[SSSD] [sssd PR#5918][+Accepted] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error 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

[SSSD] [sssd PR#5918][-Waiting for review] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation 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

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements justin-stephenson commented: """ > ``` > [root@master1 /]# sssctl analyze request list -v > NOTE: Tevent chain ID support missing, request analysis will be limited. > It is

[SSSD] [sssd PR#5918][-Changes requested] Client ca validation error

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#5918][+Waiting for review] Client ca validation error

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation 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

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error alexey-tikhonov commented: """ All CI targets are green besides Rawhide that seems to be stuck on multihost tests: ``` configure: success 00:00:45 ci-build-debug/ci-configure.log make-tests:

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread martinpitt
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error martinpitt commented: """ Thanks @ikerexxe ! Just FTR, last day (few hours) of work for this year, will check this in January. Happy EOY holidays! """ See the full comment at

[SSSD] [sssd PR#5918][synchronized] Client ca validation error

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5918 Author: ikerexxe Title: #5918: Client ca validation error Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5918/head:pr5918 git checkout pr5918 From

[SSSD] [sssd PR#5918][-Changes requested] Client ca validation error

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#5918][+Waiting for review] Client ca validation error

2021-12-17 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation 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

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread stanislavlevin
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements stanislavlevin commented: """ @justin-stephenson, I saw the message, thank you. Since there is no strict requirement on that version of libtevent I assumed the tool should work in

[SSSD] [sssd PR#5928][opened] IPA: Add password expire warning

2021-12-17 Thread elkoniu
URL: https://github.com/SSSD/sssd/pull/5928 Author: elkoniu Title: #5928: IPA: Add password expire warning Action: opened PR body: """ When LDAP is used as an access provider it can be configured to show user password expiration warning. This commit enables similar behaviour for IPA access

[SSSD] [sssd PR#5918][-Waiting for review] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation 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

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error alexey-tikhonov commented: """ Overall looks good to me. Just a couple of nitpicks. """ See the full comment at https://github.com/SSSD/sssd/pull/5918#issuecomment-996793581

[SSSD] [sssd PR#5918][+Changes requested] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements justin-stephenson commented: """ * Added one final commit (I hope) https://github.com/SSSD/sssd/pull/5863/commits/8ab51e9dbd3ec7a469f0e38046f2354297e7b7ba """ See the full

[SSSD] [sssd PR#5920][comment] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5920 Title: #5920: krb5: write kdcinfo.* file with port configuration alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5920 * `master` * 1e747fad4539ffb402010e73f78469fe57af408f - krb5: write kdcinfo.* file with port

[SSSD] [sssd PR#5920][+Pushed] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5920 Title: #5920: krb5: write kdcinfo.* file with port configuration Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#5920][-Ready to push] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5920 Title: #5920: krb5: write kdcinfo.* file with port configuration Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#5920][-Accepted] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5920 Title: #5920: krb5: write kdcinfo.* file with port configuration Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#5920][closed] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5920 Author: ikerexxe Title: #5920: krb5: write kdcinfo.* file with port configuration Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5920/head:pr5920 git checkout pr5920

[SSSD] [sssd PR#5863][synchronized] Responder and Child process tevent chain id improvements

2021-12-17 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/5863 Author: justin-stephenson Title: #5863: Responder and Child process tevent chain id improvements Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5863/head:pr5863 git

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements sumit-bose commented: """ Hi, I agree with @alexey-tikhonov, it is better to just fail hard if chain ID support is missing. bye, Sumit """ See the full comment at

[SSSD] [sssd PR#5920][comment] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5920 Title: #5920: krb5: write kdcinfo.* file with port configuration alexey-tikhonov commented: """ Debian CI failure seems to be non-relevant. """ See the full comment at https://github.com/SSSD/sssd/pull/5920#issuecomment-996953520

[SSSD] [sssd PR#5920][+Ready to push] krb5: write kdcinfo.* file with port configuration

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5920 Title: #5920: krb5: write kdcinfo.* file with port configuration Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#5918][comment] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5918 * `master` * fd0f087ada2f4a59d4b22846d06f1b19fd522e11 - ifp: improve FindByValidCertificate() error *

[SSSD] [sssd PR#5918][+Pushed] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of

[SSSD] [sssd PR#5918][-Accepted] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error 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

[SSSD] [sssd PR#5918][-Ready to push] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora

[SSSD] [sssd PR#5918][closed] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Author: ikerexxe Title: #5918: Client ca validation error Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5918/head:pr5918 git checkout pr5918

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements alexey-tikhonov commented: """ https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5863/20/fedora33/ci-make-intgcheck.log """ See the full comment at

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements justin-stephenson commented: """ > https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5863/20/fedora33/ci-make-intgcheck.log Not sure how this was caused by my PR, I'm triggering a

[SSSD] [sssd PR#5863][comment] Responder and Child process tevent chain id improvements

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5863 Title: #5863: Responder and Child process tevent chain id improvements alexey-tikhonov commented: """ > > https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5863/20/fedora33/ci-make-intgcheck.log > > Not sure how this was caused by my PR I also don't

[SSSD] [sssd PR#5918][+Ready to push] Client ca validation error

2021-12-17 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5918 Title: #5918: Client ca validation error Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora