Skip to content

feat: Add Needs Attention, Infrastructure cards to Dashboard (#11627) - #313

Open
keithchong wants to merge 1 commit into
redhat-developer:mainfrom
keithchong:11627-DashboardPart2
Open

keithchong wants to merge 1 commit into
redhat-developer:mainfrom
keithchong:11627-DashboardPart2

Conversation

@keithchong

Copy link
Copy Markdown
Collaborator

See GITOPS-11627

@openshift-ci
openshift-ci Bot requested review from aali309 and wtam2018 November 2, 2026 22:11
@coderabbitai

coderabbitai Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8a9cfc6c-ed6b-4f64-a3de-08e84993ad81

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

@codecov-commenter

codecov-commenter commented Nov 2, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 478 lines in your changes missing coverage. Please review.
✅ Project coverage is 13.35%. Comparing base (3bd1dab) to head (cabd778).
⚠️ Report is 75 commits behind head on main.

Files with missing lines Patch % Lines
...ew/infrastructure/attention/NeedsAttentionCard.tsx 0.00% 93 Missing and 1 partial ⚠️
...iew/infrastructure/operator/GitOpsOperatorCard.tsx 0.00% 63 Missing and 1 partial ⚠️
...overview/infrastructure/shared/resourceEntries.tsx 0.00% 58 Missing and 1 partial ⚠️
...view/infrastructure/argocd/ArgoCDInstancesCard.tsx 0.00% 55 Missing and 1 partial ⚠️
...frastructure/shared/InfrastructureResourceCard.tsx 0.00% 39 Missing ⚠️
...rview/infrastructure/resources/ResourcesStatus.tsx 0.00% 30 Missing and 1 partial ⚠️
...tops/components/overview/DashboardOverviewPage.tsx 0.00% 18 Missing and 3 partials ⚠️
...mponents/overview/tabs/ApplicationsOverviewTab.tsx 0.00% 14 Missing ⚠️
...onents/overview/tabs/InfrastructureOverviewTab.tsx 0.00% 13 Missing ⚠️
.../infrastructure/shared/InfrastructureCardTitle.tsx 0.00% 11 Missing ⚠️
... and 13 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #313      +/-   ##
==========================================
+ Coverage   11.30%   13.35%   +2.04%     
==========================================
  Files         169      212      +43     
  Lines        6679     7992    +1313     
  Branches     2235     2723     +488     
==========================================
+ Hits          755     1067     +312     
- Misses       5923     6667     +744     
- Partials        1      258     +257     
Flag Coverage Δ
unit-tests 13.35% <0.00%> (+2.04%) ⬆️

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@keithchong
keithchong requested a review from dkarpele November 2, 2026 22:26
@keithchong
keithchong force-pushed the 11627-DashboardPart2 branch 6 times, most recently from 146a11b to 67b324e Compare November 3, 2026 08:55
Signed-off-by: Keith Chong <kykchong@redhat.com>
@keithchong
keithchong force-pushed the 11627-DashboardPart2 branch from 67b324e to cabd778 Compare November 3, 2026 09:02
@dkarpele

dkarpele commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator

@coderabbitai review

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