Skip to content
This repository has been archived by the owner on Aug 13, 2024. It is now read-only.

Configure the extension of sites-available config files #56

Open
VoidAndAny opened this issue May 29, 2015 · 1 comment
Open

Configure the extension of sites-available config files #56

VoidAndAny opened this issue May 29, 2015 · 1 comment

Comments

@VoidAndAny
Copy link

First of all thank you for this role, it is very useful.

Is it possible to add a variable allowing to configure the extension of site definition in sites-available ?
For the moment the role always add .conf extension.

- name: Create the configurations for sites
  template: src=site.conf.j2 dest={{nginx_conf_dir}}/sites-available/{{ item }}{{nginx_config_extension|default()}}
  with_items: nginx_sites.keys()
  notify: 
   - restart nginx
  tags: [configuration,nginx]

I can make a pull request if you want, let me know if your agree with my variable name.

@jdauphant
Copy link
Owner

Thanks, the merit go to all contributor of this role.

You can made a PR but add the default in defaults/main.yml .
If it's apply only to sites configuration, the name could be nginx_sites_conf_extension: ".conf".
The use of this variable could be more uniform and apply to "independent config file" also : we can name the variable in this case ; "nginx_conf_extension"

Best regards,
Julien

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants