[Freeipa-devel] [WIP] Thin client

[Jan Cholasta]

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:
>>> Hi,
>>>
>>> I have pushed my thin client WIP branch to GitHub:
>>> <https://github.com/jcholast/freeipa/tree/trac-4739>.
>>>
>>> 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).
>>>
>>> Honza
>>>
>>
>> Hello!
>>
>> 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
> b/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):
>      server_api.finalize()
>
>      if options.setup_dns:
> -        server_api.Backend.rpcclient.connect()
>          server_api.Backend.ldap2.connect(autobind=True)
>          dns.install(False, True, options, server_api
> """

Fixed.

>
>
>> 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.
>>
>
>


-- 
Jan Cholasta