Lior Vernia has submitted this change and it was merged.
Change subject: core: Mark management network as display and migration
......................................................................
core: Mark management network as display and migration
Introducing an upgrade script to mark the management network as
display and/or migration, if no other network in the cluster has that
role. This is implicitly the case anyway because when these roles
haven't been assigned they fall back to the management network, this
makes it explicit.
Also added code to assign the roles to the management network when
another network loses the role, or when a new cluster is created.
Change-Id: I73bf3c583ae28bf32d7c602ec9f4b0fb5c7f391d
Bug-Url:
https://bugzilla.redhat.com/1023722
Signed-off-by: Lior Vernia <lvernia(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsGroupCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/AttachNetworkToVdsGroupCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/UpdateNetworkOnClusterCommand.java
A packaging/dbscripts/upgrade/03_05_0460_set_display_migration_cluster_network.sql
5 files changed, 61 insertions(+), 8 deletions(-)
Approvals:
oVirt Jenkins CI Server: Looks good to me, but someone else must approve
Lior Vernia: Verified
Moti Asayag: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/22382
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I73bf3c583ae28bf32d7c602ec9f4b0fb5c7f391d
Gerrit-PatchSet: 13
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server