Dan Kenigsberg has submitted this change and it was merged. Change subject: tests: Separate broken test from good tests ......................................................................
tests: Separate broken test from good tests In commit 29fa0abb687d (storagetests: add a test for normalize_local_path) we added a new failing test for path ending with slash. To keep the build successful, the test was marked as @brokentest. However, since this test uses permuations, all the tests were marked broken, instead of only the new test. Separate the failing tests from the other, so the good tests will fail the build when they fail. Change-Id: I5963fcd0fb25fba6305045de80346fb0cfa3666b Signed-off-by: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/57481 Continuous-Integration: Jenkins CI Reviewed-by: Freddy Rolland <froll...@redhat.com> --- M tests/storageServerTests.py 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Nir Soffer: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/57481 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5963fcd0fb25fba6305045de80346fb0cfa3666b Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Freddy Rolland <froll...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches