Change in ovirt-engine[ovirt-engine-4.1]: engine: normalize bonding options without miimon

Code Review gerrit at ovirt.org
Thu Mar 9 08:36:15 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: engine: normalize bonding options without miimon
......................................................................


engine: normalize bonding options without miimon

The bonding options are normalized by placing the option "mode"
in the first place and the option "miimon" in the second place
in the options string.

Previously, if the option "miimon" was missing, a space was added
the first place of the options string instead of the present option.

If the option "miimon" is missing, the option "mode" is still
positioned in the first place.

Change-Id: Ie3417fe2dc5bffb1b98c6a664da1c0be87fc54be
Bug-Url: https://bugzilla.redhat.com/1429810
Signed-off-by: Dominik Holler <dholler at redhat.com>
(cherry picked from commit a07cfabcaefc1dc9a5da607be9ab369bf118d7e6)
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
1 file changed, 42 insertions(+), 39 deletions(-)

Approvals:
  Dominik Holler: Verified
  Martin Mucha: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie3417fe2dc5bffb1b98c6a664da1c0be87fc54be
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Dominik Holler <dholler at redhat.com>
Gerrit-Reviewer: Dominik Holler <dholler at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>


More information about the Engine-commits mailing list