Skip to content

ROX-36447: Enable both clair-scan and roxctl-scan - #1573

Merged
vickeybrown merged 1 commit into
mainfrom
migrate-to-roxctl-scan
Sep 3, 2026
Merged

ROX-36447: Enable both clair-scan and roxctl-scan#1573
vickeybrown merged 1 commit into
mainfrom
migrate-to-roxctl-scan

Conversation

@vickeybrown

@vickeybrown vickeybrown commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Description

Replace clair-scan task with roxctl-scan to provide better security coverage as per Konflux Integration Service Team migration plan.

Changes:

  • Updated task name from clair-scan to roxctl-scan
  • Updated bundle reference to task-roxctl-scan:0.1

Checklist

  • Patch has a change log entry OR does not need one.
  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

If any of these don't apply, please comment below.

Testing Performed

CI is sufficient

@vickeybrown
vickeybrown requested review from a team and rhacs-bot as code owners August 25, 2026 16:14
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Team

Run ID: 41703f98-86d6-4c7f-b8bc-8e5671612a7f


Comment @coderabbitai help to get the list of available commands.

@vickeybrown vickeybrown changed the title Migrate from clair-scan to roxctl-scan ROX-36447: Migrate from clair-scan to roxctl-scan Aug 25, 2026
@codecov-commenter

codecov-commenter commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.20%. Comparing base (2276b55) to head (8fda436).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1573   +/-   ##
=======================================
  Coverage   33.20%   33.20%           
=======================================
  Files          22       22           
  Lines        3499     3499           
  Branches     3499     3499           
=======================================
  Hits         1162     1162           
  Misses       2332     2332           
  Partials        5        5           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vickeybrown

Copy link
Copy Markdown
Contributor Author

/retest

@vickeybrown
vickeybrown force-pushed the migrate-to-roxctl-scan branch 2 times, most recently from d66c678 to 05f7260 Compare September 1, 2026 18:44
Replace clair-scan task with roxctl-scan to provide better security
coverage as per Konflux Integration Service Team migration plan.

Changes:
- Updated task name from clair-scan to roxctl-scan
- Updated bundle reference to task-roxctl-scan:0.1

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

# Conflicts:
#	.tekton/fact-component-pipeline.yaml

# Conflicts:
#	.tekton/fact-component-pipeline.yaml
@vickeybrown
vickeybrown force-pushed the migrate-to-roxctl-scan branch from 05f7260 to 8fda436 Compare September 3, 2026 14:51
@msugakov msugakov changed the title ROX-36447: Migrate from clair-scan to roxctl-scan ROX-36447: Enable both clair-scan and roxctl-scan - #22452 Sep 3, 2026
@msugakov msugakov changed the title ROX-36447: Enable both clair-scan and roxctl-scan - #22452 ROX-36447: Enable both clair-scan and roxctl-scan Sep 3, 2026
@vickeybrown
vickeybrown merged commit 6c6e937 into main Sep 3, 2026
42 of 43 checks passed
@vickeybrown
vickeybrown deleted the migrate-to-roxctl-scan branch September 3, 2026 16:24
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