Conversation
Use PatternFly's warning variant on the Builds ready and Tests passed steps when no snapshot or test data exists. Increase spacing between the signal row and progress stepper. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Resolve merge conflicts from stashed CTRF refactor against the has_tests bug fix. Replace snapshot_test_results with test_suites and test_cases tables, simplify SnapshotRecord by removing trigger/released fields, and preserve the has_tests computed field for missing-test detection. - Add internal/ctrf package for CTRF report types - Update S3 sync to discover and parse CTRF reports - Replace JUnit XML parsing pipeline entirely - Update frontend to display test suites with tool info - Regenerate sqlc from updated schema and queries Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Brady Pratt <bpratt@redhat.com>
Ingest Clair vulnerability scan reports from S3 during the snapshot sync loop, store per-component/arch vulnerability data in SQLite, and render a new "Security Scans" tab on the release detail page. Backend: - Add vulnerability_reports and vulnerabilities tables with sqlc queries - Create internal/clair package for Clair JSON parse types - Parse scans/summary.json and per-arch clair-report-*.json from S3 - Extend sync.go Store interface and ingest() to persist scan data - Load vulnerability data in GetSnapshotByName alongside test results Frontend: - Add VulnerabilitiesTable component with CVE search, severity filters, and expandable description rows - Add VulnerabilityReport/Vulnerability types to SnapshotRecord - Security Scans tab with severity-colored labels and fix availability - Switch tabs to isFilled layout for better visual balance Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Brady Pratt <bpratt@redhat.com>
Collapse the flat (component, arch) row layout into one row per component with aggregated severity counts. Expanding a component reveals filled PatternFly tabs for each architecture, showing per-arch severity breakdown and the existing CVE table. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Brady Pratt <bpratt@redhat.com>
Signed-off-by: Brady Pratt <bpratt@redhat.com>
Marcusk19
approved these changes
Mar 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.