Update Serving Runtime Config

lucferbux commented 1 year ago

Description

How Has This Been Tested?

Follow up of #301

Deploy the operator
Check that a new configmap called servingruntimes-config its created with a key called default-config

Merge criteria:

[X] The commits are squashed in a cohesive manner and have meaningful messages.
[X] JIRA link(s): https://issues.redhat.com/browse/RHODS-6330
[X] The Jira story is acked.
[X] Live build image: quay.io/rmartine/rhods-operator-live-catalog:1.22.0-1
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work.
[ ] QE contact acknowledges that this has been tested and is approved for merge.

VaishnaviHire commented 1 year ago

/lgtm

lucferbux commented 1 year ago

I have tested the changes, it's creating the configmap with the openvino image:

    - name: ovms
      image: quay.io/rmartine/odh-openvino-servingruntime-live:1.22.0-1

pablofelix commented 1 year ago

/lgtm

openshift-ci[bot] commented 1 year ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: pablofelix Once this PR has been reviewed and has the lgtm label, please ask for approval from lucferbux by writing /assign @lucferbux in a comment. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/red-hat-data-services/odh-deployer/blob/rhods-1.21/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

red-hat-data-services / odh-deployer