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 <sbona...@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 <sbona...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadg...@redhat.com>
Gerrit-Reviewer: Leonid Natapov <leonidnata...@gmail.com>
Gerrit-Reviewer: Lev Veyde <lve...@gmail.com>
Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to