857109 - Setup New Relic

Status: VERIFIED FIXED

(Websites :: Firefox Flicks, defect)

Description

[Ben (:bensternthal)]

I'd like to have new relic running on flicks.

I am not sure if this will help us diagnosing celery, but it does look like I can setup alerts for other types of errors.

Comment 1

[Ben (:bensternthal)]

Looks like we can add monitoring for things like celery under background tasks.

Let's do it!

Comment 2

[Osmose [:osmose, :mkelly]]

pmac: Any helpful commits/PRs/wiki pages/info for getting this set up?

Comment 3

[Paul [:pmac] McLanahan]

Yes indeed.

This is the change to bedrock to enable:

https://github.com/mozilla/bedrock/compare/cb994114c5271ec7d61b9628f8cc5721fb6ec786...8f53cdf5acc80b9b96dac91e74fe7ee57ef537a4

You only need to do this one thing to the wsgi file to enable NR. IT needs to setup an ENV var in apache for that ini file path and for the app name, but that's easy.

As for celery workers I've got bug 857057 open now to do that for basket. We'll see how it goes. The docs are simple (https://newrelic.com/docs/python/python-agent-and-celery).

Comment 4

[[github robot]]

Commit pushed to master at https://github.com/mozilla/firefox-flicks

https://github.com/mozilla/firefox-flicks/commit/1b1a20d6268b7d8e605ec4a155337669c70a65a0
Bug 857109: Add New Relic code to Flicks WSGI file.

Comment 5

[Osmose [:osmose, :mkelly]]

This looks goods on dev. I was able to get data to show up in NR and the IT side is complete.

Marking as qa- as there's no specific testing to be done here except for seeing if data is showing up in NR.

Comment 6

[Rebecca Billings [:rbillings]]

Closed as [qa-]