Skip to content

OCPBUGS-70344: change OperatorHub strings to Software Catalog#15891

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
rhamilto:OCPBUGS-70344
Jan 13, 2026
Merged

OCPBUGS-70344: change OperatorHub strings to Software Catalog#15891
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
rhamilto:OCPBUGS-70344

Conversation

@rhamilto
Copy link
Copy Markdown
Member

@rhamilto rhamilto commented Jan 6, 2026

Note this PR only updates the instances of OperatorHub that appear directly in the UI. I did not update instances that appear in console.log since those are for debugging purposes.

@openshift-ci-robot openshift-ci-robot added jira/severity-low Referenced Jira bug's severity is low for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Jan 6, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@rhamilto: This pull request references Jira Issue OCPBUGS-70344, 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

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

Details

In response to this:

Note this PR only updates the instances of OperatorHub that appear directly in the UI. I did not update instances that appear in console.log since those are for debugging purposes.

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.

@openshift-ci openshift-ci Bot requested a review from yapei January 6, 2026 14:16
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 6, 2026

Walkthrough

Replace user-facing occurrences of "OperatorHub" with "Software Catalog" across localization files and React components; no logic, control flow, or exported API changes.

Changes

Cohort / File(s) Summary
Console App
frontend/packages/console-app/locales/en/console-app.json, frontend/packages/console-app/src/components/lightspeed/Lightspeed.tsx
Updated localization and button label from "Get started in OperatorHub" to "Get started in Software Catalog"
OLM Localization
frontend/packages/operator-lifecycle-manager/locales/en/olm.json
Replaced OperatorHub references with Software Catalog across multiple localized messages and help text
OLM Catalog Components
frontend/packages/operator-lifecycle-manager/src/components/catalog-source.tsx, frontend/packages/operator-lifecycle-manager/src/components/create-catalog-source.tsx
Updated DisabledPopover message and CreateCatalogSource help text to reference Software Catalog
OLM Modal Components
frontend/packages/operator-lifecycle-manager/src/components/modals/disable-default-source-modal.tsx, frontend/packages/operator-lifecycle-manager/src/components/modals/edit-default-sources-modal.tsx
Updated modal and warning copy to replace OperatorHub with Software Catalog
OLM Operator Hub UI
frontend/packages/operator-lifecycle-manager/src/components/operator-hub/operator-hub-items.tsx, frontend/packages/operator-lifecycle-manager/src/components/operator-hub/operator-hub-page.tsx
Updated empty-state titles/messages and minor formatting; "No OperatorHub items" → "No Software Catalog items"

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between 114aaf7 and 568606a.

📒 Files selected for processing (9)
  • frontend/packages/console-app/locales/en/console-app.json
  • frontend/packages/console-app/src/components/lightspeed/Lightspeed.tsx
  • frontend/packages/operator-lifecycle-manager/locales/en/olm.json
  • frontend/packages/operator-lifecycle-manager/src/components/catalog-source.tsx
  • frontend/packages/operator-lifecycle-manager/src/components/create-catalog-source.tsx
  • frontend/packages/operator-lifecycle-manager/src/components/modals/disable-default-source-modal.tsx
  • frontend/packages/operator-lifecycle-manager/src/components/modals/edit-default-sources-modal.tsx
  • frontend/packages/operator-lifecycle-manager/src/components/operator-hub/operator-hub-items.tsx
  • frontend/packages/operator-lifecycle-manager/src/components/operator-hub/operator-hub-page.tsx
🚧 Files skipped from review as they are similar to previous changes (5)
  • frontend/packages/operator-lifecycle-manager/src/components/modals/edit-default-sources-modal.tsx
  • frontend/packages/console-app/locales/en/console-app.json
  • frontend/packages/operator-lifecycle-manager/src/components/operator-hub/operator-hub-page.tsx
  • frontend/packages/operator-lifecycle-manager/src/components/create-catalog-source.tsx
  • frontend/packages/operator-lifecycle-manager/src/components/operator-hub/operator-hub-items.tsx
