Merge pull request #1462 from edx/e0d/fix-race-condition

fixes nginx access log rotate issue

playbooks/edx-east/retire_host.yml

 # ansible-playbook -i ./lifecycle_inventory.py ./retire_host.yml 
 # -e@/vars/env.yml --limit Terminating_Wait
 
-- name: Server retirement workflow
-  hosts: all
+#
+# This is separate because it's use of handlers
+# leads to various race conditions.
+#
+- name: Stop all services
+  hosts: Terminating_Wait
   sudo: True
   gather_facts: False
   vars:
     STOP_ALL_EDX_SERVICES_EXTRA_ARGS: "--no-wait"
   roles:
     - stop_all_edx_services
+
+- name: Server retirement workflow
+  hosts: Terminating_Wait
+  sudo: True
+  gather_facts: False
   tasks:
     - name: Force a log rotation
       command: /usr/sbin/logrotate -f /etc/logrotate.d/{{ item }}
@@ -34,12 +43,13 @@
         - "tracking.log"
         - "edx-services"
     - name: Terminate existing s3 log sync
-      command: /usr/bin/pkill send-logs-to-s3
+      command: /usr/bin/pkill send-logs-to-s3 || true
     - name: Send logs to s3
       command: /edx/bin/send-logs-to-s3
 
+      
 - name: Run minos verification
-  hosts: all
+  hosts: Terminating_Wait
   sudo: True
   gather_facts: False
   tasks: