Change in ovirt-engine[master]: engine: Avoid storing external network label in oVirt-engine DB

masayag at redhat.com masayag at redhat.com
Sun Apr 27 13:38:55 UTC 2014


Moti Asayag has submitted this change and it was merged.

Change subject: engine: Avoid storing external network label in oVirt-engine DB
......................................................................


engine: Avoid storing external network label in oVirt-engine DB

Network labels for oVirt-engine and for Neutron should not be mixed when
creating a network and labelling it. For oVirt-engine network only
oVirt-engine labels should be presented for newly created network
and for Neutron network only Neutron labels.
External network label should not be stored in oVirt-engine DB.
As soon as we won't store the external network labels it would never be
mixed up with the oVirt-engine managed network labels.

Change-Id: Ic3c041d6900a8af852f1d186c626be315d205be6
Bug-Url: https://bugzilla.redhat.com/1077132
Signed-off-by: Yevgeny Zaspitsky <yzaspits at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/network/AddNetworkOnProviderCommand.java
A packaging/dbscripts/upgrade/03_05_0330_nullify_external_network_labels.sql
2 files changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Moti Asayag: Looks good to me, approved
  Yevgeny Zaspitsky: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic3c041d6900a8af852f1d186c626be315d205be6
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list