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

Revert "UPSTREAM <carry>: Rebase code to kfp master branch - commit: 65d1d79fb7c77db076de0e9799516269c49e01a1" #138

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

mprahl
Copy link

@mprahl mprahl commented Feb 13, 2025

Reverts #134 due to TLS issues when connecting to an external database.

@openshift-ci openshift-ci bot requested review from gmfrasca and hbelmiro February 13, 2025 22:16
Copy link

openshift-ci bot commented Feb 13, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from mprahl. For more information see the 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:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mprahl
Copy link
Author

mprahl commented Feb 13, 2025

/cc @HumairAK

Copy link

openshift-ci bot commented Feb 13, 2025

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the opendatahub-io org. You can then trigger verification by writing /verify-owners in a comment.

  • zijianjoy
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • frontend/OWNERS
    • manifests/gcp_marketplace/OWNERS
    • manifests/kustomize/OWNERS
    • samples/contrib/kubeflow-e2e-mnist/OWNERS
  • ojarjur
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
  • aronchick
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/azure/OWNERS
  • yhwang
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/kserve/OWNERS
    • components/kubeflow/kfserving/OWNERS
  • NikeNano
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
  • SinaChavoshi
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/OWNERS
    • tools/diagnose/OWNERS
  • eedorenko
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/azure/OWNERS
    • manifests/kustomize/env/azure/OWNERS
  • jlyaoyuli
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • frontend/OWNERS
    • test/OWNERS
  • StefanoFioravanzo
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
  • Jeffwan
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
  • connor-mccarthy
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • api/OWNERS
    • docs/OWNERS
    • kubernetes_platform/OWNERS
    • samples/OWNERS
    • sdk/OWNERS
    • test/OWNERS
  • surajkota
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/aws/OWNERS
    • components/aws/sagemaker/OWNERS
    • samples/contrib/aws-samples/OWNERS
  • ananth102
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/aws/OWNERS
    • components/aws/sagemaker/OWNERS
    • samples/contrib/aws-samples/OWNERS
  • johnugeorge
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • samples/contrib/kubeflow-e2e-mnist/OWNERS
    • samples/contrib/kubeflow-katib/OWNERS
  • gkcalat
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • kubernetes_platform/OWNERS
    • manifests/gcp_marketplace/OWNERS
    • manifests/kustomize/OWNERS
    • test/OWNERS
  • numerology
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • samples/contrib/OWNERS
    • test/sample-test/OWNERS
  • gaoning777
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • test/sample-test/OWNERS
  • chensun
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • api/OWNERS
    • backend/OWNERS
    • docs/OWNERS
    • kubernetes_platform/OWNERS
    • samples/OWNERS
    • sdk/OWNERS
    • test/OWNERS
    • test/api-integration-test/OWNERS
    • third_party/OWNERS
  • IronPan
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/OWNERS
    • proxy/OWNERS
    • test/imagebuilder/OWNERS
    • test/initialization-test/OWNERS
    • third_party/OWNERS
    • tools/bazel_builder/OWNERS
  • jagadeeshi2i
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/PyTorch/pytorch-kfp-components/OWNERS
    • samples/contrib/pytorch-samples/OWNERS
  • rileyjbauer
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • test/frontend-integration-test/OWNERS
    • test/initialization-test/OWNERS
  • dushyanthsc
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • third_party/OWNERS
  • ryansteakley
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/aws/OWNERS
    • components/aws/sagemaker/OWNERS
    • samples/contrib/aws-samples/OWNERS
  • yilun-msft
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/azure/OWNERS
  • gaocegege
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/kubeflow/launcher/OWNERS
    • samples/contrib/kubeflow-e2e-mnist/OWNERS
    • samples/contrib/kubeflow-katib/OWNERS
  • adrian555
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • samples/contrib/ibm-samples/OWNERS
  • andreyvelich
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • samples/contrib/kubeflow-e2e-mnist/OWNERS
    • samples/contrib/kubeflow-katib/OWNERS
  • hougangliu
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/kubeflow/launcher/OWNERS
  • PatrickXYS
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
  • neuromage
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/OWNERS
    • tools/bazel_builder/OWNERS
  • Ark-kun
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • backend/metadata_writer/OWNERS
    • samples/OWNERS
    • samples/contrib/OWNERS
    • test/sample-test/OWNERS
  • animeshsingh
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/ibm-components/OWNERS
    • components/kubeflow/kfserving/OWNERS
    • samples/OWNERS
    • samples/contrib/OWNERS
    • samples/contrib/ibm-samples/OWNERS
  • tomcli
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/ibm-components/OWNERS
    • samples/contrib/ibm-samples/OWNERS
  • jsitu777
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/aws/OWNERS
    • components/aws/sagemaker/OWNERS
    • samples/contrib/aws-samples/OWNERS
  • sudivate
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/azure/OWNERS
    • manifests/kustomize/env/azure/OWNERS
  • berndverst
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/azure/OWNERS
  • dtzar
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/azure/OWNERS
    • manifests/kustomize/env/azure/OWNERS
  • hongye-sun
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
  • luotigerlsx
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • samples/contrib/mnist/OWNERS
  • saurabh24292
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • samples/contrib/mnist/OWNERS
  • Tomcli
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • backend/OWNERS
    • components/kserve/OWNERS
    • components/kubeflow/kfserving/OWNERS
  • chauhang
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/PyTorch/pytorch-kfp-components/OWNERS
    • samples/contrib/pytorch-samples/OWNERS
  • shrinath-suresh
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • components/PyTorch/pytorch-kfp-components/OWNERS
    • samples/contrib/pytorch-samples/OWNERS

