Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

error in vhosts.conf requires manual removal #22

Closed
robotops opened this issue Jun 18, 2015 · 1 comment
Closed

error in vhosts.conf requires manual removal #22

robotops opened this issue Jun 18, 2015 · 1 comment

Comments

@robotops
Copy link

If an error is accidentally injected into the vhosts.conf file, fixing it in the playbook won't take effect until the vhosts file is explicitly removed from the target host. I think this occurs because the nginx restart is attempted before the include of vhosts.yml. When there's an error in the vhosts file on the target host, the startup fails and the playbook halts before the new vhost.conf can be generated with the fix.

  • name: Ensure nginx is started and enabled to start at boot.
    service: name=nginx state=started enabled=yes
  • include: vhosts.yml
@geerlingguy
Copy link
Owner

This was fixed as part of #36 - sorry about not updating this issue sooner!

ivangrynenko pushed a commit to ivangrynenko/ansible-role-nginx that referenced this issue Jun 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant