Skip to content

ROX-27490: Validate Results#15218

Merged
lvalerom merged 4 commits intomasterfrom
lvm/rox-27490-handle-failed-clusters-in-results-validation
May 26, 2025
Merged

ROX-27490: Validate Results#15218
lvalerom merged 4 commits intomasterfrom
lvm/rox-27490-handle-failed-clusters-in-results-validation

Conversation

@lvalerom
Copy link
Contributor

@lvalerom lvalerom commented May 7, 2025

Description

Validate the ScanResults and generate the failedClusters map.

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

@openshift-ci
Copy link

openshift-ci bot commented May 7, 2025

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

@lvalerom lvalerom changed the base branch from master to lvm/rox-27490-add-failed-clusters-to-storage-protos May 7, 2025 18:37
@rhacs-bot
Copy link
Contributor

rhacs-bot commented May 7, 2025

Images are ready for the commit at 002dd10.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.8.x-742-g002dd10aa4.

@codecov
Copy link

codecov bot commented May 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.25%. Comparing base (ad1344e) to head (002dd10).
Report is 29 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #15218      +/-   ##
==========================================
+ Coverage   49.13%   49.25%   +0.12%     
==========================================
  Files        2570     2574       +4     
  Lines      188682   188908     +226     
==========================================
+ Hits        92701    93049     +348     
+ Misses      88668    88540     -128     
- Partials     7313     7319       +6     
Flag Coverage Δ
go-unit-tests 49.25% <100.00%> (+0.12%) ⬆️

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.

@lvalerom lvalerom mentioned this pull request May 8, 2025
11 tasks
@lvalerom lvalerom force-pushed the lvm/rox-27490-add-failed-clusters-to-storage-protos branch from 13bef89 to d4d7962 Compare May 8, 2025 10:56
@lvalerom lvalerom force-pushed the lvm/rox-27490-handle-failed-clusters-in-results-validation branch from 5e2be20 to 92c0ad5 Compare May 8, 2025 10:58
@lvalerom lvalerom marked this pull request as ready for review May 9, 2025 10:32
@lvalerom lvalerom requested review from dashrews78, mtodor and vikin91 May 9, 2025 10:38
@lvalerom lvalerom force-pushed the lvm/rox-27490-add-failed-clusters-to-storage-protos branch from d4d7962 to 4f402ec Compare May 9, 2025 10:47
@lvalerom lvalerom requested a review from a team as a code owner May 9, 2025 10:47
@lvalerom lvalerom force-pushed the lvm/rox-27490-handle-failed-clusters-in-results-validation branch from 92c0ad5 to f491ed5 Compare May 9, 2025 10:49
@lvalerom lvalerom force-pushed the lvm/rox-27490-add-failed-clusters-to-storage-protos branch 2 times, most recently from e9a3923 to 9d8eb96 Compare May 16, 2025 11:10
Base automatically changed from lvm/rox-27490-add-failed-clusters-to-storage-protos to master May 16, 2025 16:19
@lvalerom lvalerom force-pushed the lvm/rox-27490-handle-failed-clusters-in-results-validation branch from f491ed5 to 13315f8 Compare May 16, 2025 16:56
@lvalerom lvalerom force-pushed the lvm/rox-27490-handle-failed-clusters-in-results-validation branch from 3a42aaa to 37069e5 Compare May 19, 2025 10:05
@lvalerom lvalerom requested a review from mtodor May 21, 2025 11:59
@openshift-ci
Copy link

openshift-ci bot commented May 21, 2025

@lvalerom: 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 f491ed5 link false /test gke-operator-e2e-tests
ci/prow/gke-scanner-v4-install-tests f491ed5 link false /test gke-scanner-v4-install-tests
ci/prow/ocp-4-18-scanner-v4-install-tests f491ed5 link false /test ocp-4-18-scanner-v4-install-tests
ci/prow/ocp-4-18-ui-e2e-tests f491ed5 link false /test ocp-4-18-ui-e2e-tests
ci/prow/ocp-4-18-qa-e2e-tests f491ed5 link false /test ocp-4-18-qa-e2e-tests
ci/prow/ocp-4-12-ui-e2e-tests f491ed5 link false /test ocp-4-12-ui-e2e-tests
ci/prow/ocp-4-18-operator-e2e-tests f491ed5 link false /test ocp-4-18-operator-e2e-tests
ci/prow/ocp-4-12-scanner-v4-install-tests f491ed5 link false /test ocp-4-12-scanner-v4-install-tests
ci/prow/ocp-4-12-operator-e2e-tests f491ed5 link false /test ocp-4-12-operator-e2e-tests
ci/prow/ocp-4-12-qa-e2e-tests f491ed5 link false /test ocp-4-12-qa-e2e-tests
ci/prow/gke-qa-e2e-tests 002dd10 link false /test gke-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.

@lvalerom lvalerom merged commit c4d0830 into master May 26, 2025
88 of 89 checks passed
@lvalerom lvalerom deleted the lvm/rox-27490-handle-failed-clusters-in-results-validation branch May 26, 2025 16:15
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.

3 participants