On 11/24/2015 12:17 PM, Petr Vobornik wrote:
On 11/24/2015 12:10 PM, Ludwig Krispenz wrote:
I'm testing these patches.Two observations so far:
- in Topology->IPA Servers I see a table of my servers and the managed
suffix column I see both suffixes, ipaca and the realm,
but if I select one of the servers I Only see the realm suffix, this was
different in the demo video
implemented in patch 927 (separate thread)
- the graph layou is not stable. If I arrange it and the try some
actions on a segment, eg try to remove one (which is rejected), the
graph does change again-
Not sure if I understand it correctly. One can set static position to a
node by double clicking on it (this needs to be somehow communicated)
otherwise it is random - each refresh.
Maybe we should not refresh on failed operation because there is no change.
+1, failed operations on segments should not trigger graph redraws.
On 11/20/2015 04:00 PM, Petr Vobornik wrote:
Patches 928-931 are prerequisites.
Patches 932-934 implements the visualization
Patches 935-936 adds 'add' and 'remove' segment functionality to the
I have tested the new feature a bit (I have looked also on the code, but
1.) If you first setup topology w/ domain level 0 and try to render
topology graph, you are correctly informed that managed topology is
available on domain level 1 only. IF you raise your domainlevel and
click refresh, you get the same error message, probably because the
request to get domainlevel is sent only in the beginning of session. You
have to logout and then login back to see the graph.
2.) If I add a new segment using Add dialog and clicking 'Add & edit'
button, I get the following error:
IPA Error 3007: RequirementError
'topologysuffixcn' is required
However the segment is created normally. I guess that there is some bug
when viewing the segment immediately after creation.
3.) The key with suffix colors should be more descriptive. I would like
to propose the following format:
Color coding of segments by suffix:
4.) It would be nice if the graph canvas could be dynamically resized
based on the container size, but I am not sure how difficult it is to
(Graphical nitpick: I would change the color of nodes, green does not
fit very well with the bluish/grey palette of other WebUI elements)
Otherwise I think that it is a good start and additional features can be
built on top of this patchset.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code