Change in ovirt-hosted-engine-setup[master]: packaging: setup: handle bond and bridge interfaces

[sbonazzo at redhat.com]

Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: setup: handle bond and bridge interfaces
......................................................................


packaging: setup: handle bond and bridge interfaces

Previously all existing bond and bridge interfaces where
listed as possible nics for creating the management bridge.
If a bond interface was specified the bridge creation failed.

Now the bridge are not listed anymore and bonds are listed
only if they have slave interfaces.
Enslaved interfaces and interfaces used by other bridges
are not listed anymore as available for management bridge
creation.

If a bond interface with slaves is specified now should work
correctly.

Change-Id: If19dfa2c2b9b02ba8d997ad1758614fad3470eb5
Bug-Url: https://bugzilla.redhat.com/999577
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M ovirt-hosted-engine-setup.spec.in
M src/plugins/ovirt-hosted-engine-setup/network/bridge.py
2 files changed, 57 insertions(+), 5 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved
  Yedidyah Bar David: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/18406
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If19dfa2c2b9b02ba8d997ad1758614fad3470eb5
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Leonid Natapov <leonidnatapov at gmail.com>
Gerrit-Reviewer: Lev Veyde <lveyde at gmail.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>