Change in ovirt-engine[master]: core: Pull up storage domain static and dynamic in fixtures.xml

Code Review gerrit at ovirt.org
Thu Dec 29 10:47:17 UTC 2016


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

Tal Nisan has submitted this change and it was merged.

Change subject: core: Pull up storage domain static and dynamic in fixtures.xml
......................................................................


core: Pull up storage domain static and dynamic in fixtures.xml

Since introducing VM leases the VmStatic entity can hold a reference to a
storage domain with the column 'lease_sd_id', in fixtures.xml the
storage_domain_static table is defined after the vm_static table causing
any VM that holds a reference to a storage domain to fail the test
database creation on a foreign key constraint since the
storage_domain_static table is not created yet.
This patch moves the storage_domain_static and storage_domain_dynamic
table up, right after storage_pool to avoid this problem, this is also
more logical given the heirarchy of storage domain in the system.

Change-Id: I508876248ec5867974fe2cf3de4f6ecc17b5c42e
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M backend/manager/modules/dal/src/test/resources/fixtures.xml
1 file changed, 287 insertions(+), 286 deletions(-)

Approvals:
  Tal Nisan: Verified
  Jenkins CI: Passed CI tests
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I508876248ec5867974fe2cf3de4f6ecc17b5c42e
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list