StackStorm / chef-stackstorm

DEPRECATED! Community-maintained Chef Cookbook to deploy StackStorm, IFTTT for Ops
https://supermarket.chef.io/cookbooks/stackstorm
Apache License 2.0
16 stars 10 forks source link

Recipe retouch for st2web #57

Closed arm4b closed 7 years ago

arm4b commented 7 years ago

Address notes missed in https://github.com/StackStorm/chef-stackstorm/pull/45. Finalize work on https://github.com/StackStorm/chef-stackstorm/issues/24

arm4b commented 7 years ago

Related fixes made in upstream chef-nginx cookbook: https://github.com/chef-cookbooks/chef_nginx/pull/55 https://github.com/chef-cookbooks/chef_nginx/issues/57 https://github.com/chef-cookbooks/chef_nginx/pull/59

CentOS7 CI is failing due to different reason: https://github.com/StackStorm/chef-stackstorm/issues/58 (same for master)

shortdudey123 commented 7 years ago

I think you can override the merge even though the CI test for centos7 fails since you are an admin

I will see what i can find for https://github.com/StackStorm/chef-stackstorm/issues/58

arm4b commented 7 years ago

Merged.

@shortdudey123 Thanks for taking a look!