Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

CI: Change to the rolling channel for operator installs in CI tests #656

Merged
merged 1 commit into from
Aug 19, 2022

Conversation

LaVLaS
Copy link
Contributor

@LaVLaS LaVLaS commented Aug 18, 2022

Signed-off-by: Landon LaSmith LLaSmith@redhat.com

Description

Fixes an issue where the CI test container would not install the operator when running on OCP > 4.8

  • Updates the CI tests to install the ODH operator from the rolling OLM channel. beta channel was removed starting with ODH v1.1.1 which added an install requirement for OCP 4.9+
  • Updates the OC cli version to 4.10

This should resolve failures in openshift/release#31350

How Has This Been Tested?

Ran the CI peak container against a 4.11 cluster where the odh-operator was not installed

Merge criteria:

CI test runs successfully

  • The commits are squashed in a cohesive manner and have meaningful messages.
  • Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has manually tested the changes and verified that the changes work

@LaVLaS
Copy link
Contributor Author

LaVLaS commented Aug 19, 2022

/retest

2 similar comments
@LaVLaS
Copy link
Contributor Author

LaVLaS commented Aug 19, 2022

/retest

@LaVLaS
Copy link
Contributor Author

LaVLaS commented Aug 19, 2022

/retest

tests/Dockerfile Outdated Show resolved Hide resolved
Signed-off-by: Landon LaSmith <LLaSmith@redhat.com>
@LaVLaS
Copy link
Contributor Author

LaVLaS commented Aug 19, 2022

/approve

@openshift-ci
Copy link

openshift-ci bot commented Aug 19, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LaVLaS

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

The pull request process is described here

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

@VaishnaviHire
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Aug 19, 2022
@openshift-merge-robot openshift-merge-robot merged commit 28dfc47 into opendatahub-io:master Aug 19, 2022
@LaVLaS LaVLaS deleted the fix-ci-test branch August 19, 2022 18:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants