Change in ovirt-hosted-engine-setup[master]: packaging: setup: handle bridge on additional hosts

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Nov 4 12:11:59 UTC 2013


Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: setup: handle bridge on additional hosts
......................................................................


packaging: setup: handle bridge on additional hosts

Previously on an additional host hosted-engine-setup checked
for ip addr of the bridge instead of the ip addr of any existing
device. But on additional host the bridge creation is done by
the manager when adding the host.

Enabled the NIC selection for bridge creation also on additional
hosts if the bridge is not found and use its ip

Bug-Url: https://bugzilla.redhat.com/1025253
Change-Id: I81a5edada72d397c3ce30410d903840b1392034e
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py
M src/plugins/ovirt-hosted-engine-setup/network/bridge.py
2 files changed, 28 insertions(+), 13 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved
  Martin Sivák: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I81a5edada72d397c3ce30410d903840b1392034e
Gerrit-PatchSet: 1
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: Martin Sivák <msivak 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>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list