The test suite leaves temp files around
Bug #476418 reported by
Free Ekanayaka
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Released
|
Medium
|
Free Ekanayaka | ||
landscape-client (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
Running the Landscape Client test suite leaves temporary files in the
system temporary directory (/tmp).
This is a problem for long running machines running the suite
(e.g. buildbot slaves) that eventually run out of available file
descriptors.
affects landscape-client
status inprogress
importance medium
assignee free.ekanayaka
milestone 1.4.1
Related branches
tags: | added: needs-testing |
tags: | removed: needs-testing |
Changed in landscape-client: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
This is ready for review in the attached branch. For now I've decided to leak a bit the implementation of MockerTestCase by calling its private __cleanup method in run_isolated, because it makes the fix very simple and isolated.
So if you look at your /tmp after the test suite has run you'll notice that from the hundreds of files we had before we're down to only one, which is a pipe, probably left open by some script execution test. I'll have to investigate that further.