[Freeipa-devel] [freeipa PR#153][comment] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/153 Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 mbasti-rh commented: """ Fixed upstream ipa-4-4: https://fedorahosted.org/freeipa/changeset/2b2fc1abf1844b807e50b99f0912fa10d9169eca """

[Freeipa-devel] [freeipa PR#153][closed] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/153 Author: martbab Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/153/head:pr153 git

[Freeipa-devel] [freeipa PR#154][+pushed] [ipa-4-4] Rebase: Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/154 Title: #154: [ipa-4-4] Rebase: Tests: Fix cert revocation tests 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#154][closed] [ipa-4-4] Rebase: Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/154 Author: mirielka Title: #154: [ipa-4-4] Rebase: Tests: Fix cert revocation tests Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/154/head:pr154 git checkout pr154

[Freeipa-devel] [freeipa PR#142][+ack] CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/142 Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling 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#140][comment] Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/140 Title: #140: Tests: Fix cert revocation tests mbasti-rh commented: """ `Tests: Certificate revocation` doesn't apply to ipa-4-4 branch, please open separate PR against IPA 4.4 """ See the full comment at http

[Freeipa-devel] [freeipa PR#152][+pushed] Fix warnings reported by pylint in rawhide

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/152 Title: #152: Fix warnings reported by pylint in rawhide 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#142][comment] CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/142 Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/fb85230e25bd37a2a02a9d90793f337aad40a037

[Freeipa-devel] [freeipa PR#142][+pushed] CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/142 Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to

[Freeipa-devel] [freeipa PR#142][closed] CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/142 Author: dkupka Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling 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#154][comment] [ipa-4-4] Rebase: Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/154 Title: #154: [ipa-4-4] Rebase: Tests: Fix cert revocation tests mbasti-rh commented: """ Fixed upstream ipa-4-4: https://fedorahosted.org/freeipa/changeset/afabdd365a35e0e454997ff021152422bcbcf785 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#152][closed] Fix warnings reported by pylint in rawhide

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/152 Author: martbab Title: #152: Fix warnings reported by pylint in rawhide Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/152/head:pr152 git checkout pr152 --

[Freeipa-devel] [freeipa PR#152][comment] Fix warnings reported by pylint in rawhide

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/152 Title: #152: Fix warnings reported by pylint in rawhide mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/29829cc55a6be697abf881ea7867ef834bb66be7 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#137][comment] Test: disabled wrong client domain tests for domlevel 0

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/137 Title: #137: Test: disabled wrong client domain tests for domlevel 0 mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/8b0faa25d1c47f605bc6c91933469bb2370276c1 ipa-4-4: https://fedorahos

[Freeipa-devel] [freeipa PR#137][+pushed] Test: disabled wrong client domain tests for domlevel 0

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/137 Title: #137: Test: disabled wrong client domain tests for domlevel 0 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#137][closed] Test: disabled wrong client domain tests for domlevel 0

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/137 Author: ofayans Title: #137: Test: disabled wrong client domain tests for domlevel 0 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/137/head:pr137 git checkout

[Freeipa-devel] [freeipa PR#153][+ack] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/153 Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 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#153][+pushed] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/153 Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 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#136][synchronized] Fix KRA install tests

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/136 Author: mbasti-rh Title: #136: Fix KRA install tests Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/136/head:pr136 git checkout pr136 From

[Freeipa-devel] [freeipa PR#127][comment] Move ipa-otpd to $libexecdir/ipa, purge ffextension

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/127 Title: #127: Move ipa-otpd to $libexecdir/ipa, purge ffextension mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/6c53765ac1746ea3cb82554775a37fe43af062e8 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#127][closed] Move ipa-otpd to $libexecdir/ipa, purge ffextension

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/127 Author: tjaalton Title: #127: Move ipa-otpd to $libexecdir/ipa, purge ffextension Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/127/head:pr127 git checkout

[Freeipa-devel] [freeipa PR#127][+pushed] Move ipa-otpd to $libexecdir/ipa, purge ffextension

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/127 Title: #127: Move ipa-otpd to $libexecdir/ipa, purge ffextension 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#156][+pushed] cert: add revocation reason back to cert-find output

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/156 Title: #156: cert: add revocation reason back to cert-find output 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#156][closed] cert: add revocation reason back to cert-find output

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/156 Author: jcholast Title: #156: cert: add revocation reason back to cert-find output Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/156/head:pr156 git checkout

[Freeipa-devel] [freeipa PR#156][comment] cert: add revocation reason back to cert-find output

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/156 Title: #156: cert: add revocation reason back to cert-find output mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/16dad1c3cb09acee946bc5b2409447279a8bc0de ipa-4-4: https://fedorahos

[Freeipa-devel] [freeipa PR#161][comment] CI: workaround: wait for dogtag before replica-prepare

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/161 Title: #161: CI: workaround: wait for dogtag before replica-prepare mbasti-rh commented: """ It looks that 30 seconds is not enough, majority of ipa-replica-prepare passed, but I had a few test where it is still failing (ran

[Freeipa-devel] [freeipa PR#127][+ack] Move ipa-otpd to $libexecdir/ipa, purge ffextension

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/127 Title: #127: Move ipa-otpd to $libexecdir/ipa, purge ffextension 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#136][edited] Fix KRA install tests

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/136 Author: mbasti-rh Title: #136: Fix KRA install tests Action: edited Changed field: body Original value: """ - in test_installation testsuite KRA related tests were duplicated, this PR removes it - in test_installation test su

[Freeipa-devel] [bind-dyndb-ldap PR#2][comment] fix ldif syntax and add idnsTemplateAttribute

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/2 Title: #2: fix ldif syntax and add idnsTemplateAttribute mbasti-rh commented: """ Hello, I wrote inline comments. Please set proper author name in commit (no root please) """ See the full comment at https:/

[Freeipa-devel] [freeipa PR#152][+ack] Fix warnings reported by pylint in rawhide

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/152 Title: #152: Fix warnings reported by pylint in rawhide 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#159][comment] spec file: clean up BuildRequires

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires mbasti-rh commented: """ Please update BUILD.txt with how to run pylint with build, probably freeipa.org should be updated as well """ See the full comment at http

[Freeipa-devel] [freeipa PR#127][comment] Move ipa-otpd to $libexecdir/ipa, purge ffextension

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/127 Title: #127: Move ipa-otpd to $libexecdir/ipa, purge ffextension mbasti-rh commented: """ Hello @tjaalton, I cannot apply second commit, it needs rebase. """ See the full comment at https://github.com/freeipa

[Freeipa-devel] [freeipa PR#160][edited] Reverted the assertion for replica uninstall returncode

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/160 Author: ofayans Title: #160: Reverted the assertion for replica uninstall returncode Action: edited Changed field: title Original value: """ Reverted the essertion for replica uninstall returncode """ -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#126][comment] Fix ipa migrate-ds when it finds a search reference

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/126 Title: #126: Fix ipa migrate-ds when it finds a search reference mbasti-rh commented: """ Hello, LGTM but PR needs rebase """ See the full comment at https://github.com/freeipa/freeipa/pull/126#issuecom

[Freeipa-devel] [freeipa PR#148][comment] Unaccessible variable self.attrs in Tracker

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker mbasti-rh commented: """ You can check last step in this howto http://www.freeipa.org/page/Pull_request_on_Github """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#167][+pushed] Move ipa.1 man file

2016-10-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/167 Title: #167: Move ipa.1 man file 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#157][comment] git: Add commit template

2016-10-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/157 Title: #157: git: Add commit template mbasti-rh commented: """ I disagree here with Honza, I liked those comments more at bottom. Why do we even need that comments? git commit command with vim set as editor is doing tha

[Freeipa-devel] [freeipa PR#167][closed] Move ipa.1 man file

2016-10-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/167 Author: tiran Title: #167: Move ipa.1 man file Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/167/head:pr167 git checkout pr167 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#167][comment] Move ipa.1 man file

2016-10-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/167 Title: #167: Move ipa.1 man file mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/b9d68b5c3503bb708f637be6bb173a742b4105b4 """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#155][comment] Build system cleanup

2016-10-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/155 Title: #155: Build system cleanup mbasti-rh commented: """ Needs rebase """ See the full comment at https://github.com/freeipa/freeipa/pull/155#issuecomment-254240565 -- Manage your subscription for the Fre

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

2016-11-29 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements mbasti-rh commented: """ @tiran You can split patch to useful part and please send unneeded bumping of requires as separate pull request, we can continue with discussion there

[Freeipa-devel] [freeipa PR#101][+rejected] Improved vault-show error message

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/101 Title: #101: Improved vault-show error message Label: +rejected -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#101][closed] Improved vault-show error message

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/101 Author: stlaz Title: #101: Improved vault-show error message Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/101/head:pr101 git checkout pr101 -- Manage your

[Freeipa-devel] [freeipa PR#317][comment] Unify password generation across FreeIPA

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ > The passwords should have around the same entropy now. SHA-1 actually > produces 160bit outputs (hence 40-characters long hexadecimal digest

[Freeipa-devel] [freeipa PR#206][closed] Properly handle multiple cookies in rpcclient

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/206 Author: simo5 Title: #206: Properly handle multiple cookies in rpcclient Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/206/head:pr206 git checkout pr206 --

[Freeipa-devel] [freeipa PR#206][+pushed] Properly handle multiple cookies in rpcclient

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/206 Title: #206: Properly handle multiple cookies in rpcclient 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#206][comment] Properly handle multiple cookies in rpcclient

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/206 Title: #206: Properly handle multiple cookies in rpcclient mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/560ab9e3176af8e59163155207cc2c1d631915dd https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#318][+pushed] server install: fix external CA install

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/318 Title: #318: server install: fix external CA install 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#318][closed] server install: fix external CA install

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/318 Author: jcholast Title: #318: server install: fix external CA install Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/318/head:pr318 git checkout pr318 -- Manage

[Freeipa-devel] [freeipa PR#318][comment] server install: fix external CA install

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/318 Title: #318: server install: fix external CA install mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/4fff09978eab520d130d87c0112b5caac907e651 """ See the full comment

[Freeipa-devel] [freeipa PR#206][+ack] Properly handle multiple cookies in rpcclient

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/206 Title: #206: Properly handle multiple cookies in rpcclient 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#317][comment] Unify password generation across FreeIPA

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ @tiran @simo5 If you read my comments properly I was happy with removing sha1() and I pointed out that ipa_generate_password() must generate

[Freeipa-devel] [freeipa PR#320][comment] add missing attribute to ipaca replica during CA topology update

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/320 Title: #320: add missing attribute to ipaca replica during CA topology update mbasti-rh commented: """ IMO #322 this might be related, @martbab can you please check it? """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#317][comment] Unify password generation across FreeIPA

2016-12-08 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ @tiran IMO you need check `length > uppercase + lowercase + num + special`, otherwise infinity loop but generally LGTM """

[Freeipa-devel] [freeipa PR#313][comment] ipaclient.plugins: Use api_version from internally called commands

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/313 Title: #313: ipaclient.plugins: Use api_version from internally called commands mbasti-rh commented: """ Can you please provide better commit description in commit message? What, why and how it fixes? Think about our fu

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ ACK, but travis runs tests under PY3 and should be, I tried locally and tests works for me (ipa-run-tests is still

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ ACK, but travis runs tests under PY3 and should be, I tried locally and tests works for me (ipa-run-tests is still

[Freeipa-devel] [freeipa PR#307][comment] Lowered the version of gettext

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/307 Title: #307: Lowered the version of gettext mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/e5686c0ccb4a71fa0ab460fd45280d9adba91cca """ See the full comment at https://g

[Freeipa-devel] [freeipa PR#307][closed] Lowered the version of gettext

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/307 Author: pvomacka Title: #307: Lowered the version of gettext Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/307/head:pr307 git checkout pr307 -- Manage your

[Freeipa-devel] [freeipa PR#307][+pushed] Lowered the version of gettext

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/307 Title: #307: Lowered the version of gettext 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#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ `make install` put `#!/usr/bin/python` into `ipa-run-tests` and it causes that on F25 our tests are running by default

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ @martbab Yes for this PR travis run tests under PY3, see tracebacks it is from python3.5 site-packages ""&quo

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ @martbab Yes for this PR travis run tests under PY3, see tracebacks it is from python3.5 site-packages ""&quo

[Freeipa-devel] [freeipa PR#317][comment] Unify password generation across FreeIPA

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ @tiran True, I would like to have there at least assert witch prevents devs to use more than let say 15% per class and more than 50% together, to

[Freeipa-devel] [freeipa PR#317][comment] Unify password generation across FreeIPA

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ @tiran True, I would like to have there at least assert witch prevents devs to use more than let say 15% per class and more than 50% together, to

[Freeipa-devel] [freeipa PR#282][+ack] replicainstall: give correct error message on DL mismatch

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/282 Title: #282: replicainstall: give correct error message on DL mismatch 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#282][comment] replicainstall: give correct error message on DL mismatch

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/282 Title: #282: replicainstall: give correct error message on DL mismatch mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/2dcbc9416f1de708336a9afe92a138da9360ec5b """

[Freeipa-devel] [freeipa PR#282][+pushed] replicainstall: give correct error message on DL mismatch

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/282 Title: #282: replicainstall: give correct error message on DL mismatch 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#282][closed] replicainstall: give correct error message on DL mismatch

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/282 Author: stlaz Title: #282: replicainstall: give correct error message on DL mismatch Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/282/head:pr282 git checkout

[Freeipa-devel] [freeipa PR#323][opened] ipactl: pass api as argument to services

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/323 Author: mbasti-rh Title: #323: ipactl: pass api as argument to services Action: opened PR body: """ Commit 6409abf1 removes hard dependency of ipalib in ipalatform to avoid cyclic dependenies, this commit updates ipactl accor

[Freeipa-devel] [freeipa PR#320][+ack] add missing attribute to ipaca replica during CA topology update

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/320 Title: #320: add missing attribute to ipaca replica during CA topology update 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#317][comment] Unify password generation across FreeIPA

2016-12-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/317 Title: #317: Unify password generation across FreeIPA mbasti-rh commented: """ It generates random tokens than can be used as: - passwords - anything else that should be random It is written in class docstring Yes we can ra

[Freeipa-devel] [freeipa PR#334][opened] Py3: Fix ToASCII method

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/334 Author: mbasti-rh Title: #334: Py3: Fix ToASCII method Action: opened PR body: """ in Py2 to_text method returns Py2 non-unicode string, but in Py3 to_text method returns Py3 default (unicode) string. So only in Py2 we hav

[Freeipa-devel] [freeipa PR#333][comment] Remove named-pkcs11 workarounds from DNSSEC tests.

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/333 Title: #333: Remove named-pkcs11 workarounds from DNSSEC tests. mbasti-rh commented: """ Please fix the issue reported by pylint ``` * Module ipatests.test_integration.test_dnssec ipatests/test_integration/test_dnss

[Freeipa-devel] [freeipa PR#334][synchronized] Py3: Fix ToASCII method

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/334 Author: mbasti-rh Title: #334: Py3: Fix ToASCII method Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/334/head:pr334 git checkout pr334 From

[Freeipa-devel] [freeipa PR#334][edited] Py3: Fix ToASCII method

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/334 Author: mbasti-rh Title: #334: Py3: Fix ToASCII method Action: edited Changed field: body Original value: """ in Py2 to_text method returns Py2 non-unicode string, but in Py3 to_text method returns Py3 default (unicode) string.

[Freeipa-devel] [freeipa PR#329][+rejected] experiment: did pull/177 break ci?

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/329 Title: #329: experiment: did pull/177 break ci? Label: +rejected -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#313][comment] ipaclient.plugins: Use api_version from internally called commands

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/313 Title: #313: ipaclient.plugins: Use api_version from internally called commands mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/d841a79dc104521f736469eff7154c2f4266082b ipa-4-4: https://f

[Freeipa-devel] [freeipa PR#313][+pushed] ipaclient.plugins: Use api_version from internally called commands

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/313 Title: #313: ipaclient.plugins: Use api_version from internally called commands 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#313][closed] ipaclient.plugins: Use api_version from internally called commands

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/313 Author: dkupka Title: #313: ipaclient.plugins: Use api_version from internally called commands Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/313/head:pr313 git

[Freeipa-devel] [freeipa PR#334][synchronized] Py3: Fix ToASCII method

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/334 Author: mbasti-rh Title: #334: Py3: Fix ToASCII method Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/334/head:pr334 git checkout pr334 From

[Freeipa-devel] [freeipa PR#335][+ack] Add compatibility code to retrieve headers

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/335 Title: #335: Add compatibility code to retrieve headers 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#335][comment] Add compatibility code to retrieve headers

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/335 Title: #335: Add compatibility code to retrieve headers mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/397f2be9dfd6475127742c0b710b37b443d97d67 """ See the full comment

[Freeipa-devel] [freeipa PR#335][+pushed] Add compatibility code to retrieve headers

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/335 Title: #335: Add compatibility code to retrieve headers 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#335][closed] Add compatibility code to retrieve headers

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/335 Author: simo5 Title: #335: Add compatibility code to retrieve headers Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/335/head:pr335 git checkout pr335 -- Manage

[Freeipa-devel] [freeipa PR#344][edited] [4.4] password policy: Add explicit default password policy for hosts and services

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/344 Author: dkupka Title: #344: [4.4] password policy: Add explicit default password policy for hosts and services Action: edited Changed field: title Original value: """ password policy: Add explicit default password policy for hosts and

[Freeipa-devel] [freeipa PR#341][+ack] certprofile-mod: correctly authorise config update

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/341 Title: #341: certprofile-mod: correctly authorise config update 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#341][+pushed] certprofile-mod: correctly authorise config update

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/341 Title: #341: certprofile-mod: correctly authorise config update 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#341][comment] certprofile-mod: correctly authorise config update

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/341 Title: #341: certprofile-mod: correctly authorise config update mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/fec4c32ff15a96736740cf7d2f713a21af0b227e ipa-4-4: https://fedorahos

[Freeipa-devel] [freeipa PR#341][closed] certprofile-mod: correctly authorise config update

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/341 Author: mbasti-rh Title: #341: certprofile-mod: correctly authorise config update Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/341/head:pr341 git checkout

[Freeipa-devel] [freeipa PR#343][+ack] [4.3] certprofile-mod: correctly authorise config update

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/343 Title: #343: [4.3] certprofile-mod: correctly authorise config update 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#343][closed] [4.3] certprofile-mod: correctly authorise config update

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/343 Author: mbasti-rh Title: #343: [4.3] certprofile-mod: correctly authorise config update Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/343/head:pr343 git

[Freeipa-devel] [freeipa PR#343][+pushed] [4.3] certprofile-mod: correctly authorise config update

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/343 Title: #343: [4.3] certprofile-mod: correctly authorise config update 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#343][comment] [4.3] certprofile-mod: correctly authorise config update

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/343 Title: #343: [4.3] certprofile-mod: correctly authorise config update mbasti-rh commented: """ Fixed upstream ipa-4-3: https://fedorahosted.org/freeipa/changeset/278d7cf4708f1c6ac05d5fcb21db582d9aa7bab3 """

[Freeipa-devel] [freeipa PR#339][edited] freeipa-4.4.3: update translations

2016-12-14 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/339 Author: mbasti-rh Title: #339: freeipa-4.4.3: update translations Action: edited Changed field: body Original value: """ """ -- Manage your subscription for the Freeipa-devel mailing list: https://www.redha

[Freeipa-devel] [freeipa PR#336][comment] [py3] pki: add missing depedency pki-base[-python3]

2016-12-15 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/336 Title: #336: [py3] pki: add missing depedency pki-base[-python3] mbasti-rh commented: """ It has Py3 Build dependency Fixed. """ See the full comment at https://github.com/freeipa/freeipa/pull/336#issuecom

[Freeipa-devel] [freeipa PR#336][synchronized] [py3] pki: add missing depedency pki-base[-python3]

2016-12-15 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/336 Author: mbasti-rh Title: #336: [py3] pki: add missing depedency pki-base[-python3] Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/336/head:pr336 git

[Freeipa-devel] [freeipa PR#336][comment] [py3] pki: add missing depedency pki-base[-python3]

2016-12-15 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/336 Title: #336: [py3] pki: add missing depedency pki-base[-python3] mbasti-rh commented: """ Ah my previous patch was right then. BuildRequires version is lower because build works with lower version. I can raise that version

[Freeipa-devel] [freeipa PR#336][synchronized] [py3] pki: add missing depedency pki-base[-python3]

2016-12-15 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/336 Author: mbasti-rh Title: #336: [py3] pki: add missing depedency pki-base[-python3] Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/336/head:pr336 git

[Freeipa-devel] [freeipa PR#336][synchronized] [py3] pki: add missing depedency pki-base[-python3]

2016-12-15 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/336 Author: mbasti-rh Title: #336: [py3] pki: add missing depedency pki-base[-python3] Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/336/head:pr336 git

<    3   4   5   6   7   8   9   10   >