[Freeipa-devel] [freeipa PR#296][+ack] Require python-gssapi >= 1.2.0, take 2

2016-12-01 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/296 Title: #296: Require python-gssapi >= 1.2.0, take 2 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/Co

[Freeipa-devel] [freeipa PR#294][comment] client, platform: Use paths.SSH* instead of get_config_dir().

2016-12-01 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/294 Title: #294: client, platform: Use paths.SSH* instead of get_config_dir(). martbab commented: """ @tiran I agree, PEP8 errors are often overlooked in the slew of other stuff that gets printed out in the container. I will address

[Freeipa-devel] [freeipa PR#290][closed] Require python-cryptography >= 1.3.1

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/290 Author: tiran Title: #290: Require python-cryptography >= 1.3.1 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/290/head:pr290 git checkout pr290 -- Manage your s

[Freeipa-devel] [freeipa PR#290][comment] Require python-cryptography >= 1.3.1

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/290 Title: #290: Require python-cryptography >= 1.3.1 martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/289982e02fa6bef700fe2c1900ddbed864876faa """ See the full comment at h

[Freeipa-devel] [freeipa PR#290][+ack] Require python-cryptography >= 1.3.1

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/290 Title: #290: Require python-cryptography >= 1.3.1 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#290][+pushed] Require python-cryptography >= 1.3.1

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/290 Title: #290: Require python-cryptography >= 1.3.1 Label: +pushed -- 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/C

[Freeipa-devel] [freeipa PR#263][comment] Backwards compatibility with setuptools 0.9.8

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/263 Title: #263: Backwards compatibility with setuptools 0.9.8 martbab commented: """ Please reabse this PR and add ticket to the commit message. """ See the full comment at https://github.com/freeipa/freeipa/p

[Freeipa-devel] [freeipa PR#289][+pushed] Require python-gssapi >= 1.2.0

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/289 Title: #289: Require python-gssapi >= 1.2.0 Label: +pushed -- 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#290][comment] Require python-cryptography >= 1.3.1

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/290 Title: #290: Require python-cryptography >= 1.3.1 martbab commented: """ Please rebase the PR so we can do clean merge, it should be simple conflict resolution. """ See the full comment at https://gi

[Freeipa-devel] [freeipa PR#289][comment] Require python-gssapi >= 1.2.0

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/289 Title: #289: Require python-gssapi >= 1.2.0 martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/8559791e0d520f4a3503e35d1975ac31448b1390 """ See the full comment at https:

[Freeipa-devel] [freeipa PR#289][closed] Require python-gssapi >= 1.2.0

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/289 Author: tiran Title: #289: Require python-gssapi >= 1.2.0 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/289/head:pr289 git checkout pr289 -- Manage your subscri

[Freeipa-devel] [freeipa PR#289][+ack] Require python-gssapi >= 1.2.0

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/289 Title: #289: Require python-gssapi >= 1.2.0 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#283][closed] [ipa-4-4] Prevent denial of replication updates during CA replica install

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/283 Author: martbab Title: #283: [ipa-4-4] Prevent denial of replication updates during CA replica install Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/283

[Freeipa-devel] [freeipa PR#283][+pushed] [ipa-4-4] Prevent denial of replication updates during CA replica install

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/283 Title: #283: [ipa-4-4] Prevent denial of replication updates during CA replica install Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: ht

[Freeipa-devel] [freeipa PR#283][comment] [ipa-4-4] Prevent denial of replication updates during CA replica install

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/283 Title: #283: [ipa-4-4] Prevent denial of replication updates during CA replica install martbab commented: """ Fixed upstream ipa-4-4: https://fedorahosted.org/freeipa/changeset/8c6a10ceddb4fce9a3dd4a334e6804800b5c89f9 https://f

[Freeipa-devel] [freeipa PR#255][closed] Adjustments for setup requirements

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/255 Author: tiran Title: #255: Adjustments for setup requirements Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/255/head:pr255 git checkout pr255 -- Manage your sub

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/ed9645b2ac58fd4664810f05970ea258c7948420 """ See the full comment at http

[Freeipa-devel] [freeipa PR#255][+pushed] Adjustments for setup requirements

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements Label: +pushed -- 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/Cod

[Freeipa-devel] [freeipa PR#255][+ack] Adjustments for setup requirements

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements 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#255][comment] Adjustments for setup requirements

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements martbab commented: """ Installing python-wheel worked, thanks. I have discovered some other missing dependencies in minimal Docker container. I will investigate them some more and o

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements martbab commented: """ Thank you. It seems that 'bdist_wheel' target is broken in your PR: ``` # make bdist_wheel mkdir -p ./dist/wheels for dir in ipaclient ipalib

[Freeipa-devel] [freeipa PR#275][comment] Enhance __repr__ method of Principal

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Title: #275: Enhance __repr__ method of Principal martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/38cc40ddb5bf965801500bb4f66fd965b12e3c88 """ See the full comment at http

[Freeipa-devel] [freeipa PR#275][closed] Enhance __repr__ method of Principal

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Author: martbab Title: #275: Enhance __repr__ method of Principal Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/275/head:pr275 git checkout pr275 -- Manage

[Freeipa-devel] [freeipa PR#275][+pushed] Enhance __repr__ method of Principal

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Title: #275: Enhance __repr__ method of Principal Label: +pushed -- 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/C

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements martbab commented: """ As I said, if 0.9 break your PyPI work feel freee to bump it but please split the version bumps into a separate commit on top of ipasetup fixes. ""&

[Freeipa-devel] [freeipa PR#281][+ack] Accept server host names resolvable only using /etc/hosts

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/281 Title: #281: Accept server host names resolvable only using /etc/hosts 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

[Freeipa-devel] [freeipa PR#281][closed] Accept server host names resolvable only using /etc/hosts

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/281 Author: pspacek Title: #281: Accept server host names resolvable only using /etc/hosts Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/281/head:pr281 git checkout

[Freeipa-devel] [freeipa PR#281][+pushed] Accept server host names resolvable only using /etc/hosts

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/281 Title: #281: Accept server host names resolvable only using /etc/hosts Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.

[Freeipa-devel] [freeipa PR#281][comment] Accept server host names resolvable only using /etc/hosts

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/281 Title: #281: Accept server host names resolvable only using /etc/hosts martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/0e093f938d8126f11fed920b7381ba6e3d07da5b ipa-4-4: https://fedorahos

[Freeipa-devel] [freeipa PR#281][comment] Accept server host names resolvable only using /etc/hosts

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/281 Title: #281: Accept server host names resolvable only using /etc/hosts martbab commented: """ Ok I am fine with this. """ See the full comment at https://github.com/freeipa/freeipa/pull/281#issuecomment-26364018

[Freeipa-devel] [freeipa PR#225][+pushed] tests: Added basic tests for certs in idoverrides

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/225 Title: #225: tests: Added basic tests for certs in idoverrides Label: +pushed -- 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

[Freeipa-devel] [freeipa PR#225][comment] tests: Added basic tests for certs in idoverrides

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/225 Title: #225: tests: Added basic tests for certs in idoverrides martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/ccd3677b50eab2223ddf1e1b6682c20fc695ad24 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#225][closed] tests: Added basic tests for certs in idoverrides

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/225 Author: ofayans Title: #225: tests: Added basic tests for certs in idoverrides Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/225/head:pr225 git checkout pr225 --

[Freeipa-devel] [freeipa PR#275][comment] Enhance __repr__ method of Principal

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Title: #275: Enhance __repr__ method of Principal martbab commented: """ That sound like a good idea. Added such assert to the unit tests. """ See the full comment at https://github.com/freeipa/freeipa/pull/275#iss

[Freeipa-devel] [freeipa PR#275][synchronized] Enhance __repr__ method of Principal

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Author: martbab Title: #275: Enhance __repr__ method of Principal Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/275/head:pr275 git checkout pr275 From

[Freeipa-devel] [freeipa PR#255][comment] Adjustments for setup requirements

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements martbab commented: """ Well from our (as upstream) POV 0.9 and later is required for Custodia to work correctly. This requirement was introduced

[Freeipa-devel] [freeipa PR#275][comment] Enhance __repr__ method of Principal

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Title: #275: Enhance __repr__ method of Principal martbab commented: """ Sorry I somehow botched that, but it worked nevertheless. I have re-worked the PR according to your comments. ``` In [1]: import ipapython.ker

[Freeipa-devel] [freeipa PR#281][comment] Accept server host names resolvable only using /etc/hosts

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/281 Title: #281: Accept server host names resolvable only using /etc/hosts martbab commented: """ Thinking of this some more, shouldn't be `--no-host-dns` option used and advertised if you want to set unresolvable hostname duri

[Freeipa-devel] [freeipa PR#281][comment] Accept server host names resolvable only using /etc/hosts

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/281 Title: #281: Accept server host names resolvable only using /etc/hosts martbab commented: """ I see. I guess we can live with the fact that we may break such eccentric DNS topologies. I think we cannot really handle all t

[Freeipa-devel] [freeipa PR#283][opened] [ipa-4-4] Prevent denial of replication updates during CA replica install

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/283 Author: martbab Title: #283: [ipa-4-4] Prevent denial of replication updates during CA replica install Action: opened PR body: """ This is https://github.com/freeipa/freeipa/pull/269 rebased on top of ipa-4-4 branch. https://f

[Freeipa-devel] [freeipa PR#281][comment] Accept server host names resolvable only using /etc/hosts

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/281 Title: #281: Accept server host names resolvable only using /etc/hosts martbab commented: """ So can you imagine some scenario where this behavior may cause issues? Some exotic DNS setup maybe? """ See the full

[Freeipa-devel] [freeipa PR#273][closed] Build: workaround bug while calling parallel make from rpmbuild

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/273 Author: pspacek Title: #273: Build: workaround bug while calling parallel make from rpmbuild Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/273/head:pr273 git che

[Freeipa-devel] [freeipa PR#273][comment] Build: workaround bug while calling parallel make from rpmbuild

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/273 Title: #273: Build: workaround bug while calling parallel make from rpmbuild martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/132b475c2586f3ced68724355e9c45722dccf604 """

[Freeipa-devel] [freeipa PR#273][+pushed] Build: workaround bug while calling parallel make from rpmbuild

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/273 Title: #273: Build: workaround bug while calling parallel make from rpmbuild Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.fr

[Freeipa-devel] [freeipa PR#278][synchronized] Restore the original functionality of `env` and `plugins` commands

2016-11-28 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/278 Author: martbab Title: #278: Restore the original functionality of `env` and `plugins` commands Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/278

[Freeipa-devel] [freeipa PR#278][edited] Restore the original functionality of `env` and `plugins` commands

2016-11-28 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/278 Author: martbab Title: #278: Restore the original functionality of `env` and `plugins` commands Action: edited Changed field: body Original value: """ This reverts commit 1166fbc4946596fcc2ed51a1ec6990fc7dae8964 "Add '

[Freeipa-devel] [freeipa PR#278][synchronized] Restore the original functionality of `env` and `plugins` commands

2016-11-28 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/278 Author: martbab Title: #278: Restore the original functionality of `env` and `plugins` commands Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/278

[Freeipa-devel] [freeipa PR#278][synchronized] Restore the original functionality of `env` and `plugins` commands

2016-11-28 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/278 Author: martbab Title: #278: Restore the original functionality of `env` and `plugins` commands Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/278

[Freeipa-devel] [freeipa PR#278][opened] Restore the original functionality of `env` and `plugins` commands

2016-11-28 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/278 Author: martbab Title: #278: Restore the original functionality of `env` and `plugins` commands Action: opened PR body: """ This reverts commit 1166fbc4946596fcc2ed51a1ec6990fc7dae8964 "Add 'ipa localenv' sub

[Freeipa-devel] [freeipa PR#275][opened] Enhance __repr__ method of Principal

2016-11-28 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Author: martbab Title: #275: Enhance __repr__ method of Principal Action: opened PR body: """ `__repr__` now returns more descriptive string containing the actual principal name while keeping the ability to reconstruct the object

[Freeipa-devel] [freeipa PR#274][opened] Improve the robustness FreeIPA's i18n module and its tests

2016-11-25 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/274 Author: martbab Title: #274: Improve the robustness FreeIPA's i18n module and its tests Action: opened PR body: """ Prevent false positive errors reported by `ipatests/i18n.py` and `ipatests/test_ipalib/test_text.py` when L

[Freeipa-devel] [freeipa PR#231][comment] Do not log DM password in ca/kra installation logs

2016-11-25 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/231 Title: #231: Do not log DM password in ca/kra installation logs martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/e617f895e70e6812836870f504af6e22a5dc7def """ See the full

[Freeipa-devel] [freeipa PR#231][+pushed] Do not log DM password in ca/kra installation logs

2016-11-25 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/231 Title: #231: Do not log DM password in ca/kra installation logs Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/pag

[Freeipa-devel] [freeipa PR#231][closed] Do not log DM password in ca/kra installation logs

2016-11-25 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/231 Author: stlaz Title: #231: Do not log DM password in ca/kra installation logs Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/231/head:pr231 git checkout pr231 --

[Freeipa-devel] [freeipa PR#231][+ack] Do not log DM password in ca/kra installation logs

2016-11-25 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/231 Title: #231: Do not log DM password in ca/kra installation logs 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/C

[Freeipa-devel] [freeipa PR#211][comment] IPA Allows Password Reuse with History value defined when admin reset…

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/211 Title: #211: IPA Allows Password Reuse with History value defined when admin reset… martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/c223130d5f429278202aaf8bf87af53911a3b448 ""&

[Freeipa-devel] [freeipa PR#211][+pushed] IPA Allows Password Reuse with History value defined when admin reset…

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/211 Title: #211: IPA Allows Password Reuse with History value defined when admin reset… Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http:

[Freeipa-devel] [freeipa PR#211][closed] IPA Allows Password Reuse with History value defined when admin reset…

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/211 Author: tbordaz Title: #211: IPA Allows Password Reuse with History value defined when admin reset… Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/211/head:pr211

[Freeipa-devel] [freeipa PR#211][+ack] IPA Allows Password Reuse with History value defined when admin reset…

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/211 Title: #211: IPA Allows Password Reuse with History value defined when admin reset… Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://w

[Freeipa-devel] [freeipa PR#252][comment] Use namespace-aware meta importer for ipaplatform

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/252 Title: #252: Use namespace-aware meta importer for ipaplatform martbab commented: """ Is this PR valid given that we discussed to remove ipaplatform dependency from client-side modules? """ See the full comment

[Freeipa-devel] [freeipa PR#270][comment] Test: uniqueness of certificate renewal master

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/270 Title: #270: Test: uniqueness of certificate renewal master martbab commented: """ I have suggested some improvements in your code inline. Also, can we actually do this test as a part of other test suite or is it necessary to cr

[Freeipa-devel] [freeipa PR#254][comment] Replace LooseVersion with pkg_resource.parse_version

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with pkg_resource.parse_version martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/2cbaf156045769b54150e4d4c3c1071f164a16fb """ See the full

[Freeipa-devel] [freeipa PR#254][closed] Replace LooseVersion with pkg_resource.parse_version

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Author: tiran Title: #254: Replace LooseVersion with pkg_resource.parse_version Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/254/head:pr254 git checkout pr254 -

[Freeipa-devel] [freeipa PR#254][+pushed] Replace LooseVersion with pkg_resource.parse_version

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with pkg_resource.parse_version Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/p

[Freeipa-devel] [freeipa PR#254][+ack] Replace LooseVersion with pkg_resource.parse_version

2016-11-24 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with pkg_resource.parse_version 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

[Freeipa-devel] [freeipa PR#269][opened] Prevent denial of replication updates during CA replica install

2016-11-23 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/269 Author: martbab Title: #269: Prevent denial of replication updates during CA replica install Action: opened PR body: """ This PR fixes a case when CA replica install against upgraded topology hangs due to incorrectly configure

[Freeipa-devel] [freeipa PR#254][comment] Replace LooseVersion with pkg_resource.parse_version

2016-11-23 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with pkg_resource.parse_version martbab commented: """ LGTM but please add the relevant ticket number into the commit message. """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#122][comment] Acceptance tests

2016-11-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/122 Title: #122: Acceptance tests martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/3e53bbcc34bd256da36209fd8cf8ac5d33ec8093 https://fedorahosted.org/freeipa/changeset/4225484356426a73cc11211bc

[Freeipa-devel] [freeipa PR#122][+pushed] Acceptance tests

2016-11-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/122 Title: #122: Acceptance tests Label: +pushed -- 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#122][closed] Acceptance tests

2016-11-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/122 Author: dkupka Title: #122: Acceptance tests Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/122/head:pr122 git checkout pr122 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#254][comment] Replace LooseVersion with pkg_resource.parse_version

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with pkg_resource.parse_version martbab commented: """ It seems that your changes broke IPA upgrade: ``` Done configuring the web interface (httpd). Applying LDAP updates Upgrading IPA:

[Freeipa-devel] [freeipa PR#262][closed] upgrade: do not explicitly set principal for services

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/262 Author: tomaskrizek Title: #262: upgrade: do not explicitly set principal for services Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/262/head:pr262 git checkout

[Freeipa-devel] [freeipa PR#262][comment] upgrade: do not explicitly set principal for services

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/262 Title: #262: upgrade: do not explicitly set principal for services martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/2793cdc8593c40d8318ec3685408ade6bf9a5320 """ See the full

[Freeipa-devel] [freeipa PR#262][+pushed] upgrade: do not explicitly set principal for services

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/262 Title: #262: upgrade: do not explicitly set principal for services Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/

[Freeipa-devel] [freeipa PR#262][+ack] upgrade: do not explicitly set principal for services

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/262 Title: #262: upgrade: do not explicitly set principal for services 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/pag

[Freeipa-devel] [freeipa PR#261][comment] upgrade: do not set HTTP and DS principals explicitly

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/261 Title: #261: upgrade: do not set HTTP and DS principals explicitly martbab commented: """ Closing this as duplicate of https://github.com/freeipa/freeipa/pull/262 """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#261][closed] upgrade: do not set HTTP and DS principals explicitly

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/261 Author: martbab Title: #261: upgrade: do not set HTTP and DS principals explicitly Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/261/head:pr261 git checkout

[Freeipa-devel] [freeipa PR#262][comment] upgrade: do not explicitly set principal for services

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/262 Title: #262: upgrade: do not explicitly set principal for services martbab commented: """ Heh, I have opened https://github.com/freeipa/freeipa/pull/262 for this same issue recently. Since you assigned yourself to the ticket

[Freeipa-devel] [freeipa PR#262][comment] upgrade: do not explicitly set principal for services

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/262 Title: #262: upgrade: do not explicitly set principal for services martbab commented: """ Heh, I have opened PR#261 for this same issue recently. Since you assigned yourself to the ticket and filled all required fields yo

[Freeipa-devel] [freeipa PR#231][comment] Do not log DM password in ca/kra installation logs

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/231 Title: #231: Do not log DM password in ca/kra installation logs martbab commented: """ Well that is what I was pointing at, by adding both DM and admin passwords to the parent method's default `nolog_list`, you are futu

[Freeipa-devel] [freeipa PR#191][+pushed] Exclude testing ipa.pot file from zanata

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/191 Title: #191: Exclude testing ipa.pot file from zanata Label: +pushed -- 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/Contribu

[Freeipa-devel] [freeipa PR#191][closed] Exclude testing ipa.pot file from zanata

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/191 Author: mbasti-rh Title: #191: Exclude testing ipa.pot file from zanata Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/191/head:pr191 git checkout pr191 -- Manag

[Freeipa-devel] [freeipa PR#191][comment] Exclude testing ipa.pot file from zanata

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/191 Title: #191: Exclude testing ipa.pot file from zanata martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/ad32bf147ed6996c0967bb8e8cfb803113ceaf5f ipa-4-4: https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#191][+ack] Exclude testing ipa.pot file from zanata

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/191 Title: #191: Exclude testing ipa.pot file from zanata 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/

[Freeipa-devel] [freeipa PR#196][comment] ipatests: unresolvable nested netgroups

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/196 Title: #196: ipatests: unresolvable nested netgroups martbab commented: """ I am strongly opposed to keeping this particular test in XMLRPC suite since it actually does not test any XMLRPC calls but is, in fact, an integration

[Freeipa-devel] [freeipa PR#261][opened] upgrade: do not set HTTP and DS principals explicitly

2016-11-21 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/261 Author: martbab Title: #261: upgrade: do not set HTTP and DS principals explicitly Action: opened PR body: """ In ipa-server-upgrade, HTTP and DS principals are explicitly constructed from realm, fqdn, and service prefix. Thi

[Freeipa-devel] [freeipa PR#247][+pushed] Add 'ipa local-env' subcommand

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/247 Title: #247: Add 'ipa local-env' subcommand Label: +pushed -- 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#247][comment] Add 'ipa local-env' subcommand

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/247 Title: #247: Add 'ipa local-env' subcommand martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/1166fbc4946596fcc2ed51a1ec6990fc7dae8964 """ See the full comment at

[Freeipa-devel] [freeipa PR#247][closed] Add 'ipa local-env' subcommand

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/247 Author: tiran Title: #247: Add 'ipa local-env' subcommand Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/247/head:pr247 git checkout pr247 -- Manage your subscri

[Freeipa-devel] [freeipa PR#247][+ack] Add 'ipa local-env' subcommand

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/247 Title: #247: Add 'ipa local-env' subcommand 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#254][comment] Replace LooseVersion with parse_ipa_version()

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with parse_ipa_version() martbab commented: """ Oh right, ok. My code reading skills are sub-par today. """ See the full comment at https://github.com/freeipa/freeipa/pull/254#iss

[Freeipa-devel] [freeipa PR#254][comment] Replace LooseVersion with parse_ipa_version()

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with parse_ipa_version() martbab commented: """ Upon closer inspection of the affected code it seems that all the code in ipalib/ipaclient/ipapython actually parses and compares API versions, whic

[Freeipa-devel] [freeipa PR#254][comment] Replace LooseVersion with parse_ipa_version()

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with parse_ipa_version() martbab commented: """ I was thinking about this a bit, and was wondering whether the platform-specific idiosyncracies of the version handling could be safely confined

[Freeipa-devel] [freeipa PR#254][comment] Replace LooseVersion with pkg_resources.parse_version

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with pkg_resources.parse_version martbab commented: """ Yes that's why we resorted to a direct CFFI call to RPM libs during server version check in upgrade. We simply could not win aside

[Freeipa-devel] [freeipa PR#254][comment] Replace LooseVersion with pkg_resources.parse_version

2016-11-18 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with pkg_resources.parse_version martbab commented: """ I have a sneaky suspicion that the parse_version couldn't cope correctly with some of the downstream versioning schemes like RHEL z

[Freeipa-devel] [freeipa PR#248][opened] Fix the naming of ipa-dnskeysyncd service principal

2016-11-16 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/248 Author: martbab Title: #248: Fix the naming of ipa-dnskeysyncd service principal Action: opened PR body: """ https://fedorahosted.org/freeipa/ticket/6405 """ To pull the PR as Git branch: git remote add ghfree

[Freeipa-devel] [freeipa PR#243][comment] Don't modify redhat_system_units

2016-11-16 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/243 Title: #243: Don't modify redhat_system_units martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/94a9dfb9d72ecd25a01316febbf2ffec50912e2e """ See the full comment at http

[Freeipa-devel] [freeipa PR#243][closed] Don't modify redhat_system_units

2016-11-16 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/243 Author: tiran Title: #243: Don't modify redhat_system_units Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/243/head:pr243 git checkout pr243 -- Manage your subsc

[Freeipa-devel] [freeipa PR#243][+pushed] Don't modify redhat_system_units

2016-11-16 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/243 Title: #243: Don't modify redhat_system_units Label: +pushed -- 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#243][+ack] Don't modify redhat_system_units

2016-11-16 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/243 Title: #243: Don't modify redhat_system_units 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

<    1   2   3   4   5   6   7   8   >