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 <tsa...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmosk...@redhat.com>
Gerrit-Reviewer: Martin Sivák <msi...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Tomer Saban <tsa...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to