OLS-3210: Remove pf5/4-19 console images from related_images.json and bundle by blublinsky · Pull Request #1730 · openshift/lightspeed-operator

blublinsky · 2026-06-18T08:37:50Z

Description

Summary
Bundle follow-up for OLS-3210 / epic OLS-3205: align the OLM bundle with the single-console-image operator merged in #1705.

Operator code no longer accepts --console-image-pf5 or --console-image-4-19. The CSV still passed those flags and listed pf5/4-19 in relatedImages, which caused OLM installs to CrashLoop. This PR updates related_images.json and regenerates the bundle.

Changes
Remove lightspeed-console-plugin-pf5 and lightspeed-console-plugin-4-19 from related_images.json
Regenerate bundle (make bundle BUNDLE_TAG=1.1.1):
Drop --console-image-pf5 and --console-image-4-19 from the operator deployment args in the CSV
Drop pf5/4-19 entries from spec.relatedImages
Keep a single --console-image and lightspeed-console-plugin related imageSummary
Bundle follow-up for OLS-3210 / epic OLS-3205: align the OLM bundle with the single-console-image operator merged in #1705.

Operator code no longer accepts --console-image-pf5 or --console-image-4-19. The CSV still passed those flags and listed pf5/4-19 in relatedImages, which caused OLM installs to CrashLoop. This PR updates related_images.json and regenerates the bundle.

Changes
Remove lightspeed-console-plugin-pf5 and lightspeed-console-plugin-4-19 from related_images.json
Regenerate bundle (make bundle BUNDLE_TAG=1.1.1):
Drop --console-image-pf5 and --console-image-4-19 from the operator deployment args in the CSV
Drop pf5/4-19 entries from spec.relatedImages
Keep a single --console-image and lightspeed-console-plugin related image

Type of change

Related Tickets & Documents

Related Issue #
https://redhat.atlassian.net/browse/OLS-3210
closes

https://redhat.atlassian.net/browse/OLS-3210

Checklist before requesting a review

I have performed a self-review of my code.
PR has passed all pre-merge test jobs.
If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

Summary by CodeRabbit

Chores
- Updated the operator console plugin image references by consolidating to a single main image variant.
Documentation
- Clarified TLS certificate secret behavior for KeyCertSecretRef when the optional ca.crt key is not present.
Tests
- Removed the Tekton integration E2E pipeline used for console pre-commits on ephemeral clusters.

openshift-ci-robot · 2026-06-18T08:37:54Z

coderabbitai · 2026-06-18T08:38:03Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: fdb4347d-6173-4a5f-bdb7-20fc4646f851

📥 Commits

Reviewing files that changed from the base of the PR and between 0884eb4 and fcfc033.

⛔ Files ignored due to path filters (1)

related_images.json is excluded by !related_images.json

📒 Files selected for processing (2)

.tekton/integration-tests/pipelines/lightspeed-console-e2e-test-pipeline-pf5.yaml
bundle/manifests/lightspeed-operator.clusterserviceversion.yaml

💤 Files with no reviewable changes (1)

.tekton/integration-tests/pipelines/lightspeed-console-e2e-test-pipeline-pf5.yaml

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

bundle/manifests/lightspeed-operator.clusterserviceversion.yaml

📝 Walkthrough

Walkthrough

The ClusterServiceVersion manifest updates the TLS certificate secret reference CRD description to clarify behavior when the optional ca.crt key is absent, and removes the pf5 and 4-19 console-plugin variant image entries from the relatedImages list. The Tekton pipeline for console E2E testing on pf5 environments is entirely deleted.

Changes

CSV Manifest: Console Plugin Consolidation

Layer / File(s)	Summary
Console plugin images and ca.crt documentation `bundle/manifests/lightspeed-operator.clusterserviceversion.yaml`	The `relatedImages` section removes the `pf5` and `4-19` variant entries for `lightspeed-console-plugin`, leaving only the primary image entry. The CRD description for the optional `ca.crt` key in TLS certificate secret references is updated to specify that the console proxy falls back to the default system trust store when `ca.crt` is not provided.

Tekton Pipeline Removal

