Moti Asayag has submitted this change and it was merged. Change subject: engine: Allow engine to configure management network ...................................................................... engine: Allow engine to configure management network The patch eliminates* the need of ovirt-host-deploy to create the management network on top of the installed host. The management network will be created as the last step of the installation process, by the logical network definition in the data-center. Since the network creation relies on the setupNetworks, no further action will be taken for non-supported cluster versions. * ovirt-nodes are the exceptional in this process and for them (regardless their version), the ovirt-host-deploy will create the management bridge (regardless the logical network definition). If the network is defined otherwise, the admin will have to use the 'Sync Network' option via the setupNetworks dialog to enable it. Once ovirt-node will not create the br* bridges, the engine will be able to use the same logic for configuring the management network on the node. Change-Id: Iaf82e10481e595d690e7ce894283b4ed5b9b3269 Bug-Url: https://bugzilla.redhat.com/891056 Signed-off-by: Moti Asayag <masayag(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/NetworkConfigurator.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VersionSupport.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 6 files changed, 415 insertions(+), 51 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14230 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf82e10481e595d690e7ce894283b4ed5b9b3269 Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com&gt; Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com&gt; Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt; Gerrit-Reviewer: Livnat Peer <lpeer(a)redhat.com&gt; Gerrit-Reviewer: Martin Pavlik <xpavli44(a)gmail.com&gt; Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com&gt; Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com&gt; Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;