Change in ovirt-engine[ovirt-engine-4.1]: core: use default mac pool for clusters without DC

Code Review gerrit at ovirt.org
Sun Jan 15 09:03:16 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: core: use default mac pool for clusters without DC
......................................................................


core: use default mac pool for clusters without DC

There might exist clusters, which are not linked to any data center.
For those first update statement does not work, leaving possible null
values in cluster.mac_pool_id column, and because of that adding of
constraint will fail.

We have to select some mac pool to be used for such clusters, and
having no information which should be used, we can use only default
mac pool.

Change-Id: If33a212d6c8f730bae8ebe46bc1d6359e95c0495
Bug-Url: https://bugzilla.redhat.com/1410189
Signed-off-by: Martin Mucha <mmucha at redhat.com>
---
M packaging/dbscripts/upgrade/04_01_0010_add_mac_pool_id_to_vds_group.sql
1 file changed, 13 insertions(+), 0 deletions(-)

Approvals:
  Martin Peřina: Looks good to me, approved
  Martin Mucha: Verified
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/70108
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If33a212d6c8f730bae8ebe46bc1d6359e95c0495
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list