Tal Nisan has uploaded a new change for review.
Change subject: core: Fix problem with changing host cluster
......................................................................
core: Fix problem with changing host cluster
The VDS entity vds_group_id property was changed to vdsGroupId, in the
canDo action of ChangeVDSCluster the field change validation was hard coded
to the old property name thus failing on host cluster change even if the
host is in maintenance state, changed the check to use the new field name
Change-Id: Ia5789c931e69bc2ed5f7840d8503dcf843a102ed
Signed-off-by: Tal Nisan <tni...@redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/78/11978/1
diff --git
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
index 1fc4f42..8c44f5b 100644
---
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
+++
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
@@ -58,7 +58,7 @@
addCanDoActionMessage(VdcBllMessages.VDS_INVALID_SERVER_ID);
return false;
}
- if (!ObjectIdentityChecker.CanUpdateField(vds, "vds_group_id",
vds.getStatus())) {
+ if (!ObjectIdentityChecker.CanUpdateField(vds, "vdsGroupId",
vds.getStatus())) {
addCanDoActionMessage(VdcBllMessages.VDS_STATUS_NOT_VALID_FOR_UPDATE);
return false;
}
--
To view, visit http://gerrit.ovirt.org/11978
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia5789c931e69bc2ed5f7840d8503dcf843a102ed
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tni...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
