[SSSD] [sssd PR#202][comment] T3315 infopipe group users master

2017-07-27 Thread celestian
URL: https://github.com/SSSD/sssd/pull/202 Title: #202: T3315 infopipe group users master celestian commented: """ The issue was that getent shows user test_user in test_group, but dbus call doesn't. How I did it is described in my description. But I don't know if i

[SSSD] [sssd PR#214][comment] UTIL: Set udp_preference_limit=0 in krb5 snippet

2017-07-27 Thread celestian
URL: https://github.com/SSSD/sssd/pull/214 Title: #214: UTIL: Set udp_preference_limit=0 in krb5 snippet celestian commented: """ @fidencio Oh, I see -- now I understand what are you looking for. Maybe it is little confusing, there on github, that it is not visible that my pa

[SSSD] [sssd PR#214][comment] UTIL: Set udp_preference_limit=0 in krb5 snippet

2017-07-27 Thread celestian
URL: https://github.com/SSSD/sssd/pull/214 Title: #214: UTIL: Set udp_preference_limit=0 in krb5 snippet celestian commented: """ @fidencio I am totally out of scope of this PR. I just assume that I addressed @frozencemetery 's comment from Mar 28. The conditional set

[SSSD] [sssd PR#206][comment] IFP: Fix of names in GetUserGroups method

2017-04-12 Thread celestian
URL: https://github.com/SSSD/sssd/pull/206 Title: #206: IFP: Fix of names in GetUserGroups method celestian commented: """ Bump. """ See the full comment at https://github.com/SSSD/sssd/pull/206#issuecomment-293582645 _

[SSSD] [sssd PR#202][comment] T3315 infopipe group users master

2017-04-03 Thread celestian
URL: https://github.com/SSSD/sssd/pull/202 Title: #202: T3315 infopipe group users master celestian commented: """ IMO, this patch set fix method org.freedesktop.sssd.infopipe.Groups.Group.UpdateMemberList() which you need call if you would like to see members of group. &q

[SSSD] [sssd PR#208][comment] IFP: Filter with * in Users.ListByName method

2017-04-03 Thread celestian
URL: https://github.com/SSSD/sssd/pull/208 Title: #208: IFP: Filter with * in Users.ListByName method celestian commented: """ CI passed: http://sssd-ci.duckdns.org/logs/job/67/31/summary.html """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#208][comment] IFP: Filter with * in Users.ListByName method

2017-04-03 Thread celestian
URL: https://github.com/SSSD/sssd/pull/208 Title: #208: IFP: Filter with * in Users.ListByName method celestian commented: """ Yes. It is rebased now. This PR has to go before #211 IFP: Fix of limit = 0 (unlimited result). """ See the full comment at https:/

[SSSD] [sssd PR#208][synchronized] IFP: Filter with * in Users.ListByName method

2017-04-03 Thread celestian
URL: https://github.com/SSSD/sssd/pull/208 Author: celestian Title: #208: IFP: Filter with * in Users.ListByName method Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/208/head:pr208 git checkout pr208 From

[SSSD] [sssd PR#218][comment] TEST: Adding krb5-libs to dependencies

2017-03-31 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Title: #218: TEST: Adding krb5-libs to dependencies celestian commented: """ @lslebodn thanks. I think I see it now. New patch pushed. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#218][synchronized] TEST: Adding krb5-libs to dependencies

2017-03-31 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Author: celestian Title: #218: TEST: Adding krb5-libs to dependencies Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/218/head:pr218 git checkout pr218 From

[SSSD] [sssd PR#218][comment] TEST: Adding krb5-libs to dependencies

2017-03-30 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Title: #218: TEST: Adding krb5-libs to dependencies celestian commented: """ I see ``` install-deps: success 00:01:07 ci-install-deps.log autoreconf: success 00:00:34 ci-autoreconf.

[SSSD] [sssd PR#214][comment] UTIL: Set udp_preference_limit=0 in krb5 snippet

2017-03-30 Thread celestian
URL: https://github.com/SSSD/sssd/pull/214 Title: #214: UTIL: Set udp_preference_limit=0 in krb5 snippet celestian commented: """ I see ``` Warning: Permanently added '172.19.2.156' (ECDSA) to the list of known hosts. install-deps: success 00:01:07 ci-in

[SSSD] [sssd PR#214][comment] UTIL: Set udp_preference_limit=0 in krb5 snippet

2017-03-30 Thread celestian
URL: https://github.com/SSSD/sssd/pull/214 Title: #214: UTIL: Set udp_preference_limit=0 in krb5 snippet celestian commented: """ I see ``` Warning: Permanently added '172.19.2.156' (ECDSA) to the list of known hosts. install-deps: success 00:01:07 ci-in

[SSSD] [sssd PR#218][edited] TEST: Adding krb5-libs to dependencies

2017-03-30 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Author: celestian Title: #218: TEST: Adding krb5-libs to dependencies Action: edited Changed field: title Original value: """ TEST: Adding krb5-kdc to dependencies """ ___ s

[SSSD] [sssd PR#218][synchronized] TEST: Adding krb5-kdc to dependencies

2017-03-30 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Author: celestian Title: #218: TEST: Adding krb5-kdc to dependencies Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/218/head:pr218 git checkout pr218 From

[SSSD] [sssd PR#218][edited] TEST: Adding krb5-kdc to dependencies

2017-03-30 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Author: celestian Title: #218: TEST: Adding krb5-kdc to dependencies Action: edited Changed field: title Original value: """ TEST: Adding paython-requests to dependencies """ _

[SSSD] [sssd PR#218][synchronized] TEST: Adding paython-requests to dependencies

2017-03-30 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Author: celestian Title: #218: TEST: Adding paython-requests to dependencies Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/218/head:pr218 git checkout pr218 From

[SSSD] [sssd PR#218][opened] TEST: Adding paython-requests to dependencies

2017-03-30 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Author: celestian Title: #218: TEST: Adding paython-requests to dependencies Action: opened PR body: """ Resolves: https://pagure.io/SSSD/sssd/issue/3353 Note: I am not sure if this is the correct dependency which we were looking

[SSSD] [sssd PR#214][opened] UTIL: Set udp_preference_limit=0 in krb5 snippet

2017-03-28 Thread celestian
URL: https://github.com/SSSD/sssd/pull/214 Author: celestian Title: #214: UTIL: Set udp_preference_limit=0 in krb5 snippet Action: opened PR body: """ We add udp_preference_limit = 0 to krb5 snippet. This option enable TCP connection before UDP, when sending a message to the

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-03-28 Thread celestian
URL: https://github.com/SSSD/sssd/pull/211 Title: #211: IFP: Fix of limit = 0 (unlimited result) celestian commented: """ @lslebodn thanks, that's good point. I synced new version of the patch. """ See the full comment at https://github.com/SSSD/s

[SSSD] [sssd PR#211][synchronized] IFP: Fix of limit = 0 (unlimited result)

2017-03-28 Thread celestian
URL: https://github.com/SSSD/sssd/pull/211 Author: celestian Title: #211: IFP: Fix of limit = 0 (unlimited result) Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/211/head:pr211 git checkout pr211 From

[SSSD] [sssd PR#211][opened] IFP: Fix of limit = 0 (unlimited result)

2017-03-28 Thread celestian
URL: https://github.com/SSSD/sssd/pull/211 Author: celestian Title: #211: IFP: Fix of limit = 0 (unlimited result) Action: opened PR body: """ If we set limit to 0 it means that result is unlimited. Internally we restrict number of result by allocation of result array. In unlim

[SSSD] [sssd PR#208][comment] IFP: Filter with * in Users.ListByName method

2017-03-24 Thread celestian
URL: https://github.com/SSSD/sssd/pull/208 Title: #208: IFP: Filter with * in Users.ListByName method celestian commented: """ The patch updated. How to test (updated): ``` # We have sssd configured and there are some users # Wee need also ifp service enabled in sssd.conf

[SSSD] [sssd PR#208][synchronized] IFP: Filter with * in Users.ListByName method

2017-03-24 Thread celestian
URL: https://github.com/SSSD/sssd/pull/208 Author: celestian Title: #208: IFP: Filter with * in Users.ListByName method Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/208/head:pr208 git checkout pr208 From

[SSSD] [sssd PR#206][comment] IFP: Fix of names in GetUserGroups method

2017-03-24 Thread celestian
URL: https://github.com/SSSD/sssd/pull/206 Title: #206: IFP: Fix of names in GetUserGroups method celestian commented: """ Updated -- I just added "resolves" link to the commit message. """ See the full comment at https://github.com/S

[SSSD] [sssd PR#206][synchronized] IFP: Fix of names in GetUserGroups method

2017-03-24 Thread celestian
URL: https://github.com/SSSD/sssd/pull/206 Author: celestian Title: #206: IFP: Fix of names in GetUserGroups method Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/206/head:pr206 git checkout pr206 From

[SSSD] [sssd PR#208][opened] IFP: Filter with * in Users.ListByName method

2017-03-23 Thread celestian
URL: https://github.com/SSSD/sssd/pull/208 Author: celestian Title: #208: IFP: Filter with * in Users.ListByName method Action: opened PR body: """ This patch fixed asterisk in filter of Users.ListByName method. Resolves: https://pagure.io/SSSD/sssd/issue/3305 How to test ```

[SSSD] [sssd PR#206][comment] IFP: Fix of names in GetUserGroups method

2017-03-22 Thread celestian
URL: https://github.com/SSSD/sssd/pull/206 Title: #206: IFP: Fix of names in GetUserGroups method celestian commented: """ How to test: We need setup sssd as client of FreeIPA server with AD trust. And we need add user from AD domain to FreeIPA group (ask me if you need help

[SSSD] [sssd PR#206][opened] IFP: Fix of names in GetUserGroups method

2017-03-22 Thread celestian
URL: https://github.com/SSSD/sssd/pull/206 Author: celestian Title: #206: IFP: Fix of names in GetUserGroups method Action: opened PR body: """ This patch adds code which chooses the right domain for creation of output group's name. """ To pull the PR a

[SSSD] [sssd PR#202][opened] T3315 infopipe group users master

2017-03-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/202 Author: celestian Title: #202: T3315 infopipe group users master Action: opened PR body: """ Reproducer is: ``` # PREPARING ipa user-add --first=Test --last=User --email=u...@test-domain.sssd test_user ipa group-add test_group # REPRO

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing celestian commented: """ LGTM and I pushed it to CI test. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#153][-Changes requested] sss_cache: User/groups invalidation in domain cache

2017-03-07 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@list

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-03-07 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ New version uploaded. Thanks for review. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#153][synchronized] sss_cache: User/groups invalidation in domain cache

2017-03-07 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: sss_cache: User/groups invalidation in domain cache Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/153/head:pr153 git checkout pr153 From

[SSSD] [sssd PR#170][comment] PROXY: Remove duplicit users from group

2017-03-02 Thread celestian
URL: https://github.com/SSSD/sssd/pull/170 Title: #170: PROXY: Remove duplicit users from group celestian commented: """ So, @lslebodn and me looked at how to test this patch. Unfortunately we found out that proxy code uses ```nss_files_getgrnam_r``` which is not mocked by ``

[SSSD] [sssd PR#153][-Changes requested] sss_cache: User/groups invalidation in domain cache

2017-03-01 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@list

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-03-01 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ I updated the patch. 1. The issue with enum is addressed. 1. I added function ```sysdb_invalidate_cache_entry()``` to sysdb API and removed t

[SSSD] [sssd PR#153][synchronized] sss_cache: User/groups invalidation in domain cache

2017-03-01 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: sss_cache: User/groups invalidation in domain cache Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/153/head:pr153 git checkout pr153 From

[SSSD] [sssd PR#170][comment] PROXY: Remove duplicit users from group

2017-02-27 Thread celestian
URL: https://github.com/SSSD/sssd/pull/170 Title: #170: PROXY: Remove duplicit users from group celestian commented: """ There is the first version of patch. I would like to ask @jhrozek or @lslebodn if our cwrap tests has capability to test this patch. I need ```/etc/group```

[SSSD] [sssd PR#170][opened] PROXY: Remove duplicit users from group

2017-02-27 Thread celestian
URL: https://github.com/SSSD/sssd/pull/170 Author: celestian Title: #170: PROXY: Remove duplicit users from group Action: opened PR body: """ It is possible to have duplicit members in local files (/etc/group). This patch removes duplicity in groups in proxy provider.

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-02-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ New version pushed. CI passed: http://sssd-ci.duckdns.org/logs/job/63/02/summary.html """ See the full comment at https:/

[SSSD] [sssd PR#153][synchronized] sss_cache: User/groups invalidation in domain cache

2017-02-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: sss_cache: User/groups invalidation in domain cache Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/153/head:pr153 git checkout pr153 From

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-02-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ New version pushed, but CI said "NO": http://sssd-ci.duckdns.org/logs/job/62/98/summary.html """ See the full comment a

[SSSD] [sssd PR#153][synchronized] sss_cache: User/groups invalidation in domain cache

2017-02-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: sss_cache: User/groups invalidation in domain cache Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/153/head:pr153 git checkout pr153 From

[SSSD] [sssd PR#158][opened] IPA_SUDO: Unused value fix

2017-02-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/158 Author: celestian Title: #158: IPA_SUDO: Unused value fix Action: opened PR body: """ Unused value was immediately overwritten. Resolves: https://fedorahosted.org/sssd/ticket/3309 """ To pull the PR as Git branc

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-02-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ The are results from our CI http://sssd-ci.duckdns.org/logs/job/62/90/summary.html How we can see this patch fails on test_ts_cache.py integrati

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-02-15 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ Fixed patch is pushed. I sent it to our CI and I will share the result. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#153][synchronized] sss_cache: User/groups invalidation in domain cache

2017-02-15 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: sss_cache: User/groups invalidation in domain cache Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/153/head:pr153 git checkout pr153 From

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-02-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ So, dnf repositories work again. I am able to test the functionality of my patch set. Unfortunately it doesnt work. I will fix it. ""&

[SSSD] [sssd PR#153][+Changes requested] sss_cache: User/groups invalidation in domain cache

2017-02-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@list

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-02-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ FYI http://sssd-ci.duckdns.org/logs/job/62/59/summary.html """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-02-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ I pushed new version (#2). I addressed Lukáš's comment. Now it works only for sss_cache case. In detail, I added functions ``` sysdb_invalid

[SSSD] [sssd PR#153][comment] sss_cache: User/groups invalidation in domain cache

2017-02-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: sss_cache: User/groups invalidation in domain cache celestian commented: """ I pushed new version (#2). I addressed Lukáš's comment. Now it works only for sss_cache case. In detail, I added functions ``` sysdb_invalid

[SSSD] [sssd PR#153][edited] sss_cache: User/groups invalidation in domain cache

2017-02-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: sss_cache: User/groups invalidation in domain cache Action: edited Changed field: title Original value: """ SYSDB: Changing dataExpireTimestamp in domain cache """ __

[SSSD] [sssd PR#153][synchronized] SYSDB: Changing dataExpireTimestamp in domain cache

2017-02-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: SYSDB: Changing dataExpireTimestamp in domain cache Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/153/head:pr153 git checkout pr153 From

[SSSD] [sssd PR#153][comment] SYSDB: Changing dataExpireTimestamp in domain cache

2017-02-13 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Title: #153: SYSDB: Changing dataExpireTimestamp in domain cache celestian commented: """ Thanks for comments. In my opinion it would be better to have the same value of dataExpireTimestamp only if we use sss_cache. The question is whet

[SSSD] [sssd PR#153][opened] SYSDB: Changing dataExpireTimestamp in domain cache

2017-02-13 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: SYSDB: Changing dataExpireTimestamp in domain cache Action: opened PR body: """ When a group/users are invalidated from sss cache, the group/user information in Domain (cache_LDAP.ldb) and times

[SSSD] [sssd PR#153][synchronized] SYSDB: Changing dataExpireTimestamp in domain cache

2017-02-13 Thread celestian
URL: https://github.com/SSSD/sssd/pull/153 Author: celestian Title: #153: SYSDB: Changing dataExpireTimestamp in domain cache Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/153/head:pr153 git checkout pr153 From

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-02-01 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ bump """ See the full comment at https://github.com/SSSD/sssd/pull/85#issuecomment-276602905

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ New version is pushed. Thanks, @sumit-bose """ See the full comment at https://github.com/SSSD/sssd/

[SSSD] [sssd PR#85][-Changes requested] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.f

[SSSD] [sssd PR#85][synchronized] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Author: celestian Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/85/head:pr85 git checkout pr85 From

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-19 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ I pushed new version of the patch set. I addressed @sumit-bose notes, I hope in right manner. Unfortunately ```test_user_is_from_another_domain()`

[SSSD] [sssd PR#85][synchronized] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-19 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Author: celestian Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/85/head:pr85 git checkout pr85 From

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ I pushed new version. Let me update the situation: There are three commits: ``` [1] SYSDB: Removing of sysdb_try_to_find_expecte

[SSSD] [sssd PR#85][synchronized] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Author: celestian Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/85/head:pr85 git checkout pr85 From

[SSSD] [sssd PR#125][closed] RESPONDER: Adding of return value checking

2017-01-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/125 Author: celestian Title: #125: RESPONDER: Adding of return value checking Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/125/head:pr125 git checkout pr125

[SSSD] [sssd PR#125][comment] RESPONDER: Adding of return value checking

2017-01-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/125 Title: #125: RESPONDER: Adding of return value checking celestian commented: """ Oh yes, I came too late :( Closing this pull request. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#125][opened] RESPONDER: Adding of return value checking

2017-01-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/125 Author: celestian Title: #125: RESPONDER: Adding of return value checking Action: opened PR body: """ """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/125/head

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-13 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ There is new version if somebody would like to look how I fight. The positive test case ```test_user_is_on_batch``` is ready, the neg

[SSSD] [sssd PR#85][synchronized] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-13 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Author: celestian Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/85/head:pr85 git checkout pr85 From

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-10 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ Solved, thanks to @lslebodn . I will prepare new version. """ See the full comment at https://github.com/SSSD/sssd/

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-10 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ WIP of tests added. I have issue with proeprly setting up test environment. This call (at line 95): ``` test_ctx->initgr_state->opts = m

[SSSD] [sssd PR#85][synchronized] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2017-01-10 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Author: celestian Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/85/head:pr85 git checkout pr85 From

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

2016-11-28 Thread celestian
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-11-28 Thread celestian
URL: https://github.com/SSSD/sssd/pull/93 Title: #93: SSH: Use default_domain_suffix for users' authorized keys celestian commented: """ Code LGTM. CI: http://sssd-ci.duckdns.org/logs/job/57/87/summary.html There is a failure on test ```test_sanity_rfc2307``` -- I think it

[SSSD] [sssd PR#85][+Changes requested] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2016-11-28 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.f

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2016-11-28 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ So, I will rewrite tests for sysdb_try_to_find_expected_dn() to suitable form for sdap_object_in_domain(). """ See the full comme

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

2016-11-23 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ Squashed version pushed. """ See the full comment at https://github.com/SSSD/sssd/

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

2016-11-23 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Author: celestian Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Action: synchronized 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 From

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

2016-11-23 Thread celestian
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][-Changes requested] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-23 Thread celestian
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-23 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ I pushed new version. The patch is the same plus I added back-ported patch from #80 (with cerry-pick tag). """ See the full

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

2016-11-23 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Author: celestian Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Action: synchronized 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 From

[SSSD] [sssd PR#85][comment] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2016-11-23 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() celestian commented: """ Reproducer: We need AD domain and it's AD subdomain. If we type in SSSD box connected to AD domain: ``` id Administrator@ ``` it resolves betwee

[SSSD] [sssd PR#85][opened] SYSDB: Removing of sysdb_try_to_find_expected_dn()

2016-11-23 Thread celestian
URL: https://github.com/SSSD/sssd/pull/85 Author: celestian Title: #85: SYSDB: Removing of sysdb_try_to_find_expected_dn() Action: opened PR body: """ Currently in order to match multiple LDAP search results we use two different functions - we have sysdb_try_to_find_expecte

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

2016-11-23 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ Thanks for CR. After pushing it is important to cherry pick #80 as well. """ See the full comment at https://github.com/

[SSSD] [sssd PR#70][comment] check_duplicate: check name member before using it

2016-11-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/70 Title: #70: check_duplicate: check name member before using it celestian commented: """ @lslebodn, Lukas, are you satisfied by Sumit's explanation? """ See the full comment at https://github.com/SSSD/

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

2016-11-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ So, I pushed new version. Now ```sysdb_get_sudo_filter()``` uses ```nameAlias``` values. (And after pushing #80 I will cherry-pick it to 1.13

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

2016-11-21 Thread celestian
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][synchronized] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Author: celestian Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Action: synchronized 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 From

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

2016-11-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/80 Author: celestian Title: #80: SYSDB: Fixing of sudorule without a sudoUser Action: edited Changed field: title Original value: """ SYSDB: Sudorule without a sudoUser returns EINVAL """

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

2016-11-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/80 Title: #80: SYSDB: Sudorule without a sudoUser returns EINVAL celestian commented: """ New version pushed. """ See the full comment at https://github.com/SSSD/sssd/pull/80#issuecomment-261919172 __

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

2016-11-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/80 Author: celestian Title: #80: SYSDB: Sudorule without a sudoUser returns EINVAL Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/80/head:pr80 git checkout pr80 From

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

2016-11-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/80 Author: celestian Title: #80: SYSDB: Sudorule without a sudoUser returns EINVAL Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/80/head:pr80 git checkout pr80 From

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

2016-11-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/80 Author: celestian Title: #80: SYSDB: Sudorule without a sudoUser returns EINVAL Action: opened PR body: """ This patch solved a regression caused by the recent patches to lowercase sudoUser -- in case sudoUser is missing completel

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

2016-11-16 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ I am afraid there is the same issue as in https://fedorahosted.org/sssd/ticket/3241. The patch will be added soon. """

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

2016-11-16 Thread celestian
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][-Changes requested] RESPONDER: Enable sudoRule in case insen. domains (1.13)

2016-11-14 Thread celestian
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-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ We discussed this issue with @jhrozek. I misunderstood the case -- the right is -- user is ```Administrator```, the sudoRule is w

  1   2   3   >