[Freeipa-devel] [freeipa PR#680][opened] ipa-otpd.socket.in: Use a platform specific value for KDC service file

2017-03-31 Thread tjaalton
URL: https://github.com/freeipa/freeipa/pull/680 Author: tjaalton Title: #680: ipa-otpd.socket.in: Use a platform specific value for KDC service file Action: opened PR body: """ https://pagure.io/freeipa/issue/6845 """ To pull the PR as Git branch: git remote add ghfreeipa

[Freeipa-devel] [freeipa PR#679][opened] Make sure remote hosts have our keys

2017-03-31 Thread simo5
URL: https://github.com/freeipa/freeipa/pull/679 Author: simo5 Title: #679: Make sure remote hosts have our keys Action: opened PR body: """ In complex replication setups a replica may try to obtain CA keys from a host that is not the master we initially create the keys against. In this case

[Freeipa-devel] [freeipa PR#679][comment] Make sure remote hosts have our keys

2017-03-31 Thread simo5
URL: https://github.com/freeipa/freeipa/pull/679 Title: #679: Make sure remote hosts have our keys simo5 commented: """ I haven't tested this yet ... but what could possibily go wrong? :-) """ See the full comment at https://github.com/freeipa/freeipa/pull/679#issuecomment-290762100 --

[Freeipa-devel] [freeipa PR#679][synchronized] Make sure remote hosts have our keys

2017-03-31 Thread simo5
URL: https://github.com/freeipa/freeipa/pull/679 Author: simo5 Title: #679: Make sure remote hosts have our keys Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/679/head:pr679 git checkout pr679 From

[Freeipa-devel] [freeipa PR#679][synchronized] Make sure remote hosts have our keys

2017-03-31 Thread simo5
URL: https://github.com/freeipa/freeipa/pull/679 Author: simo5 Title: #679: Make sure remote hosts have our keys Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/679/head:pr679 git checkout pr679 From

[Freeipa-devel] [freeipa PR#618][synchronized] [WIP] Tox testing support for client wheel packages

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/618 Author: tiran Title: #618: [WIP] Tox testing support for client wheel packages Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/618/head:pr618 git checkout

[Freeipa-devel] [freeipa PR#397][comment] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing tiran commented: """ @MartinBasti ```dbus-devel``` is in the ```with_wheels``` section. Documentation is part of https://pagure.io/freeipa/issue/6842 . """ See

[Freeipa-devel] [freeipa PR#397][synchronized] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/397 Author: tiran Title: #397: Improve wheel building and provide ipaserver wheel for local testing Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#675][synchronized] [WIP] Fix PKCS11 helper

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/675 Author: MartinBasti Title: #675: [WIP] Fix PKCS11 helper Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/675/head:pr675 git checkout pr675 From

[Freeipa-devel] [freeipa PR#593][+pushed] Add make devcheck for developers

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers 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#593][comment] Add make devcheck for developers

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers MartinBasti commented: """ master: * e357133fd7b276ccabfe1896ee948f2bb3541d94 Add make devcheck for developers * 6c092c24b2bfbba0a3f263d88f7a0dbf83f24869 Skip test_session_storage in ipaclient

[Freeipa-devel] [freeipa PR#593][closed] Add make devcheck for developers

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/593 Author: tiran Title: #593: Add make devcheck for developers Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/593/head:pr593 git checkout pr593 -- Manage your

[Freeipa-devel] [freeipa PR#397][comment] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing MartinBasti commented: """ So put it into specfile to `with_wheels` section """ See the full comment at

[Freeipa-devel] [freeipa PR#397][comment] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing MartinBasti commented: """ And document in `BUILD.txt` how to build wheels """ See the full comment at

[Freeipa-devel] [freeipa PR#397][comment] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing tiran commented: """ You need dbus-devel package. I opened https://pagure.io/freeipa/issue/6842 to track lack of documentation. """ See the full comment at

[Freeipa-devel] [freeipa PR#397][comment] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing MartinBasti commented: """ Build failed: ``` make wheel_bundle IPA_SERVER_WHEELS=1 ... checking for DBUS... no configure: error: Package requirements (dbus-1

[Freeipa-devel] [freeipa PR#593][synchronized] Add make devcheck for developers

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/593 Author: tiran Title: #593: Add make devcheck for developers Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/593/head:pr593 git checkout pr593 From

[Freeipa-devel] [freeipa PR#480][comment] Hide request_type doc string in cert-request help

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/480 Title: #480: Hide request_type doc string in cert-request help MartinBasti commented: """ @Akasurde you are welcome """ See the full comment at https://github.com/freeipa/freeipa/pull/480#issuecomment-290680413 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers MartinBasti commented: """ Ah right the description hasn't been updated """ See the full comment at https://github.com/freeipa/freeipa/pull/593#issuecomment-290678887 -- Manage your subscription

[Freeipa-devel] [freeipa PR#480][comment] Hide request_type doc string in cert-request help

2017-03-31 Thread Akasurde
URL: https://github.com/freeipa/freeipa/pull/480 Title: #480: Hide request_type doc string in cert-request help Akasurde commented: """ @MartinBasti @frasertweedale @HonzaCholasta Thanks """ See the full comment at https://github.com/freeipa/freeipa/pull/480#issuecomment-290679480 -- Manage

[Freeipa-devel] [freeipa PR#593][edited] Add make devcheck for developers

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/593 Author: tiran Title: #593: Add make devcheck for developers Action: edited Changed field: body Original value: """ Ticket 6604 makes pylint and jsl optional dependencies. The change is controversal, because some developers prefer that pylint

[Freeipa-devel] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers MartinBasti commented: """ Needs rebase """ See the full comment at https://github.com/freeipa/freeipa/pull/593#issuecomment-290679169 -- Manage your subscription for the Freeipa-devel mailing

[Freeipa-devel] [freeipa PR#480][comment] Hide request_type doc string in cert-request help

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/480 Title: #480: Hide request_type doc string in cert-request help MartinBasti commented: """ master: * a1bb442054936113369a88b49483e914664712e7 Hide request_type doc string in cert-request help ipa-4-5: *

[Freeipa-devel] [freeipa PR#480][closed] Hide request_type doc string in cert-request help

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/480 Author: Akasurde Title: #480: Hide request_type doc string in cert-request help Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/480/head:pr480 git checkout pr480

[Freeipa-devel] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers MartinBasti commented: """ @stlaz why is this ACKed when it depends on #670 ? """ See the full comment at https://github.com/freeipa/freeipa/pull/593#issuecomment-290678060 -- Manage your

[Freeipa-devel] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers stlaz commented: """ @MartinBasti #670 was ACKed already and the commit was originally a part of this. """ See the full comment at

[Freeipa-devel] [freeipa PR#480][+pushed] Hide request_type doc string in cert-request help

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/480 Title: #480: Hide request_type doc string in cert-request help 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#490][closed] certdb: use certutil and match_hostname for cert verification

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/490 Author: HonzaCholasta Title: #490: certdb: use certutil and match_hostname for cert verification Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/490/head:pr490

[Freeipa-devel] [freeipa PR#490][comment] certdb: use certutil and match_hostname for cert verification

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/490 Title: #490: certdb: use certutil and match_hostname for cert verification MartinBasti commented: """ master: * 9183cf2a7505624235b255b1406702cdaa65bb38 certdb: use certutil and match_hostname for cert verification *

[Freeipa-devel] [freeipa PR#490][+pushed] certdb: use certutil and match_hostname for cert verification

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/490 Title: #490: certdb: use certutil and match_hostname for cert verification 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#621][comment] Add --password-expiration to allow an admin to force a password change

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/621 Title: #621: Add --password-expiration to allow an admin to force a password change MartinBasti commented: """ master: * 274b0bcf5ff2408739d94ba1b1b4bca69f310dfc Add --password-expiration to allow admin to force user password expiration """

[Freeipa-devel] [freeipa PR#621][+pushed] Add --password-expiration to allow an admin to force a password change

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/621 Title: #621: Add --password-expiration to allow an admin to force a password change 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#621][closed] Add --password-expiration to allow an admin to force a password change

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/621 Author: redhatrises Title: #621: Add --password-expiration to allow an admin to force a password change Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#670][closed] [Py3] session storage parameters must be bytes

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/670 Author: tiran Title: #670: [Py3] session storage parameters must be bytes Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/670/head:pr670 git checkout pr670 --

[Freeipa-devel] [freeipa PR#670][comment] [Py3] session storage parameters must be bytes

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/670 Title: #670: [Py3] session storage parameters must be bytes MartinBasti commented: """ master: * d06315de6b1e951d6cce7d7d6495a32b44216274 session storage parameters must be bytes """ See the full comment at

[Freeipa-devel] [freeipa PR#670][+pushed] [Py3] session storage parameters must be bytes

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/670 Title: #670: [Py3] session storage parameters must be bytes 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#678][+pushed] ipa-ca-install man page: Add domain level 1 help

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/678 Title: #678: ipa-ca-install man page: Add domain level 1 help 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#678][comment] ipa-ca-install man page: Add domain level 1 help

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/678 Title: #678: ipa-ca-install man page: Add domain level 1 help MartinBasti commented: """ master: * b96a942cdca09496be9f911499036bee60084aee ipa-ca-install man page: Add domain level 1 help ipa-4-4: * 1734e143582843ef1d397a4929687b1068bdf413

[Freeipa-devel] [freeipa PR#678][closed] ipa-ca-install man page: Add domain level 1 help

2017-03-31 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/678 Author: flo-renaud Title: #678: ipa-ca-install man page: Add domain level 1 help Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/678/head:pr678 git checkout pr678

[Freeipa-devel] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers stlaz commented: """ Thanks, ACK. """ See the full comment at https://github.com/freeipa/freeipa/pull/593#issuecomment-290673932 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#593][+ack] Add make devcheck for developers

2017-03-31 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers 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] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers tiran commented: """ I split the changes to session storage tests into a separate commit. The other commit is in #670 """ See the full comment at

[Freeipa-devel] [freeipa PR#593][synchronized] Add make devcheck for developers

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/593 Author: tiran Title: #593: Add make devcheck for developers Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/593/head:pr593 git checkout pr593 From

[Freeipa-devel] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers stlaz commented: """ Whichever is ok with you, I don't mind if it's in the same PR if it is related to the same ticket. """ See the full comment at

[Freeipa-devel] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers tiran commented: """ ```test_session_storage``` is not a unit test or functional test. It is an integration test that depends on a valid Kerberos configuration and session. Do you prefer a separate

[Freeipa-devel] [freeipa PR#593][comment] Add make devcheck for developers

2017-03-31 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/593 Title: #593: Add make devcheck for developers stlaz commented: """ The changes to Makefile and configure.ac are just fine. I understand that changes in the `ipapython/session_storage.py` are done elsewhere so once that is pushed, we'll need a

[Freeipa-devel] [freeipa PR#490][+ack] certdb: use certutil and match_hostname for cert verification

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/490 Title: #490: certdb: use certutil and match_hostname for cert verification 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#678][+ack] ipa-ca-install man page: Add domain level 1 help

2017-03-31 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/678 Title: #678: ipa-ca-install man page: Add domain level 1 help 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#621][+ack] Add --password-expiration to allow an admin to force a password change

2017-03-31 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/621 Title: #621: Add --password-expiration to allow an admin to force a password change 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#397][comment] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing tiran commented: """ Thanks @MartinBasti I rebased the PR and added a small workaround for ```dbus-python```. The package uses make to compile some of its

[Freeipa-devel] [freeipa PR#593][synchronized] Add make devcheck for developers

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/593 Author: tiran Title: #593: Add make devcheck for developers Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/593/head:pr593 git checkout pr593 From

[Freeipa-devel] [freeipa PR#593][synchronized] Add make devcheck for developers

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/593 Author: tiran Title: #593: Add make devcheck for developers Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/593/head:pr593 git checkout pr593 From

[Freeipa-devel] [freeipa PR#593][edited] Add make devcheck for developers

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/593 Author: tiran Title: #593: Add make devcheck for developers Action: edited Changed field: body Original value: """ Ticket 6604 makes pylint and jsl optional dependencies. The change is controversal, because some developers prefer that pylint

[Freeipa-devel] [freeipa PR#397][comment] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing HonzaCholasta commented: """ LGTM. """ See the full comment at https://github.com/freeipa/freeipa/pull/397#issuecomment-290632228 -- Manage your subscription

[Freeipa-devel] [freeipa PR#397][synchronized] Improve wheel building and provide ipaserver wheel for local testing

2017-03-31 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/397 Author: tiran Title: #397: Improve wheel building and provide ipaserver wheel for local testing Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa