Skip to content
This repository was archived by the owner on Jun 21, 2022. It is now read-only.

Conversation

@groovecoder
Copy link
Contributor

This is how I started it. It should work like this, but some follow-ups could be to see if we can add more granular monitoring/profiling into macros and macro workers?

@groovecoder
Copy link
Contributor Author

Oops, just noticed I need to make the app name come from an environment variable too.

@groovecoder groovecoder force-pushed the add-newrelic-1005013 branch from a8b6afa to fb0c001 Compare August 25, 2015 20:53
@groovecoder
Copy link
Contributor Author

Updated newrelic.js to pull all config from environment variables:

NEW_RELIC_APP_NAME
NEW_RELIC_LICENSE_KEY
NEW_RELIC_HIGH_SECURITY

It still works. And when I left my local vm running some background render tasks, I'm starting to see some good data about which pages take the longest for KumaScript to execute:
mdn-local-ks-nr

@darkwing
Copy link
Contributor

R+. Builds properly, tests pass, eye-test passes.

darkwing added a commit that referenced this pull request Aug 27, 2015
bug 1005013 - add newrelic to server process
@darkwing darkwing merged commit 5ac70d8 into master Aug 27, 2015
@jwhitlock jwhitlock deleted the add-newrelic-1005013 branch August 11, 2017 14:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants