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

Code Review gerrit at ovirt.org
Thu Jan 26 11:12:45 UTC 2017


>From Moti Asayag <masayag at redhat.com>:

Moti Asayag 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
Related-To: https://bugzilla.redhat.com/1412563
Signed-off-by: Dominik Holler <dholler at redhat.com>
---
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/70923
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie3417fe2dc5bffb1b98c6a664da1c0be87fc54be
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dominik Holler <dholler at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dominik Holler <dholler at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>


More information about the Engine-commits mailing list