Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Newrelic plugins #113

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Newrelic plugins #113

wants to merge 4 commits into from

Conversation

emachnic
Copy link
Contributor

Description of your patch

Due to the Python requirements, this is only compatible with new Gentoo stack

This installs the MeetMe newrelic_plugin_agent on Engine Yard environments. Users can use the base recipe and customize based on which plugins need to be enabled.

Estimated risk

Low

Components involved

  • Installs pip with easy_install
  • Uses pip to install newrelic-plugin-agent
  • Adds configuration file to /etc
  • Creates init.d script and adds newrelic_plugin_agent to the default run level
  • Uses monit to monitor the service

Description of testing done

  • Tested locally with chefspec and foodcritic
  • Uploaded and applied recipes to solo and cluster, stable-v4 environments

QA Instructions

  1. Make sure New Relic is enabled
  2. Add license key to newrelic_plugins/recipes/default.rb
  3. Uncomment 'redis' section in newrelic_plugins/templates/default/newrelic_plugin_agent.yml file
  4. Include the recipe in the main recipe
  5. Upload and apply to environment
  6. Verify that files are in place, daemon is running, and Redis shows up in New Relic dash. Can also use redis-cli to verify that data is going into New Relic

@emachnic
Copy link
Contributor Author

@helgi I'm an idiot. Fixed 😉

@dennisjbell
Copy link
Contributor

Please add the standard cookbook/main/recipes/default.rb boilerplate

# Uncomment to include NewRelic Plugins recipe (Gentoo 12.11)
# include_recipe 'newrelic_plugins'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants