[Freeipa-devel] [freeipa PR#258][+pushed] Break ipaplatform / ipalib import cycle of hell

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/258 Title: #258: Break ipaplatform / ipalib import cycle of hell 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#258][comment] Break ipaplatform / ipalib import cycle of hell

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/258 Title: #258: Break ipaplatform / ipalib import cycle of hell mbasti-rh commented: """ ACK from me if @jcholast is not against it """ See the full comment at https://github.com/freeipa/freeipa/pull/258#issuecom

[Freeipa-devel] [freeipa PR#264][-ack] Python3 pylint fixes

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/264 Title: #264: Python3 pylint fixes 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#263][comment] Backwards compatibility with setuptools 0.9.8

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/263 Title: #263: Backwards compatibility with setuptools 0.9.8 mbasti-rh commented: """ Please add ticket to commit message """ See the full comment at https://github.com/freeipa/freeipa/pull/263#issuecomment-26278695

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

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/263 Title: #263: Backwards compatibility with setuptools 0.9.8 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#264][+ack] Python3 pylint fixes

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/264 Title: #264: Python3 pylint fixes 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#265][comment] Add main guards to a couple of Python scripts

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/265 Title: #265: Add main guards to a couple of Python scripts mbasti-rh commented: """ Is this part of PIP effort? If yes then commit misses ticket otherwise ACK """ See the full comment at https://github.com/freeipa

[Freeipa-devel] [freeipa PR#257][comment] Don't ship install subpackages with wheels

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/257 Title: #257: Don't ship install subpackages with wheels mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/526bcea705d04895aa6b09bce996ac340783d1d0 """ See the full comment

[Freeipa-devel] [freeipa PR#257][closed] Don't ship install subpackages with wheels

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/257 Author: tiran Title: #257: Don't ship install subpackages with wheels Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/257/head:pr257 git checkout pr257 -- Manage

[Freeipa-devel] [freeipa PR#257][+ack] Don't ship install subpackages with wheels

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/257 Title: #257: Don't ship install subpackages with wheels 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#257][+pushed] Don't ship install subpackages with wheels

2016-11-24 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/257 Title: #257: Don't ship install subpackages with wheels 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#257][comment] Don't ship install subpackages with wheels

2016-11-23 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/257 Title: #257: Don't ship install subpackages with wheels mbasti-rh commented: """ ACK, please rebase """ See the full comment at https://github.com/freeipa/freeipa/pull/257#issuecomment-262560605 -- Manage you

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

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management mbasti-rh commented: """ NACK 1) view My User Vaults/Add vault There is no marked radio button and all fields are shown which are mutual exclusive. A one option from radio group should be mark

[Freeipa-devel] [freeipa PR#259][comment] Minor fixes for IPAVersion class

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/259 Title: #259: Minor fixes for IPAVersion class mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/29947fe1a304ff6f913e5d94d56d8108a7c94087 """ See the full comment at http

[Freeipa-devel] [freeipa PR#259][+pushed] Minor fixes for IPAVersion class

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/259 Title: #259: Minor fixes for IPAVersion class 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#259][closed] Minor fixes for IPAVersion class

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/259 Author: tiran Title: #259: Minor fixes for IPAVersion class Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/259/head:pr259 git checkout pr259 -- Manage your

[Freeipa-devel] [freeipa PR#259][+ack] Minor fixes for IPAVersion class

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/259 Title: #259: Minor fixes for IPAVersion class 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#256][comment] Pylint: whitelist packages with extension modules

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/256 Title: #256: Pylint: whitelist packages with extension modules mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/573eee444e1746fd5949897294c96a1793e74511 """ See the full

[Freeipa-devel] [freeipa PR#235][+pushed] Remove unused Knob function

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function 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#235][comment] Remove unused Knob function

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/55b14abcb561422cf48755dae6b0638656535fe5 """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#256][closed] Pylint: whitelist packages with extension modules

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/256 Author: tiran Title: #256: Pylint: whitelist packages with extension modules Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/256/head:pr256 git checkout pr256 --

[Freeipa-devel] [freeipa PR#256][+pushed] Pylint: whitelist packages with extension modules

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/256 Title: #256: Pylint: whitelist packages with extension modules 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#256][+ack] Pylint: whitelist packages with extension modules

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/256 Title: #256: Pylint: whitelist packages with extension modules 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#235][+ack] Remove unused Knob function

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function 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#235][comment] Remove unused Knob function

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Remove unused Knob function mbasti-rh commented: """ @jcholast ping """ See the full comment at https://github.com/freeipa/freeipa/pull/235#issuecomment-262279274 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#223][+pushed] LDAP refactoring: remove admin_conn

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn 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#223][comment] LDAP refactoring: remove admin_conn

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/68295bf8cfd57333deb50f58df1b336a4b48ffe7 https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#223][closed] LDAP refactoring: remove admin_conn

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/223 Author: tomaskrizek Title: #223: LDAP refactoring: remove admin_conn Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/223/head:pr223 git checkout pr223 -- Manage

[Freeipa-devel] [freeipa PR#238][comment] Build system refactoring phase 8: update translation system

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/238 Title: #238: Build system refactoring phase 8: update translation system mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/4842231074683ff68be50b147560f5383aa305b6 https://fedorahos

[Freeipa-devel] [freeipa PR#238][+pushed] Build system refactoring phase 8: update translation system

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/238 Title: #238: Build system refactoring phase 8: update translation system 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#238][closed] Build system refactoring phase 8: update translation system

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/238 Author: pspacek Title: #238: Build system refactoring phase 8: update translation system Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/238/head:pr238 git

[Freeipa-devel] [freeipa PR#223][+ack] LDAP refactoring: remove admin_conn

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn 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#238][+ack] Build system refactoring phase 8: update translation system

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/238 Title: #238: Build system refactoring phase 8: update translation system 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#122][+ack] Acceptance tests

2016-11-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/122 Title: #122: Acceptance tests 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#223][comment] LDAP refactoring: remove admin_conn

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn mbasti-rh commented: """ LGTM and Works for me, but I have to make sure that things I wrote inline won''t happen """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#223][comment] LDAP refactoring: remove admin_conn

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn mbasti-rh commented: """ LGTM and Works for me, but I have to make sure that things I wrote inline didn't happen """ See the full comment at https://githu

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

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/255 Title: #255: Adjustments for setup requirements mbasti-rh commented: """ I found some changes in versions of dependencies I don't like, because there is no explanation why it is needed and it is out of sync between specfil

[Freeipa-devel] [freeipa PR#249][+pushed] Remove references to ds_newinst.pl

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/249 Title: #249: Remove references to ds_newinst.pl 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#244][comment] Add templating to ipaplatform path [RFC]

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/244 Title: #244: Add templating to ipaplatform path [RFC] mbasti-rh commented: """ Can you finish this PR? It can be tested and possibly merged """ See the full comment at https://github.com/freeipa/freeipa/p

[Freeipa-devel] [freeipa PR#249][closed] Remove references to ds_newinst.pl

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/249 Author: frasertweedale Title: #249: Remove references to ds_newinst.pl Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/249/head:pr249 git checkout pr249 --

[Freeipa-devel] [freeipa PR#249][comment] Remove references to ds_newinst.pl

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/249 Title: #249: Remove references to ds_newinst.pl mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/687ebd18a1927cd6dcbb6cb884b979096c8a44aa """ See the full comment at http

[Freeipa-devel] [freeipa PR#212][+pushed] KRA: don't add KRA container when KRA replica

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/212 Title: #212: KRA: don't add KRA container when KRA replica 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#212][closed] KRA: don't add KRA container when KRA replica

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/212 Author: mbasti-rh Title: #212: KRA: don't add KRA container when KRA replica Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/212/head:pr212 git checkout pr212

[Freeipa-devel] [freeipa PR#212][comment] KRA: don't add KRA container when KRA replica

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/212 Title: #212: KRA: don't add KRA container when KRA replica mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/61094a2a20f5cacdb7c87940d0db8d8593a87505 """ See the full

[Freeipa-devel] [freeipa PR#256][comment] Pylint: whitelist packages with extension modules

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/256 Title: #256: Pylint: whitelist packages with extension modules mbasti-rh commented: """ LGTM, I'll test it tomorrow """ See the full comment at https://github.com/freeipa/freeipa/pull/256#issuecomment-26199789

[Freeipa-devel] [freeipa PR#258][comment] Break ipaplatform / ipalib import cycle of hell

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/258 Title: #258: Break ipaplatform / ipalib import cycle of hell mbasti-rh commented: """ LGTM, except the inline comment I made, I'll test it tomorrow """ See the full comment at https://github.com/freeipa/freeipa/p

[Freeipa-devel] [freeipa PR#259][comment] Minor fixes for IPAVersion class

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/259 Title: #259: Minor fixes for IPAVersion class mbasti-rh commented: """ LGTM, I'll test tomorrow """ See the full comment at https://github.com/freeipa/freeipa/pull/259#issuecomment-261995775 -- Manage your subs

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

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/261 Title: #261: upgrade: do not set HTTP and DS principals explicitly 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#260][closed] Build: fix path in ipa-ods-exporter.socket unit file

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/260 Author: pspacek Title: #260: Build: fix path in ipa-ods-exporter.socket unit file Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/260/head:pr260 git checkout

[Freeipa-devel] [freeipa PR#260][+ack] Build: fix path in ipa-ods-exporter.socket unit file

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/260 Title: #260: Build: fix path in ipa-ods-exporter.socket unit file 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#209][comment] Enumerate available options in IPA installer

2016-11-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/209 Title: #209: Enumerate available options in IPA installer mbasti-rh commented: """ @jcholast I know, but it doesn't fill `metavar` with choices. I don't know when we will migrate to argparse, so I think until that we can ex

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

2016-11-18 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/254 Title: #254: Replace LooseVersion with parse_ipa_version() mbasti-rh commented: """ Thank you for PY3 fix, it actually belongs to this ticket https://fedorahosted.org/freeipa/ticket/6473 """ See the full comment

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

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

[Freeipa-devel] [freeipa PR#250][closed] ipapython and ipatest no longer require lxml

2016-11-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/250 Author: tiran Title: #250: ipapython and ipatest no longer require lxml Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/250/head:pr250 git checkout pr250 --

[Freeipa-devel] [freeipa PR#250][+pushed] ipapython and ipatest no longer require lxml

2016-11-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/250 Title: #250: ipapython and ipatest no longer require lxml 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#250][comment] ipapython and ipatest no longer require lxml

2016-11-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/250 Title: #250: ipapython and ipatest no longer require lxml mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/c93bfda594723357f3ff9f4eb8191f3d76df680f """ See the full comment

[Freeipa-devel] [freeipa PR#250][+ack] ipapython and ipatest no longer require lxml

2016-11-17 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/250 Title: #250: ipapython and ipatest no longer require lxml 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#206][comment] Properly handle multiple cookies in rpcclient

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/206 Title: #206: Properly handle multiple cookies in rpcclient mbasti-rh commented: """ At the end of server install many times and it is also printed by each ipa command ``` Forwarding 'ca_is_enabled' to json server 'https://vm-058-0

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

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/126 Author: flo-renaud Title: #126: Fix ipa migrate-ds when it finds a search reference Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/126/head:pr126 git checkout

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

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/126 Title: #126: Fix ipa migrate-ds when it finds a search reference 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#126][comment] Fix ipa migrate-ds when it finds a search reference

2016-11-16 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: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/efb3700389ff46244189fa95779484eb099d63b4 """ See the full

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

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/126 Title: #126: Fix ipa migrate-ds when it finds a search reference 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#237][closed] Update man page for ipa-adtrust-install by removing --no-msdcs option

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/237 Author: pspacek Title: #237: Update man page for ipa-adtrust-install by removing --no-msdcs option Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/237/head:pr237

[Freeipa-devel] [freeipa PR#237][comment] Update man page for ipa-adtrust-install by removing --no-msdcs option

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/237 Title: #237: Update man page for ipa-adtrust-install by removing --no-msdcs option mbasti-rh commented: """ Fixed upstream ipa-4-4: https://fedorahosted.org/freeipa/changeset/ef988aab6c756d5fec4513c182d702fb0a1db249 "&quo

[Freeipa-devel] [freeipa PR#234][comment] Always use GSSAPI to set up initial replication

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/234 Title: #234: Always use GSSAPI to set up initial replication mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/9d7943f3da7fb84975cc8f45047aafee13bf85dc https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#234][+pushed] Always use GSSAPI to set up initial replication

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/234 Title: #234: Always use GSSAPI to set up initial replication 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#234][+ack] Always use GSSAPI to set up initial replication

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/234 Title: #234: Always use GSSAPI to set up initial replication 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#234][closed] Always use GSSAPI to set up initial replication

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/234 Author: martbab Title: #234: Always use GSSAPI to set up initial replication Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/234/head:pr234 git checkout pr234 --

[Freeipa-devel] [freeipa PR#187][+pushed] Register entry points of Custodia plugins

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/187 Title: #187: Register entry points of Custodia plugins 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#187][+ack] Register entry points of Custodia plugins

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/187 Title: #187: Register entry points of Custodia plugins 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#187][comment] Register entry points of Custodia plugins

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/187 Title: #187: Register entry points of Custodia plugins mbasti-rh commented: """ Rebased, ticket added to commit message, pushed Fixed upstream master: https://fedorahosted.org/freeipa/changeset/9102fb3b02fbe55480428e60fb8df4fd668

[Freeipa-devel] [freeipa PR#187][closed] Register entry points of Custodia plugins

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/187 Author: tiran Title: #187: Register entry points of Custodia plugins Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/187/head:pr187 git checkout pr187 -- Manage

[Freeipa-devel] [freeipa PR#241][+ack] Port ipapython.dnssec.odsmgr to xml.etree

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/241 Title: #241: Port ipapython.dnssec.odsmgr to xml.etree 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#241][comment] Port ipapython.dnssec.odsmgr to xml.etree

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/241 Title: #241: Port ipapython.dnssec.odsmgr to xml.etree mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/64af88fee4a482b3f393d38ff2c7f9494e689a7b https://fedorahosted.org/fr

[Freeipa-devel] [freeipa PR#241][+pushed] Port ipapython.dnssec.odsmgr to xml.etree

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/241 Title: #241: Port ipapython.dnssec.odsmgr to xml.etree 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#241][closed] Port ipapython.dnssec.odsmgr to xml.etree

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/241 Author: tiran Title: #241: Port ipapython.dnssec.odsmgr to xml.etree Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/241/head:pr241 git checkout pr241 -- Manage

[Freeipa-devel] [freeipa PR#241][comment] Port ipapython.dnssec.odsmgr to xml.etree

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/241 Title: #241: Port ipapython.dnssec.odsmgr to xml.etree mbasti-rh commented: """ There is already DNSSEC issue that is not caused by this PR https://fedorahosted.org/freeipa/ticket/6495 """ See the full comment

[Freeipa-devel] [freeipa PR#195][+pushed] Make ipaclient pip install-able

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/195 Title: #195: Make ipaclient pip install-able 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#195][comment] Make ipaclient pip install-able

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/195 Title: #195: Make ipaclient pip install-able mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/8346e1b067483d4d836627a267805bbe8d6e7efa """ See the full comment at https://g

[Freeipa-devel] [freeipa PR#195][closed] Make ipaclient pip install-able

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/195 Author: tiran Title: #195: Make ipaclient pip install-able Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/195/head:pr195 git checkout pr195 -- Manage your

[Freeipa-devel] [freeipa PR#195][+ack] Make ipaclient pip install-able

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/195 Title: #195: Make ipaclient pip install-able 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#248][+pushed] Fix the naming of ipa-dnskeysyncd service principal

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/248 Title: #248: Fix the naming of ipa-dnskeysyncd service principal 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#248][comment] Fix the naming of ipa-dnskeysyncd service principal

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/248 Title: #248: Fix the naming of ipa-dnskeysyncd service principal mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/6ca96b3db03d4f3c5dbf465ca3d36bd563771c47 """ See the full

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

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/248 Title: #248: Fix the naming of ipa-dnskeysyncd service principal 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#246][+pushed] Build: ignore rpmbuild for lint target

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/246 Title: #246: Build: ignore rpmbuild for lint target 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#246][closed] Build: ignore rpmbuild for lint target

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/246 Author: tomaskrizek Title: #246: Build: ignore rpmbuild for lint target Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/246/head:pr246 git checkout pr246 --

[Freeipa-devel] [freeipa PR#246][comment] Build: ignore rpmbuild for lint target

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/246 Title: #246: Build: ignore rpmbuild for lint target mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/28c5e128c823f32787a4bde87e6b248928a2cd0a """ See the full comment

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

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/247 Title: #247: Add 'ipa local-env' subcommand mbasti-rh commented: """ You have PEP8 error there ``` ./ipalib/cli.py:57:1: E402 module level import not at top of file ``` """ See the full comment at https://githu

[Freeipa-devel] [freeipa PR#244][comment] Add templating to ipaplatform path [RFC]

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/244 Title: #244: Add templating to ipaplatform path [RFC] mbasti-rh commented: """ Edit: wrong PR """ See the full comment at https://github.com/freeipa/freeipa/pull/244#issuecomment-260964435 -- Manage your subs

[Freeipa-devel] [freeipa PR#244][comment] Add templating to ipaplatform path [RFC]

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/244 Title: #244: Add templating to ipaplatform path [RFC] mbasti-rh commented: """ You have PEP8 error there ``` ./ipalib/cli.py:57:1: E402 module level import not at top of file ``` """ See the full comment at http

[Freeipa-devel] [freeipa PR#180][closed] Make api.env.nss_dir relative to api.env.confdir

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/180 Author: tiran Title: #180: Make api.env.nss_dir relative to api.env.confdir Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/180/head:pr180 git checkout pr180 --

[Freeipa-devel] [freeipa PR#180][comment] Make api.env.nss_dir relative to api.env.confdir

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/180 Title: #180: Make api.env.nss_dir relative to api.env.confdir mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/9006ed34bb1edfaafc3345c1128800dc802c14ff """ See the full

[Freeipa-devel] [freeipa PR#180][+pushed] Make api.env.nss_dir relative to api.env.confdir

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/180 Title: #180: Make api.env.nss_dir relative to api.env.confdir 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#180][+ack] Make api.env.nss_dir relative to api.env.confdir

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/180 Title: #180: Make api.env.nss_dir relative to api.env.confdir 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#247][comment] Add 'ipa show_env' subcommand

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/247 Title: #247: Add 'ipa show_env' subcommand mbasti-rh commented: """ So then it should be called `local-env` to be clear what does it do """ See the full comment at https://github.com/freeipa/freeipa/pull/247#iss

[Freeipa-devel] [freeipa PR#244][comment] Add templating to ipaplatform path [RFC]

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/244 Title: #244: Add templating to ipaplatform path [RFC] mbasti-rh commented: """ LGTM then, I'll wait for final version for testing """ See the full comment at https://github.com/freeipa/freeipa/pull/244#issuecom

[Freeipa-devel] [freeipa PR#241][comment] Port ipapython.dnssec.odsmgr to xml.etree

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/241 Title: #241: Port ipapython.dnssec.odsmgr to xml.etree mbasti-rh commented: """ Commit has no ticket `Use xml.etree in ipa-client-automount script` """ See the full comment at https://github.com/freeipa/freeipa/p

[Freeipa-devel] [freeipa PR#244][comment] Add templating to ipaplatform path [RFC]

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/244 Title: #244: Add templating to ipaplatform path [RFC] mbasti-rh commented: """ It has missing ticket """ See the full comment at https://github.com/freeipa/freeipa/pull/244#issuecomment-260937077 -- Manage you

[Freeipa-devel] [freeipa PR#182][comment] Use env var IPA_CONFDIR to get confdir for 'cli' context

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/182 Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context mbasti-rh commented: """ This should have test because it is completely new so it is not part of any current test suites. However final review will wai

[Freeipa-devel] [freeipa PR#223][comment] LDAP refactoring: remove admin_conn

2016-11-16 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/223 Title: #223: LDAP refactoring: remove admin_conn mbasti-rh commented: """ ACK: removing admin_conn connections - just for future, python doesn't like to have too many dereference from performance point fo view. So you should in

<    1   2   3   4   5   6   7   8   9   10   >