Tomer Saban has posted comments on this change.

Change subject: webadmin: Fixed VM moving between clusters error.
......................................................................


Patch Set 1:

In the file "ExistingVmModelBehavior.java", I changed the indentation That's 
why Gerrit says that so many lines were changed. So I noting the changes that 
were done:

Removed line 196:
updateCpuProfile(vm.getVdsGroupId(), vm.getVdsGroupCompatibilityVersion(), 
vm.getCpuProfileI
d());

Added from line 204:

@Override
public void dataCenterWithClusterSelectedItemChanged()
{
    super.dataCenterWithClusterSelectedItemChanged();
    if (getModel().getSelectedCluster() != null)
    {
        updateCpuProfile(getModel().getSelectedCluster().getId(),
                         getClusterCompatibilityVersion(),
                         vm.getCpuProfileId());
    }
}

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I191425b7d6ac53d0f6f9c3d5a3ba53d52e7e8c42
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomer Saban <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Jiří Moskovčák <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Tomer Saban <[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