[SSSD] [sssd PR#198][comment] secrets: support https in proxy provider

2017-03-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/198 Title: #198: secrets: support https in proxy provider lslebodn commented: """ Build failed on rehl6 and rhel7 http://sssd-ci.duckdns.org/logs/job/65/71/summary.html rhel7 ``` make-tests /var/lib/jenkins/workspace/ci/label/rhel7/src/tests/tcurl_test_too

[SSSD] [sssd PR#198][+Changes requested] secrets: support https in proxy provider

2017-03-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/198 Title: #198: secrets: support https in proxy provider Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahos

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ Okay, new series pushed based on @pbrezina review. I've moved the code that made sense to me to cache_req_domain.[ch]. Not all of cod

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ @pbrezina, @sumit-bose, As this PR wasn't pushed yet I'm just updating it with the patches for domainResolutionOrder as well. For te

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ @pbrezina, @sumit-bose, As this PR wasn't pushed yet I'm just updating it with the patches for domainResolutionOrder as well. For te

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

2017-03-27 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/136 Author: spbnick Title: #136: Tlog integration Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/136/head:pr136 git checkout pr136 From 871c2b596de3ae5238750ba6bd4a008a40e98138 M

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

2017-03-27 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/136 Title: #136: Tlog integration spbnick commented: """ Alright. Aside from `dp_req_initgr_pp` not letting us report a failure, I think this is close to what we need, and is ready for a proper review. """ See the full comment at https://github.com/SSSD/

[SSSD] [sssd PR#136][edited] Tlog integration WIP

2017-03-27 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/136 Author: spbnick Title: #136: Tlog integration WIP Action: edited Changed field: title Original value: """ Tlog integration WIP """ ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubs

[SSSD] [sssd PR#210][comment] Kcm configure

2017-03-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/210 Title: #210: Kcm configure lslebodn commented: """ I dropped 2nd patch for libcurl error message because it is changed in #198 """ See the full comment at https://github.com/SSSD/sssd/pull/210#issuecomment-289473136 __

[SSSD] [sssd PR#210][synchronized] Kcm configure

2017-03-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/210 Author: lslebodn Title: #210: Kcm configure Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/210/head:pr210 git checkout pr210 From 23fbd75966081def5322ef71154b419b45d908ff Mon

[SSSD] [sssd PR#198][-Changes requested] secrets: support https in proxy provider

2017-03-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/198 Title: #198: secrets: support https in proxy provider Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahos

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ On Mon, Mar 27, 2017 at 2:25 PM, Pavel Březina wrote: > Please, put the cache req domain stuff into a new cache req module instead >

[SSSD] [sssd PR#198][comment] secrets: support https in proxy provider

2017-03-27 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/198 Title: #198: secrets: support https in proxy provider pbrezina commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/198#issuecomment-289439821 ___ sssd-devel mailin

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-27 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname pbrezina commented: """ Please, put the cache req domain stuff into a new cache req module instead of mixing it with responder common. Patches looks good othe

[SSSD] [sssd PR#198][comment] secrets: support https in proxy provider

2017-03-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/198 Title: #198: secrets: support https in proxy provider lslebodn commented: """ Just for the record. Crash was caused by setting `long` value into variable `response_code` which had type `int`. And variables `req` and `response_code` were on the stack n

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ Ah! Please, set SELinux to permissive in case you're updating freeipa-server! """ See the full comment at https://github.com/SSSD/sss

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ @pbrezina, @sumit-bose, As this PR wasn't pushed yet I'm just updating it with the patches for domainResolutionOrder as well. For te

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ retest please """ See the full comment at https://github.com/SSSD/sssd/pull/187#issuecomment-289415727 __

[SSSD] [sssd PR#210][opened] Kcm configure

2017-03-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/210 Author: lslebodn Title: #210: Kcm configure Action: opened PR body: """ I am not sure about 2nd patch "CONFIGURE: Update error messages for missing curl" because IIRC secrets responder will require curl as well. """ To pull the PR as Git branch: git

[SSSD] [sssd PR#197][closed] KCM responder

2017-03-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/197 Author: jhrozek Title: #197: KCM responder Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/197/head:pr197 git checkout pr197 ___ sssd-dev

[SSSD] [sssd PR#197][+Pushed] KCM responder

2017-03-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder 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#197][comment] KCM responder

2017-03-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder jhrozek commented: """ Pushed to master: * e89ba95737202d551db2c9524127e6c4cf308796 * 2b5518eeaacc6245cfa77ee4a7086f16208060fc * 35c9dfe9ba78d3a635cd1af0fb6349ba44344623 * cac0db2f8004ae88b9263dc3888a11a2d3d3d114 * c9db8b