Fixing cron syntax pr devel 1.x (#748)

* Php unix socket pr 1.x (#724)

* Moving cachetool adapter setup to the installer role and adding a test.

* Updating cachetool documentation.

* Php unix socket pr 1.x (#726)

* Moving cachetool adapter setup to the installer role and adding a test.

* Updating cachetool documentation.

* Making sure _cachetool_adapter is always set.

* Optional-drush-cr-before-import (#728)

* 73171 fixing cachetool checks fpm pr 1.x (#729)

* skipping cachetool check when no fpm

* remove empty line

---------

Co-authored-by: filip <filip.rupic@codeenigma.com>

* asg_management_fix_and_improve (#733)

* asg_management_fix_and_improve

* asg_management_fix_and_improve

* asg_management_fix_and_improve

* asg_management_fix_and_improve

* asg_management_fix_and_improve

* Adding-delay-opcache-status-check (#737)

* Adding-delay-opcache-status-check

* Removing-failed-when

* fix_var_name_for_asg_management (#739)

* Fixing-cron-syntax

* Fixing-cron-syntax-PR-devel-1.x

* Fixing-cron-syntax-PR-devel-1.x

* Another-cron-test

* Fixing-cron-syntax-PR-devel-1.x

* Fixing-cron-syntax-PR-devel-1.x

* Update main.yml

---------

Co-authored-by: Greg Harvey <greg.harvey@gmail.com>
Co-authored-by: Filip Rupic <123341158+filiprupic@users.noreply.github.com>
Co-authored-by: filip <filip.rupic@codeenigma.com>
Co-authored-by: tymofiisobchenko <104431720+tymofiisobchenko@users.noreply.github.com>

Author: 5 people

roles/cron/cron_drupal7/defaults/main.yml

@@ -19,6 +19,6 @@ drupal:
   # If defer is set to true, the Ansible target must be declared with defer_target. If using a group, include the index. For example, _ce_www_dev[0]
   defer_target: ""
   drush_location: "{{ drush_bin }}" # you might specify another location, e.g. "{{ deploy_path }}/vendor/bin/drush"
-  cron_error_filter: "> /tmp/cron_output.txt 2>&1; rc=\\$?; if [ \\$rc -ne 0 ] || grep -iE \"error|fail|fatal|critical|exception\" /tmp/cron_output.txt | grep -v -E \"(\\\"rc\\\": -13|MODULE FAILURE)\" | grep -q .; then [ -n \\\"{{ entry.cron_error_mailto }}\\\" ] && mail -s \\\"Cron Error - \\$(hostname)\\\" {{ entry.cron_error_mailto }} < /tmp/cron_output.txt; fi"
-  cron_error_email: ""
+  cron_error_filter: "> /tmp/cron_output.txt 2>&1; rc=\\$?; if [ \\$rc -ne 0 ] || grep -iE 'error|fail|fatal|critical|exception' /tmp/cron_output.txt | grep -v -E '(\\\"rc\\\": -13|MODULE FAILURE)' | grep -q .; then fi"
+  cron_error_email: "{% if item.cron_error_mailto is defined and item.cron_error_mailto|length > 0 %}mail -s 'Cron Error - $(hostname)' {{ item.cron_error_mailto }} < /tmp/cron_output.txt;{% endif %}"
   python_interpreter: "-e 'ansible_python_interpreter=/home/controller/ce-python/bin/python'"

roles/cron/cron_drupal8/defaults/main.yml

@@ -22,6 +22,6 @@ drupal:
   # If defer is set to true, the Ansible target must be declared with defer_target. If using a group, include the index. For example, _ce_www_dev[0]
   defer_target: ""
   drush_location: "{{ drush_bin }}" # you might specify another location, e.g. "{{ deploy_path }}/vendor/bin/drush"
-  cron_error_filter: "> /tmp/cron_output.txt 2>&1; rc=\\$?; if [ \\$rc -ne 0 ] || grep -iE \"error|fail|fatal|critical|exception\" /tmp/cron_output.txt | grep -v -E \"(\\\"rc\\\": -13|MODULE FAILURE)\" | grep -q .; then [ -n \\\"{{ entry.cron_error_mailto }}\\\" ] && mail -s \\\"Cron Error - \\$(hostname)\\\" {{ entry.cron_error_mailto }} < /tmp/cron_output.txt; fi"
-  cron_error_email: ""
+  cron_error_filter: "> /tmp/cron_output.txt 2>&1; rc=\\$?; if [ \\$rc -ne 0 ] || grep -iE 'error|fail|fatal|critical|exception' /tmp/cron_output.txt | grep -v -E '(\\\"rc\\\": -13|MODULE FAILURE)' | grep -q .; then fi"
+  cron_error_email: "{% if item.cron_error_mailto is defined and item.cron_error_mailto|length > 0 %}mail -s 'Cron Error - $(hostname)' {{ item.cron_error_mailto }} < /tmp/cron_output.txt;{% endif %}"
   python_interpreter: "-e 'ansible_python_interpreter=/home/controller/ce-python/bin/python'"