From 2c2783e127cb40e4462b86590a3108bcb76bb81c Mon Sep 17 00:00:00 2001
From: Ralph Bean <rbean@redhat.com>
Date: Fri, 23 Aug 2013 15:17:25 +0000
Subject: [PATCH] Add logs and logrotate for badge award cronjobs on
 badges-backend01.

---
 .../files/cron/award-libravatar-badge.cron           |  2 +-
 .../files/cron/award-oldschool-badges.cron           |  2 +-
 .../badges-backend/files/cron/cron-badges-logrotate  | 12 ++++++++++++
 roles/badges-backend/tasks/main.yml                  | 10 ++++++++++
 4 files changed, 24 insertions(+), 2 deletions(-)
 create mode 100644 roles/badges-backend/files/cron/cron-badges-logrotate

diff --git a/roles/badges-backend/files/cron/award-libravatar-badge.cron b/roles/badges-backend/files/cron/award-libravatar-badge.cron
index 0357c150af..e4f2a4d036 100644
--- a/roles/badges-backend/files/cron/award-libravatar-badge.cron
+++ b/roles/badges-backend/files/cron/award-libravatar-badge.cron
@@ -1 +1 @@
-40 * * * * fedmsg /usr/share/badges/cronjobs/award-libravatar-badge > /dev/null 2>&1
+40 * * * * fedmsg /usr/share/badges/cronjobs/award-libravatar-badge > /var/log/fedmsg/cron-award-libravatar-badge.log 2>&1
diff --git a/roles/badges-backend/files/cron/award-oldschool-badges.cron b/roles/badges-backend/files/cron/award-oldschool-badges.cron
index 33c91e9328..6fce244621 100644
--- a/roles/badges-backend/files/cron/award-oldschool-badges.cron
+++ b/roles/badges-backend/files/cron/award-oldschool-badges.cron
@@ -1 +1 @@
-25 * * * * fedmsg /usr/share/badges/cronjobs/award-oldschool-badges > /dev/null 2>&1
+25 * * * * fedmsg /usr/share/badges/cronjobs/award-oldschool-badges > /var/log/fedmsg/cron-award-oldschool-badges.log 2>&1
diff --git a/roles/badges-backend/files/cron/cron-badges-logrotate b/roles/badges-backend/files/cron/cron-badges-logrotate
new file mode 100644
index 0000000000..856d8cd428
--- /dev/null
+++ b/roles/badges-backend/files/cron/cron-badges-logrotate
@@ -0,0 +1,12 @@
+/var/log/fedmsg/cron-award-libravatar-badge.log {
+   missingok
+   notifempty
+   rotate 4
+   weekly
+}
+/var/log/fedmsg/cron-award-oldschool-badges.log {
+   missingok
+   notifempty
+   rotate 4
+   weekly
+}
diff --git a/roles/badges-backend/tasks/main.yml b/roles/badges-backend/tasks/main.yml
index 2abe9dd790..e5e26e5d1e 100644
--- a/roles/badges-backend/tasks/main.yml
+++ b/roles/badges-backend/tasks/main.yml
@@ -109,6 +109,16 @@
   - config
   - cron
 
+- name: logrotate for badge award cronjobs
+  copy: >
+    src=cron/cron-badges-logrotate
+    dest=/etc/logrotate.d/cron-badges-logrotate
+    owner=root
+    mode=644
+  tags:
+  - config
+  - cron
+
 - name: copy over the badge-admin one-off scripts
   copy: >
     src=$item