Closed
Bug 1034980
Opened 11 years ago
Closed 11 years ago
mozpool dbcron should remove its tempfile when done
Categories
(Infrastructure & Operations :: RelOps: Puppet, task)
Infrastructure & Operations
RelOps: Puppet
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dustin, Assigned: dustin)
Details
Attachments
(1 file)
|
1.64 KB,
patch
|
arich
:
review+
dustin
:
checked-in+
|
Details | Diff | Splinter Review |
Particularly since running the cron as root creates that temp file as root, preventing it from running as apache later.
|
Assignee | |
Comment 1•11 years ago
|
||
Attachment #8457011 -
Flags: review?(arich)
|
||
Comment 2•11 years ago
|
||
Comment on attachment 8457011 [details] [diff] [review]
bug1034980.patch
Review of attachment 8457011 [details] [diff] [review]:
-----------------------------------------------------------------
I've never personally used trap to catch successful exits, but I'm going to trust that you've done this before, tested it, and know that it works that way.
Attachment #8457011 -
Flags: review?(arich) → review+
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8457011 -
Flags: checked-in+
|
|
Assignee | |
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•