OCPBUGS-69647: icon is missing from Console plugins list table #15856

krishagarwal278 · 2025-12-17T13:20:07Z

Fixed Status capitalization issue so Status component can recognize icons

Summary by CodeRabbit

Bug Fixes
- Fixed the console operator status display to show properly capitalized status text while maintaining error message visibility when status is "failed".

_{✏️ Tip: You can customize this high-level summary in your review settings.}

openshift-ci-robot · 2025-12-17T13:20:14Z

@krishagarwal278: This pull request references Jira Issue OCPBUGS-69647, which is invalid:

expected the bug to target the "4.22.0" version, but no target version was set

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

Fixed Status capitalization issue so Status component can recognize icons

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 openshift-eng/jira-lifecycle-plugin repository.

coderabbitai · 2025-12-17T13:20:27Z

Walkthrough

ConsoleOperatorConfig component modified to format plugin status strings by capitalizing the first character when passing to the Status component, while maintaining existing error message-based title handling for failed states.

Changes

Cohort / File(s)	Summary
Status display formatting `frontend/packages/console-app/src/components/console-operator/ConsoleOperatorConfig.tsx`	ConsolePluginStatus now uppercases status string first character (e.g., "failed" → "Failed") for consistent UI display; preserves errorMessage-based title logic when status is "failed"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: fixing Status capitalization so the Status component can recognize icons for console plugins, which directly addresses the missing icon issue.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

📝 Generate docstrings

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting

📥 Commits

Reviewing files that changed from the base of the PR and between b44673a and 67f40e8.

📒 Files selected for processing (1)

frontend/packages/console-app/src/components/console-operator/ConsoleOperatorConfig.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

frontend/packages/console-app/src/components/console-operator/ConsoleOperatorConfig.tsx

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

krishagarwal278 · 2025-12-17T13:20:50Z

/jira refresh

openshift-ci-robot · 2025-12-17T13:20:59Z

@krishagarwal278: This pull request references Jira Issue OCPBUGS-69647, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target version (4.22.0) matches configured target version for branch (4.22.0)
bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @yapei

Details

In response to this:

/jira refresh

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 openshift-eng/jira-lifecycle-plugin repository.

sg00dwin

/lgtm

frontend/packages/console-app/src/components/console-operator/ConsoleOperatorConfig.tsx

openshift-ci · 2025-12-17T19:05:58Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: krishagarwal278, logonoff, sg00dwin

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~frontend/OWNERS~~ [logonoff]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

krishagarwal278 · 2025-12-18T08:50:37Z

/retest

krishagarwal278 · 2025-12-18T10:02:59Z

/retest

krishagarwal278 · 2025-12-22T06:50:38Z

/retest

rhamilto · 2025-12-22T13:03:28Z

/test e2e-gcp-console

openshift-ci · 2025-12-22T18:43:08Z

New changes are detected. LGTM label has been removed.

krishagarwal278 · 2025-12-22T19:06:40Z

/retest

krishagarwal278 · 2025-12-23T14:41:19Z

/retest

krishagarwal278 · 2025-12-23T19:32:48Z

/retest

krishagarwal278 · 2026-01-12T05:18:03Z

/retest

krishagarwal278 · 2026-01-12T15:41:12Z

/retest

krishagarwal278 · 2026-01-13T09:39:21Z

/retest

Leo6Leo · 2026-01-14T15:57:00Z

/cc @Leo6Leo

…cons

openshift-ci-robot · 2026-01-14T20:22:06Z

@krishagarwal278: This pull request references Jira Issue OCPBUGS-69647, which is valid.

3 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target version (4.22.0) matches configured target version for branch (4.22.0)
bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @yapei

Details

In response to this:

Fixed Status capitalization issue so Status component can recognize icons

Summary by CodeRabbit

Bug Fixes

Fixed the console operator status display to show properly capitalized status text while maintaining error message visibility when status is "failed".

