[SSSD] [sssd PR#274][comment] Merge sss_cache and sss_debuglevel into sssctl

2017-05-19 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/274 Title: #274: Merge sss_cache and sss_debuglevel into sssctl jhrozek commented: """ I still see value in having sssctl manage the cache expiration because especially for sudo rules, we not only need to expire the rules in the cache, but also tell the

[SSSD] [sssd PR#230][comment] Failure to filter out AD subdomain with flatname is used

2017-05-19 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/230 Title: #230: Failure to filter out AD subdomain with flatname is used justin-stephenson commented: """ @lslebodn yes it looks the same, I think we can close this PR and associated ticket `3365` """ See the full comment at

[SSSD] [sssd PR#230][closed] Failure to filter out AD subdomain with flatname is used

2017-05-19 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/230 Author: justin-stephenson Title: #230: Failure to filter out AD subdomain with flatname is used Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/230/head:pr230 git checkout pr230

[SSSD] [sssd PR#274][comment] Merge sss_cache and sss_debuglevel into sssctl

2017-05-19 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/274 Title: #274: Merge sss_cache and sss_debuglevel into sssctl justin-stephenson commented: """ @lslebodn Yes, you make a good point. Is there a suggested way to handle this or do you prefer to drop the `sss_cache` commits altogether? """ See the full

[SSSD] [sssd PR#255][comment] Use default configuration with missing sssd.conf

2017-05-19 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/255 Title: #255: Use default configuration with missing sssd.conf jhrozek commented: """ yes, feel free to push """ See the full comment at https://github.com/SSSD/sssd/pull/255#issuecomment-302748782 ___

[SSSD] [sssd PR#255][+Accepted] Use default configuration with missing sssd.conf

2017-05-19 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/255 Title: #255: Use default configuration with missing sssd.conf Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#255][comment] Use default configuration with missing sssd.conf

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/255 Title: #255: Use default configuration with missing sssd.conf lslebodn commented: """ > sssd is still run on start up (but is not supposed to) it should be just > disabled on systemd level... @jhrozek I hope you agree with Pavel :-) """ See the full

[SSSD] [sssd PR#255][comment] Use default configuration with missing sssd.conf

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/255 Title: #255: Use default configuration with missing sssd.conf lslebodn commented: """ > sssd is still run on start up (but is not supposed to) it should be just > disabled on systemd level... @jhrozek I hope you agree with Pavel :-) """ See the full

[SSSD] [sssd PR#266][+Changes requested] IPA: Fix the PAM error code that auth code expects to start migration

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/266 Title: #266: IPA: Fix the PAM error code that auth code expects to start migration Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#230][comment] Failure to filter out AD subdomain with flatname is used

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/230 Title: #230: Failure to filter out AD subdomain with flatname is used lslebodn commented: """ @justin-stephenson Is my assumption in previous comment correct? """ See the full comment at https://github.com/SSSD/sssd/pull/230#issuecomment-302728816

[SSSD] [sssd PR#274][+Changes requested] Merge sss_cache and sss_debuglevel into sssctl

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/274 Title: #274: Merge sss_cache and sss_debuglevel into sssctl Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#270][+Pushed] Subdomains direct integration

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/270 Title: #270: Subdomains direct 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#270][comment] Subdomains direct integration

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/270 Title: #270: Subdomains direct integration lslebodn commented: """ master: * c4ddb9ccab670f9c0d0377680237b62f9f91c496 * b4ca0da4d8d70bcfbd4f809f3b3b094d43d64cfc """ See the full comment at https://github.com/SSSD/sssd/pull/270#issuecomment-302722677

[SSSD] [sssd PR#270][closed] Subdomains direct integration

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/270 Author: mzidek-rh Title: #270: Subdomains direct integration Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/270/head:pr270 git checkout pr270

[SSSD] [sssd PR#136][comment] Tlog integration

2017-05-19 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/136 Title: #136: Tlog integration spbnick commented: """ This update rebases on latest master and also changes "tlog-rec" to "tlog-rec-session", as the latter is now supposed to be used as the login shell in upstream tlog. """ See the full comment at

[SSSD] [sssd PR#270][comment] Subdomains direct integration

2017-05-19 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/270 Title: #270: Subdomains direct integration sumit-bose commented: """ Since it is a fallback to the old default behavior I think a fallback is fine here. """ See the full comment at https://github.com/SSSD/sssd/pull/270#issuecomment-302693689

[SSSD] [sssd PR#270][comment] Subdomains direct integration

2017-05-19 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/270 Title: #270: Subdomains direct integration mzidek-rh commented: """ Ok, I changed the level from CRIT_FAILURE to MINOR_FAILURE. Or does someone prefer to fail completely in that case? The reason I decided to continue is that in most cases the

[SSSD] [sssd PR#270][synchronized] Subdomains direct integration

2017-05-19 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/270 Author: mzidek-rh Title: #270: Subdomains direct integration Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/270/head:pr270 git checkout pr270 From

[SSSD] [sssd PR#270][comment] Subdomains direct integration

2017-05-19 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/270 Title: #270: Subdomains direct integration lslebodn commented: """ On (19/05/17 04:49), mzidek-rh wrote: >Commit message updated (otherwise no changes). > And another small nitpick. There are CRITICAL debug messages even though we fall back to

[SSSD] Re: Fragile test enumeration

2017-05-19 Thread Lukas Slebodnik
On (31/03/17 11:58), Lukas Slebodnik wrote: >ehlo, > >we pushed recently many patches and I noticed increased amount of failures >in enumeration test. > >http://sssd-ci.duckdns.org/logs/job/66/82/summary.html >http://sssd-ci.duckdns.org/logs/job/66/83/summary.html

[SSSD] [sssd PR#270][comment] Subdomains direct integration

2017-05-19 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/270 Title: #270: Subdomains direct integration mzidek-rh commented: """ Commit message updated (otherwise no changes). """ See the full comment at https://github.com/SSSD/sssd/pull/270#issuecomment-302682307

[SSSD] [sssd PR#270][synchronized] Subdomains direct integration

2017-05-19 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/270 Author: mzidek-rh Title: #270: Subdomains direct integration Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/270/head:pr270 git checkout pr270 From

[SSSD] [sssd PR#270][comment] Subdomains direct integration

2017-05-19 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/270 Title: #270: Subdomains direct integration sumit-bose commented: """ The patch looks good, Alexander's comments are included and I was able to properly overwrite the search bases for subdomains, so ACK. Before pushing you might want to fix a few

[SSSD] [sssd PR#270][comment] Subdomains direct integration

2017-05-19 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/270 Title: #270: Subdomains direct integration sumit-bose commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/270#issuecomment-302673988 ___ sssd-devel mailing list