🧰 Additional context used
📓 Path-based instructions (1)
**

⚙️ CodeRabbit configuration file

-Focus on major issues impacting performance, readability, maintainability and security. Avoid nitpicks and avoid verbosity.

Files:

  • frontend/packages/operator-lifecycle-manager/src/components/catalog-source.tsx
  • frontend/packages/operator-lifecycle-manager/src/components/modals/disable-default-source-modal.tsx
  • frontend/packages/operator-lifecycle-manager/locales/en/olm.json
  • frontend/packages/console-app/src/components/lightspeed/Lightspeed.tsx
🧬 Code graph analysis (1)
frontend/packages/console-app/src/components/lightspeed/Lightspeed.tsx (1)
frontend/__mocks__/i18next.ts (1)
  • t (8-14)
🔇 Additional comments (4)
frontend/packages/console-app/src/components/lightspeed/Lightspeed.tsx (1)

135-135: LGTM! Terminology updated consistently.

The button label has been correctly updated to reference "Software Catalog" instead of "OperatorHub", aligning with the PR objectives.

frontend/packages/operator-lifecycle-manager/src/components/catalog-source.tsx (1)

425-427: LGTM! User-facing message updated correctly.

The DisabledPopover message now references "Software Catalog" instead of "OperatorHub", maintaining clarity while aligning with the new terminology.

frontend/packages/operator-lifecycle-manager/src/components/modals/disable-default-source-modal.tsx (1)

59-61: LGTM! Modal warning updated with correct terminology.

The modal body text now uses "Software Catalog" for the UI terminology while correctly preserving the technical resource names "OperatorSource and CatalogSource".

frontend/packages/operator-lifecycle-manager/locales/en/olm.json (1)

51-51: LGTM! All locale strings updated consistently.

The translation strings have been systematically updated to use "Software Catalog" instead of "OperatorHub" for user-facing terminology, while correctly preserving technical resource names like "CatalogSource". The updates are coherent and align with the PR objectives.

Also applies to: 110-110, 191-191, 301-301, 305-306


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

@openshift-ci openshift-ci Bot requested review from TheRealJon and spadgett January 6, 2026 14:16
@openshift-ci openshift-ci Bot added component/core Related to console core functionality component/olm Related to OLM approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/i18n Indicates issue or PR relates to internationalization or has content that needs to be translated labels Jan 6, 2026
@rhamilto
Copy link
Copy Markdown
Member Author

rhamilto commented Jan 6, 2026

/cherry-pick release-4.21

@openshift-cherrypick-robot
Copy link
Copy Markdown

@rhamilto: once the present PR merges, I will cherry-pick it on top of release-4.21 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-4.21

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.

@rhamilto
Copy link
Copy Markdown
Member Author

rhamilto commented Jan 6, 2026

/retest

3 similar comments
@rhamilto
Copy link
Copy Markdown
Member Author

rhamilto commented Jan 6, 2026

/retest

@rhamilto
Copy link
Copy Markdown
Member Author

rhamilto commented Jan 7, 2026

/retest

@jhadvig
Copy link
Copy Markdown
Member

jhadvig commented Jan 7, 2026

/retest

@jhadvig
Copy link
Copy Markdown
Member

jhadvig commented Jan 7, 2026

/override ci/prow/okd-scos-images

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jan 7, 2026

@jhadvig: Overrode contexts on behalf of jhadvig: ci/prow/okd-scos-images

Details

In response to this:

/override ci/prow/okd-scos-images

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.

@rhamilto
Copy link
Copy Markdown
Member Author

rhamilto commented Jan 7, 2026

/test backend

@rhamilto
Copy link
Copy Markdown
Member Author

rhamilto commented Jan 7, 2026

/override ci/prow/okd-scos-images

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jan 7, 2026

