Skip to content

fix(tests): use latest catalog index to resolve/workaround e2e issues - #4065

Merged
openshift-merge-bot[bot] merged 7 commits into
redhat-developer:mainfrom
christoph-jerolimov:use-latest-catalog-index
Jan 26, 2026
Merged

fix(tests): use latest catalog index to resolve/workaround e2e issues#4065
openshift-merge-bot[bot] merged 7 commits into
redhat-developer:mainfrom
christoph-jerolimov:use-latest-catalog-index

Conversation

@christoph-jerolimov

@christoph-jerolimov christoph-jerolimov commented Jan 26, 2026

Copy link
Copy Markdown
Member

Description

This changes switches the used container image for the catalog index to the latest catalog CI build version instead of the one that is linked from the latest versioned helm chart.

This removes one level of indirection from rhdh => rhdh-helm-chart => catalog index.

Which issue(s) does this PR fix

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

@openshift-ci
openshift-ci Bot requested review from albarbaro and josephca January 26, 2026 12:02
@christoph-jerolimov christoph-jerolimov changed the title fix(tests): use latest catalog index until a new helm chart is released that uses that version fix(tests): use latest catalog index until a new helm chart is released that uses that version to resolve/workaround e2e issues Jan 26, 2026
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
@christoph-jerolimov christoph-jerolimov changed the title fix(tests): use latest catalog index until a new helm chart is released that uses that version to resolve/workaround e2e issues fix(tests): use latest catalog index to resolve/workaround e2e issues Jan 26, 2026
@christoph-jerolimov christoph-jerolimov self-assigned this Jan 26, 2026
@github-actions

Copy link
Copy Markdown
Contributor

@nickboldt nickboldt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm
this will pull from the latest index TAG instead of the pinned/digested one in the latest quay.io/rhdh/chart -CI tagged chart.

christoph-jerolimov and others added 2 commits January 26, 2026 14:19
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
@github-actions

Copy link
Copy Markdown
Contributor

🚫 Image Push Skipped.

The container image push was skipped because the build was skipped (either due to [skip-build] tag or no relevant changes with existing image)

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
@github-actions

Copy link
Copy Markdown
Contributor

🚫 Image Push Skipped.

The container image push was skipped because the build was skipped (either due to [skip-build] tag or no relevant changes with existing image)

@github-actions

Copy link
Copy Markdown
Contributor

🚫 Image Push Skipped.

The container image push was skipped because the build was skipped (either due to [skip-build] tag or no relevant changes with existing image)

@github-actions

Copy link
Copy Markdown
Contributor

🚫 Image Push Skipped.

The container image push was skipped because the build was skipped (either due to [skip-build] tag or no relevant changes with existing image)

@github-actions

Copy link
Copy Markdown
Contributor

🚫 Image Push Skipped.

The container image push was skipped because the build was skipped (either due to [skip-build] tag or no relevant changes with existing image)

…ds are replicated in the custom values file

They will be overridden otherwise. See https://issues.redhat.com/browse/RHDHPLAN-869
@github-actions

Copy link
Copy Markdown
Contributor

🚫 Image Push Skipped.

The container image push was skipped because the build was skipped (either due to [skip-build] tag or no relevant changes with existing image)

@sonarqubecloud

Copy link
Copy Markdown

@rm3l rm3l left a comment

Copy link
Copy Markdown
Member

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 commented Jan 26, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nickboldt, rm3l

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

@openshift-merge-bot
openshift-merge-bot Bot merged commit 5e2ed32 into redhat-developer:main Jan 26, 2026
18 checks passed
@christoph-jerolimov
christoph-jerolimov deleted the use-latest-catalog-index branch January 26, 2026 18:38
@zdrapela zdrapela mentioned this pull request Feb 5, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants