Skip to content

ci: revert "fix(ci): temporarily pin catalog index image in CI to 1.10-51 to work around wrong lightspeed OCI refs [RHDHBUGS-3095]"#401

Merged
rm3l merged 1 commit into
redhat-developer:mainfrom
rm3l:chore/revert_pinning_catalog_index_image_in_ci
May 17, 2026
Merged

ci: revert "fix(ci): temporarily pin catalog index image in CI to 1.10-51 to work around wrong lightspeed OCI refs [RHDHBUGS-3095]"#401
rm3l merged 1 commit into
redhat-developer:mainfrom
rm3l:chore/revert_pinning_catalog_index_image_in_ci

Conversation

@rm3l

@rm3l rm3l commented May 17, 2026

Copy link
Copy Markdown
Member

Description of the change

This reverts commit 8e40fbf.

Which issue(s) does this PR fix or relate to

  • JIRA_issue_link

How to test changes / Special notes to the reviewer

Checklist

  • For each Chart updated, version bumped in the corresponding Chart.yaml according to Semantic Versioning.
  • For each Chart updated, variables are documented in the values.yaml and added to the corresponding README.md. The pre-commit utility can be used to generate the necessary content. Run pre-commit run --all-files to run the hooks and then push any resulting changes. The pre-commit Workflow will enforce this and warn you if needed.
  • JSON Schema template updated and re-generated the raw schema via the pre-commit hook.
  • Tests pass using the Chart Testing tool and the ct lint command.
  • If you updated the orchestrator-infra chart, make sure the versions of the Knative CRDs are aligned with the versions of the CRDs installed by the OpenShift Serverless operators declared in the values.yaml file. See Installing Knative Eventing and Knative Serving CRDs for more details.

…1` to work around wrong lightspeed OCI refs [RHDHBUGS-3095] (redhat-developer#387)"

This reverts commit 8e40fbf.
@rm3l rm3l requested a review from a team as a code owner May 17, 2026 21:00
@openshift-ci openshift-ci Bot requested review from gazarenkov and zdrapela May 17, 2026 21:00
@rhdh-qodo-merge

rhdh-qodo-merge Bot commented May 17, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@sonarqubecloud

Copy link
Copy Markdown

@rhdh-qodo-merge

rhdh-qodo-merge Bot commented May 17, 2026

Copy link
Copy Markdown

Review Summary by Qodo

Revert catalog index image tag pinning in CI workflows

📦 Other

Grey Divider

Walkthroughs

Description
• Removes temporary catalog index image tag pinning from CI workflows
• Reverts workaround for lightspeed OCI refs issue ([RHDHBUGS-3095](https://redhat.atlassian.net/browse/RHDHBUGS-3095))
• Simplifies Helm arguments in nightly and test workflows
• Allows catalog index to use latest version in CI
Diagram
flowchart LR
  A["CI Workflows<br/>nightly.yaml & test.yaml"] -->|Remove pinning| B["catalogIndex.image.tag=1.10-51"]
  B -->|Revert| C["Use latest catalog index"]
  C -->|Result| D["Simplified Helm args"]
Loading

Grey Divider

File Changes

1. .github/workflows/nightly.yaml ⚙️ Configuration changes +1/-2

Remove catalog index tag pinning from nightly workflow

• Removes --set global.catalogIndex.image.tag=1.10-51 from extra_helm_args
• Deletes TODO comment referencing [RHDHBUGS-3095](https://redhat.atlassian.net/browse/RHDHBUGS-3095) workaround
• Keeps other Helm arguments for backstage image configuration

.github/workflows/nightly.yaml


2. .github/workflows/test.yaml ⚙️ Configuration changes +1/-2

Remove catalog index tag pinning from test workflow

• Removes --set global.catalogIndex.image.tag=1.10-51 from conditional Helm args
• Deletes TODO comment referencing [RHDHBUGS-3095](https://redhat.atlassian.net/browse/RHDHBUGS-3095) workaround
• Simplifies main branch condition to only set backstage image parameters
• Changes fallback from pinned tag to empty string for non-main branches

.github/workflows/test.yaml


Grey Divider

Qodo Logo

@rm3l rm3l merged commit f7a70d8 into redhat-developer:main May 17, 2026
5 of 6 checks passed
@rm3l rm3l deleted the chore/revert_pinning_catalog_index_image_in_ci branch May 17, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant