Merge lp:~david-goetz/swift/auditor_bug into lp:~hudson-openstack/swift/trunk
Proposed by
David Goetz
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | John Dickinson | ||||
Approved revision: | 350 | ||||
Merged at revision: | 350 | ||||
Proposed branch: | lp:~david-goetz/swift/auditor_bug | ||||
Merge into: | lp:~hudson-openstack/swift/trunk | ||||
Diff against target: |
93 lines (+47/-24) 2 files modified
swift/obj/auditor.py (+27/-24) test/unit/obj/test_auditor.py (+20/-0) |
||||
To merge this branch: | bzr merge lp:~david-goetz/swift/auditor_bug | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
John Dickinson | Approve | ||
gholt (community) | Approve | ||
Review via email: mp+73449@code.launchpad.net |
Description of the change
Fix for object auditor. It doesn't close files that are quarantined for certain reasons, zero byte files for one, which will cause it to eventually crash due to keeping too many files open. Thanks David Kranz for finding / reporting this!!
To post a comment you must log in.
Your was_df stuff should have a try/finally