Closed Bug 1340216 Opened 7 years ago Closed 5 years ago

Make pulse_listener_jobs and pulse_listener_pushes report to New Relic

Categories

(Tree Management :: Treeherder: Infrastructure, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

The New Relic Python agent knows how to annotate:
* Django wsgi requests
* Celery tasks
* Django management commands that are finite duration (require enabling manually, see bug 1340203)

However for other tasks (or Django management commands that are infinite loops, like read_pulse_jobs and read_pulse_resultsets, and so unsuited to the management commands hook in bug 1340203), the agent doesn't know what functions are the individual finite-duration transactions.

As such these require manual annotations:
https://docs.newrelic.com/docs/agents/python-agent/supported-features/python-background-tasks#wrapping

In the case of `read_pulse_{jobs,resultsets}` it seems like the most appropriate function to annotate is `on_message()`:
https://github.com/mozilla/treeherder/blob/15941c3d048104016d3fb8566bb9fae659d6dce3/treeherder/etl/pulse_consumer.py#L92-L116
Blocks: 1529404

Updating summary to reflect the new names for the scripts after bug 1530206.

Priority: P3 → P2
Summary: Make read_pulse_jobs and read_pulse_resultsets report to New Relic → Make pulse_listener_jobs and pulse_listener_pushes report to New Relic
Assignee: nobody → emorley
Status: NEW → ASSIGNED
Priority: P2 → P1
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: