Nir Soffer has posted comments on this change. Change subject: utils: Add execCmd sync mode failing stress tests ......................................................................
Patch Set 2: The new tests always fail with current code. They always pass if rebased on http://gerrit.ovirt.org/27553. I think we need another patch before this patch, disabling the stress tests by default, since it does not make sense to run stress test each time you run the tests, and forcing everyone to run with some SKIP_STRESS_TESTS flag is crazy. When these tests are enabled, we may like more iterations and more delays between functions calls. In previous life, I used to run stress tests that run for few days. There is no reason to limit them to 30 seconds or so because they happen to be in the test suit. -- To view, visit http://gerrit.ovirt.org/27784 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I57ebd5faf028ccfc928a8b2e7ae47008f0970c23 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: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches