Skip to content

Commit

Permalink
Merge pull request #552 from sous-chefs/use_nginx
Browse files Browse the repository at this point in the history
Depend on nginx instead of chef_nginx
  • Loading branch information
tas50 authored Sep 18, 2017
2 parents 4e0292b + de52d04 commit 751d871
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ The functionality that was previously in the nagios::client recipe has been move

- apache2 4.0 or greater
- build-essential
- chef_nginx
- nginx
- php
- yum-epel

Expand Down
3 changes: 2 additions & 1 deletion metadata.rb
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,11 @@
recipe 'nagios::pagerduty', 'Integrates contacts w/ PagerDuty API'

depends 'apache2', '>= 4.0'
depends 'nginx', '>= 7.0'
depends 'php-fpm', '>= 0.7.9'
depends 'zap', '>= 0.6.0'

%w( build-essential php chef_nginx yum-epel nrpe ).each do |cb|
%w( build-essential php yum-epel nrpe ).each do |cb|
depends cb
end

Expand Down
2 changes: 1 addition & 1 deletion recipes/nginx.rb
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

node.default['nagios']['server']['web_server'] = 'nginx'

include_recipe 'chef_nginx'
include_recipe 'nginx'

node.default['php-fpm']['pools']['www']['user'] = node['nginx']['user']
node.default['php-fpm']['pools']['www']['group'] = node['nginx']['group']
Expand Down

0 comments on commit 751d871

Please sign in to comment.