[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#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#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#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][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][+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][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#160][comment] Reverted the assertion for replica uninstall returncode

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/160 Title: #160: Reverted the assertion for replica uninstall returncode mbasti-rh commented: """ Ticket https://fedorahosted.org/freeipa/ticket/5725 is in already closed milestone, please create a new one (I suppose you want ba

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

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/155 Title: #155: Build system cleanup 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#155][comment] Build system cleanup

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/155 Title: #155: Build system cleanup mbasti-rh commented: """ Works for me, Not pushing yet to give time to others to disagree """ See the full comment at https://github.com/freeipa/freeipa/pull/155#issuecom

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

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/161 Author: mbasti-rh Title: #161: CI: workaround: wait for dogtag before replica-prepare Action: opened PR body: """ In domain level 0 ipa-replica-prepare fails because dogtag is not ready so soon after final restart during inst

[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#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#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] [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#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#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#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#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][+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#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#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#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#140][closed] Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/140 Author: mirielka Title: #140: 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/140/head:pr140 git checkout pr140 -- Manage your

[Freeipa-devel] [freeipa PR#140][+pushed] 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 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#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: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/c9c92e3a7f4961d91e0395daf17f5aeb34c20178 https://fedorahosted.org/fr

[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#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][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#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#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#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#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#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#132][comment] Draft for a new setup.py (WIP)

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/132 Title: #132: Draft for a new setup.py (WIP) mbasti-rh commented: """ I cannot reinstall packages using your commits ``` dnf reinstall <IPA*>.rpm ... Error: Transaction check error: file /usr/lib/python2.7/site-packages/ip

[Freeipa-devel] [freeipa PR#146][+pushed] WebUI: fix API Browser menu label

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/146 Title: #146: WebUI: fix API Browser menu label 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#146][comment] WebUI: fix API Browser menu label

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/146 Title: #146: WebUI: fix API Browser menu label mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/28c7644980186f50ee007cd5c2f2edcf103eb942 """ See the full comment at http

[Freeipa-devel] [freeipa PR#146][closed] WebUI: fix API Browser menu label

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/146 Author: pvomacka Title: #146: WebUI: fix API Browser menu label Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/146/head:pr146 git checkout pr146 -- Manage your

[Freeipa-devel] [freeipa PR#146][+ack] WebUI: fix API Browser menu label

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/146 Title: #146: WebUI: fix API Browser menu label 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#140][edited] Tests: Fix cert revocation tests

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/140 Author: mirielka Title: #140: Tests: Fix cert revocation tests Action: edited Changed field: title Original value: """ Tests: Remove invalid certplugin tests """ -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#136][comment] Fix KRA install tests

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/136 Title: #136: Fix KRA install tests mbasti-rh commented: """ Self NACK, I should not remove tests, I have to fix tasks.py to install replica properly with KRA, because we actually support --setup-kra option """

[Freeipa-devel] [freeipa PR#133][comment] Tests: print what was expected from exceptions and callables in xmlrpc_tests

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/133 Title: #133: Tests: print what was expected from exceptions and callables in xmlrpc_tests mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/8683cbf1242230fff8cd94da8aa27b27e9307535 &q

[Freeipa-devel] [freeipa PR#133][+pushed] Tests: print what was expected from exceptions and callables in xmlrpc_tests

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/133 Title: #133: Tests: print what was expected from exceptions and callables in xmlrpc_tests 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#133][closed] Tests: print what was expected from exceptions and callables in xmlrpc_tests

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/133 Author: pspacek Title: #133: Tests: print what was expected from exceptions and callables in xmlrpc_tests 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#133][+ack] Tests: print what was expected from exceptions and callables in xmlrpc_tests

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/133 Title: #133: Tests: print what was expected from exceptions and callables in xmlrpc_tests 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#144][+pushed] Pylint: remove unused values - the last part

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/144 Title: #144: Pylint: remove unused values - the last part 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#144][closed] Pylint: remove unused values - the last part

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/144 Author: mbasti-rh Title: #144: Pylint: remove unused values - the last part Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/144/head:pr144 git checkout pr144

[Freeipa-devel] [freeipa PR#144][comment] Pylint: remove unused values - the last part

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/144 Title: #144: Pylint: remove unused values - the last part mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/49b29591aa979560068449b78fd547915420ff08 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#134][closed] DNS URI support

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/134 Author: pspacek Title: #134: DNS URI support Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/134/head:pr134 git checkout pr134 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#134][+pushed] DNS URI support

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/134 Title: #134: DNS URI support 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#134][comment] DNS URI support

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/134 Title: #134: DNS URI support mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/f363dfbeed7aeba51d694a60b29389d94c7bda44 https://fedorahosted.org/freeipa/changeset/bf96b80200c3e8d1795a913db

[Freeipa-devel] [freeipa PR#134][+ack] DNS URI support

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/134 Title: #134: DNS URI support 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#138][comment] Fix ipa-cacert-manage man page

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/138 Title: #138: Fix ipa-cacert-manage man page mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/eb75578cbb2447fc2fafb8a79d8500b27e3edff0 """ See the full comment at https://g

[Freeipa-devel] [freeipa PR#138][+pushed] Fix ipa-cacert-manage man page

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/138 Title: #138: Fix ipa-cacert-manage man page 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#138][closed] Fix ipa-cacert-manage man page

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/138 Author: flo-renaud Title: #138: Fix ipa-cacert-manage man page Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/138/head:pr138 git checkout pr138 -- Manage your

[Freeipa-devel] [freeipa PR#141][closed] Tests: Fix failing test_ipalib/test_parameters

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/141 Author: mirielka Title: #141: Tests: Fix failing test_ipalib/test_parameters Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/141/head:pr141 git checkout pr141 --

[Freeipa-devel] [freeipa PR#141][+pushed] Tests: Fix failing test_ipalib/test_parameters

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/141 Title: #141: Tests: Fix failing test_ipalib/test_parameters 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#141][comment] Tests: Fix failing test_ipalib/test_parameters

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/141 Title: #141: Tests: Fix failing test_ipalib/test_parameters mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/c3e3130a35f05348405701731ec2d5b85a772997 """ See the full

[Freeipa-devel] [freeipa PR#150][closed] Fix compatibility with python-dns 1.15.0

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/150 Author: pspacek Title: #150: Fix compatibility with python-dns 1.15.0 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/150/head:pr150 git checkout pr150 -- Manage

[Freeipa-devel] [freeipa PR#150][comment] Fix compatibility with python-dns 1.15.0

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/150 Title: #150: Fix compatibility with python-dns 1.15.0 mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/8e02652e7c889ce7d32b592b5235917a0f519503 ipa-4-4: https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#150][+pushed] Fix compatibility with python-dns 1.15.0

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/150 Title: #150: Fix compatibility with python-dns 1.15.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#150][+ack] Fix compatibility with python-dns 1.15.0

2016-10-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/150 Title: #150: Fix compatibility with python-dns 1.15.0 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#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#145][comment] [WIP] Refactoring: LDAP Connection Management

2016-10-10 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/145 Title: #145: [WIP] Refactoring: LDAP Connection Management mbasti-rh commented: """ I did some inline comments, I'm not fully satisfied with implementation of the connection manager, I'll think about it tomorrow. "&quo

[Freeipa-devel] [freeipa PR#145][comment] [WIP] Refactoring: LDAP Connection Management

2016-10-10 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/145 Title: #145: [WIP] Refactoring: LDAP Connection Management mbasti-rh commented: """ Please fix PEP8 ``` ./ipapython/ipaldap.py:766:49: E231 missing whitespace after ',' ./ipapython/ipaldap.py:822:9: E265 block comme

[Freeipa-devel] [freeipa PR#147][+rejected] Tests: Unaccessible variable self.attrs in Tracker

2016-10-10 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/147 Title: #147: Tests: Unaccessible variable self.attrs in Tracker 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#134][comment] DNS URI support

2016-10-07 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/134 Title: #134: DNS URI support mbasti-rh commented: """ I'm so sorry, I didn't noticed earlier, but you forgot to bump API in VERSION Otherwise LGTM and works for me """ See the full comment at https://githu

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

2016-10-07 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: """ Works for me, I'm not pushing this yet if somebody wants to double check pickle implementation "&quo

[Freeipa-devel] [freeipa PR#108][comment] Bump pki min version and add commentary about sub-CA revocation on delete

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/108 Title: #108: Bump pki min version and add commentary about sub-CA revocation on delete mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/6b3f4984296f3caff8f29490eae3ed1dca64b8c3 https://f

[Freeipa-devel] [freeipa PR#108][+pushed] Bump pki min version and add commentary about sub-CA revocation on delete

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/108 Title: #108: Bump pki min version and add commentary about sub-CA revocation on delete 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#108][closed] Bump pki min version and add commentary about sub-CA revocation on delete

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/108 Author: frasertweedale Title: #108: Bump pki min version and add commentary about sub-CA revocation on delete 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#115][closed] Don't show traceback when ipa config file is not an absolute path

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/115 Author: tomaskrizek Title: #115: Don't show traceback when ipa config file is not an absolute path Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/115/head:pr115

[Freeipa-devel] [freeipa PR#115][comment] Don't show traceback when ipa config file is not an absolute path

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/115 Title: #115: Don't show traceback when ipa config file is not an absolute path mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/d7a2dfddbc2dc9ae4cea7d65e56d61a6a4d2b928 https://fedorahos

[Freeipa-devel] [freeipa PR#123][comment] Tests: Remove silent deleting and creating entries by tracker

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/123 Title: #123: Tests: Remove silent deleting and creating entries by tracker mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/74e52e86867372365d1d63561f7d1ff961b89ee0 """

[Freeipa-devel] [freeipa PR#123][+pushed] Tests: Remove silent deleting and creating entries by tracker

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/123 Title: #123: Tests: Remove silent deleting and creating entries by tracker 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#139][comment] WebUI: Vault Management

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management mbasti-rh commented: """ Yeah, and I forgot to write: 11) There should be an information in webUI, that secrets can be added/retrieved to vault only by using vault-archive and vault-re

[Freeipa-devel] [freeipa PR#139][comment] WebUI: Vault Management

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management mbasti-rh commented: """ 1) I created shared vault, but I cannot see it in 'Shared Vaults', it is show only in 'My Vaults' i.e. it was created ad user vault according CLI 2) 'My Vau

[Freeipa-devel] [freeipa PR#139][comment] WebUI: Vault Management

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management mbasti-rh commented: """ I'm not sure if this is done on purpose, but Vault section is shown there even I have no KRA installed in topology, and I'm getting error ``` An error has occurred

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

2016-10-06 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: """ IMO here (__init__ of CheckedIPAddress) is missing self._net = addr._net it may cause issues ``` if

[Freeipa-devel] [freeipa PR#133][comment] Tests: print what was expected from exceptions and callables in xmlrpc_tests

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/133 Title: #133: Tests: print what was expected from exceptions and callables in xmlrpc_tests mbasti-rh commented: """ Please set proper patch author, otherwise LGTM """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#134][comment] DNS URI support

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/134 Title: #134: DNS URI support mbasti-rh commented: """ I was able to add an invalid URI record ``` [root@vm-058-017 ~]# ipa dnsrecord-add test.zone. --uri-rec='0 0 trolo"lo' Record name: test2 Record name: test2 URI r

[Freeipa-devel] [freeipa PR#134][comment] DNS URI support

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/134 Title: #134: DNS URI support mbasti-rh commented: """ NACK, please see inline comments """ See the full comment at https://github.com/freeipa/freeipa/pull/134#issuecomment-251916849 -- Manage your subscription

[Freeipa-devel] [freeipa PR#108][+ack] Bump pki min version and add commentary about sub-CA revocation on delete

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/108 Title: #108: Bump pki min version and add commentary about sub-CA revocation on delete 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#115][+ack] Don't show traceback when ipa config file is not an absolute path

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/115 Title: #115: Don't show traceback when ipa config file is not an absolute path 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#132][comment] Draft for a new setup.py (WIP)

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/132 Title: #132: Draft for a new setup.py (WIP) mbasti-rh commented: """ This WIP works for me, I like that we get rid of setup.py.in files. I'm looking forward to final version Please fix PEP8 reported error and my inline co

[Freeipa-devel] [freeipa PR#135][comment] Pylint: remove unused variables

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/135 Title: #135: Pylint: remove unused variables mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/d9375881460d63cdd696bb0705da0ac205db9870 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#135][+pushed] Pylint: remove unused variables

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/135 Title: #135: Pylint: remove unused variables 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#135][closed] Pylint: remove unused variables

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/135 Author: mbasti-rh Title: #135: Pylint: remove unused variables Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/135/head:pr135 git checkout pr135 -- Manage your

[Freeipa-devel] [freeipa PR#125][closed] Add iSecStore.span

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/125 Author: tiran Title: #125: Add iSecStore.span Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/125/head:pr125 git checkout pr125 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#125][+pushed] Add iSecStore.span

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/125 Title: #125: Add iSecStore.span 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#128][+pushed] Properly handle LDAP socket closures in ipa-otpd

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/128 Title: #128: Properly handle LDAP socket closures in ipa-otpd 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#128][closed] Properly handle LDAP socket closures in ipa-otpd

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/128 Author: npmccallum Title: #128: Properly handle LDAP socket closures in ipa-otpd Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/128/head:pr128 git checkout pr128

[Freeipa-devel] [freeipa PR#138][comment] Fix ipa-cacert-manage man page

2016-10-06 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/138 Title: #138: Fix ipa-cacert-manage man page mbasti-rh commented: """ Is this written in IdM guide, if not IMO it would be nice to open doc bug in BZ and add this info there as well """ See the full comment at http

[Freeipa-devel] [freeipa PR#125][+ack] Add iSecStore.span

2016-10-05 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/125 Title: #125: Add iSecStore.span 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#136][edited] Fix KRA install tests

2016-10-05 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/136 Author: mbasti-rh Title: #136: Fix KRA install tests Action: edited Changed field: title Original value: """ [WIP] Fix KRA install tests """ -- Manage your subscription for the Freeipa-devel mailing list:

<    2   3   4   5   6   7   8   9   10   >