[SSSD] [sssd PR#342][comment] SELINUX: Use getseuserbyname to get IPA seuser

2017-08-28 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/342 Title: #342: SELINUX: Use getseuserbyname to get IPA seuser justin-stephenson commented: """ I agree with Fabiano, I could not find an equivalent libselinux user delete function. If there is a one i'm missing then I don't mind updating the PR. """ See

[SSSD] [sssd PR#355][+Pushed] localauth plugin: change return code of sss_an2ln

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/355 Title: #355: localauth plugin: change return code of sss_an2ln Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahoste

[SSSD] [sssd PR#355][closed] localauth plugin: change return code of sss_an2ln

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/355 Author: sumit-bose Title: #355: localauth plugin: change return code of sss_an2ln Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/355/head:pr355 git checkout pr355 _

[SSSD] [sssd PR#355][comment] localauth plugin: change return code of sss_an2ln

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/355 Title: #355: localauth plugin: change return code of sss_an2ln jhrozek commented: """ * master: b4e45531b3e98efce868d8a01ebd2dbe54348217 3f94a979eebd1c9496b49b4e07b7823550dec97e """ See the full comment at https://github.com/SSSD/sssd/pull/355#issueco

[SSSD] [sssd PR#355][comment] localauth plugin: change return code of sss_an2ln

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/355 Title: #355: localauth plugin: change return code of sss_an2ln jhrozek commented: """ The change is a one-liner, matching the ticket. There is a unit test and neither CI nor Coverity complain -> ACK. """ See the full comment at https://github.com/SSS

[SSSD] [sssd PR#355][+Accepted] localauth plugin: change return code of sss_an2ln

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/355 Title: #355: localauth plugin: change return code of sss_an2ln Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahos

[SSSD] [sssd PR#332][+Changes requested] sydb: index improvements

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/332 Title: #332: sydb: index improvements 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#332][comment] sydb: index improvements

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/332 Title: #332: sydb: index improvements jhrozek commented: """ Setting changes requested as per the review. (note that this is NOT a nudge for anyone to work faster, I'm just reconciling the PR statuses) """ See the full comment at https://github.com/

[SSSD] [sssd PR#351][comment] NSS: Look for name attribute also in nss_cmd_getsidbyid

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/351 Title: #351: NSS: Look for name attribute also in nss_cmd_getsidbyid jhrozek commented: """ Setting Changes requested as per @pbrezina's last comment (note that this is NOT a nudge for anyone to work faster, I'm just reconciling the PR statuses) """

[SSSD] [sssd PR#351][+Changes requested] NSS: Look for name attribute also in nss_cmd_getsidbyid

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/351 Title: #351: NSS: Look for name attribute also in nss_cmd_getsidbyid Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@

[SSSD] [sssd PR#325][+Changes requested] MAN: Improve description of 'trusted domain section' in sssd.conf's man page

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/325 Title: #325: MAN: Improve description of 'trusted domain section' in sssd.conf's man page Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an emai

[SSSD] [sssd PR#347][+Pushed] Fixes related to negative cache and "root" user/group

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/347 Title: #347: Fixes related to negative cache and "root" user/group Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorah

[SSSD] [sssd PR#347][comment] Fixes related to negative cache and "root" user/group

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/347 Title: #347: Fixes related to negative cache and "root" user/group jhrozek commented: """ * master: 6c3841099addb84bf3e9a2f85e96dffae1b94623 5883b99fa0d13368f6e79fdb40b6637d36ed1801 137e105ac8ca3476d2f74d24ae13860774937000 b4b3d0642120ca05f63959fe2f317a

[SSSD] [sssd PR#347][closed] Fixes related to negative cache and "root" user/group

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/347 Author: fidencio Title: #347: Fixes related to negative cache and "root" user/group Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/347/head:pr347 git checkout pr347 ___

[SSSD] [sssd PR#241][closed] FleetCommander Integration

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/241 Author: fidencio Title: #241: FleetCommander Integration Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/241/head:pr241 git checkout pr241 __

[SSSD] [sssd PR#241][comment] FleetCommander Integration

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/241 Title: #241: FleetCommander Integration jhrozek commented: """ * master: 4a311702045b065a97a0c0fc0ccc7a1fc84b38cf 85517b57685809ff96818bbd3e3b4678ac74b461 85a93ca67ae020607006cd035170c9360fb0a450 684a13e8de1526257ca2e40b6bf2e05585d4eaca 4b37ee7d37000351

[SSSD] [sssd PR#241][+Pushed] FleetCommander Integration

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/241 Title: #241: FleetCommander Integration Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#358][comment] Requesting a pull to SSSD:master from jhrozek:pep8

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/358 Title: #358: Requesting a pull to SSSD:master from jhrozek:pep8 jhrozek commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/358#issuecomment-325437251 ___ sssd-dev

[SSSD] [sssd PR#347][comment] Fixes related to negative cache and "root" user/group

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/347 Title: #347: Fixes related to negative cache and "root" user/group jhrozek commented: """ CI: http://vm-058-233.XXX/logs/job/74/35/summary.html (there was one failure related to the buggy leak check that we already track, but that's unrelated to this

[SSSD] [sssd PR#347][+Accepted] Fixes related to negative cache and "root" user/group

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/347 Title: #347: Fixes related to negative cache and "root" user/group Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedor

[SSSD] [sssd PR#358][opened] Requesting a pull to SSSD:master from jhrozek:pep8

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/358 Author: jhrozek Title: #358: Requesting a pull to SSSD:master from jhrozek:pep8 Action: opened PR body: """ To reprouce, run: pep8 src/tests/intg/*.py No functional changes are present. """ To pull the PR as Git branch: git remote add ghsssd ht

[SSSD] [sssd PR#241][+Accepted] FleetCommander Integration

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/241 Title: #241: FleetCommander Integration Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#241][comment] FleetCommander Integration

2017-08-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/241 Title: #241: FleetCommander Integration jhrozek commented: """ ACK, CI: http://vm-058-233.XXX/logs/job/74/33/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/241#issuecomment-325391191

[SSSD] [sssd PR#357][opened] A few fixes/improvements related to issue #2976

2017-08-28 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/357 Author: fidencio Title: #357: A few fixes/improvements related to issue #2976 Action: opened PR body: """ While digging into the sdap_id_op.c code, I've noticed a missing debug log that could be useful and a fishy situation where we can mark the whol

[SSSD] [sssd PR#321][comment] certmap: add OpenSSL implementation

2017-08-28 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/321 Title: #321: certmap: add OpenSSL implementation sumit-bose commented: """ Thank you for the review, I addressed all your comments. About the tickets, the validation is currently done in two places, the ssh responder and p11_child. So I think https://

[SSSD] [sssd PR#321][synchronized] certmap: add OpenSSL implementation

2017-08-28 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/321 Author: sumit-bose Title: #321: certmap: add OpenSSL implementation Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/321/head:pr321 git checkout pr321 From 08faaa9a420f796c93b4

[SSSD] [sssd PR#241][comment] FleetCommander Integration

2017-08-28 Thread olivergs
URL: https://github.com/SSSD/sssd/pull/241 Title: #241: FleetCommander Integration olivergs commented: """ Tested the latest build for the patches from @fidencio COPR repo and it worked as expected. """ See the full comment at https://github.com/SSSD/sssd/pull/241#issuecomment-325337915 __

[SSSD] [sssd PR#241][comment] FleetCommander Integration

2017-08-28 Thread olivergs
URL: https://github.com/SSSD/sssd/pull/241 Title: #241: FleetCommander Integration olivergs commented: """ Tested the latest build for the patches from @fidencio COPR repo and it worked as expected. On Thu, Aug 17, 2017 at 4:23 PM, fidencio wrote: > Patch set updated. > > This is the patch