Tomas Jelinek has posted comments on this change.

Change subject: webadmin: new VM guide dialog does not get updated
......................................................................


Patch Set 2: Verified+1

Good catch Daniel! I have not realized that the NewGuideVmInterfaceModel is not 
used because it seems to work as the new VM dialog already adds the first 
interface.

The NewGuideVmInterfaceModel has been introduced by 
4b9cb568535fb3cdf4ec0ef20a8e28af0217dd45 but it has never been used (not even 
in that patch => the refresh did not work since that patch). So I have adjusted 
the VmGuideModel to use also this one and verified that it does the refresh 
correctly.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7f6c38cdc316b8f9b3501e04980d7cb2e3908136
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[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