[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

2016-11-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) jhrozek commented: """ On Thu, Nov 03, 2016 at 11:19:02PM -0700, celestian wrote: > So, new version is pushed. It uses only ```sudoUser``` attribute. Thank you, t

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

2016-11-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) jhrozek commented: """ On Fri, Nov 04, 2016 at 12:06:08AM -0700, celestian wrote: > I found function: > ``` > char * > sss_get_cas

[SSSD] [sssd PR#46][comment] sss_client: Defer thread cancellation until completion of nss/pam operations

2016-11-06 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/46 Title: #46: sss_client: Defer thread cancellation until completion of nss/pam operations jhrozek commented: """ Well, if the old code would be disabled and there would be nobody using the new config option from the start, I'm fi

[SSSD] [sssd PR#43][+Accepted] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

2016-11-07 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fed

[SSSD] [sssd PR#71][opened] MONITOR: Do not set up watchdog for monitor

2016-11-07 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/71 Author: jhrozek Title: #71: MONITOR: Do not set up watchdog for monitor Action: opened PR body: """ It makes little sense to set up watchdog for monitor because there is no entity that would restart the monitor. Therefore we sho

[SSSD] [sssd PR#72][opened] Remove two unused methods from monitor's sbus interface

2016-11-07 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/72 Author: jhrozek Title: #72: Remove two unused methods from monitor's sbus interface Action: opened PR body: """ None """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch gh

[SSSD] [sssd PR#71][closed] MONITOR: Do not set up watchdog for monitor

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/71 Author: jhrozek Title: #71: MONITOR: Do not set up watchdog for monitor Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/71/head:pr71 git checkout pr71

[SSSD] [sssd PR#71][+Pushed] MONITOR: Do not set up watchdog for monitor

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/71 Title: #71: MONITOR: Do not set up watchdog for monitor 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#71][comment] MONITOR: Do not set up watchdog for monitor

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/71 Title: #71: MONITOR: Do not set up watchdog for monitor jhrozek commented: """ master: fbe6644aa28d93f492434950680c5618eb567712 sssd-1-14: 2d88a121918e800b266d018d43dad9bd374b10a7 """ See the full comment at https:

[SSSD] [sssd PR#73][+Accepted] AD_PROVIDER: Enabled subdomains (1.13)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/73 Title: #73: AD_PROVIDER: Enabled subdomains (1.13) 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#73][comment] AD_PROVIDER: Enabled subdomains (1.13)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/73 Title: #73: AD_PROVIDER: Enabled subdomains (1.13) jhrozek commented: """ I tested the patches with a setup that consists of three domains (win.trust.test, child.win.trust.test and sibling.win.trust.test). The patch works good, I test

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) jhrozek commented: """ btw the downstream test against a 389DS server also passed (I'm not sure if we have some downstream tests for sssd+sudo+AD..), so I

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) jhrozek commented: """ master: f4a1046bb88d7a0ab3617e49ae94bfa849d10645 23637e2fd2b1fe42bdd2335893a11ac8016f56bc sssd-1-14: 143b1dcbbe865a139616a22b1

[SSSD] [sssd PR#43][-Accepted] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fed

[SSSD] [sssd PR#43][+Pushed] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedor

[SSSD] [sssd PR#43][closed] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/43 Author: celestian Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/43/head:pr43 git checkout pr43 ___

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ On Tue, Nov 08, 2016 at 04:13:43AM -0800, celestian wrote: > I pushed new version, only one difference -- I fix cherry-pick pointer. > Th

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-08 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ On Tue, Nov 08, 2016 at 05:06:41AM -0800, celestian wrote: > Yes, the second patch explicitly qualifies the names. I don't know if there

[SSSD] [sssd PR#77][opened] Qualify ghost users with RFC2307bis and ldap_nesting_level=0

2016-11-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/77 Author: jhrozek Title: #77: Qualify ghost users with RFC2307bis and ldap_nesting_level=0 Action: opened PR body: """ None """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetc

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ oops...clicked send to early. I meant to say "So the filter never matches the lowercase sudoUser". """ See the full

[SSSD] [sssd PR#39][+Changes requested] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@l

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ Are you sure this is enough? Because when the patch is applied, I see that we only match the sudoUser value with the original case. Don't we

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for IPA and AD providers

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for IPA and AD providers jhrozek commented: """ Either way, I think these patches can be re-reviewed and pushed separately. """ See the full comment at https://github.com/

[SSSD] [sssd PR#53][comment] Fixes in the config API related to secrets responder

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder jhrozek commented: """ On Tue, Nov 08, 2016 at 05:40:21AM -0800, lslebodn wrote: > On (08/11/16 05:15), fidencio wrote: > >@jhrozek: > >For the first patch th

[SSSD] [sssd PR#53][+Changes requested] Fixes in the config API related to secrets responder

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists

[SSSD] [sssd PR#68][+Accepted] MAN: Document different defaults for IPA and AD providers

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for IPA and AD providers Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fed

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for IPA and AD providers

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for IPA and AD providers jhrozek commented: """ ACK. I won't push the patches until tomorrow though in case @lslebodn or someone else wanted to take a look as well. I'll also wait

[SSSD] [sssd PR#69][+Changes requested] krb5: Use command line arguments instead env vars for krb5_child

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/69 Title: #69: krb5: Use command line arguments instead env vars for krb5_child Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel

[SSSD] [sssd PR#74][+Pushed] IPA/AD: check auth ctx before using it

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/74 Title: #74: IPA/AD: check auth ctx before using it 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#74][-Accepted] IPA/AD: check auth ctx before using it

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/74 Title: #74: IPA/AD: check auth ctx before using it 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#74][comment] IPA/AD: check auth ctx before using it

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/74 Title: #74: IPA/AD: check auth ctx before using it jhrozek commented: """ master: ea11ed3ea6291488dd762033246edc4ce3951aeb sssd-1-14: 37e070c8c2ea79d8d84bae3da3a34c81212744ab """ See the full comment at https:

[SSSD] [sssd PR#53][comment] Fixes in the config API related to secrets responder

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder jhrozek commented: """ Thanks, ACK """ See the full comment at https://github.com/SSSD/sssd/pull/53#issuecomment-259810451

[SSSD] [sssd PR#53][+Accepted] Fixes in the config API related to secrets responder

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedoraho

[SSSD] [sssd PR#53][+Pushed] Fixes in the config API related to secrets responder

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahost

[SSSD] [sssd PR#53][comment] Fixes in the config API related to secrets responder

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder jhrozek commented: """ Since there is no ticket, I only pushed the patches to master: 682c9c3467055c2149af28826f7458b857b0f8c4 da8801c363716533f60bc78e10f3a2100cebc3a1 &q

[SSSD] [sssd PR#53][-Accepted] Fixes in the config API related to secrets responder

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedoraho

[SSSD] [sssd PR#53][closed] Fixes in the config API related to secrets responder

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/53 Author: fidencio Title: #53: Fixes in the config API related to secrets responder Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/53/head:pr53 git checkout pr53 _

[SSSD] [sssd PR#74][closed] IPA/AD: check auth ctx before using it

2016-11-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/74 Author: sumit-bose Title: #74: IPA/AD: check auth ctx before using it Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/74/head:pr74 git checkout pr74 _

[SSSD] [sssd PR#78][comment] ipa: Nested netgroups do not work

2016-11-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/78 Title: #78: ipa: Nested netgroups do not work jhrozek commented: """ btw just to help with the review, I also fired up a test job using our downstream tests with a 7.3 package and this test. I'll let you know how the test wen

[SSSD] [sssd PR#78][+Accepted] ipa: Nested netgroups do not work

2016-11-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/78 Title: #78: ipa: Nested netgroups do not work 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#78][comment] ipa: Nested netgroups do not work

2016-11-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/78 Title: #78: ipa: Nested netgroups do not work jhrozek commented: """ The downstream tests passed (the job ID is 1594128 if anyone wants to take a look). So I'm adding the Accepted tag. """ See the full comment at h

[SSSD] [sssd PR#78][closed] ipa: Nested netgroups do not work

2016-11-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/78 Author: mzidek-rh Title: #78: ipa: Nested netgroups do not work Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/78/head:pr78 git checkout pr78 ___

[SSSD] [sssd PR#78][+Pushed] ipa: Nested netgroups do not work

2016-11-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/78 Title: #78: ipa: Nested netgroups do not work 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#78][comment] ipa: Nested netgroups do not work

2016-11-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/78 Title: #78: ipa: Nested netgroups do not work jhrozek commented: """ master: ff565da1011aa4312847e28e7af66e57fccf8b90 sssd-1-14: 7de33877c7e39f9a5cae6baf815dc18ae5a18597 sssd-1-13: 9c4c3a204e1631f61378982d39cdba0ce25ee262 ""&

[SSSD] [sssd PR#78][-Accepted] ipa: Nested netgroups do not work

2016-11-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/78 Title: #78: ipa: Nested netgroups do not work 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#68][closed] MAN: Document different defaults for IPA and AD providers

2016-11-15 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Author: justin-stephenson Title: #68: MAN: Document different defaults for IPA and AD providers Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/68/head:pr68 git checkout pr68 ___

[SSSD] [sssd PR#68][-Accepted] MAN: Document different defaults for IPA and AD providers

2016-11-15 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for IPA and AD providers Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fed

[SSSD] [sssd PR#68][+Pushed] MAN: Document different defaults for IPA and AD providers

2016-11-15 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for IPA and AD providers Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedor

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for IPA and AD providers

2016-11-15 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for IPA and AD providers jhrozek commented: """ master: 6e27e8572f671de575d9ac2a34a677d9efc24fbc 8caf7ba5005b3be5447311713ad2b58169f9d32f """ See the full comment at https:

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-17 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ Apart from fixing ticket #3241, why does sysdb_get_sudo_filter add its own lowercased name and does not add all nameAlias values instead? &q

[SSSD] [sssd PR#80][comment] SYSDB: Sudorule without a sudoUser returns EINVAL

2016-11-17 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/80 Title: #80: SYSDB: Sudorule without a sudoUser returns EINVAL jhrozek commented: """ This would work, but wouldn't it be better to return a custom error code instead of EINVAL so that we are sure we are not skipping some error fr

[SSSD] [sssd PR#81][opened] Please see the commit message, the fix is hopefully simple

2016-11-18 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/81 Author: jhrozek Title: #81: Please see the commit message, the fix is hopefully simple Action: opened PR body: """ None """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetc

[SSSD] [sssd PR#80][+Accepted] SYSDB: Fixing of sudorule without a sudoUser

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/80 Title: #80: SYSDB: Fixing of sudorule without a sudoUser 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#80][comment] SYSDB: Fixing of sudorule without a sudoUser

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/80 Title: #80: SYSDB: Fixing of sudorule without a sudoUser jhrozek commented: """ ACK, the rule without a sudoUser attribute is properly skipped and the other rules are normally stored. """ See the full comment at https:

[SSSD] [sssd PR#39][+Accepted] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedo

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ ACK, this version works for me. I will run also downstream tests to be sure, but my manual testing passed. """ See the full comme

[SSSD] [sssd PR#82][+Accepted] KRB5: Remove spurious warning in logs

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/82 Title: #82: KRB5: Remove spurious warning in logs 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#81][synchronized] Please see the commit message, the fix is hopefully simple

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/81 Author: jhrozek Title: #81: Please see the commit message, the fix is hopefully simple Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/81/head:pr81 git checkout pr81 From

[SSSD] [sssd PR#81][-Changes requested] Please see the commit message, the fix is hopefully simple

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/81 Title: #81: Please see the commit message, the fix is hopefully simple Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...

[SSSD] [sssd PR#81][comment] Please see the commit message, the fix is hopefully simple

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/81 Title: #81: Please see the commit message, the fix is hopefully simple jhrozek commented: """ Thanks for the review, I pushed a new version. I did test the patch (otherwise the conditional build of the files provider where I added AM_CON

[SSSD] [sssd PR#80][comment] SYSDB: Fixing of sudorule without a sudoUser

2016-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/80 Title: #80: SYSDB: Fixing of sudorule without a sudoUser jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/57/23/summary.html the rawhide failure is not related, it's a know bug in the latest python-urllib releas

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes jhrozek commented: """ On Wed, Nov 23, 2016 at 06:05:10AM -0800, lslebodn wrote: > @jhrozek Do you pan to review the patch? Feel free to take over the review, I'm currently busy with other w

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ For some reason the downstream tests are stuck and time out, even with known-good packages. I will keep trying but for downstream's sake I&

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ regardless of what we choose, the patch for PR #80 does not apply atop this patch, can we have a version that applies to the 1.13 branch, pleas

[SSSD] [sssd PR#39][+Changes requested] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@l

[SSSD] [sssd PR#39][-Accepted] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedo

[SSSD] [sssd PR#75][+Changes requested] Add configuirable max payload size limit of a secret

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/75 Title: #75: Add configuirable max payload size limit of a secret Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists

[SSSD] [sssd PR#75][comment] Add configuirable max payload size limit of a secret

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/75 Title: #75: Add configuirable max payload size limit of a secret jhrozek commented: """ Apart from the nitpick, I don't have any other comments. Unless someone else has, I can even squash the nitpick and just push the patches,

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ ack, this version works for me """ See the full comment at https://github.com/SSSD/sssd/

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ (CI pending) """ See the full comment at https://github.com/SSSD/sssd/pull/39#issuecomment-262709772

[SSSD] [sssd PR#39][+Accepted] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedo

[SSSD] [sssd PR#75][comment] Add configuirable max payload size limit of a secret

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/75 Title: #75: Add configuirable max payload size limit of a secret jhrozek commented: """ ACK, I'll push when CI finishes """ See the full comment at https://github.com/SSSD/

[SSSD] [sssd PR#75][+Accepted] Add configuirable max payload size limit of a secret

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/75 Title: #75: Add configuirable max payload size limit of a secret Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedoraho

[SSSD] [sssd PR#75][comment] Add configuirable max payload size limit of a secret

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/75 Title: #75: Add configuirable max payload size limit of a secret jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/57/36/summary.html the failure on rawhide is unrelated. """ See the full comment at https:

[SSSD] [sssd PR#75][closed] Add configuirable max payload size limit of a secret

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/75 Author: fidencio Title: #75: Add configuirable max payload size limit of a secret Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/75/head:pr75 git checkout pr75 _

[SSSD] [sssd PR#75][comment] Add configuirable max payload size limit of a secret

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/75 Title: #75: Add configuirable max payload size limit of a secret jhrozek commented: """ master: 9ba53ac52b12e2b9fdd3981d28ae56a8fece0131 7171a7584dda534dde5409f3e7f4657e845ece15 """ See the full comment at https:

[SSSD] [sssd PR#75][+Pushed] Add configuirable max payload size limit of a secret

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/75 Title: #75: Add configuirable max payload size limit of a secret Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahost

[SSSD] [sssd PR#39][-Accepted] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedo

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/57/35/summary.html sssd-1-13: b6d0b0a14c7f09371cbb2afd0347f6a16fcfc8dd """ See the full

[SSSD] [sssd PR#39][+Pushed] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedora

[SSSD] [sssd PR#39][closed] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/39 Author: celestian Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/39/head:pr39 git checkout pr39

[SSSD] [sssd PR#86][closed] SYSDB: check aliases before using them (sssd-1-13 only)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/86 Author: jhrozek Title: #86: SYSDB: check aliases before using them (sssd-1-13 only) Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/86/head:pr86 git checkout pr86

[SSSD] [sssd PR#86][comment] SYSDB: check aliases before using them (sssd-1-13 only)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/86 Title: #86: SYSDB: check aliases before using them (sssd-1-13 only) jhrozek commented: """ apparently I can't use the hub tool, sorry """ See the full comment at https://github.com/SSSD/

[SSSD] [sssd PR#87][opened] SYSDB: Only process aliases if they are non-NULL

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/87 Author: jhrozek Title: #87: SYSDB: Only process aliases if they are non-NULL Action: opened PR body: """ This bug was found by downstream coverity check. The aliases array is created by sysdb_get_sudo_user_info() which returns a NULL

[SSSD] [sssd PR#87][edited] SYSDB: Only process aliases if they are non-NULL (sssd-1-13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/87 Author: jhrozek Title: #87: SYSDB: Only process aliases if they are non-NULL (sssd-1-13) Action: edited Changed field: title Original value: """ SYSDB: Only process aliases if th

[SSSD] [sssd PR#87][closed] SYSDB: Only process aliases if they are non-NULL (sssd-1-13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/87 Author: jhrozek Title: #87: SYSDB: Only process aliases if they are non-NULL (sssd-1-13) Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/87/head:pr87 git checkout pr87

[SSSD] [sssd PR#87][+Pushed] SYSDB: Only process aliases if they are non-NULL (sssd-1-13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/87 Title: #87: SYSDB: Only process aliases if they are non-NULL (sssd-1-13) Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fe

[SSSD] [sssd PR#87][comment] SYSDB: Only process aliases if they are non-NULL (sssd-1-13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/87 Title: #87: SYSDB: Only process aliases if they are non-NULL (sssd-1-13) jhrozek commented: """ * sssd-1-13: 6ec33f5acfba6fa266c1dd7722c0c7bb88333f7f """ See the full comment at https://github.com/SSSD/sssd/

[SSSD] [sssd PR#87][-Accepted] SYSDB: Only process aliases if they are non-NULL (sssd-1-13)

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/87 Title: #87: SYSDB: Only process aliases if they are non-NULL (sssd-1-13) Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.

[SSSD] [sssd PR#93][opened] SSH: Use default_domain_suffix for users' authorized keys

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Author: jhrozek Title: #93: SSH: Use default_domain_suffix for users' authorized keys Action: opened PR body: """ None """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git

[SSSD] [sssd PR#93][edited] SSH: Use default_domain_suffix for users' authorized keys

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Author: jhrozek Title: #93: SSH: Use default_domain_suffix for users' authorized keys Action: edited ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#93][edited] SSH: Use default_domain_suffix for users' authorized keys

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Author: jhrozek Title: #93: SSH: Use default_domain_suffix for users' authorized keys Action: edited Changed field: body Original value: """ In commit eeecc48 we disabled default_domain_suffix for the SSH responder, but in a wron

[SSSD] [sssd PR#93][synchronized] SSH: Use default_domain_suffix for users' authorized keys

2016-11-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Author: jhrozek Title: #93: SSH: Use default_domain_suffix for users' authorized keys Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/93/head:pr93 git checkout pr93

[SSSD] [sssd PR#95][opened] STAP: Check the number of transactions in the stap aggregate before dereferencing it.

2016-11-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/95 Author: jhrozek Title: #95: STAP: Check the number of transactions in the stap aggregate before dereferencing it. Action: opened PR body: """ None """ To pull the PR as Git branch: git remote add ghsssd https://github.

[SSSD] [sssd PR#93][closed] SSH: Use default_domain_suffix for users' authorized keys

2016-12-06 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Author: jhrozek Title: #93: SSH: Use default_domain_suffix for users' authorized keys Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/93/head:pr93 git checkout

[SSSD] [sssd PR#93][+Pushed] SSH: Use default_domain_suffix for users' authorized keys

2016-12-06 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Title: #93: SSH: Use default_domain_suffix for users' authorized keys Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedor

[SSSD] [sssd PR#93][-Accepted] SSH: Use default_domain_suffix for users' authorized keys

2016-12-06 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Title: #93: SSH: Use default_domain_suffix for users' authorized keys Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fed

[SSSD] [sssd PR#93][comment] SSH: Use default_domain_suffix for users' authorized keys

2016-12-06 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Title: #93: SSH: Use default_domain_suffix for users' authorized keys jhrozek commented: """ master: ed71fba97dfcf5b3f0f1834c06660c481b9ab3ce sssd-1-14: 2949fe58ac344c44d756ca309d4b2b7f3590cee3 """ See the full

[SSSD] [sssd PR#93][comment] SSH: Use default_domain_suffix for users' authorized keys

2016-12-06 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/93 Title: #93: SSH: Use default_domain_suffix for users' authorized keys jhrozek commented: """ On Mon, Nov 28, 2016 at 05:06:54AM -0800, Pavel Březina wrote: > Can you also prepare a patch to handle this inside cache_req? Ideally o

[SSSD] [sssd PR#104][opened] Suppress a confusing error message from timestamp cache updates

2016-12-07 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/104 Author: jhrozek Title: #104: Suppress a confusing error message from timestamp cache updates Action: opened PR body: """ 0 is not an error, let's use a second variable instead to track the timestamp update retval and only emit a

<    1   2   3   4   5   6   7   8   9   10   >