@dsp-developers
Copy link

A set of new images have been built to help with testing out this PR:
API Server: quay.io/opendatahub/ds-pipelines-api-server:pr-138
DSP DRIVER: quay.io/opendatahub/ds-pipelines-driver:pr-138
DSP LAUNCHER: quay.io/opendatahub/ds-pipelines-launcher:pr-138
Persistence Agent: quay.io/opendatahub/ds-pipelines-persistenceagent:pr-138
Scheduled Workflow Manager: quay.io/opendatahub/ds-pipelines-scheduledworkflow:pr-138
MLMD Server: quay.io/opendatahub/mlmd-grpc-server:latest
MLMD Envoy Proxy: registry.redhat.io/openshift-service-mesh/proxyv2-rhel8:2.3.9-2
UI: quay.io/opendatahub/ds-pipelines-frontend:pr-138

@dsp-developers
Copy link

An OCP cluster where you are logged in as cluster admin is required.

The Data Science Pipelines team recommends testing this using the Data Science Pipelines Operator. Check here for more information on using the DSPO.

To use and deploy a DSP stack with these images (assuming the DSPO is deployed), first save the following YAML to a file named dspa.pr-138.yaml:

apiVersion: datasciencepipelinesapplications.opendatahub.io/v1alpha1
kind: DataSciencePipelinesApplication
metadata:
  name: pr-138
spec:
  dspVersion: v2
  apiServer:
    image: "quay.io/opendatahub/ds-pipelines-api-server:pr-138"
    argoDriverImage: "quay.io/opendatahub/ds-pipelines-driver:pr-138"
    argoLauncherImage: "quay.io/opendatahub/ds-pipelines-launcher:pr-138"
  persistenceAgent:
    image: "quay.io/opendatahub/ds-pipelines-persistenceagent:pr-138"
  scheduledWorkflow:
    image: "quay.io/opendatahub/ds-pipelines-scheduledworkflow:pr-138"
  mlmd:  
    deploy: true  # Optional component
    grpc:
      image: "quay.io/opendatahub/mlmd-grpc-server:latest"
    envoy:
      image: "registry.redhat.io/openshift-service-mesh/proxyv2-rhel8:2.3.9-2"
  mlpipelineUI:
    deploy: true  # Optional component 
    image: "quay.io/opendatahub/ds-pipelines-frontend:pr-138"
  objectStorage:
    minio:
      deploy: true
      image: 'quay.io/opendatahub/minio:RELEASE.2019-08-14T20-37-41Z-license-compliance'

Then run the following:

cd $(mktemp -d)
git clone git@github.com:opendatahub-io/data-science-pipelines.git
cd data-science-pipelines/
git fetch origin pull/138/head
git checkout -b pullrequest 1ca38faa866697330411f5a0a87c5a431edd7385
oc apply -f dspa.pr-138.yaml

More instructions here on how to deploy and test a Data Science Pipelines Application.

@HumairAK HumairAK merged commit f595a7b into master Feb 14, 2025
6 of 7 checks passed
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.

3 participants