Skip to content
This repository was archived by the owner on Aug 8, 2019. It is now read-only.

One click to delete all errs#44

Open
sodabrew wants to merge 1 commit into
brightroll:masterfrom
sodabrew:delete_all_errs
Open

One click to delete all errs#44
sodabrew wants to merge 1 commit into
brightroll:masterfrom
sodabrew:delete_all_errs

Conversation

@sodabrew

Copy link
Copy Markdown
Collaborator

Implements #29

screen shot 2014-01-25 at 12 24 42 pm
screen shot 2014-01-25 at 12 24 29 pm

@sodabrew

Copy link
Copy Markdown
Collaborator Author

I wonder if it would be better to delete by date rather than all or nothing.

@bwitt

bwitt commented Jan 29, 2014

Copy link
Copy Markdown
Contributor

Usually we're archiving them all (to /rq/tmp/queuename). Can we do that instead? Also if /rq/tmp/queuename is owned by root or otherwise unwriteable, we just put into /rq/tmp (this is what the clean_rq_errors.sh script does)

@sodabrew

Copy link
Copy Markdown
Collaborator Author

At what point do the errors actually get deleted? This seems like sweeping under the rug.

I could make the button run a cleaner script, and then the script defines the behavior (possibly with options in the main rq config file).

@bwitt

bwitt commented Jan 29, 2014

Copy link
Copy Markdown
Contributor

We have a cron that cleans them after 2 weeks. Yeah maybe just run the cleaner script.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants