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

Reply via email to