Skip to content

Hotfix: Add the missing SBOM stage for release status#137

Merged
yeyisan merged 3 commits intomasterfrom
hotfix-kdt-sbom-release-status
Nov 27, 2025
Merged

Hotfix: Add the missing SBOM stage for release status#137
yeyisan merged 3 commits intomasterfrom
hotfix-kdt-sbom-release-status

Conversation

@yeyisan
Copy link
Member

@yeyisan yeyisan commented Nov 26, 2025


Add SBOM Support to Release Status Command

Changes

  1. client/projects.go
    - Added SBOM field to ReleaseStatus struct
  2. cmd/release.go
    - Added --sbom flag for filtering SBOM-specific status checks
    - Updated status table output to include SBOM column
    - Added SBOM failure detection in isReleaseFailed function
    - Fixed verbose mode to skip scan detail fetch for SBOM (SBOM has no scan_id field)

Testing

  • ✅ Build successful
  • ✅ All existing tests pass
  • ✅ Verified --sbom flag appears in help output
  • ✅ No compilation errors

Expected Behavior After Fix

  • kdt release will display SBOM status in the output table
  • Command will properly fail with non-zero exit code when SBOM criteria fails
  • Aligns pipeline behavior with UI status display
  • Supports --sbom flag for specific SBOM status checking
  • Works correctly in verbose mode without "scan id cannot be empty" errors

s1: the release status is failed due to SBOM rules
Screenshot 2025-11-26 at 17 20 12

s2: the SBOM related rules are passed successfully
Screenshot 2025-11-26 at 17 21 36

canack
canack previously approved these changes Nov 26, 2025
nerginturgut
nerginturgut previously approved these changes Nov 26, 2025
Copy link
Contributor

@nerginturgut nerginturgut left a comment

Choose a reason for hiding this comment

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

it looks good 💯

@yeyisan yeyisan dismissed stale reviews from nerginturgut and canack via f2f8f09 November 26, 2025 15:29
@sonarqubecloud
Copy link

@yeyisan yeyisan merged commit c1a828a into master Nov 27, 2025
4 checks passed
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.

3 participants