Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add manifests for odh-model-controller and model-mesh #639

Merged
merged 6 commits into from
Oct 20, 2023
Merged

add manifests for odh-model-controller and model-mesh #639

merged 6 commits into from
Oct 20, 2023

Conversation

VedantMahabaleshwarkar
Copy link
Contributor

@VedantMahabaleshwarkar VedantMahabaleshwarkar commented Oct 17, 2023

Description

  • Added model-mesh and odh-model-controller manifests to get_all_manifests.sh
  • added logic to allow kserve and model-mesh to share a singular odh-model-controller deployment

Testing

The testing for this PR is to be done along with the testing for this and this PR since they target the same set of overall changes.
The ODH operator image has been built with the following changes :

How Has This Been Tested?

ModelMesh testing

  • cluster login
  • git clone git@github.com:VedantMahabaleshwarkar/opendatahub-operator.git
  • cd opendatahub-operator
  • switch to branch transition
  • make deploy -e IMG=quay.io/vedantm/opendatahub-operator:version_builtin
  • This will install the ODH operator in NS opendatahub-operator-system with the image quay.io/vedantm/opendatahub-operator:version_builtin
  • create the following DSC in NS opendatahub
apiVersion: datasciencecluster.opendatahub.io/v1
kind: DataScienceCluster
metadata:
  name: example
spec:
  components:
    modelmeshserving:
      managementState: Managed
  • deploy and test a model with modelmesh

Kserve testing

Note: Cleanup kserve CRDs before testing modelmesh
  • cluster login
  • install ODH Kserve + dependencies stack by following instructions
Note: Only install the dependencies, the ODH operator will be a custom install according to next steps
  • git clone git@github.com:VedantMahabaleshwarkar/opendatahub-operator.git
  • cd opendatahub-operator
  • switch to branch transition
  • make deploy -e IMG=quay.io/vedantm/opendatahub-operator:version_builtin
  • This will install the ODH operator in NS opendatahub-operator-system with the image quay.io/vedantm/opendatahub-operator:version_builtin
  • create the following DSC in NS opendatahub
apiVersion: datasciencecluster.opendatahub.io/v1
kind: DataScienceCluster
metadata:
  name: default
spec:
  components:
    kserve:
      managementState: Managed

Merge criteria:

  • 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

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>
@openshift-ci
Copy link

openshift-ci bot commented Oct 17, 2023

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@VedantMahabaleshwarkar
Copy link
Contributor Author

Custom ODH operator image was built with the following settings in get_all_manifests.sh for the operator :

["odh-model-controller"]="VedantMahabaleshwarkar:odh-model-controller:manifests_transition_test:config:odh-model-controller"
["model-mesh"]="VedantMahabaleshwarkar:modelmesh-serving:manifests_overlay:config:model-mesh"

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>
components/kserve/kserve.go Outdated Show resolved Hide resolved
@VaishnaviHire
Copy link
Member

@VedantMahabaleshwarkar Can you also fix the linter issues?

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>
Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>
Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>
@openshift-ci openshift-ci bot added the lgtm label Oct 20, 2023
@openshift-ci
Copy link

openshift-ci bot commented Oct 20, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: VaishnaviHire

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

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>
@openshift-ci openshift-ci bot removed the lgtm label Oct 20, 2023
Copy link
Member

@VaishnaviHire VaishnaviHire left a comment

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 added the lgtm label Oct 20, 2023
@openshift-ci
Copy link

openshift-ci bot commented Oct 20, 2023

@VedantMahabaleshwarkar: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/opendatahub-operator-e2e 6d33c15 link true /test opendatahub-operator-e2e

Full PR test history. Your PR dashboard.

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/test-infra repository. I understand the commands that are listed here.

@VaishnaviHire
Copy link
Member

/test opendatahub-operator-e2e

@VaishnaviHire
Copy link
Member

Tests successful locally

/lgtm

@VaishnaviHire VaishnaviHire merged commit 2231cba into opendatahub-io:incubation Oct 20, 2023
VaishnaviHire pushed a commit to VaishnaviHire/opendatahub-operator that referenced this pull request Oct 20, 2023
…#639)

* add manifests for odh-model-controller and model-mesh

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>

* update correct release branches for modelmesh

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>

* fix odh-model-controller not getting uninstalled

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>

* fix lint

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>

* correct release branch for modelmesh to 0.11.0

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>

* support adding multiple repos in modelmesh devFlags

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>

---------

Signed-off-by: Vedant Mahabaleshwarkar <vmahabal@redhat.com>
(cherry picked from commit 2231cba)
@zdtsw zdtsw added the odh-1.11 label Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants