Merge pull request #360 from edx/jarv/logrotate

logrotate is not a service

playbooks/roles/common/handlers/main.yml

@@ -2,9 +2,3 @@
 - name: common | restart rsyslogd
   service: name=rsyslog state=restarted
   sudo: True
-
-- name: common | restart logrotate
-  service: name=logrotate state=restarted
-  sudo: True
-
-

playbooks/roles/nginx/tasks/main.yml

@@ -67,14 +67,12 @@
 
 - name: nginx | Set up nginx access log rotation
   template: dest=/etc/logrotate.d/nginx-access src=edx_logrotate_nginx_access.j2 owner=root group=root mode=644
-  notify: common | restart logrotate
   tags:
   - logging
   - update
 
 - name: nginx | Set up nginx access log rotation
   template: dest=/etc/logrotate.d/nginx-error src=edx_logrotate_nginx_error.j2 owner=root group=root mode=644
-  notify: common | restart logrotate
   tags:
   - logging
   - update
@@ -85,7 +83,7 @@
   tags:
   - nginx
   - update
-  
+
 # If tasks that notify restart nginx don't change the state of the remote system
 # their corresponding notifications don't get run.  If nginx has been stopped for
 # any reason, this will ensure that it is started up again.