Merge pull request #3 from edx/jarv/nginx

fixing yaml syntax for include statement

playbooks/roles/nginx/tasks/main.yml

@@ -12,8 +12,9 @@
   notify: restart nginx
 # Standard configuration that is common across all roles
 # Default values for these variables are set in group_vars/all
-- include: nginx_site.yml state="{{ nginx_cfg.sites_enabled.edx_release }}" site_name=edx-release
-- include: nginx_site.yml state="{{ nginx_cfg.sites_enabled.basic_auth }}" site_name=basic-auth
+# Note: remove spaces in {{..}}, otherwise you will get a template parsing error.
+- include: nginx_site.yml state={{nginx_cfg.sites_enabled.edx_release}} site_name=edx-release
+- include: nginx_site.yml state={{nginx_cfg.sites_enabled.basic_auth}} site_name=basic-auth
 # Default htpassword file, required for basic auth
 - copy: content={{ nginx_cfg.htpasswd }} dest=/etc/nginx/nginx.htpasswd
   sudo: True

playbooks/roles/nginx/tasks/nginx_site.yml

 # Requires nginx package
 ---
-- name: Copying nginx config
+- name: Copying nginx config {{ site_name }}
   sudo: True
   template: src={{ site_name }}.j2 dest=/etc/nginx/sites-available/{{ site_name }}
   notify: restart nginx
 
-- name: Creating nginx config link
+- name: Creating nginx config link {{ site_name }}
   sudo: True
   file: src=/etc/nginx/sites-available/{{ site_name }} dest=/etc/nginx/sites-enabled/{{ site_name }} state={{ state }} owner=root group=root
   notify: restart nginx