Cookbook Cleanup - Githubissues

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

Cookbook Cleanup #42

Closed arm4b closed 7 years ago

arm4b commented 7 years ago

Work on https://github.com/StackStorm/chef-stackstorm/issues/31

Remove outdated logic left from the old installer (pre-packages era).

[x] Main cleanup
[x] Verify configure SSH & sudo logic
[x] Enable st2 services
[x] Services restart handler on st2 config change
[x] Add missing ChefSpec unit tests
[x] Test with CentOS 7.0
[x] README re-check :eyes:

arm4b commented 7 years ago

The main cleanup stage is nearly done. As expected, after removing all unused config settings and outdated logic this CookBook becomes as simple as Installing StackStorm package + manage configs + start/stop services.

I've adeed TODO checklist in the first message of this PR to keep better visibility.

arm4b commented 7 years ago

Alright, I'm going to re-check Readme again and merge it.

shortdudey123 commented 7 years ago

Not sure where all of the platform_version lines are coming from in the chefspec test on travis

arm4b commented 7 years ago

@shortdudey123 Yeah, noticed that Travis thing too. After some research, found what's that. Described here: https://github.com/StackStorm/chef-stackstorm/issues/44