Moti Asayag has uploaded a new change for review.

Change subject: engine: Validate host provider on host update command
......................................................................

engine: Validate host provider on host update command

When setting the host provider for an existing host, the provider
should be verified for existence and for type compliance.

Change-Id: I1ded3b710957bade0f8f91a39ff9cdef9e4c2147
Bug-Url: https://bugzilla.redhat.com/1157377
Signed-off-by: Moti Asayag <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/55/37555/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java
index 230b661..becf0c7 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java
@@ -80,6 +80,8 @@
                 && validate(validator.updatePortAllowed())
                 && validate(validator.clusterNotChanged())
                 && validate(validator.changeProtocolAllowed())
+                && validate(validator.hostProviderExists())
+                && validate(validator.hostProviderTypeMatches())
                 && validateNetworkProviderConfiguration()
                 && 
isPowerManagementLegal(getParameters().getVdsStaticData().isPmEnabled(),
                         getParameters().getFenceAgents(),


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1ded3b710957bade0f8f91a39ff9cdef9e4c2147
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to