Updating-cron-path-fixing-mailto

Author: drazenCE

roles/cron/cron_drupal7/defaults/main.yml

@@ -12,7 +12,7 @@ drupal:
           # month:
           job: cron
           # disabled: true
-          mailto: "{{ drupal.cron_mailto | default('') }}"
+          # mailto: "{{ drupal.cron_mailto | default('') }}" # Each cron can have it's own mailto and can be configured to use different e-mail addresses.
   cron_mailto: ""
   # If the sites are being deployed to an ASG, setting defer to true will create the crontab entry on the deploy server rather than all of the app servers.
   defer: false

roles/cron/cron_drupal7/tasks/job.yml

@@ -1,7 +1,7 @@
 ---
 - name: Define cron job command.
   ansible.builtin.set_fact:
-    _cron_job_command: "cd {{ deploy_path }}/{{ webroot }}/sites/{{ site.folder }} && {{ drupal.drush_location }} {{ entry.job }}"
+    _cron_job_command: "cd {{ live_symlink_dest }}/{{ webroot }}/sites/{{ site.folder }} && {{ drupal.drush_location }} {{ entry.job }} > /dev/null 2>&1"
 
 - name: Define cron job command if deferred (ASG).
   ansible.builtin.set_fact:
@@ -31,5 +31,5 @@
     job: "{{ _cron_job_command }}"
     state: present
     disabled: "{{ entry.disabled | default(omit) }}"
-    mailto: "{{ entry.mailto if entry.mailto is defined else omit }}"
+    mailto: "{{ entry.mailto | default(omit) }}"
   delegate_to: "{{ 'localhost' if drupal.defer else inventory_hostname }}"

roles/cron/cron_drupal8/defaults/main.yml

@@ -12,7 +12,7 @@ drupal:
           # month:
           job: cron
           # disabled: true
-          mailto: "{{ drupal.cron_mailto | default('') }}"
+          # mailto: "{{ drupal.cron_mailto | default('') }}" # Each cron can have it's own mailto and can be configured to use different e-mail addresses.
   cron_mailto: ""
   # If the sites are being deployed to an ASG, setting defer to true will create the crontab entry on the deploy server rather than all of the app servers.
   defer: false

roles/cron/cron_drupal8/tasks/job.yml

@@ -1,14 +1,9 @@
 ---
 - name: Define cron job command.
   ansible.builtin.set_fact:
-    _cron_job_command: "cd {{ deploy_path }}/{{ webroot }}/sites/{{ site.folder }} && {{ drupal.drush_location }} {{ entry.job }}"
+    _cron_job_command: "cd {{ live_symlink_dest }}/{{ webroot }}/sites/{{ site.folder }} && {{ drupal.drush_location }} {{ entry.job }} > /dev/null 2>&1"
   when: deploy_operation == "deploy"
 
-- name: Revert cron job command.
-  ansible.builtin.set_fact:
-    _cron_job_command: "cd {{ previous_deploy_path }}/{{ webroot }}/sites/{{ site.folder }} && {{ drupal.drush_location }} {{ entry.job }}"
-  when: deploy_operation == "revert"
-
 - name: Define cron job command if deferred (ASG).
   ansible.builtin.set_fact:
     _cron_job_command: "cd {{ _ce_deploy_base_dir }} && {{ _ce_deploy_ansible_location }} {{ drupal.defer_target }} -m shell -a \"{{ _cron_job_command }}\""
@@ -37,5 +32,5 @@
     job: "{{ _cron_job_command }}"
     state: present
     disabled: "{{ entry.disabled | default(omit) }}"
-    mailto: "{{ entry.mailto if entry.mailto is defined else omit }}"
+    mailto: "{{ entry.mailto | default(omit) }}"
   delegate_to: "{{ 'localhost' if drupal.defer else inventory_hostname }}"