[SSSD] [sssd PR#237][comment] providers: Move hostid from ipa to sdap

2017-04-20 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/237 Title: #237: providers: Move hostid from ipa to sdap lslebodn commented: """ On (20/04/17 13:20), Hristo Venev wrote: >UUIDs are exposed in LDAP for users and groups. They are not exposed. They are just available in ldap provider. But do not have defau

[SSSD] [sssd PR#237][comment] providers: Move hostid from ipa to sdap

2017-04-20 Thread hvenev
URL: https://github.com/SSSD/sssd/pull/237 Title: #237: providers: Move hostid from ipa to sdap hvenev commented: """ UUIDs are exposed in LDAP for users and groups. Not exposing `memberOf` would require having completely different attribute maps in LDAP and IPA. I don't think this can be done

[SSSD] [sssd PR#237][comment] providers: Move hostid from ipa to sdap

2017-04-20 Thread hvenev
URL: https://github.com/SSSD/sssd/pull/237 Title: #237: providers: Move hostid from ipa to sdap hvenev commented: """ UUIDs are exposed in LDAP for users and groups. Not exposing `memberOf` would require having completely different attribute maps in LDAP and IPA. I don't think this can be done

[SSSD] [sssd PR#237][comment] providers: Move hostid from ipa to sdap

2017-04-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/237 Title: #237: providers: Move hostid from ipa to sdap jhrozek commented: """ Hi, I'm sorry the review takes so long. We're swamped with fixing bugs at the moment and the Easter holidays didn't help either. I think since the patch moves quite a bit of

[SSSD] [sssd PR#233][-Changes requested] SDAP: Fix handling of search bases

2017-04-20 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/233 Title: #233: SDAP: Fix handling of search bases Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.or

[SSSD] [sssd PR#233][comment] SDAP: Fix handling of search bases

2017-04-20 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/233 Title: #233: SDAP: Fix handling of search bases mzidek-rh commented: """ Updated patch pushed. """ See the full comment at https://github.com/SSSD/sssd/pull/233#issuecomment-295791320 ___ sssd-devel mailing

[SSSD] [sssd PR#233][synchronized] SDAP: Fix handling of search bases

2017-04-20 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/233 Author: mzidek-rh Title: #233: SDAP: Fix handling of search bases Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/233/head:pr233 git checkout pr233 From e975a673b812a2f7e4e523

[SSSD] [sssd PR#233][+Changes requested] SDAP: Fix handling of search bases

2017-04-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/233 Title: #233: SDAP: Fix handling of search bases Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.or

[SSSD] [sssd PR#233][comment] SDAP: Fix handling of search bases

2017-04-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/233 Title: #233: SDAP: Fix handling of search bases jhrozek commented: """ nack, I'm getting a valgrind error with this patch: ``` ==31975== Conditional jump or move depends on uninitialised value(s) ==31975==at 0xDB1543E: ad_set_search_bases (ad_common

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

2017-04-20 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/136 Title: #136: Tlog integration spbnick commented: """ Pavel, after your suggestion on IRC I replaced `sysdb_initgroups` with `sysdb_initgroups_with_views` in the data provider, and also switched to using `sss_view_ldb_msg_find_attr_as_uint64` and `sysd

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

2017-04-20 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/136 Title: #136: Tlog integration spbnick commented: """ Pavel, after your suggestion on IRC I replaced `sysdb_initgroups` with `sysdb_initgroups_with_views` in the data provider, and also switched to using `sss_view_ldb_msg_find_attr_as_uint64` and `sysd

[SSSD] [sssd PR#236][+Pushed] README: Update links to mailing lists

2017-04-20 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/236 Title: #236: README: Update links to mailing lists 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#236][closed] README: Update links to mailing lists

2017-04-20 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/236 Author: lslebodn Title: #236: README: Update links to mailing lists Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/236/head:pr236 git checkout pr236 ___

[SSSD] [sssd PR#236][comment] README: Update links to mailing lists

2017-04-20 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/236 Title: #236: README: Update links to mailing lists lslebodn commented: """ On (20/04/17 05:10), Pavel Březina wrote: >Ack. > master: * 363e4c407085ea5623850b1dadb1344f2edd3c34 LS """ See the full comment at https://github.com/SSSD/sssd/pull/236#issu

[SSSD] [sssd PR#237][comment] providers: Move hostid from ipa to sdap

2017-04-20 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/237 Title: #237: providers: Move hostid from ipa to sdap pbrezina commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/237#issuecomment-295709734 ___ sssd-devel mailing list --

[SSSD] [sssd PR#236][comment] README: Update links to mailing lists

2017-04-20 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/236 Title: #236: README: Update links to mailing lists pbrezina commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/236#issuecomment-295710621 ___ sssd-devel mailing list -- sssd-de

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

2017-04-20 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/240 Title: #240: IFP: Fix of names in GetUserGroups method pbrezina commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/240#issuecomment-295650831 ___ sssd-devel mailing list -- sss

[SSSD] [sssd PR#238][comment] SECRESTS: remove unused variable

2017-04-20 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/238 Title: #238: SECRESTS: remove unused variable pbrezina commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/238#issuecomment-295627197 ___ sssd-devel mailing list -- sssd-devel@l