Change in ovirt-engine[master]: engine: Verify host provider exists when it is set

masayag at redhat.com masayag at redhat.com
Mon Feb 9 14:27:26 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list