Change in ovirt-engine[master]: engine: Allow engine to configure management network

masayag at redhat.com masayag at redhat.com
Thu May 30 06:30:07 UTC 2013


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 at redhat.com>
---
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 at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Martin Pavlik <xpavli44 at gmail.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>



More information about the Engine-commits mailing list