Skip to content

Add endpoint for displaying multiple groups of log detective runs - #3191

Merged
centosinfra-prod-github-app[bot] merged 1 commit into
packit:mainfrom
Jany26:ld-group-endpoint
Sep 23, 2026
Merged

centosinfra-prod-github-app[bot] merged 1 commit into
packit:mainfrom
Jany26:ld-group-endpoint

Conversation

@Jany26

@Jany26 Jany26 commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Adding /api/log-detective/groups?page=X&per_page=Y endpoint (works similarly to /api/log-detective/) that shows multiple groups of Log Detective runs

This is to improve the dashboard readability, so that we can show the Log Detective runs from the same trigger on the same line.

I opted to not modify the original endpoint, but dashboard is probably the only consumer.

Merge before packit/dashboard#544

RELEASE NOTES BEGIN

Exposing an endpoint to get a list of groups of Log Detective runs.

RELEASE NOTES END

@Jany26
Jany26 requested a review from a team as a code owner September 22, 2026 14:16
@Jany26
Jany26 requested review from betulependule and removed request for a team September 22, 2026 14:16
@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

@mfocko mfocko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

To improve the dashboard readability, so that we show the Log
Detective runs from the same trigger on the same line.

Signed-off-by: Jan Matufka <jmatufka@redhat.com>
@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

@Jany26 Jany26 added the mergeit Merge via Zuul label Sep 23, 2026
@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app
centosinfra-prod-github-app Bot merged commit 86351c0 into packit:main Sep 23, 2026
8 checks passed
@github-project-automation github-project-automation Bot moved this from New to Done in Packit pull requests Sep 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergeit Merge via Zuul

Projects

Development

Successfully merging this pull request may close these issues.

4 participants