@rhamilto: Overrode contexts on behalf of rhamilto: ci/prow/okd-scos-images

Details

In response to this:

/override ci/prow/okd-scos-images

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.

@rhamilto
Copy link
Copy Markdown
Member Author

rhamilto commented Jan 8, 2026

@TheRealJon, mind reviewing? @yapei, mind verifying?

Copy link
Copy Markdown
Member

@jhadvig jhadvig left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jan 8, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jan 8, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jhadvig, rhamilto

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:

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

@jhadvig jhadvig added plugin-api-approved Indicates a PR with plugin API changes has been approved by an API reviewer and removed lgtm Indicates that a PR is ready to be merged. labels Jan 8, 2026
@yapei
Copy link
Copy Markdown
Contributor

yapei commented Jan 9, 2026

changes in the PR looks good
Screenshot 2026-01-09 at 3 36 31 PM
Screenshot 2026-01-09 at 3 33 52 PM
Screenshot 2026-01-09 at 3 32 03 PM
Screenshot 2026-01-09 at 3 29 13 PM

I see some other occurrence of OperatorHub in cypress test, the important one is here

      nav.sidenav.shouldNotHaveNavSection(['Ecosystem', 'OperatorHub']);

shall we also update the test together? I'm also fine to leave test as it is and update when needed ~

@rhamilto
Copy link
Copy Markdown
Member Author

rhamilto commented Jan 9, 2026

I see some other occurrence of OperatorHub in cypress test, the important one is here

I am thinking we should create a separate bug for the tests since they're not failing but are clearly outdated.

@yapei
Copy link
Copy Markdown
Contributor

yapei commented Jan 12, 2026

I see some other occurrence of OperatorHub in cypress test, the important one is here

I am thinking we should create a separate bug for the tests since they're not failing but are clearly outdated.

Thanks for the confirmation!

/verified by @yapei

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jan 12, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@yapei: This PR has been marked as verified by @yapei.

Details

In response to this:

I see some other occurrence of OperatorHub in cypress test, the important one is here

I am thinking we should create a separate bug for the tests since they're not failing but are clearly outdated.

Thanks for the confirmation!

/verified by @yapei

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.

@jhadvig jhadvig added the lgtm Indicates that a PR is ready to be merged. label Jan 12, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD 54bb8be and 2 for PR HEAD 568606a in total

@logonoff
Copy link
Copy Markdown
Member

/retest

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

/retest-required

Remaining retests: 0 against base HEAD 4e7a8f7 and 1 for PR HEAD 568606a in total

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jan 13, 2026

@rhamilto: all tests passed!

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-merge-bot openshift-merge-bot Bot merged commit db91e2e into openshift:main Jan 13, 2026
8 checks passed
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@rhamilto: Jira Issue Verification Checks: Jira Issue OCPBUGS-70344
✔️ This pull request was pre-merge verified.
✔️ All associated pull requests have merged.
✔️ All associated, merged pull requests were pre-merge verified.

Jira Issue OCPBUGS-70344 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓

Details

In response to this:

Note this PR only updates the instances of OperatorHub that appear directly in the UI. I did not update instances that appear in console.log since those are for debugging purposes.

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.

@openshift-cherrypick-robot
Copy link
Copy Markdown

@rhamilto: new pull request created: #15913

Details

In response to this:

/cherry-pick release-4.21

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.

@openshift-merge-robot
Copy link
Copy Markdown
Contributor

Fix included in accepted release 4.22.0-0.nightly-2026-01-19-085729

@rhamilto rhamilto deleted the OCPBUGS-70344 branch January 19, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. component/core Related to console core functionality component/olm Related to OLM jira/severity-low Referenced Jira bug's severity is low for the branch this PR is targeting. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. kind/i18n Indicates issue or PR relates to internationalization or has content that needs to be translated lgtm Indicates that a PR is ready to be merged. plugin-api-approved Indicates a PR with plugin API changes has been approved by an API reviewer verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants