Roll `service_report` function into class

GetDotaStats / site

The GetDotaStats site in all of its glory

http://getdotastats.com

MIT License

7 stars 2 forks source link

Roll `service_report` function into class #219

Closed jimmydorry closed 8 years ago

jimmydorry commented 8 years ago

Doh. When I updated the crons recently, I forgot that a class had been made.

[x] Implemented
[x] Tested
[x] Fix bugs
[ ] Option to turn off specific checks

jimmydorry commented 8 years ago

[11-13 10:02:04] [CRON] [CMG] Warning: First time running cron `s2_cron_cmg_111`! It may be new! || http://getdotastats.com/s2/routine/log_hourly.html?1447372912

Check how the class determines service name. Something went wrong here.

jimmydorry commented 8 years ago

Probs done https://github.com/GetDotaStats/site/commit/fcd45fde7297fd4dd5bbb56e0a7e0050ed7e1007

jimmydorry commented 8 years ago

Be able to turn off specific checks, so good increases don't get reported like:

[CRON] [WORKSHOP] Warning: Major increase (>1%) in performance index #1! 20 successful scrapes to 21 successful scrapes

jimmydorry commented 8 years ago

No longer relevant. New reporting class needs to be built into new cron system.