Skip to content

Add pickup and duration time metrics for processed jobs #4312

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

johha
Copy link
Contributor

@johha johha commented Apr 16, 2025

This change adds a new delayed job plugin which generates the following metrics after a cc-worker processed a job.

  • cc_job_pickup_delay_seconds: the difference of run_at and locked_at
  • cc_job_duration_seconds: Time difference of locked_at and current time.

Both metrics are labeled with the name of the worker and the queue of the job to allow further aggregation etc.

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@johha johha force-pushed the job-latency-metrics branch 3 times, most recently from 9668b40 to 6a6bb42 Compare April 17, 2025 09:01
This change adds a new delayed job plugin which generates the following
metrics after a cc-worker processed a job.
- cc_job_pickup_delay_seconds: the difference of `run_at` and `locked_at`
- cc_job_duration_seconds: Time difference of `locked_at` and current time.

Both metrics are labeled with the name of the worker and the queue of the job to allow further aggregation etc.
@johha johha force-pushed the job-latency-metrics branch 2 times, most recently from 919d9a4 to 1523372 Compare April 17, 2025 11:06
@johha johha force-pushed the job-latency-metrics branch from 1523372 to cf47949 Compare April 17, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant