Einav Cohen has posted comments on this change.

Change subject: webadmin: Fix System tree when in Gluster-only mode
......................................................................


Patch Set 7:

Thanks for posting the screen-shots, Anmol - much appreciated. 

A couple of clarifications (and I apologize in advance if you have already 
answered that in here one way or another): 

- In "Both" mode: will we have the 'root' "Clusters" node as well as the 
"Data-Centers" 'root' node (assuming we have both 'virt' Data-Centers and 
gluster-only Clusters in the setup)? 

- Will a specific Cluster appear twice in the tree if it is enabled for both 
the 'virt' service and the 'gluster' service (once beneath the "Clusters" 
'root' node and once beneath the "Clusters" node of the specific Data-Center to 
which this Cluster belongs)? 

Have you tested your code in the particular cases above to see that the UI is 
behaving as expected?

Thanks!

-- 
To view, visit https://gerrit.ovirt.org/39363
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c85bc2b78f885511a8e53212984d93343ffd6a8
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: anmolbabu <[email protected]>
Gerrit-Reviewer: Alexander Wels <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Greg Sheremeta <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
Gerrit-Reviewer: anmolbabu <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to