Skip to content

ROX-27490: Add failed clusters to storage protos#15201

Merged
lvalerom merged 1 commit intomasterfrom
lvm/rox-27490-add-failed-clusters-to-storage-protos
May 16, 2025
Merged

ROX-27490: Add failed clusters to storage protos#15201
lvalerom merged 1 commit intomasterfrom
lvm/rox-27490-add-failed-clusters-to-storage-protos

Conversation

@lvalerom
Copy link
Contributor

@lvalerom lvalerom commented May 6, 2025

Description

Storage proto changes to add the information of failed clusters when a report is generated.

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 6, 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-information May 6, 2025 20:48
@lvalerom lvalerom changed the base branch from lvm/rox-27490-add-failed-clusters-information to lvm/rox-26974-add-failed-clusters-to-the-api-response May 6, 2025 20:49
@rhacs-bot
Copy link
Contributor

rhacs-bot commented May 6, 2025

Images are ready for the commit at 9d8eb96.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.8.x-661-ge9a3923152.

@codecov
Copy link

codecov bot commented May 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.11%. Comparing base (6f8bf50) to head (9d8eb96).
Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #15201      +/-   ##
==========================================
- Coverage   49.12%   49.11%   -0.01%     
==========================================
  Files        2568     2568              
  Lines      188577   188577              
==========================================
- Hits        92630    92619      -11     
- Misses      88641    88649       +8     
- Partials     7306     7309       +3     
Flag Coverage Δ
go-unit-tests 49.11% <ø> (-0.01%) ⬇️

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 force-pushed the lvm/rox-26974-add-failed-clusters-to-the-api-response branch from e47be13 to d0223ad Compare May 8, 2025 10:54
@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 marked this pull request as ready for review May 9, 2025 10:24
@lvalerom lvalerom requested a review from a team as a code owner May 9, 2025 10:24
@lvalerom lvalerom requested review from dashrews78, mtodor and vikin91 May 9, 2025 10:37
@lvalerom lvalerom force-pushed the lvm/rox-26974-add-failed-clusters-to-the-api-response branch from d0223ad to d54ead3 Compare May 9, 2025 10:46
@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
Copy link
Contributor

@vikin91 vikin91 left a comment

Choose a reason for hiding this comment

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

Looks simple and good!

Base automatically changed from lvm/rox-26974-add-failed-clusters-to-the-api-response to master May 14, 2025 15:50
@lvalerom lvalerom force-pushed the lvm/rox-27490-add-failed-clusters-to-storage-protos branch from 4f402ec to e9a3923 Compare May 16, 2025 11:09
@lvalerom lvalerom force-pushed the lvm/rox-27490-add-failed-clusters-to-storage-protos branch from e9a3923 to 9d8eb96 Compare May 16, 2025 11:10
@lvalerom
Copy link
Contributor Author

@vikin91 I changed reason for repeated reasons as per #13361 (comment)

@vikin91
Copy link
Contributor

vikin91 commented May 16, 2025

@vikin91 I changed reason for repeated reasons as per #13361 (comment)

Great! Thanks for addressing that :)

@lvalerom lvalerom merged commit 0c967a0 into master May 16, 2025
98 checks passed
@lvalerom lvalerom deleted the lvm/rox-27490-add-failed-clusters-to-storage-protos branch May 16, 2025 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants