Merge lp:~simpoir/landscape-client/fix_tempfile_cleaning into lp:~landscape/landscape-client/trunk
Proposed by
Simon Poirier
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Simon Poirier | ||||
Approved revision: | 919 | ||||
Merged at revision: | 917 | ||||
Proposed branch: | lp:~simpoir/landscape-client/fix_tempfile_cleaning | ||||
Merge into: | lp:~landscape/landscape-client/trunk | ||||
Diff against target: |
320 lines (+44/-43) 10 files modified
landscape/broker/tests/test_store.py (+2/-10) landscape/lib/tests/test_persist.py (+7/-7) landscape/manager/aptsources.py (+6/-1) landscape/manager/scriptexecution.py (+3/-6) landscape/monitor/tests/test_cephusage.py (+2/-4) landscape/monitor/tests/test_usermonitor.py (+2/-3) landscape/package/tests/test_changer.py (+1/-1) landscape/tests/helpers.py (+18/-8) landscape/tests/test_service.py (+1/-1) landscape/tests/test_watchdog.py (+2/-2) |
||||
To merge this branch: | bzr merge lp:~simpoir/landscape-client/fix_tempfile_cleaning | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
🤖 Landscape Builder | test results | Approve | |
Chris Glass (community) | Approve | ||
Benji York (community) | Approve | ||
Review via email:
|
Commit message
This branches does some cleanup in test tempfiles and fixes their broken
removal in test helpers.
Description of the change
This branches does some cleanup in test tempfiles and fixes their broken
removal in test helpers.
Testing instructions:
export TMP=$(mktemp -d)
trial -j8 landscape
ls $TMP
To post a comment you must log in.
Command: TRIAL_ARGS=-j4 make check /ci.lscape. net/job/ latch-test/ 7857/
Result: Success
Revno: 918
Branch: lp:~simpoir/landscape-client/fix_tempfile_cleaning
Jenkins: https:/