ROX-36481: [UI] Add Top prefix to aggregate CVSS/CVE Severity columns - #22517
ROX-36481: [UI] Add Top prefix to aggregate CVSS/CVE Severity columns#22517alkmim wants to merge 1 commit into
Conversation
|
Skipping CI for Draft Pull Request. |
11b3603 to
9f831bd
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughSummary by CodeRabbit
WalkthroughThe pull request renames workload CVE table labels to distinguish top severity and CVSS values. The related Cypress assertion now expects the updated severity label. ChangesWorkload CVE label updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This change clarifies aggregated vulnerability column headings without altering the underlying values or behavior, and no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description follows the required template, identifies the affected pages, records documentation and quality checks, explains the modified existing test, and documents manual validation with screenshots.
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
🚀 Build Images ReadyImages are ready for commit 929ab3f. To use with deploy scripts: export MAIN_IMAGE_TAG=5.0.x-135-g929ab3f3ea |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #22517 +/- ##
===========================================
+ Coverage 50.06% 51.38% +1.32%
===========================================
Files 791 2876 +2085
Lines 62717 180438 +117721
===========================================
+ Hits 31397 92714 +61317
- Misses 28003 79615 +51612
- Partials 3317 8109 +4792
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
ab8c426 to
5a1d808
Compare
383ab8d to
ace54e3
Compare
|
Is this not also true for virtual machine vulnerabilities? |
ace54e3 to
ed0d99b
Compare
ed0d99b to
929ab3f
Compare
Description
"CVSS" and "CVE severity" column headings changed to "Top CVSS" and "Top CVE Severity" on pages where the values are aggregated, pages:
User-facing documentation
Testing and quality
Automated testing
How I validated my change
Deployed the UI and manually checked the fields. I also confirmed that filtering the columns is working.
CVE page
Deployment Page
Image Page