Skip to content

feat(block-scheduler): status page shows completed jobs - #15580

Merged
owen-d merged 4 commits into
mainfrom
scheduler-completed-status
Jan 3, 2025
Merged

owen-d merged 4 commits into
mainfrom
scheduler-completed-status

Conversation

@owen-d

@owen-d owen-d commented Jan 2, 2025

Copy link
Copy Markdown
Contributor

The scheduler keeps a circular buffer of the last n completed jobs. This PR now includes this information in the status page for easier tracking of system behavior. it also adds a test utility to play around with this locally. Here's an example:
image

@owen-d
owen-d requested a review from a team as a code owner January 2, 2025 19:02
Comment thread pkg/blockbuilder/scheduler/status_preview_test.go Outdated

// TestPreview is a utility test that runs a local server with the status page.
// Run it with: go test -tags=preview -v -run TestPreview
func TestPreview(t *testing.T) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: i guess we could just remove status_test.go since this is a better way to preview the page

@ashwanthgoli ashwanthgoli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

Co-authored-by: Ashwanth <iamashwanth@gmail.com>
Signed-off-by: Owen Diehl <ow.diehl@gmail.com>
@owen-d
owen-d merged commit 10194f7 into main Jan 3, 2025
@owen-d
owen-d deleted the scheduler-completed-status branch January 3, 2025 17:12
mveitas pushed a commit to mveitas/loki that referenced this pull request Jan 6, 2025
Signed-off-by: Owen Diehl <ow.diehl@gmail.com>
Co-authored-by: Ashwanth <iamashwanth@gmail.com>
felix0102 pushed a commit to felix0102/loki that referenced this pull request Aug 2, 2026
Signed-off-by: Owen Diehl <ow.diehl@gmail.com>
Co-authored-by: Ashwanth <iamashwanth@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants