Change in ovirt-engine[master]: core: Change method name in AbstractDiskVmCommand

tnisan at redhat.com tnisan at redhat.com
Mon Apr 11 13:51:55 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: core: Change method name in AbstractDiskVmCommand
......................................................................


core: Change method name in AbstractDiskVmCommand

The method isDiskExist name is misleading since we are testing not only
the disk existence but also whether it's attached to the VM, the method
name was changed to reflect that

Change-Id: Icf6f40d43a6b9eac44eb51234e3f53296f96e56e
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/AbstractDiskVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/DetachDiskFromVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/HotPlugDiskToVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/UpdateVmDiskCommand.java
4 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Tal Nisan: Verified; Passed CI tests
  Allon Mureinik: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/55760
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icf6f40d43a6b9eac44eb51234e3f53296f96e56e
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list