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

Better Support for New Relic Server Monitoring #316

Merged
merged 4 commits into from May 20, 2016
Merged

Conversation

ttmc
Copy link
Contributor

@ttmc ttmc commented May 18, 2016

  • Tested the existing code for installing the New Relic agent on all AWS instances, and made some improvements. For example, the New Relic license key is now read from an environment variable named NEWRELIC_KEY and there's a helpful error message if that isn't set.
  • Added documentation about what New Relic Server Monitoring is and how to use it to monitor a BigchainDB cluster on AWS.

@ttmc ttmc self-assigned this May 18, 2016
@ttmc ttmc mentioned this pull request May 18, 2016
9 tasks
@ttmc ttmc changed the title Better Support for New Relic Better Support for New Relic Server Monitoring May 18, 2016
@@ -5,6 +5,9 @@

from __future__ import with_statement, unicode_literals

from os import environ # a mapping (like a dict)
from sys import exit as sys_exit
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can just do:

import sys

and then in the code:

sys.exit('...')

it's simpler

@ttmc
Copy link
Contributor Author

ttmc commented May 20, 2016

@vrde Thanks, I changed it to import sys and sys.exit('...

@vrde vrde merged commit 046f39f into master May 20, 2016
@vrde vrde deleted the better-newrelic-support branch May 20, 2016 13:42
@vrde vrde assigned vrde and unassigned ttmc May 20, 2016
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.

None yet

2 participants