Moti Asayag has submitted this change and it was merged.
Change subject: engine: Verify host provider exists when it is set
......................................................................
engine: Verify host provider exists when it is set
The external host provider should be examined for
existence when it is provided for the host, instead
when it is only changed (which fails for the case
of changing provider from an existing to missing
provider).
Change-Id: Ifb91e124f690edbc8f85d86dd26e2ec379fc24e3
Signed-off-by: Moti Asayag <masayag(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/UpdateHostValidator.java
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/UpdateHostValidatorTest.java
2 files changed, 1 insertion(+), 14 deletions(-)
Approvals:
Moti Asayag: Verified
Yair Zaslavsky: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/37606
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifb91e124f690edbc8f85d86dd26e2ec379fc24e3
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org