Layer / File(s)	Summary
Console E2E test pipeline deletion `.tekton/integration-tests/pipelines/lightspeed-console-e2e-test-pipeline-pf5.yaml`	The entire `ols-console-pre-commits` Tekton Pipeline resource is removed. This pipeline provisioned ephemeral clusters, installed operator bundles, ran Cypress E2E tests, and exported logs for pf5-variant console plugin environments.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

openshift/lightspeed-operator#1691: Modifies console-plugin image entries in the same CSV file, including earlier changes to pf5 and 4-19 variant references.
openshift/lightspeed-operator#1705: Aligns with the removal of multi-image console selection flags and consolidation of relatedImages to a single entry.
openshift/lightspeed-operator#1710: Modifies the same CSV file to update image digests and containerArgs for lightspeed-console-plugin variants.

Suggested reviewers

xrajesh

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly references the main objective: removing pf5/4-19 console images from related_images.json and the bundle, which aligns with the actual changes made to the ClusterServiceVersion manifest.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

… bundle

openshift-ci · 2026-06-19T09:06:25Z

@blublinsky: 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.

JoaoFula · 2026-06-19T09:10:19Z

/lgtm

blublinsky · 2026-06-19T10:08:23Z

/approve

openshift-ci · 2026-06-19T10:08:38Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: blublinsky

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:

~~OWNERS~~ [blublinsky]

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

openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 18, 2026

openshift-ci Bot requested review from bparees and raptorsun June 18, 2026 08:38

blublinsky force-pushed the bundle-console-multi-image branch 2 times, most recently from 0884eb4 to ab7b72d Compare June 18, 2026 10:51

openshift-ci Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 18, 2026

blublinsky mentioned this pull request Jun 18, 2026

OLS-3212: Remove pf5/419 Konflux artifacts and obsolete Cypress console pipeline #1734

Merged

11 tasks

blublinsky force-pushed the bundle-console-multi-image branch from ab7b72d to a3e5cbd Compare June 18, 2026 14:19

openshift-ci Bot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jun 18, 2026

OLS-3210: Remove pf5/4-19 console images from related_images.json and…

fcfc033

… bundle

blublinsky force-pushed the bundle-console-multi-image branch from a3e5cbd to fcfc033 Compare June 19, 2026 07:37

openshift-ci Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 19, 2026

openshift-ci Bot assigned JoaoFula Jun 19, 2026

openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 19, 2026

openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 19, 2026

openshift-merge-bot Bot merged commit b09aced into openshift:main Jun 19, 2026
14 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

OLS-3210: Remove pf5/4-19 console images from related_images.json and bundle#1730

OLS-3210: Remove pf5/4-19 console images from related_images.json and bundle#1730
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
blublinsky:bundle-console-multi-image

blublinsky commented Jun 18, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

openshift-ci-robot commented Jun 18, 2026 •

edited by openshift-ci Bot

Loading

Description

Type of change

Related Tickets & Documents

Checklist before requesting a review

Testing

Uh oh!

coderabbitai Bot commented Jun 18, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

openshift-ci Bot commented Jun 19, 2026

Uh oh!

JoaoFula commented Jun 19, 2026

Uh oh!

blublinsky commented Jun 19, 2026

Uh oh!

openshift-ci Bot commented Jun 19, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

blublinsky commented Jun 18, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of change

Related Tickets & Documents

Checklist before requesting a review

Testing

Summary by CodeRabbit

Uh oh!

openshift-ci-robot commented Jun 18, 2026 • edited by openshift-ci Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of change

Related Tickets & Documents

Checklist before requesting a review

Testing

Uh oh!

coderabbitai Bot commented Jun 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

openshift-ci Bot commented Jun 19, 2026

Uh oh!

JoaoFula commented Jun 19, 2026

Uh oh!

blublinsky commented Jun 19, 2026

Uh oh!

openshift-ci Bot commented Jun 19, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

blublinsky commented Jun 18, 2026 •

edited by coderabbitai Bot

Loading

openshift-ci-robot commented Jun 18, 2026 •

edited by openshift-ci Bot

Loading

coderabbitai Bot commented Jun 18, 2026 •

edited

Loading