[Freeipa-devel] [bind-dyndb-ldap PR#15][opened] ignore what configure and make produced

2017-04-06 Thread mingzym
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/15 Author: mingzym Title: #15: ignore what configure and make produced Action: opened PR body: """ let's ignore all the files configure and make created. """ To pull the PR as Git branch: git remote add ghbind-dyndb-ldap

[Freeipa-devel] [freeipa PR#694][comment] RFC: implement local PKINIT deployment in server/replica install

2017-04-06 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/694 Title: #694: RFC: implement local PKINIT deployment in server/replica install MartinBasti commented: """ Should be anon keytab removed by upgrade, are there any leftovers in LDAP to be removed during upgrade? """ See the full comment at

[Freeipa-devel] [freeipa PR#688][comment] Update get_attr_filter in LDAPSearch to handle nsaccountlock user searches

2017-04-06 Thread redhatrises
URL: https://github.com/freeipa/freeipa/pull/688 Title: #688: Update get_attr_filter in LDAPSearch to handle nsaccountlock user searches redhatrises commented: """ @HonzaCholasta ready for your review. """ See the full comment at

[Freeipa-devel] [freeipa PR#694][comment] RFC: implement local PKINIT deployment in server/replica install

2017-04-06 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/694 Title: #694: RFC: implement local PKINIT deployment in server/replica install martbab commented: """ I have re-worked the PR and implemented most of the missing steps (except for API for querying PKINIT status in topology). I have also removed

[Freeipa-devel] [freeipa PR#694][edited] RFC: implement local PKINIT deployment in server/replica install

2017-04-06 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/694 Author: martbab Title: #694: RFC: implement local PKINIT deployment in server/replica install Action: edited Changed field: body Original value: """ This PR implements a basic local PKINIT functionality for server install with '--no-pkinit'

[Freeipa-devel] [freeipa PR#694][edited] RFC: implement local PKINIT deployment in server/replica install

2017-04-06 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/694 Author: martbab Title: #694: RFC: implement local PKINIT deployment in server/replica install Action: edited Changed field: body Original value: """ This PR implements a basic local PKINIT functionality for server install with '--no-pkinit'

[Freeipa-devel] [freeipa PR#694][edited] RFC: implement local PKINIT deployment in server/replica install

2017-04-06 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/694 Author: martbab Title: #694: RFC: implement local PKINIT deployment in server/replica install Action: edited Changed field: body Original value: """ This PR implements a basic local PKINIT functionality for server install with '--no-pkinit'

[Freeipa-devel] [freeipa PR#694][synchronized] RFC: implement local PKINIT deployment in server/replica install

2017-04-06 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/694 Author: martbab Title: #694: RFC: implement local PKINIT deployment in server/replica install Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/694/head:pr694

[Freeipa-devel] [bind-dyndb-ldap PR#11][comment] Coverity: fix REVERSE_INULL for pevent->inst

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/11 Title: #11: Coverity: fix REVERSE_INULL for pevent->inst tomaskrizek commented: """ master: - 13b185182aeb48562cf63251b84bcf910b57a0fc """ See the full comment at https://github.com/freeipa/bind-dyndb-ldap/pull/11#issuecomment-292221521

[Freeipa-devel] [bind-dyndb-ldap PR#11][+pushed] Coverity: fix REVERSE_INULL for pevent->inst

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/11 Title: #11: Coverity: fix REVERSE_INULL for pevent->inst Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [bind-dyndb-ldap PR#11][closed] Coverity: fix REVERSE_INULL for pevent->inst

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/11 Author: tomaskrizek Title: #11: Coverity: fix REVERSE_INULL for pevent->inst Action: closed To pull the PR as Git branch: git remote add ghbind-dyndb-ldap https://github.com/freeipa/bind-dyndb-ldap git fetch ghbind-dyndb-ldap

[Freeipa-devel] [bind-dyndb-ldap PR#12][+pushed] README.md: fix markdown formatting

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/12 Title: #12: README.md: fix markdown formatting Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [bind-dyndb-ldap PR#12][comment] README.md: fix markdown formatting

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/12 Title: #12: README.md: fix markdown formatting tomaskrizek commented: """ master: - a9ffcc8eb190d5dc01e018abb9f8bba2013ab5e2 - 55c2ffc2b3aef3e9cf7c9131e40a1057b032527b """ See the full comment at

[Freeipa-devel] [bind-dyndb-ldap PR#11][+ack] Coverity: fix REVERSE_INULL for pevent->inst

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/11 Title: #11: Coverity: fix REVERSE_INULL for pevent->inst Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [bind-dyndb-ldap PR#12][closed] README.md: fix markdown formatting

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/12 Author: tomaskrizek Title: #12: README.md: fix markdown formatting Action: closed To pull the PR as Git branch: git remote add ghbind-dyndb-ldap https://github.com/freeipa/bind-dyndb-ldap git fetch ghbind-dyndb-ldap pull/12/head:pr12 git

[Freeipa-devel] [bind-dyndb-ldap PR#12][+ack] README.md: fix markdown formatting

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/12 Title: #12: README.md: fix markdown formatting Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [bind-dyndb-ldap PR#14][closed] time_t maybe unsigned or long

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/14 Author: mingzym Title: #14: time_t maybe unsigned or long Action: closed To pull the PR as Git branch: git remote add ghbind-dyndb-ldap https://github.com/freeipa/bind-dyndb-ldap git fetch ghbind-dyndb-ldap pull/14/head:pr14 git checkout

[Freeipa-devel] [bind-dyndb-ldap PR#14][+ack] time_t maybe unsigned or long

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/14 Title: #14: time_t maybe unsigned or long Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [bind-dyndb-ldap PR#14][comment] time_t maybe unsigned or long

2017-04-06 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/14 Title: #14: time_t maybe unsigned or long tomaskrizek commented: """ Thanks for the contribution! """ See the full comment at https://github.com/freeipa/bind-dyndb-ldap/pull/14#issuecomment-292218042 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#697][comment] Create system users for FreeIPA services during package installation

2017-04-06 Thread tjaalton
URL: https://github.com/freeipa/freeipa/pull/697 Title: #697: Create system users for FreeIPA services during package installation tjaalton commented: """ if I understood the sysusers.d file format correctly, ipa.sysusers.debian.conf would need this line added: m www-data ipaapi as you can

[Freeipa-devel] [bind-dyndb-ldap PR#14][opened] time_t maybe unsigned or long

2017-04-06 Thread mingzym
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/14 Author: mingzym Title: #14: time_t maybe unsigned or long Action: opened PR body: """ when compiling with heimdal, the krb5_timestamp is set to long instead of the unsigned in mit-krb5. """ To pull the PR as Git branch: git remote add

[Freeipa-devel] [freeipa PR#697][opened] Create system users for FreeIPA services during package installation

2017-04-06 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/697 Author: dkupka Title: #697: Create system users for FreeIPA services during package installation Action: opened PR body: """ Previously system users needed by FreeIPA server services was created during ipa-server-install. This led to problem

[Freeipa-devel] [freeipa PR#632][comment] ipa-sam: create the gidNumber attribute in the trusted domain entry

2017-04-06 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/632 Title: #632: ipa-sam: create the gidNumber attribute in the trusted domain entry abbra commented: """ LGTM. `nltest /sc_verify:ipa.example.test` works thanks to this pull request: ``` C:\Users\Administrator>nltest /sc_query:ipa.example.test

[Freeipa-devel] [freeipa PR#632][+ack] ipa-sam: create the gidNumber attribute in the trusted domain entry

2017-04-06 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/632 Title: #632: ipa-sam: create the gidNumber attribute in the trusted domain entry Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#677][comment] cert: defer cert-find result post-processing

2017-04-06 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/677 Title: #677: cert: defer cert-find result post-processing stlaz commented: """ The patched IPA works better than the current 4.4 and 4.5 branches in terms of options logic, that's good. From the code I am not sure which searches we do miss,

[Freeipa-devel] [freeipa PR#689][comment] Sort SRV records by priority

2017-04-06 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/689 Title: #689: Sort SRV records by priority MartinBasti commented: """ ``` git reset HEAD~3 git commit add -p git commit --amend ``` should help """ See the full comment at https://github.com/freeipa/freeipa/pull/689#issuecomment-292142293 --

[Freeipa-devel] [freeipa PR#689][comment] Sort SRV records by priority

2017-04-06 Thread alex-zel
URL: https://github.com/freeipa/freeipa/pull/689 Title: #689: Sort SRV records by priority alex-zel commented: """ Sorry I think I messed up trying to squash the commits """ See the full comment at https://github.com/freeipa/freeipa/pull/689#issuecomment-292140831 -- Manage your subscription

[Freeipa-devel] [freeipa PR#689][synchronized] Sort SRV records by priority

2017-04-06 Thread alex-zel
URL: https://github.com/freeipa/freeipa/pull/689 Author: alex-zel Title: #689: Sort SRV records by priority Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/689/head:pr689 git checkout pr689 From

[Freeipa-devel] [freeipa PR#689][comment] Sort SRV records by priority

2017-04-06 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/689 Title: #689: Sort SRV records by priority MartinBasti commented: """ @alex-zel Please merge your commits into one, thanks. """ See the full comment at https://github.com/freeipa/freeipa/pull/689#issuecomment-292138392 -- Manage your

[Freeipa-devel] [freeipa PR#671][edited] Slim down dependencies

2017-04-06 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/671 Author: tiran Title: #671: Slim down dependencies Action: edited Changed field: body Original value: """ * Remove unused install requires * Correct dependencies for yubico otptoken * Properly report optional dependency for yubico otptoken *

[Freeipa-devel] [freeipa PR#671][synchronized] Slim down dependencies

2017-04-06 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/671 Author: tiran Title: #671: Slim down dependencies Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/671/head:pr671 git checkout pr671 From

[Freeipa-devel] [freeipa PR#671][edited] Slim down dependencies

2017-04-06 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/671 Author: tiran Title: #671: Slim down dependencies Action: edited Changed field: title Original value: """ [WIP] Slim down dependencies """ -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#693][comment] [tests] collect audit.log for easier selinux investigation

2017-04-06 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/693 Title: #693: [tests] collect audit.log for easier selinux investigation MartinBasti commented: """ master: * fd597f83aed53bf3281ce9ec6b94f601868cfc75 collect audit.log for easier selinux investigation """ See the full comment at

[Freeipa-devel] [freeipa PR#693][+pushed] [tests] collect audit.log for easier selinux investigation

2017-04-06 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/693 Title: #693: [tests] collect audit.log for easier selinux investigation Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#693][closed] [tests] collect audit.log for easier selinux investigation

2017-04-06 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/693 Author: MartinBasti Title: #693: [tests] collect audit.log for easier selinux investigation Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/693/head:pr693 git

[Freeipa-devel] [freeipa PR#694][comment] RFC: implement local PKINIT deployment in server/replica install

2017-04-06 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/694 Title: #694: RFC: implement local PKINIT deployment in server/replica install martbab commented: """ @MartinBasti I haven't thought about CA-less -> CA-full but in this case you would have local PKINIT and should configure full PKINIT manually

[Freeipa-devel] [freeipa PR#689][synchronized] Sort SRV records by priority

2017-04-06 Thread alex-zel
URL: https://github.com/freeipa/freeipa/pull/689 Author: alex-zel Title: #689: Sort SRV records by priority Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/689/head:pr689 git checkout pr689 From