Closed Bug 512258 Opened 15 years ago Closed 15 years ago

Need to Clear Validation cache in maintenance scripts

Categories

(addons.mozilla.org Graveyard :: Maintenance Scripts, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rjwalsh, Assigned: rjwalsh)

Details

(Whiteboard: [patch-5.1])

Attachments

(1 file, 2 obsolete files)

I was digging through the cache today and realized we never do anything to clean it.  We should clear this out, probably once a day or so to make sure that stuff doesn't build up here.
Are you talking about test_results_cache?
Target Milestone: --- → 5.1
Attachment #396245 - Flags: review?(clouserw)
Comment on attachment 396245 [details] [diff] [review]
Clear out validate-* in NETAPP_STORAGE

1) we should escape NETAPP_STORAGE because you never know what kind of crazy stuff IT puts in there ;)

2) We should capture the output and send it to debug()

3) Why are you using (!-name -prune)?  Can't you just do something like: -name "validate-*"
Attachment #396245 - Flags: review?(clouserw) → review-
Attached patch v2 (obsolete) — Splinter Review
Attachment #396245 - Attachment is obsolete: true
Attachment #396462 - Flags: review?(clouserw)
Comment on attachment 396462 [details] [diff] [review]
v2

> /dev/null gives us no output
Attachment #396462 - Flags: review?(clouserw) → review-
Attached patch v3Splinter Review
No /dev/null
Attachment #396462 - Attachment is obsolete: true
Attachment #396517 - Flags: review?(clouserw)
Comment on attachment 396517 [details] [diff] [review]
v3

I'll r+ but this won't work unless shell_exec() is all on the same line so please change that before committing.
Attachment #396517 - Flags: review?(clouserw) → review+
Whiteboard: [patch-5.1]
Fixed in r50386
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: