Vojtech Szocs has posted comments on this change.

Change subject: webadmin: manage cluster networks dialog
......................................................................


Patch Set 8:

New patch set with following updates included:

1. ClusterNetworkManageModel isRequired/setRequired now delegates to the 
corresponding "network" entity. This makes sure that during attach 
(AttachNetworkToVdsGroup), all network entities provided to this action have 
their "required" attribute set according to what user selected on UI.

2. The "set cluster display network" action (UpdateDisplayToVdsGroup) is going 
to get deprecated, see [http://gerrit.ovirt.org/3679]. Display network update 
is done during cluster network attach. Because of this, extra 
UpdateDisplayToVdsGroup-related logic was removed from ClusterNetworkListModel.

--
To view, visit http://gerrit.ovirt.org/3209
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I617a73105dcadd6733b4a854ebdece47b066d367
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Danny Rankevich <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Danny Rankevich <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to