Lior Vernia has posted comments on this change.

Change subject: webadmin: Copy VNIC profile configuration from template
......................................................................


Patch Set 1:

(1 comment)

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmModelBehavior.java
Line 25: import org.ovirt.engine.ui.uicommonweb.models.SystemTreeItemType;
Line 26: 
Line 27: public class NewVmModelBehavior extends VmModelBehaviorBase {
Line 28: 
Line 29:     private final ProfileBehavior networkBehavior = new 
EditProfileBehavior();
The logic here is of editing the profiles defined in the template rather than 
starting from scratch. However, your concern is justified in the case of the 
Blank template, whose behavior now changed to the worse slightly. Let me think 
this over :)
Line 30: 
Line 31:     @Override
Line 32:     public void initialize(SystemTreeItemModel systemTreeSelectedItem)
Line 33:     {


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iffc7f38ce068a076fd66c245c178d9ff4813155a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lver...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Lior Vernia <lver...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to