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).
⚠️ Report is 39 commits behind head on master.

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
@JoukoVirtanen JoukoVirtanen requested a review from mtodor April 5, 2026 04:36
@JoukoVirtanen
Copy link
Copy Markdown
Contributor Author

JoukoVirtanen commented Apr 6, 2026

/test jv-run-perf-scale-using-comment-perf-scale-24nodes-scale-test

@JoukoVirtanen
Copy link
Copy Markdown
Contributor Author

JoukoVirtanen commented Apr 6, 2026

/test pull-ci-stackrox-stackrox-master-ocp-4-21-scanner-v4-install-tests

@JoukoVirtanen
Copy link
Copy Markdown
Contributor Author

JoukoVirtanen commented Apr 6, 2026

pull-ci-stackrox-stackrox-master-ocp-4-21-scanner-v4-install-tests

@JoukoVirtanen
Copy link
Copy Markdown
Contributor Author

JoukoVirtanen commented Apr 8, 2026

/test gke-nongroovy-e2e-tests

davdhacs
davdhacs approved these changes Apr 15, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@davdhacs davdhacs left a comment

Choose a reason for hiding this comment

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

+1

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

Reviewers

@davdhacs davdhacs davdhacs approved these changes

@mtodor mtodor Awaiting requested review from mtodor

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JoukoVirtanen @davdhacs