Change in ovirt-engine[master]: Verify vdsm cluster level support before send getHardwareInf...

yzaslavs at redhat.com yzaslavs at redhat.com
Wed Jan 16 10:06:16 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>



More information about the Engine-commits mailing list