_{✏️ Tip: You can customize this high-level summary in your review settings.}

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 openshift-eng/jira-lifecycle-plugin repository.

krishagarwal278 · 2026-01-15T07:11:39Z

/test backend

krishagarwal278 · 2026-01-15T07:40:14Z

/retest

krishagarwal278 · 2026-01-16T12:13:08Z

/retest

openshift-ci · 2026-01-16T12:30:35Z

@krishagarwal278: The following test 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/e2e-gcp-console	`67f40e8`	link	true	`/test e2e-gcp-console`

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.

openshift-ci-robot added jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. and removed jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. labels Dec 17, 2025

openshift-ci bot requested review from rhamilto, sg00dwin and yapei December 17, 2025 13:21

openshift-ci bot added the component/core Related to console core functionality label Dec 17, 2025

sg00dwin approved these changes Dec 17, 2025

View reviewed changes

openshift-ci bot assigned sg00dwin Dec 17, 2025

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 17, 2025

rhamilto reviewed Dec 17, 2025

View reviewed changes

frontend/packages/console-app/src/components/console-operator/ConsoleOperatorConfig.tsx Show resolved Hide resolved

logonoff approved these changes Dec 17, 2025

View reviewed changes

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 17, 2025

openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 22, 2025

krishagarwal278 force-pushed the OCPBUGS-69647 branch from ec5ef08 to 58c1bf0 Compare December 22, 2025 18:43

openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Dec 22, 2025

openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 22, 2025

krishagarwal278 force-pushed the OCPBUGS-69647 branch from 58c1bf0 to e8500e7 Compare December 22, 2025 20:34

openshift-ci bot requested a review from Leo6Leo January 14, 2026 15:57

openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 14, 2026

openshift-ci bot added the kind/i18n Indicates issue or PR relates to internationalization or has content that needs to be translated label Jan 14, 2026

Fixed Status capitalization issue so Status component can recognize i…

67f40e8

…cons

krishagarwal278 force-pushed the OCPBUGS-69647 branch from b44673a to 67f40e8 Compare January 14, 2026 20:20

openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 14, 2026

OCPBUGS-69647: icon is missing from Console plugins list table #15856

Are you sure you want to change the base?

OCPBUGS-69647: icon is missing from Console plugins list table #15856

Conversation

krishagarwal278 commented Dec 17, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

openshift-ci-robot commented Dec 17, 2025

Uh oh!

coderabbitai bot commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Uh oh!

krishagarwal278 commented Dec 17, 2025

Uh oh!

openshift-ci-robot commented Dec 17, 2025

Uh oh!

sg00dwin left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

openshift-ci bot commented Dec 17, 2025

Uh oh!

krishagarwal278 commented Dec 18, 2025

Uh oh!

krishagarwal278 commented Dec 18, 2025

Uh oh!

krishagarwal278 commented Dec 22, 2025

Uh oh!

rhamilto commented Dec 22, 2025

Uh oh!

openshift-ci bot commented Dec 22, 2025

Uh oh!

krishagarwal278 commented Dec 22, 2025

Uh oh!

krishagarwal278 commented Dec 23, 2025

Uh oh!

krishagarwal278 commented Dec 23, 2025

Uh oh!

krishagarwal278 commented Jan 12, 2026

Uh oh!

krishagarwal278 commented Jan 12, 2026

Uh oh!

krishagarwal278 commented Jan 13, 2026

Uh oh!

Leo6Leo commented Jan 14, 2026

Uh oh!

openshift-ci-robot commented Jan 14, 2026

Summary by CodeRabbit

Uh oh!

krishagarwal278 commented Jan 15, 2026

Uh oh!

krishagarwal278 commented Jan 15, 2026

Uh oh!

krishagarwal278 commented Jan 16, 2026

Uh oh!

openshift-ci bot commented Jan 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

krishagarwal278 commented Dec 17, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Dec 17, 2025 •

edited

Loading