[Freeipa-devel] [PATCH] 822 webui: topology plugin

[Martin Babinsky]

On 05/27/2015 04:14 PM, Petr Vobornik wrote:
> On 05/26/2015 12:22 PM, Petr Vobornik wrote:
>> On 05/15/2015 01:50 PM, Petr Vobornik wrote:
>>> On 04/21/2015 04:09 PM, Petr Vobornik wrote:
>>>> First iteration of Topology plugin Web UI.
>>>>
>>>> It reflects current state of topology plugin python part which is
>>>> implemented in "[PATCH] manage replication topology in the shared tree"
>>>> and my wip patch.
>>>>
>>>> I expect that the server API part will change a bit therefore this will
>>>> as well.
>>>>
>>>> Graphical visualization/management (ticket 4286)  will be
>>>> implemented in
>>>> separate patch.
>>>>
>>>> https://fedorahosted.org/freeipa/ticket/4997
>>>> http://www.freeipa.org/page/V4/Manage_replication_topology
>>>>
>>>>
>>>
>>> New version attached. It requires stage user web ui patches in order to
>>> apply (I expect that user life cycle backend will be pushed sooner than
>>> topology)
>>>
>>> Changes:
>>> - Left host and Right host fields are now host comboboxes
>>> - Connectivity are radio buttons with "both, left-right, right-left,
>>> none" options
>>> - segment name is not a required field in its adder dialog
>>>
>>> IMHO "Attributes to strip", "Attributes to replicate", "Attributes for
>>> total update", "Initialize replica", "Session timeout", "Replication
>>> agreement enabled" fields should not be just free-form textboxes, but
>>> they should be more specific, e.g. a checkbox for "Replication agreement
>>> enabled" or integer for "Session timeout", but that should be modified
>>> first in the backend python plugin.
>>>
>>>
>>
>> New patchset which replaces the old patch.
>>
>> Contains Web UI for:
>> - topologysuffix, topologysegment, domain level, server
>>
>> Backend is implemented in patches:
>> - tbabej 325-9
>> - pvoborni 855, 857
>>
>>
>
> New update which reflects the API change in domain level patches.
> (domainlevel-show changed to domainlevel-get).
>
> Now it depends only on pvoborni 857-2, the rest was pushed.
>
>

The patches seem to do what they are supposed to do.

However, I have not found any UI element implementing the 
'topologysegment-refresh' functionality. Was this only an oversight or 
do you plan to implement it in next patch?

-- 
Martin^3 Babinsky