Dan Kenigsberg has submitted this change and it was merged.

Change subject: tests: Remove hack for devices with same names
......................................................................


tests: Remove hack for devices with same names

After the test isolation issues in tcTests.py and ipwrapperTests.py were
resolved, we are not expecting setup to fail because a device with same
name exists, so we can remove the hack that used to skip tests.

It is possible that slaves contains stale devices from previous run of
the broken tests, but this should be solved by cleaning the slaves, not
by hiding the error and skipping tests.

Change-Id: I675e57756859ca7ffe272607b7f528c281c65256
Signed-off-by: Nir Soffer <nsof...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/44166
Reviewed-by: Ido Barkan <ibar...@redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M tests/tcTests.py
1 file changed, 1 insertion(+), 11 deletions(-)

Approvals:
  Nir Soffer: Verified
  Ido Barkan: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I675e57756859ca7ffe272607b7f528c281c65256
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Roman Mohr <rm...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to