Update ice cookbook - Githubissues

mdsol / ice_cookbook

Cookbook to install and configure Ice, an AWS cost monitoring tool from Netflix OSS

Other

41 stars 22 forks source link

Update ice cookbook #31

Closed shortdudey123 closed 9 years ago

shortdudey123 commented 9 years ago

Add support for https://github.com/mdsol/ice_cookbook/issues/29
Allow nginx to be optional
Update to be compatible with Tomcat 7
Add testing on CentOS 7.1

harryw commented 9 years ago

Thanks @shortdudey123! @rayrod2030 do you have time to take a look at this?

rayrod2030 commented 9 years ago

Great pull request. I'll try to review it in the next couple of days.

On Mon, Jun 22, 2015 at 6:01 AM, Harry Wilkinson notifications@github.com wrote:

Thanks @shortdudey123 https://github.com/shortdudey123! @rayrod2030 https://github.com/rayrod2030 do you have time to take a look at this?

— Reply to this email directly or view it on GitHub https://github.com/mdsol/ice_cookbook/pull/31#issuecomment-114056137.

shortdudey123 commented 9 years ago

@rayrod2030 any update from your side?

rayrod2030 commented 9 years ago

Sorry for the delay. These changes look great. Merging!

shortdudey123 commented 9 years ago

No problem and thanks!