Change in ovirt-engine[master]: core: Use disk id instead of volume id in OvfUtilsTest

Code Review gerrit at ovirt.org
Tue Jun 6 16:49:20 UTC 2017


>From Maor Lipchuk <mlipchuk at redhat.com>:

Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Use disk id instead of volume id in OvfUtilsTest
......................................................................


core: Use disk id instead of volume id in OvfUtilsTest

This fix is related to test of unregistered disks.
The engine filters out disks which are attached to VMs to know
which are unregistered disks and which are not.
Since the filtering is being done with the disk id the test should
be changed to use the disk id in the OVF instead of the volume id.

Change-Id: Ib5d1f613a6ead694e93f8a29cda53b0f24d6de03
Bug-Url: https://bugzilla.redhat.com/1443641
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/utils/OvfUtilsTest.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Maor Lipchuk: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib5d1f613a6ead694e93f8a29cda53b0f24d6de03
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.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