Skip to content

Show snippets in Log Detective runs - #540

Merged
Jany26 merged 2 commits into
packit:mainfrom
Jany26:log-detective-snippets
Jul 16, 2026
Merged

Show snippets in Log Detective runs#540
Jany26 merged 2 commits into
packit:mainfrom
Jany26:log-detective-snippets

Conversation

@Jany26

@Jany26 Jany26 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Successful Log Detective run overview now shows also a list of snippets, grouped by file, in expandable sections.

I needed to do a couple of hacks to override coloring (to be visible in dark/light mode) for snippet analysis and to change the default padding of the text blocks. Some hacks are not that important, I just wanted some things to look a certain way.

Disclaimer: the snippets and analysis content in the screenshot are a mix of some actual Log Detective runs and some random stuff I added just for visual checking.

Default view:
image

Expanded view:
image

Dark mode:
image

RELEASE NOTES BEGIN

Show list of snippets grouped by file in the Log Detective run view.

RELEASE NOTES END

@usercont-release-bot

usercont-release-bot commented Jul 10, 2026

Copy link
Copy Markdown

Preview: https://packit-dashboard-pr-540.surge.sh (deployed at Fri 10 Jul 2026, 12:42 UTC)

@centosinfra-prod-github-app

Copy link
Copy Markdown

@centosinfra-prod-github-app

Copy link
Copy Markdown

@Jany26
Jany26 force-pushed the log-detective-snippets branch from 99ca0b4 to e9239c6 Compare July 10, 2026 12:41
@centosinfra-prod-github-app

Copy link
Copy Markdown

@Jany26
Jany26 requested review from jpodivin and mfocko July 10, 2026 12:51
@Jany26
Jany26 force-pushed the log-detective-snippets branch from e9239c6 to 9ef9da0 Compare July 16, 2026 12:06
@Jany26

Jany26 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor Author

last change only unified the fonts for 123: and Analysis: so they look the same as the snippet content itself -> safe to merge i guess

@centosinfra-prod-github-app

Copy link
Copy Markdown

@mfocko
mfocko force-pushed the log-detective-snippets branch from fc0edc1 to d530952 Compare July 16, 2026 12:20
@centosinfra-prod-github-app

Copy link
Copy Markdown

Jany26 added 2 commits July 16, 2026 14:36
- conditioned on status=complete
- snippets are grouped by file into expandable blocks
- occasional per-snippet analysis is also displayed in a different color
- snippet text is truncated for long snippets

Assisted-by: Claude Sonnet 4.6
Signed-off-by: Jan Matufka <jmatufka@redhat.com>
Signed-off-by: Jan Matufka <jmatufka@redhat.com>
@Jany26
Jany26 force-pushed the log-detective-snippets branch from d530952 to 499ce66 Compare July 16, 2026 12:36
@centosinfra-prod-github-app

Copy link
Copy Markdown

@Jany26 Jany26 added the mergeit Merge via Zuul label Jul 16, 2026
@Jany26
Jany26 added this pull request to the merge queue Jul 16, 2026
Merged via the queue into packit:main with commit e573903 Jul 16, 2026
5 of 8 checks passed
@Jany26
Jany26 deleted the log-detective-snippets branch July 16, 2026 13:02
@github-project-automation github-project-automation Bot moved this from New to Done in Packit pull requests Jul 16, 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