Failing without reboot on Ubuntu 16.04 LTS

[groggemans]

TASK [geerlingguy.firewall : Ensure the firewall is enabled and will start on boot.] ***
fatal: [ansible-dev-ubuntu]: FAILED! => {"changed": false, "failed": true, "msg": "Could not find the requested service \"'firewall'\": "}

It runs fine after a reboot.

[groggemans]

Also happens on newer Debian hosts.

Reason: the new service file is not detected (ansible/ansible-modules-core#191)
Should be fixed in ansible v2.2 (ansible/ansible#3660) but will require the systemd module to be used.

Maybe you could add the systemctl daemon-reload command to the role for older ansible versions.

[geerlingguy]

Wrong repo in above link :) — ansible/ansible-modules-core#3660

[geerlingguy]

I'm seeing issues like this pop up for many of my roles now, including:

• MySQL: Restart fails due to unsupported sleep parameter. ansible-role-mysql#128 (sort-of-related)
• Solr: Systemd status incorrect on RHEL/CentOS 7, causing fail ansible-role-solr#33 (sort-of-related)
• Firewall: this issue...

Just documenting here for my own sanity.

[kathgironpe]

Doesn't work with Debian either.

[geerlingguy]

Note that the automated build (runs on CentOS 6, CentOS 7, Ubuntu 12, Ubuntu 14, Ubuntu 16, and Debian 8 at this time) is passing with the latest code: https://travis-ci.org/geerlingguy/ansible-role-firewall

But there are some cases where it might not work, and it looks like Ansible 2.2 might have the fix (ref: geerlingguy/ansible-role-mysql#128), so rather than try to figure out a hacky workaround, I'm going to wait for that and then see if it fixes those issues.

Also, in some cases, if you're working from a Windows workstation, check to make sure your line endings for the files in this role are set to 'Unix' and not Windows. That's caused some consternation for people deploying VMs in Windows via Vagrant using this role and the Ansible provisioner.

[kathgironpe]

@geerlingguy I am sorry. You are right. It works! Thank you for this.

[kathgironpe]

On my part, I forgot to reference the path for variables.

[geerlingguy]

@katgironpe Great, glad I could help!

@groggemans - I'm going to close this out, but please let me know if it's still an issue.

[groggemans]

Ok, Thanks!

[geerlingguy]

Reopening due to #958. Might add a switch for systemd systems and use it for Ubuntu 16.04, CentOS 7, Debian 8, etc. so they get the proper templated file.

[geerlingguy]

I have a fix, will push up a branch soon!

[kathgironpe]

@geerlingguy thank you!

[geerlingguy]

Still not using the unit file on CentOS 7. Will fix that now. See also: geerlingguy/ansible-role-mailhog#15