Change in ovirt-engine[ovirt-engine-3.6]: engine: handle bond options mode text value

tnisan at redhat.com tnisan at redhat.com
Mon Dec 21 13:30:01 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: engine: handle  bond options mode text value
......................................................................


engine: handle  bond options mode text value

Bond options value can be both numeric and text value.
This patch adds support for the text value.
Longer description using lines' length under 72 chars.
Bond options comming from vds will be converted to
numeric values to store them uniformly.

Change-Id: If0fd5df3d02288b72e0b210849cc43a0ff74b507
Bug-Url: https://bugzilla.redhat.com/1286967
Signed-off-by: Marcin Mirecki <mmirecki at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/BondMode.java
M backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/businessentities/network/BondModeTest.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
3 files changed, 74 insertions(+), 46 deletions(-)

Approvals:
  Marcin Mirecki: Verified
  Alona Kaplan: Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/50409
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If0fd5df3d02288b72e0b210849cc43a0ff74b507
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list