Skip to content

bug 855089 - remove redundant cron scripts (NEEDS REBASING to change bug number)#1159

Closed
peterbe wants to merge 4 commits intomozilla-services:masterfrom
peterbe:bug855095-remove-redundant-cron-scripts
Closed

bug 855089 - remove redundant cron scripts (NEEDS REBASING to change bug number)#1159
peterbe wants to merge 4 commits intomozilla-services:masterfrom
peterbe:bug855095-remove-redundant-cron-scripts

Conversation

@peterbe
Copy link
Contributor

@peterbe peterbe commented Mar 26, 2013

There's some other old crufty looking code in socorro/cron like fixBrokenDumps.py and serverstatus.py which I have don't know about. What I do know is that they we're replaced by crontabber.

@ghost ghost assigned lonnen Mar 26, 2013
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure about this one? I see it active in the prod crontab still

@rhelmer
Copy link
Contributor

rhelmer commented Mar 26, 2013

prod crontab for reference:

Socorro cron job manager

*/5 * * * * socorro /data/socorro/application/scripts/crons/crontabber.sh

Generate data for /status page

*/5 * * * * socorro /data/socorro/application/scripts/crons/cron_status.sh

Modulelist report for crash-analysis

00 17 * * * socorro export JAVA_HOME=/usr/lib/jvm/java-1.6.0; /data/socorro/application/scripts/crons/cron_modulelist.sh

Correlation report for crash-analysis

05 00 * * * socorro /data/socorro/application/scripts/crons/cron_libraries.sh > /var/log/socorro/cron_libraries.log 2>&1

Daily CSV report for crash-analysis

55 03 * * * socorro /data/bin/cron_daily_reports.sh >> /var/log/socorro/cron_daily_reports.log

#fixes minidump stuff for b2g
0 * * * * socorro /data/socorro/application/scripts/crons/cron_fixbrokendumps.sh

PROD ONLY - submit crashes to stage

*/4 * * * * socorro /data/socorro/application/scripts/crons/cron_submitter.sh crash-reports.allizom.org 1000

@lonnen
Copy link
Contributor

lonnen commented Mar 27, 2013

r+

1 similar comment
@rhelmer
Copy link
Contributor

rhelmer commented Mar 27, 2013

r+

@peterbe
Copy link
Contributor Author

peterbe commented Mar 27, 2013

ce5026c

@peterbe peterbe closed this Mar 27, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants