Skip to content

ROX-34722: Switch to periodic konflux retest - #22554

Open
kurlov wants to merge 1 commit into
masterfrom
akurlov/ROX-34722-switch-to-periodic-konflux-retest
Open

ROX-34722: Switch to periodic konflux retest#22554
kurlov wants to merge 1 commit into
masterfrom
akurlov/ROX-34722-switch-to-periodic-konflux-retest

Conversation

@kurlov

@kurlov kurlov commented Sep 1, 2026

Copy link
Copy Markdown
Member

Description

Updates the Konflux retest process to run periodically, with the goal of optimizing GitHub Actions invocations.

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

test: https://github.com/stackrox/test-konflux-repo/pull/5

@coderabbitai

coderabbitai Bot commented Sep 1, 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: Team

Run ID: cd0c9a80-5b95-4a98-8c3c-9eb704cb3b68

📥 Commits

Reviewing files that changed from the base of the PR and between a61985a and 01ceed5.

📒 Files selected for processing (1)
  • .github/workflows/konflux-auto-retest.yml

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


📝 Walkthrough

Summary by CodeRabbit

  • New Features

    • Added periodic automated retesting every 10 minutes for eligible builds.
    • Added the ability to manually start a retest when needed.
    • Retesting now uses a standardized retest command for improved consistency.
  • Chores

    • Updated the automated build retest process to improve reliability and ensure failed builds are revisited regularly.

Walkthrough

The Konflux retest workflow now runs every 10 minutes or through manual dispatch. It calls the periodic retest workflow, adds read permissions, and uses the /konflux-retest command.

Changes

Konflux Retest Scheduling

Layer / File(s) Summary
Periodic retest workflow
.github/workflows/konflux-auto-retest.yml
The workflow replaces event-based triggers with a 10-minute schedule and manual dispatch. It calls the periodic reusable workflow, renames the job, updates permissions, removes the check suffix, and adds the /konflux-retest command.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 01cee

This localized workflow change is merge-ready after normal checks and review; no actionable merge-blocking risk remains.

Suggested reviewers: janisz

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title clearly and concisely describes the main change: switching Konflux retesting to a periodic process.
Description check ✅ Passed The description follows the required template, explains the change, addresses documentation, records testing status, and links to validation results. Several checklist items remain unchecked, but the …
Full details: Docstring Coverage

Explanation

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 files. (1 skipped: 1 unsupported.)

Full details: Description check

Explanation

The description follows the required template, explains the change, addresses documentation, records testing status, and links to validation results. Several checklist items remain unchecked, but the description is sufficiently complete for this workflow-only change.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch akurlov/ROX-34722-switch-to-periodic-konflux-retest

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

@codecov

codecov Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.38%. Comparing base (2bdd164) to head (01ceed5).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #22554      +/-   ##
==========================================
- Coverage   51.42%   51.38%   -0.05%     
==========================================
  Files        2876     2876              
  Lines      180446   180446              
==========================================
- Hits        92799    92720      -79     
- Misses      79560    79615      +55     
- Partials     8087     8111      +24     
Flag Coverage Δ
go-unit-tests 51.38% <ø> (-0.05%) ⬇️

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.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

🚀 Build Images Ready

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

export MAIN_IMAGE_TAG=5.0.x-136-g01ceed5a96

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

/konflux-retest operator-on-push

contains(github.event.check_run.pull_requests[0].url, format('/repos/{0}/pulls/', github.repository))
uses: stackrox/actions/.github/workflows/retest-konflux-builds.yml@9238e423c3ae1ac4eb0f254cbb98da9daae24d86 # ratchet:stackrox/actions/.github/workflows/retest-konflux-builds.yml@v1
retest:
uses: stackrox/actions/.github/workflows/periodic-retest-konflux-builds.yml@f4660e8 # ratchet:stackrox/actions/.github/workflows/periodic-retest-konflux-builds.yml@v1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this needs to be the full SHA

Suggested change
uses: stackrox/actions/.github/workflows/periodic-retest-konflux-builds.yml@f4660e8 # ratchet:stackrox/actions/.github/workflows/periodic-retest-konflux-builds.yml@v1
uses: stackrox/actions/.github/workflows/periodic-retest-konflux-builds.yml@f4660e8ce6bd7558e782e8e7cd26bad8cfbb6b9f # ratchet:stackrox/actions/.github/workflows/periodic-retest-konflux-builds.yml@v1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants