[Freeipa-devel] [freeipa PR#226][+ack] Build refactoring phase 5

2016-11-11 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/226 Title: #226: Build refactoring phase 5 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#226][closed] Build refactoring phase 5

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/226 Author: pspacek Title: #226: Build refactoring phase 5 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/226/head:pr226 git checkout pr226 -- Manage your

[Freeipa-devel] [freeipa PR#226][synchronized] Build refactoring phase 5

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/226 Author: pspacek Title: #226: Build refactoring phase 5 Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/226/head:pr226 git checkout pr226 From

[Freeipa-devel] [freeipa PR#214][comment] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/214 Title: #214: ipaldap: remove do_bind from LDAPClient jcholast commented: """ DM password may be `None` in `dns_container_exists()` and `dnssec_container_exists()` (for example in `BindInstance.setup()`), so you can't simple bind

[Freeipa-devel] [freeipa PR#214][reopened] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/214 Author: tomaskrizek Title: #214: ipaldap: remove do_bind from LDAPClient Action: reopened To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/214/head:pr214 git checkout pr214 --

[Freeipa-devel] [freeipa PR#226][+pushed] Build refactoring phase 5

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/226 Title: #226: Build refactoring phase 5 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#226][comment] Build refactoring phase 5

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/226 Title: #226: Build refactoring phase 5 mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/394edf5f055766fa0cdf70afab6d263f75d0d065

[Freeipa-devel] [freeipa PR#233][opened] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Author: pspacek Title: #233: Build phase 6: %install cleanup Action: opened PR body: """ This patch set is based on phase 5 - PR #226. Now all the installation steps (except Python2/3) are handled by make install. Python 2/3 support will

[Freeipa-devel] [freeipa PR#214][synchronized] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/214 Author: tomaskrizek Title: #214: ipaldap: remove do_bind from LDAPClient Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/214/head:pr214 git checkout pr214

[Freeipa-devel] [freeipa PR#232][+ack] Installer refactoring

2016-11-11 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/232 Title: #232: Installer refactoring 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#232][+pushed] Installer refactoring

2016-11-11 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/232 Title: #232: Installer refactoring 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#233][synchronized] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Author: pspacek Title: #233: Build phase 6: %install cleanup Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/233/head:pr233 git checkout pr233 From

[Freeipa-devel] [freeipa PR#233][comment] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Title: #233: Build phase 6: %install cleanup pspacek commented: """ Rebased on top of current master. """ See the full comment at https://github.com/freeipa/freeipa/pull/233#issuecomment-259937908 -- Manage your subscription for the

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

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/234 Author: martbab Title: #234: Always use GSSAPI to set up initial replication Action: opened PR body: """ This PR makes DS replica use common method to set up initial replication in both domain levels, namely GSSAPI. Since the workflow was

[Freeipa-devel] [freeipa PR#232][comment] Installer refactoring

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/232 Title: #232: Installer refactoring mbasti-rh commented: """ ACK, we may ignore those minor PEP8 issues, it is mainly caused by copying code to other parts. """ See the full comment at

[Freeipa-devel] [freeipa PR#207][comment] Provide user hint about IP address in IPA install

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/207 Title: #207: Provide user hint about IP address in IPA install mbasti-rh commented: """ LGTM, I'll test later """ See the full comment at https://github.com/freeipa/freeipa/pull/207#issuecomment-259938389 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#174][comment] add log module

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/174 Title: #174: add log module mbasti-rh commented: """ @shanyin Did centralized logging meet your requirements? """ See the full comment at https://github.com/freeipa/freeipa/pull/174#issuecomment-259939617 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#214][synchronized] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/214 Author: tomaskrizek Title: #214: ipaldap: remove do_bind from LDAPClient Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/214/head:pr214 git checkout pr214

[Freeipa-devel] [freeipa PR#232][comment] Installer refactoring

2016-11-11 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/232 Title: #232: Installer refactoring jcholast commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/eac6f52957c361c219ad6048b515ddb62da31154

[Freeipa-devel] [freeipa PR#232][closed] Installer refactoring

2016-11-11 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/232 Author: jcholast Title: #232: Installer refactoring Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/232/head:pr232 git checkout pr232 -- Manage your subscription

[Freeipa-devel] [freeipa PR#233][comment] Build phase 6: %install cleanup

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/233 Title: #233: Build phase 6: %install cleanup mbasti-rh commented: """ Build failed ``` Failed to open: 'freeipa.spec.in', not a valid spec file. ``` """ See the full comment at https://github.com/freeipa/freeipa/pull/233#issuecomment-259939261

[Freeipa-devel] [freeipa PR#213][edited] Build system refactoring phase 3

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/213 Author: pspacek Title: #213: Build system refactoring phase 3 Action: edited Changed field: body Original value: """ This monster patch-set refactors most of build system and moves most of the logic from SPEC file to build system. It is not

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

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/234 Title: #234: Always use GSSAPI to set up initial replication mbasti-rh commented: """ ``` Traceback (most recent call last): File "/sbin/ipa-server-install", line 23, in from ipaserver.install import ipa_server_install File

[Freeipa-devel] [freeipa PR#214][comment] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/214 Title: #214: ipaldap: remove do_bind from LDAPClient mbasti-rh commented: """ ``` ipaserver/install/ca.py:226: [W0612(unused-variable), install_step_1] Unused variable 'dm_password') ``` """ See the full comment at

[Freeipa-devel] [freeipa PR#207][+ack] Provide user hint about IP address in IPA install

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/207 Title: #207: Provide user hint about IP address in IPA install 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#207][closed] Provide user hint about IP address in IPA install

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/207 Author: Akasurde Title: #207: Provide user hint about IP address in IPA install Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/207/head:pr207 git checkout pr207

[Freeipa-devel] [freeipa PR#207][comment] Provide user hint about IP address in IPA install

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/207 Title: #207: Provide user hint about IP address in IPA install mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/28bc54f91dfbd76887180fa67ceecb46977a4fb8 """ See the full comment at

[Freeipa-devel] [freeipa PR#233][comment] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Title: #233: Build phase 6: %install cleanup pspacek commented: """ @tiran Please re-review and set review status accordingly. Thanks! """ See the full comment at https://github.com/freeipa/freeipa/pull/233#issuecomment-259955838 -- Manage

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

2016-11-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/212 Title: #212: KRA: don't add KRA container when KRA replica stlaz commented: """ configure_instance and configure_replica codes were merged, you'll need to check for self.clone value instead. """ See the full comment at

[Freeipa-devel] [freeipa PR#214][synchronized] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/214 Author: tomaskrizek Title: #214: ipaldap: remove do_bind from LDAPClient Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/214/head:pr214 git checkout pr214

[Freeipa-devel] [freeipa PR#214][synchronized] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/214 Author: tomaskrizek Title: #214: ipaldap: remove do_bind from LDAPClient Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/214/head:pr214 git checkout pr214

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

2016-11-11 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/223 Author: tomaskrizek Title: #223: LDAP refactoring: remove admin_conn Action: synchronized 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 From

[Freeipa-devel] [freeipa PR#233][synchronized] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Author: pspacek Title: #233: Build phase 6: %install cleanup Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/233/head:pr233 git checkout pr233 From

[Freeipa-devel] [freeipa PR#233][synchronized] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Author: pspacek Title: #233: Build phase 6: %install cleanup Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/233/head:pr233 git checkout pr233 From

[Freeipa-devel] [freeipa PR#233][synchronized] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Author: pspacek Title: #233: Build phase 6: %install cleanup Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/233/head:pr233 git checkout pr233 From

[Freeipa-devel] [freeipa PR#233][comment] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Title: #233: Build phase 6: %install cleanup pspacek commented: """ I've fixed incorrect use of `-D` in `install` calls above. """ See the full comment at https://github.com/freeipa/freeipa/pull/233#issuecomment-259960522 -- Manage your

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

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/196 Title: #196: ipatests: unresolvable nested netgroups martbab commented: """ Is there any particular reason why this is among XML RPC tests and not a separate integration test? IMHO it should be a CI test as it tests integration with SSSD. I get

[Freeipa-devel] [freeipa PR#233][comment] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Title: #233: Build phase 6: %install cleanup pspacek commented: """ > Failed to open: 'freeipa.spec.in', not a valid spec file. Damn it! I added the last commit with comment in SPEC file and did not run tests on that. Surprise surprise, RPM is

[Freeipa-devel] [freeipa PR#207][+pushed] Provide user hint about IP address in IPA install

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/207 Title: #207: Provide user hint about IP address in IPA 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#233][comment] Build phase 6: %install cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/233 Title: #233: Build phase 6: %install cleanup pspacek commented: """ This version fixes the fixable issues, i.e. everything mentioned above except changing file ownership. """ See the full comment at

[Freeipa-devel] [freeipa PR#230][+ack] cert-request: accept CSRs with extraneous data

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/230 Title: #230: cert-request: accept CSRs with extraneous data 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#230][comment] cert-request: accept CSRs with extraneous data

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/230 Title: #230: cert-request: accept CSRs with extraneous data martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/e1df2e0792a6a423563c4787215b284948f51582 """ See the full comment at

[Freeipa-devel] [freeipa PR#230][+pushed] cert-request: accept CSRs with extraneous data

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/230 Title: #230: cert-request: accept CSRs with extraneous data 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#230][closed] cert-request: accept CSRs with extraneous data

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/230 Author: frasertweedale Title: #230: cert-request: accept CSRs with extraneous data Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/230/head:pr230 git checkout

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

2016-11-11 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: synchronized 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

[Freeipa-devel] [freeipa PR#209][comment] Enumerate available options in IPA installer

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/209 Title: #209: Enumerate available options in IPA installer mbasti-rh commented: """ Hi, we changed a lot of code during refactoring, PR doesn't apply. IMO list of choices shown in --help should be handled in knob() if metavar is not specified

[Freeipa-devel] [freeipa PR#214][comment] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/214 Title: #214: ipaldap: remove do_bind from LDAPClient martbab commented: """ The fix works as expected. ACK. Fixed upstream master: https://fedorahosted.org/freeipa/changeset/f183f70e0183e51d569ada972bd3ec73cad76a30 """ See the full comment at

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

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/234 Author: martbab Title: #234: Always use GSSAPI to set up initial replication Action: synchronized 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

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

2016-11-11 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/182 Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context tiran commented: """ http://www.freeipa.org/page/V4/Integration_Improvements#API_for_local_configuration_directory """ See the full comment at

[Freeipa-devel] [freeipa PR#214][closed] ipaldap: remove do_bind from LDAPClient

2016-11-11 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/214 Author: tomaskrizek Title: #214: ipaldap: remove do_bind from LDAPClient Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/214/head:pr214 git checkout pr214 --

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

2016-11-11 Thread simo5
URL: https://github.com/freeipa/freeipa/pull/187 Title: #187: Register entry points of Custodia plugins simo5 commented: """ Forgot the reasons, I was probably not thinking about PEP8 back then. """ See the full comment at https://github.com/freeipa/freeipa/pull/187#issuecomment-259963079 --

[Freeipa-devel] Design document: Integration Improvements

2016-11-11 Thread Christian Heimes
Hello, I have released the first version of a new design document. It describes how I'm going to improve integration of FreeIPA's client libraries (ipalib, ipapython, ipaclient, ipaplatform) for third party developers. http://www.freeipa.org/page/V4/Integration_Improvements Regards, Christian

[Freeipa-devel] [freeipa PR#235][opened] Make Knob function deprecated

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/235 Author: mbasti-rh Title: #235: Make Knob function deprecated Action: opened PR body: """ `Knob` function is outdated and was replaced by `knob`. Make explicit note in code about this. https://fedorahosted.org/freeipa/ticket/6392 """ To pull

[Freeipa-devel] [freeipa PR#235][+ack] Make Knob function deprecated

2016-11-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Make Knob function deprecated 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] Make Knob function deprecated

2016-11-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Make Knob function deprecated stlaz commented: """ ACK, there should be note about this deprecation somewhere. Deleting Knob might be worth a ticket as well. """ See the full comment at

[Freeipa-devel] [freeipa PR#235][-ack] Make Knob function deprecated

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

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/235 Title: #235: Make Knob function deprecated mbasti-rh commented: """ I would wait with ACK, I realized that Knobs with capital K are not used anymore, so we can remove it instead of deprecating """ See the full comment at

[Freeipa-devel] [freeipa PR#236][opened] Build phase 7: cleanup

2016-11-11 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/236 Author: pspacek Title: #236: Build phase 7: cleanup Action: opened PR body: """ Depends on PR #233. - Clean-up ancient leftovers and clean minor bugs here and there. - Support --enable-silent-rules and V=0 variable for make to make the build

[Freeipa-devel] [freeipa PR#236][comment] Build phase 7: cleanup

2016-11-11 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/236 Title: #236: Build phase 7: cleanup lslebodn commented: """ You should the opposite of the last patch "Build: clean-up spurious NULL variables from Makefile.am files". The NULL should be added to each list. NACK to the last patch. It does not

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

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/235 Author: mbasti-rh Title: #235: Remove unused Knob function Action: edited Changed field: title Original value: """ Make Knob function deprecated """ -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#235][synchronized] Make Knob function deprecated

2016-11-11 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/235 Author: mbasti-rh Title: #235: Make Knob function deprecated Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/235/head:pr235 git checkout pr235 From

Re: [Freeipa-devel] Design document: Integration Improvements

2016-11-11 Thread Martin Basti
On 11.11.2016 15:25, Christian Heimes wrote: Hello, I have released the first version of a new design document. It describes how I'm going to improve integration of FreeIPA's client libraries (ipalib, ipapython, ipaclient, ipaplatform) for third party developers.

Re: [Freeipa-devel] Design document: Integration Improvements

2016-11-11 Thread Christian Heimes
On 2016-11-11 17:46, Martin Basti wrote: > > > On 11.11.2016 15:25, Christian Heimes wrote: >> Hello, >> >> I have released the first version of a new design document. It describes >> how I'm going to improve integration of FreeIPA's client libraries >> (ipalib, ipapython, ipaclient,

Re: [Freeipa-devel] Design document: Integration Improvements

2016-11-11 Thread Rob Crittenden
Martin Basti wrote: > > > On 11.11.2016 15:25, Christian Heimes wrote: >> Hello, >> >> I have released the first version of a new design document. It describes >> how I'm going to improve integration of FreeIPA's client libraries >> (ipalib, ipapython, ipaclient, ipaplatform) for third party

Re: [Freeipa-devel] Design document: Integration Improvements

2016-11-11 Thread Christian Heimes
On 2016-11-11 18:33, Rob Crittenden wrote: > Martin Basti wrote: >> 2) if I understand correctly, you want to separate client installer code >> and client CLI code. In past we had freeipa-admintools but it was >> removed because it was really tightly bounded to installed client. Do >> you want to