-
Notifications
You must be signed in to change notification settings - Fork 31
Conversation
@@ -84,6 +85,7 @@ def currency_filter(num, grouping=True): | |||
return locale.currency(num, grouping=grouping) | |||
|
|||
if __name__ == '__main__': | |||
nr_agent.initialize() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My impression from the docs is that we can either run our app prefixed with newrelic-admin run-program
or import and initialize the agent within the app. One or the other should be enough--I don't think we need both.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, @jmcarp . Yes, let's standardize on the wrapper script and take this import/initialize out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# "development", "test", "staging" or "production". | ||
# | ||
|
||
[FEC_ENVIRONMENT:development] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be NEW_RELIC_ENV
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually that should all be taken care of in the CF manifests, I think... We can probably just take out this part. At least the newrelic.ini that @dlapiduz submitted for the API had monitor_mode set to true globally.
- Set high_security to true - Removed app names (these are specified in CF manifests) - Removed redundant environ-specific settings
I got new relic working on the test instance. Still need to up the security settings.