Skip to content

chore: bump collector and fact ahead of the code freeze - #22839

Draft
Molter73 wants to merge 1 commit into
masterfrom
mauro/chore/bump-fact-collector
Draft

chore: bump collector and fact ahead of the code freeze#22839
Molter73 wants to merge 1 commit into
masterfrom
mauro/chore/bump-fact-collector

Conversation

@Molter73

@Molter73 Molter73 commented Sep 15, 2026

Copy link
Copy Markdown
Member

Description

The versions being set to the corresponding _VERSION files are the ones that will be used as part of the upcoming StackRox 5.0 release.

User-facing documentation

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • added unit tests
  • added e2e tests
  • added regression tests
  • added compatibility tests
  • modified existing tests

How I validated my change

The versions being used have been validated in the CI for the corresponding projects.

@Molter73
chore: bump collector and fact ahead of the code freeze
a96c3b6 
The versions being set to the corresponding `_VERSION` files are the
ones that will be used as part of the upcoming StackRox 5.0 release.
@github-actions github-actions Bot added the ci-all-qa-tests Tells CI to run all API tests (not just BAT). label Sep 15, 2026
@coderabbitai

coderabbitai Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 94973d7e-1f5d-4ed7-b126-104d6af95896

📥 Commits

Reviewing files that changed from the base of the PR and between 116d16d and a96c3b6.

📒 Files selected for processing (2)
  • COLLECTOR_VERSION
  • FACT_VERSION

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Chores
    • Updated the Collector component to version 3.25.7.
    • Updated the Fact component to version 0.4.0.

Walkthrough

The pull request updates COLLECTOR_VERSION to 3.25.7 and FACT_VERSION to 0.4.0.

Changes

Version updates

Layer / File(s) Summary
Update version values
COLLECTOR_VERSION, FACT_VERSION		 The collector and fact version values were updated to 3.25.7 and 0.4.0.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Suggested reviewers: rhacs-bot

Merge Risk: ⚪ Minimal · up to a96c3

The version updates match the intended release values, with no actionable merge-blocking risk identified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the version bumps for collector and fact before the code freeze.
Description check ✅ Passed The description follows the required template and explains the release purpose and validation. It does not explicitly confirm CI inspection, but the omission is non-critical for this small version-onl…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch mauro/chore/bump-fact-collector

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

@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

🚀 Build Images Ready

Images are ready for commit a96c3b6. To use with deploy scripts:

export MAIN_IMAGE_TAG=5.0.x-300-ga96c3b6df6

@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.79%. Comparing base (116d16d) to head (a96c3b6).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master   #22839      +/-   ##
==========================================
- Coverage   51.82%   51.79%   -0.04%     
==========================================
  Files        2901     2901              
  Lines      182815   182815              
==========================================
- Hits        94753    94682      -71     
- Misses      79774    79825      +51     
- Partials     8288     8308      +20
Flag Coverage Δ
go-unit-tests 51.79% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@openshift-ci

openshift-ci Bot commented Sep 15, 2026

Copy link
Copy Markdown

@Molter73: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/gke-operator-e2e-tests a96c3b6 link false /test gke-operator-e2e-tests
ci/prow/ocp-4-22-qa-e2e-tests a96c3b6 link false /test ocp-4-22-qa-e2e-tests
ci/prow/gke-nongroovy-e2e-tests a96c3b6 link true /test gke-nongroovy-e2e-tests
ci/prow/ocp-4-12-operator-e2e-tests a96c3b6 link false /test ocp-4-12-operator-e2e-tests
ci/prow/ocp-4-22-operator-e2e-tests a96c3b6 link false /test ocp-4-22-operator-e2e-tests
ci/prow/gke-qa-e2e-tests a96c3b6 link false /test gke-qa-e2e-tests
ci/prow/ocp-4-22-nongroovy-e2e-tests a96c3b6 link false /test ocp-4-22-nongroovy-e2e-tests
ci/prow/ocp-4-12-qa-e2e-tests a96c3b6 link false /test ocp-4-12-qa-e2e-tests

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@Molter73
Molter73 marked this pull request as draft September 15, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

ci-all-qa-tests Tells CI to run all API tests (not just BAT). do-not-merge/work-in-progress

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Molter73