diff --git a/nginx/init.sls b/nginx/init.sls index 07240ff..b2659aa 100644 --- a/nginx/init.sls +++ b/nginx/init.sls @@ -1,11 +1,9 @@ -{%- set nginx=pillar.get('nginx', {}) %} - include: - nginx.common -{% if nginx.get('user_auth_enabled', true) %} +{% if pillar.get('nginx', {}).get('user_auth_enabled', true) %} - nginx.users {% endif %} -{% if nginx.get('install_from_source') %} +{% if pillar.get('nginx', {}).get('install_from_source') %} - nginx.source {% else %} - nginx.package diff --git a/nginx/map.jinja b/nginx/map.jinja index b749fab..74b80e2 100644 --- a/nginx/map.jinja +++ b/nginx/map.jinja @@ -1,10 +1,8 @@ {% set nginx = salt['grains.filter_by']({ 'Debian': { - 'package': 'nginx', 'apache_utils': 'apache2-utils', }, 'RedHat': { - 'package': 'nginx', 'apache_utils': 'httpd-tools', }, -}, merge=salt['pillar.get']('nginx:lookup')) %} +}, merge=salt['pillar.get']('nginx:lookup')) %} \ No newline at end of file diff --git a/nginx/ng/install.sls b/nginx/ng/install.sls index 4333698..f538ccd 100644 --- a/nginx/ng/install.sls +++ b/nginx/ng/install.sls @@ -5,25 +5,10 @@ {% from 'nginx/ng/map.jinja' import nginx, sls_block with context %} nginx_install: - {% if nginx.install_from_source %} + {% if nginx.from_source %} ## add source compilation here {% else %} pkg.installed: {{ sls_block(nginx.package.opts) }} - name: {{ nginx.lookup.package }} {% endif %} - -{% if salt['grains.get']('os_family') == 'Debian' %} -{% if nginx.install_from_ppa %} -nginx_ppa_repo: - pkgrepo.managed: - - humanname: nginx-ppa-{{ grains['oscodename'] }} - - name: deb http://ppa.launchpad.net/nginx/stable/ubuntu {{ grains['oscodename'] }} main - - file: /etc/apt/sources.list.d/nginx-stable-{{ grains['oscodename'] }}.list - - dist: {{ grains['oscodename'] }} - - keyid: C300EE8C - - keyserver: keyserver.ubuntu.com - - require_in: - - pkg: nginx_install -{% endif %} -{% endif %} diff --git a/nginx/package.sls b/nginx/package.sls index 300dde7..fd5ae0c 100644 --- a/nginx/package.sls +++ b/nginx/package.sls @@ -1,4 +1,3 @@ -{% from "nginx/map.jinja" import nginx with context %} {% set use_upstart = pillar.get('nginx', {}).get('use_upstart', true) %} {% if use_upstart %} nginx-old-init: @@ -36,23 +35,9 @@ nginx-old-init-disable: - file: nginx-old-init {% endif %} -{% if salt['grains.get']('os_family') == 'Debian' %} -nginx-ppa-repo: - pkgrepo.managed: - - enabled: {{ salt['pillar.get']('nginx:install_from_ppa', False) }} - - humanname: nginx-ppa-{{ grains['oscodename'] }} - - name: deb http://ppa.launchpad.net/nginx/stable/ubuntu {{ grains['oscodename'] }} main - - file: /etc/apt/sources.list.d/nginx-stable-{{ grains['oscodename'] }}.list - - dist: {{ grains['oscodename'] }} - - keyid: C300EE8C - - keyserver: keyserver.ubuntu.com - - require_in: - - pkg: nginx -{% endif %} - nginx: pkg.installed: - - name: {{ nginx.package }} + - name: nginx {% if use_upstart %} file: - managed diff --git a/pillar.example b/pillar.example index 84bb0f0..757b0a4 100644 --- a/pillar.example +++ b/pillar.example @@ -1,6 +1,5 @@ nginx: install_from_source: True - install_from_ppa: False use_upstart: True user_auth_enabled: True with_luajit: False @@ -14,9 +13,6 @@ nginx: source: http://github.com/agentzh/headers-more-nginx-module/tarball/v0.21 source_hash: sha1=dbf914cbf3f7b6cb7e033fa7b7c49e2f8879113b - lookup: - package: nginx - # ======== # nginx.ng # ======== @@ -34,10 +30,7 @@ nginx: vhost_use_symlink: True # Source compilation is not currently a part of nginx.ng - install_from_source: False - - # Install nginx.ng.lookup.package from the PPA repository - install_from_ppa: True + from_source: False package: opts: {} # this partially exposes parameters of pkg.installed