Skip to content

Add job_queue_duration_seconds metric#19

Merged
Okabe-Junya merged 2 commits intomercari-masterfrom
Okabe-Junya/feature/add-job-queue-duration-metric
Apr 28, 2026
Merged

Add job_queue_duration_seconds metric#19
Okabe-Junya merged 2 commits intomercari-masterfrom
Okabe-Junya/feature/add-job-queue-duration-metric

Conversation

@Okabe-Junya
Copy link
Copy Markdown
Member

@Okabe-Junya Okabe-Junya commented Apr 24, 2026

@Okabe-Junya Okabe-Junya changed the title feat(listener): add job_queue_duration_seconds metric Add job_queue_duration_seconds metric Apr 24, 2026
Comment on lines +496 to +497
queueDuration := msg.ScaleSetAssignTime.Unix() - msg.QueueTime.Unix()
e.observeHistogram(MetricJobQueueDurationSeconds, l, float64(queueDuration))
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hope it doesn't include the concurrency wait 👍

Copy link
Copy Markdown
Member Author

@Okabe-Junya Okabe-Junya Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's merge/release rc version, and try to test in our sandbox environment :)

@Okabe-Junya Okabe-Junya merged commit 8aee61a into mercari-master Apr 28, 2026
8 of 9 checks passed
@Okabe-Junya Okabe-Junya deleted the Okabe-Junya/feature/add-job-queue-duration-metric branch April 28, 2026 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants