Arik Hadas has uploaded a new change for review. Change subject: core: remove vm fields for migration from & to ports ......................................................................
core: remove vm fields for migration from & to ports VM#migreatingToPort & VM#migreatingFromPort fields were unused so they are removed. Change-Id: I0742195dabff682943c5afae08ffa6e9539497a4 Signed-off-by: Arik Hadas <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java 2 files changed, 0 insertions(+), 24 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/73/25573/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java index a6225b3..42e41c6 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java @@ -123,8 +123,6 @@ boolean migrateSucceeded = connectLunDisks(getDestinationVdsId()) && migrateVm(); if (!migrateSucceeded) { - getVm().setMigreatingToPort(0); - getVm().setMigreatingFromPort(0); throw new VdcBLLException(VdcBllErrors.RESOURCE_MANAGER_MIGRATION_FAILED_AT_DST); } ExecutionHandler.setAsyncJob(getExecutionContext(), true); diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java index 9242d97..91b9797 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java @@ -1142,26 +1142,6 @@ vmStatistics = value; } - private int migreatingToPort; - - public int getMigreatingToPort() { - return migreatingToPort; - } - - public void setMigreatingToPort(int value) { - migreatingToPort = value; - } - - private int migreatingFromPort; - - public int getMigreatingFromPort() { - return migreatingFromPort; - } - - public void setMigreatingFromPort(int value) { - migreatingFromPort = value; - } - private String runOnVdsName; public String getRunOnVdsName() { @@ -1337,8 +1317,6 @@ result = prime * result + (int) (temp ^ (temp >>> 32)); result = prime * result + ((floppyPath == null) ? 0 : floppyPath.hashCode()); result = prime * result + ((initializationType == null) ? 0 : initializationType.hashCode()); - result = prime * result + migreatingFromPort; - result = prime * result + migreatingToPort; result = prime * result + ((privateGuestAgentVersion == null) ? 0 : privateGuestAgentVersion.hashCode()); result = prime * result + ((runOnVdsName == null) ? 0 : runOnVdsName.hashCode()); result = prime * result + (runOnce ? 1231 : 1237); -- To view, visit http://gerrit.ovirt.org/25573 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0742195dabff682943c5afae08ffa6e9539497a4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
