Change in ovirt-engine[master]: Verify vdsm cluster level support before send getHardwareInf...
Yair Zaslavsky has submitted this change and it was merged.
Change subject: Verify vdsm cluster level support before send getHardwareInfo request
......................................................................
Verify vdsm cluster level support before send getHardwareInfo request
Without verifying vdsm cluster support level the engine sends API
request that does not exist in vdsm. This raises exception in engine.
This path adds verification for vdsm cluster support.
Change-Id: I240786444bdf393f6df4e53daf94d0ee197ecb73
Bug-Id: https://bugzilla.redhat.com/show_bug.cgi?id=894317
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/FeatureSupported.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
2 files changed, 20 insertions(+), 2 deletions(-)
Approvals:
Yair Zaslavsky: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11027
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I240786444bdf393f6df4e53daf94d0ee197ecb73
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>