Skip to content

ROX-33972: Run perf-scale using comment#19810

Open
JoukoVirtanen wants to merge 4 commits intomasterfrom
jv-run-perf-scale-using-comment
Open

ROX-33972: Run perf-scale using comment#19810
JoukoVirtanen wants to merge 4 commits intomasterfrom
jv-run-perf-scale-using-comment

Conversation

@JoukoVirtanen
Copy link
Copy Markdown
Contributor

@JoukoVirtanen JoukoVirtanen commented Apr 3, 2026 •
edited
Loading

Description

This is a companion PR to an openshift/release PR that makes it possible to run perf-scale tests for stackrox/stackrox PRs using a comment. This adds a script that waits for images needed for that test to be ready.

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 changes here are validated at openshift/release#77116

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Apr 3, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 3, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.60%. Comparing base (2d5d7a2) to head (61e0dd6).

Additional details and impacted files 
@@           Coverage Diff           @@
##           master   #19810   +/-   ##
=======================================
  Coverage   49.60%   49.60%           
=======================================
  Files        2763     2763           
  Lines      208339   208339           
=======================================
  Hits       103341   103341           
  Misses      97331    97331           
  Partials     7667     7667
Flag Coverage Δ
go-unit-tests 49.60% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 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.

@JoukoVirtanen JoukoVirtanen force-pushed the jv-run-perf-scale-using-comment branch from 35ae6c0 to f611b6b Compare April 3, 2026 21:48
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026 •
edited
Loading

🚀 Build Images Ready

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

export MAIN_IMAGE_TAG=4.11.x-564-g61e0dd66a5

@JoukoVirtanen JoukoVirtanen mentioned this pull request Apr 5, 2026
Open
@JoukoVirtanen JoukoVirtanen changed the title Jv run perf scale using comment ROX-33972: Run perf-scale using comment Apr 5, 2026
@JoukoVirtanen JoukoVirtanen marked this pull request as ready for review April 5, 2026 04:25
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Apr 5, 2026

@JoukoVirtanen: The following test 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-nongroovy-e2e-tests 61e0dd6 link true /test gke-nongroovy-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.

@JoukoVirtanen JoukoVirtanen requested a review from mtodor April 5, 2026 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mtodor mtodor Awaiting requested review from mtodor

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@JoukoVirtanen