Hello,

This role fails on Debian when playing the following tasks in check_mode on a fresh server :

- name: Configure exim (Debian only).
  lineinfile:
    dest: "{{ exim_configuration_file }}"
    regexp: "{{ item.regexp }}"
    line: "{{ item.line }}"
    state: present
  with_items:
    - regexp: '^dc_eximconfig_configtype'
      line: "dc_eximconfig_configtype='{{ exim_dc_eximconfig_configtype }}'"
    - regexp: '^dc_localdelivery'
      line: "dc_localdelivery='{{ exim_dc_localdelivery }}'"
  notify: restart exim
  when: ansible_os_family == 'Debian'

- name: Configure primary_hostname (if set).
  lineinfile:
    dest: "{{ exim_configuration_file }}"
    regexp: '^#?\s?primary_hostname'
    line: "primary_hostname={{ exim_primary_hostname }}"
    state: present
  notify: restart exim
  when: exim_primary_hostname is defined and exim_primary_hostname

- name: Ensure exim is running.
  service: "name={{ exim_daemon }} state=started enabled=yes"

with the following error message :

failed: [xxx] (item={'regexp': '^dc_eximconfig_configtype', 'line': "dc_eximconfig_configtype='internet'"}) => {"ansible_loop_var": "item", "changed": false, "item": {"line": "dc_eximconfig_configtype='internet'", "regexp": "^dc_eximconfig_configtype"}, "msg": "Destination /etc/exim4/update-exim4.conf.conf does not exist !", "rc": 257}
failed: [xxx] (item={'regexp': '^dc_localdelivery', 'line': "dc_localdelivery='mail_spool'"}) => {"ansible_loop_var": "item", "changed": false, "item": {"line": "dc_localdelivery='mail_spool'", "regexp": "^dc_localdelivery"}, "msg": "Destination /etc/exim4/update-exim4.conf.conf does not exist !", "rc": 257}

and :

fatal: [xxx]: FAILED! => {"changed": false, "msg": "Could not find the requested service exim4: host"}

because exim is not installed and therefore the destination file does not exist, just like the systemd service.

I suggest adding the following line to these tasks to ignore check_mode errors :

  ignore_errors: "{{ ansible_check_mode }}"

See: https://travis-ci.org/geerlingguy/ansible-role-exim/builds/434179556

--> Action: 'idempotence'
ERROR: Idempotence test failed because of the following tasks:
* [instance] => geerlingguy.exim : Ensure exim is running.
An error occurred during the test sequence action: 'idempotence'. Cleaning up.

I would like to be able to configure, at a minimum:

• dc_eximconfig_configtype
• dc_localdelivery

The primary_hostname in the Exim config file sometimes needs to be manually configured, because Exim has trouble picking up the actual hostname of the server (even if hostname -f outputs the correct hostname).

See http://www.jeffgeerling.com/blog/2016/how-get-your-servers-emails-through-gmails-spam-filter-exim for a little more background.

It would be nice to allow this to be configured... if for no other reason than it's one less task I have to maintain in my downstream playbooks for a few LEB servers where Exim is picking up the domain without the TLD (e.g. emails delivered from apache@example instead of [email protected].

Hi,

Can you tag the last commits please