[SSSD] [sssd PR#514][synchronized] Backport several UPN related patches to sssd-1-13

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/514 Author: jhrozek Title: #514: Backport several UPN related patches to sssd-1-13 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/514/head:pr514 git checkout pr514 From

[SSSD] [sssd PR#514][comment] Backport several UPN related patches to sssd-1-13

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/514 Title: #514: Backport several UPN related patches to sssd-1-13 jhrozek commented: """ It doesn't make much sense to backport commit afadeb1 because it fixes an issue that was only uncovered by the way cache-req evaluates results. It looks like the

[SSSD] [sssd PR#514][-Changes requested] Backport several UPN related patches to sssd-1-13

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/514 Title: #514: Backport several UPN related patches to sssd-1-13 Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#519][comment] DEBUG: Print simple access provider allow and deny lists

2018-02-22 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/519 Title: #519: DEBUG: Print simple access provider allow and deny lists pbrezina commented: """ I think using highest debug level `SSSDBG_TRACE_ALL` would be better so we do not swamp the logs with information easily obtainable from a configuration

[SSSD] [sssd PR#496][comment] sysdb: sanitize search filter input - backport sssd-1-13

2018-02-22 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/496 Title: #496: sysdb: sanitize search filter input - backport sssd-1-13 sumit-bose commented: """ @lslebodn, thank you, patches are looking good. ACK. Do you think that for completeness and easier reference it would be ok if I push your patches to this

[SSSD] [sssd PR#521][comment] Avoid double semicolon warnings on older compilers

2018-02-22 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/521 Title: #521: Avoid double semicolon warnings on older compilers pbrezina commented: """ Ack. Thank you Andreas. """ See the full comment at https://github.com/SSSD/sssd/pull/521#issuecomment-367648886 ___

[SSSD] [sssd PR#521][+Accepted] Avoid double semicolon warnings on older compilers

2018-02-22 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/521 Title: #521: Avoid double semicolon warnings on older compilers Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#516][+Pushed] DESKPROFILE: Document it doesn't work when run as unprivileged user

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/516 Title: #516: DESKPROFILE: Document it doesn't work when run as unprivileged user Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#496][comment] sysdb: sanitize search filter input - backport sssd-1-13

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/496 Title: #496: sysdb: sanitize search filter input - backport sssd-1-13 lslebodn commented: """ master: * 3cf0aa1904fe791eed414bfb532c91a79ea2df9c * 0c0b9a3323dfc7988a58d4b67e9c94d0567ee82c * 76b79f5a816152d5aa07618b7825fb4926d15f1e *

[SSSD] [sssd PR#442][+Pushed] LDAP: Improve error treatment from sdap_cli_connect() in ldap_auth

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/442 Title: #442: LDAP: Improve error treatment from sdap_cli_connect() in ldap_auth Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#521][comment] Avoid double semicolon warnings on older compilers

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/521 Title: #521: Avoid double semicolon warnings on older compilers lslebodn commented: """ master: * 1ef36a878ac0f55c97e4d1c5597ac9cc1a52613a """ See the full comment at https://github.com/SSSD/sssd/pull/521#issuecomment-367676950

[SSSD] [sssd PR#521][+Pushed] Avoid double semicolon warnings on older compilers

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/521 Title: #521: Avoid double semicolon warnings on older compilers Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#521][comment] Avoid double semicolon warnings on older compilers

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/521 Title: #521: Avoid double semicolon warnings on older compilers lslebodn commented: """ comment would not work; because it would bot be expanded by macro. ACK++ """ See the full comment at https://github.com/SSSD/sssd/pull/521#issuecomment-367676908

[SSSD] [sssd PR#521][closed] Avoid double semicolon warnings on older compilers

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/521 Author: cryptomilk Title: #521: Avoid double semicolon warnings on older compilers Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/521/head:pr521 git checkout pr521

[SSSD] [sssd PR#516][closed] DESKPROFILE: Document it doesn't work when run as unprivileged user

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/516 Author: fidencio Title: #516: DESKPROFILE: Document it doesn't work when run as unprivileged user Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/516/head:pr516 git checkout pr516

[SSSD] [sssd PR#516][comment] DESKPROFILE: Document it doesn't work when run as unprivileged user

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/516 Title: #516: DESKPROFILE: Document it doesn't work when run as unprivileged user lslebodn commented: """ master: * 5150fbdc429ee9ed52e80e5f133ddb48c5b79914 """ See the full comment at https://github.com/SSSD/sssd/pull/516#issuecomment-367678121

[SSSD] [sssd PR#496][closed] sysdb: sanitize search filter input - backport sssd-1-13

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/496 Author: sumit-bose Title: #496: sysdb: sanitize search filter input - backport 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/496/head:pr496 git checkout pr496

[SSSD] [sssd PR#496][+Pushed] sysdb: sanitize search filter input - backport sssd-1-13

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/496 Title: #496: sysdb: sanitize search filter input - backport sssd-1-13 Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#428][+Pushed] CI: Ignore source file generated by systemtap in coverage

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/428 Title: #428: CI: Ignore source file generated by systemtap in coverage Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#496][comment] sysdb: sanitize search filter input - backport sssd-1-13

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/496 Title: #496: sysdb: sanitize search filter input - backport sssd-1-13 lslebodn commented: """ > Do you think that for completeness and easier reference it would be ok if I > push your patches to this PR and the 1.14 ones to PR#497? Feel free to push

[SSSD] [sssd PR#496][synchronized] sysdb: sanitize search filter input - backport sssd-1-13

2018-02-22 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/496 Author: sumit-bose Title: #496: sysdb: sanitize search filter input - backport sssd-1-13 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/496/head:pr496 git checkout pr496

[SSSD] [sssd PR#515][+Changes requested] sssctl: Showing help even when sssd not configured

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/515 Title: #515: sssctl: Showing help even when sssd not configured Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#516][+Accepted] DESKPROFILE: Document it doesn't work when run as unprivileged user

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/516 Title: #516: DESKPROFILE: Document it doesn't work when run as unprivileged user Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#496][comment] sysdb: sanitize search filter input - backport sssd-1-13

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/496 Title: #496: sysdb: sanitize search filter input - backport sssd-1-13 lslebodn commented: """ LGTM :-) I'll wait with pushing till after successful run in centos ci """ See the full comment at

[SSSD] [sssd PR#410][comment] IPA: sanitize name in override search filter - Backport to SSSD-1.13

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/410 Title: #410: IPA: sanitize name in override search filter - Backport to SSSD-1.13 jhrozek commented: """ I haven't looked at the code at all, but judging by the previous message, the state of this PR should be "Changes Requested". """ See the full

[SSSD] [sssd PR#515][comment] sssctl: Showing help even when sssd not configured

2018-02-22 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/515 Title: #515: sssctl: Showing help even when sssd not configured pbrezina commented: """ Well, currently I see some one kind of interesting code issue there that should be fixed as well. Jakub did first this change: ```c 284937e6 (Pavel Březina

[SSSD] [sssd PR#516][comment] DESKPROFILE: Document it doesn't work when run as unprivileged user

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/516 Title: #516: DESKPROFILE: Document it doesn't work when run as unprivileged user jhrozek commented: """ I think it's OK to add the Accepted label since the centos CI is passing now. """ See the full comment at

[SSSD] [sssd PR#521][comment] Avoid double semicolon warnings on older compilers

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/521 Title: #521: Avoid double semicolon warnings on older compilers lslebodn commented: """ Andreas created PR also on pagure and added comment there https://pagure.io/SSSD/sssd/pull-request/3645 Adding also here ``` -

[SSSD] [sssd PR#519][comment] DEBUG: Print simple access provider allow and deny lists

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/519 Title: #519: DEBUG: Print simple access provider allow and deny lists jhrozek commented: """ btw for debugging purposes I wonder if it was also a good idea to read and print the lists in `sssm_simple_access_init`. I think confdb already contains the

[SSSD] [sssd PR#519][+Changes requested] DEBUG: Print simple access provider allow and deny lists

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/519 Title: #519: DEBUG: Print simple access provider allow and deny lists Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#496][+Accepted] sysdb: sanitize search filter input - backport sssd-1-13

2018-02-22 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/496 Title: #496: sysdb: sanitize search filter input - backport sssd-1-13 Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#509][closed] SPEC: Fix systemd executions/requirements

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/509 Author: lslebodn Title: #509: SPEC: Fix systemd executions/requirements Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/509/head:pr509 git checkout pr509

[SSSD] [sssd PR#509][comment] SPEC: Fix systemd executions/requirements

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/509 Title: #509: SPEC: Fix systemd executions/requirements lslebodn commented: """ master: * 274ee2952fcb521222fe1eb86331c6dab505dc3a """ See the full comment at https://github.com/SSSD/sssd/pull/509#issuecomment-367754826

[SSSD] [sssd PR#509][+Pushed] SPEC: Fix systemd executions/requirements

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/509 Title: #509: SPEC: Fix systemd executions/requirements 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#513][+Accepted] SPEC: Reduce changes between upstream and downstream

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/513 Title: #513: SPEC: Reduce changes between upstream and downstream Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#513][comment] SPEC: Reduce changes between upstream and downstream

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/513 Title: #513: SPEC: Reduce changes between upstream and downstream jhrozek commented: """ CI: http://vm-031.ABC/logs/job/85/04/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/513#issuecomment-367781981

[SSSD] [sssd PR#509][+Accepted] SPEC: Fix systemd executions/requirements

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/509 Title: #509: SPEC: Fix systemd executions/requirements 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#391][+Accepted] Use dbus-daemon in cwrap enviroment for test

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/391 Title: #391: Use dbus-daemon in cwrap enviroment for test Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#391][comment] Use dbus-daemon in cwrap enviroment for test

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/391 Title: #391: Use dbus-daemon in cwrap enviroment for test jhrozek commented: """ CI: http://vm-031.$ABC/logs/job/85/06/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/391#issuecomment-367782486

[SSSD] [sssd PR#509][comment] SPEC: Fix systemd executions/requirements

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/509 Title: #509: SPEC: Fix systemd executions/requirements jhrozek commented: """ CI passed, ACK """ See the full comment at https://github.com/SSSD/sssd/pull/509#issuecomment-367748990 ___ sssd-devel mailing

[SSSD] [sssd PR#514][comment] Backport several UPN related patches to sssd-1-13

2018-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/514 Title: #514: Backport several UPN related patches to sssd-1-13 jhrozek commented: """ Hi @lslebodn could you take a look at the patches no later than Monday? I'm on PTO the rest of the next week and I'd like the patches to be merged downstream

[SSSD] [sssd PR#513][comment] SPEC: Reduce changes between upstream and downstream

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/513 Title: #513: SPEC: Reduce changes between upstream and downstream lslebodn commented: """ master: * 65afba536e4535b1381ae5468edb16b3c9b89578 """ See the full comment at https://github.com/SSSD/sssd/pull/513#issuecomment-367799871

[SSSD] [sssd PR#513][closed] SPEC: Reduce changes between upstream and downstream

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/513 Author: lslebodn Title: #513: SPEC: Reduce changes between upstream and downstream Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/513/head:pr513 git checkout pr513

[SSSD] [sssd PR#513][+Pushed] SPEC: Reduce changes between upstream and downstream

2018-02-22 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/513 Title: #513: SPEC: Reduce changes between upstream and downstream Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#522][opened] Prepare SSSD to support IPA in trust to Samba AD

2018-02-22 Thread abbra
URL: https://github.com/SSSD/sssd/pull/522 Author: abbra Title: #522: Prepare SSSD to support IPA in trust to Samba AD Action: opened PR body: """ This pull request prepares SSSD ipa provider to support IPA in trust to Samba AD but the same changes are needed for a properly working

[SSSD] [sssd PR#522][comment] Prepare SSSD to support IPA in trust to Samba AD

2018-02-22 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/522 Title: #522: Prepare SSSD to support IPA in trust to Samba AD centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/522#issuecomment-367925811

[SSSD] [sssd PR#522][comment] Prepare SSSD to support IPA in trust to Samba AD

2018-02-22 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/522 Title: #522: Prepare SSSD to support IPA in trust to Samba AD centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/522#issuecomment-367925813