diff --git a/roles/www_options/tasks/php-settings.yml b/roles/www_options/tasks/php-settings.yml index 887b05c40..18fb606f6 100644 --- a/roles/www_options/tasks/php-settings.yml +++ b/roles/www_options/tasks/php-settings.yml @@ -6,12 +6,6 @@ # roles/pbx/tasks/freepbx.yml # roles/wordpress/tasks/install.yml -# Ansible's ini_file would normally be best, to tweak .ini files: -# https://docs.ansible.com/ansible/latest/collections/community/general/ini_file_module.html -# -# But in this case, explanatory comments (inserted by lineinfile below) offer -# important context to implementers modifying both php.ini files after the fact. - # 2022-12-30: FYI ansible_date_time.tz provides TZ ABBREVIATIONS (equivalent # to 'date +%Z' output) which leads to serious ambiguity -- and not just (A) @@ -26,9 +20,9 @@ # timedatectl show -p Timezone --value # # This takes care of essentially everything (e.g. output "America/New_York") -# by checking (1) symlink /etc/localtime then (2) text file /etc/timezone -# then (3) if neither exist, "UTC" is declated (correctly!) One drawback: -# timedatectl if not easily usable within chroot environments. +# by checking (1) symlink /etc/localtime then (2) text file /etc/timezone if +# nec, then (3) if neither exist, "UTC" is declated (correctly!) Potential +# drawback: timedatectl is not easily usable within chroot environments. - name: Extract Time Zone from symlink /etc/localtime &/or text file /etc/timezone (or lack thereof!) command: timedatectl show -p "Timezone" --value @@ -116,6 +110,13 @@ when: nginx_high_php_limits or moodle_install +# To tweak .ini files, Ansible's ini_file is normally better than lineinfile: +# https://docs.ansible.com/ansible/latest/collections/community/general/ini_file_module.html +# +# But for the 6 * 4 above, explanatory comments (inserted by lineinfile) offer +# important context to implementers modifying both php.ini files after the fact. + + - name: Restart 'php{{ php_version }}-fpm' systemd service systemd: name: php{{ php_version }}-fpm