On 08/06/16 14:44, Jan Cholasta wrote:
On 8.6.2016 14:40, Martin Babinsky wrote:
On 06/08/2016 02:11 PM, David Kupka wrote:
On 28/04/16 14:45, Jan Cholasta wrote:

I have pushed my thin client WIP branch to GitHub:

All commits up to "ipalib: use relative imports for cross-plugin
imports" should be good for review. The rest is subject to change
(WARNING: I will force push into this branch).



Patch set:

spec file: require correct packages to get API plugins
schema: fix typo
schema: fix topic command output
replica install: use remote server API to create service entries
This one is not complete since it breaks replica install w/ --setup-dns.
Removing this line seems to fix this case:

diff --git a/ipaserver/install/server/replicainstall.py
index 41eee96..d695a15 100644
--- a/ipaserver/install/server/replicainstall.py
+++ b/ipaserver/install/server/replicainstall.py
@@ -1478,7 +1478,6 @@ def promote(installer):

     if options.setup_dns:
-        server_api.Backend.rpcclient.connect()
         dns.install(False, True, options, server_api


schema: do not validate unrequested params in command_defaults

fixes some regressions introduced by previous patches. Works for me and
I haven't met any new regression or bug, ACK.

Thanks for the catch, Martin. Now it's really fixed and can be pushed. Obligatory keyword to trigger the push: ACK

BTW, I've discovered other related issue [1] that is there since 4.2. I will send patch soon.

[1] https://fedorahosted.org/freeipa/ticket/5945

David Kupka

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to