From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has posted comments on this change.
Change subject: tests: Add thread leak check for pytest based tests
......................................................................
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/78881/1//COMMIT_MSG
Commit Message:
PS1, Line 9: pytest-threadleak
hmmm, the upstream author of this seems a bit dubious. should we trust him not to sneak evil code into vdsm tests?
seriously, the commit message should mention what is this plugin, who wrote it, when, and why there's no --processleak nor --fdleak yet!
--
To view, visit
https://gerrit.ovirt.org/78881
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ida95106d4333e1fe826d0f9df46ebfbe03791fd7
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer
nsoffer@redhat.com
Gerrit-Reviewer: Adam Litke
alitke@redhat.com
Gerrit-Reviewer: Dan Kenigsberg
danken@redhat.com
Gerrit-Reviewer: Edward Haas
edwardh@redhat.com
Gerrit-Reviewer: Francesco Romani
fromani@redhat.com
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer
nsoffer@redhat.com
Gerrit-Reviewer: Piotr Kliczewski
piotr.kliczewski@gmail.com
Gerrit-Reviewer: gerrit-hooks
automation@ovirt.org
Gerrit-HasComments: Yes