From Dan Kenigsberg <danken(a)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(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: Yes