Skip to content

More job metrics, less tables #20

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

Merged
merged 22 commits into from
Oct 13, 2023
Merged

More job metrics, less tables #20

merged 22 commits into from
Oct 13, 2023

Conversation

jessarcher
Copy link
Member

@jessarcher jessarcher commented Oct 2, 2023

  • Removes the QueueSizes recorder and table. This approach took snapshots every 15 seconds and only captured the current number of pending and failed jobs for connection and queue.
  • Updates the Jobs recorder to capture the first time the job began processing, the time it failed, and the time it was processed. This now gives us the stats for the Queues and the SlowJobs cards. It also allows us to display additional stats, such as the number of processed jobs in a period.
  • Updates the Queues card to display graphs for the current period, instead of a table with only the current values:

image
image

@jessarcher jessarcher force-pushed the job-improvements branch 4 times, most recently from df756c4 to 32f9ae0 Compare October 6, 2023 05:59
@jessarcher jessarcher force-pushed the job-improvements branch 4 times, most recently from c38e9cd to dcc6df0 Compare October 10, 2023 01:36
@driesvints
Copy link
Member

@jessarcher the first image with the light background could do with a slightly darker background for the metric box so it's a bit different than the white background of the card.

@jessarcher
Copy link
Member Author

@jessarcher the first image with the light background could do with a slightly darker background for the metric box so it's a bit different than the white background of the card.

Good call - I've updated it (and the screenshot above)

@jessarcher jessarcher force-pushed the job-improvements branch 3 times, most recently from b200cd7 to 3fb1d7d Compare October 13, 2023 03:06
@jessarcher jessarcher marked this pull request as ready for review October 13, 2023 05:01
@jessarcher jessarcher merged commit d86237f into master Oct 13, 2023
@jessarcher jessarcher deleted the job-improvements branch October 13, 2023 05:01
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