Gilad Chaplik has posted comments on this change.

Change subject: webadmin: fix cpu profile changes for import
......................................................................


Patch Set 1:

(1 comment)

@Doron, it's not related to REST, There is a UI bug in the import dialog, not 
changing the cpu profile when changing cluster.

http://gerrit.ovirt.org/#/c/35931/1/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmFromExportDomainModel.java
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmFromExportDomainModel.java:

Line 283:     private void fetchCpuProfiles(Guid clusterId) {
Line 284:         
Frontend.getInstance().runQuery(VdcQueryType.GetCpuProfilesByClusterId,
Line 285:                 new IdQueryParameters(clusterId),
Line 286:                 new AsyncQuery(new INewAsyncCallback() {
Line 287:                     
> ???
Done
Line 288:                     @Override
Line 289:                     public void onSuccess(Object model, Object 
returnValue) {
Line 290:                         List<CpuProfile> cpuProfiles =
Line 291:                                 (List<CpuProfile>) 
((VdcQueryReturnValue) returnValue).getReturnValue();


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia524e72fcff6d3cc655e6aee9f078b9365faaba3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
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