Skip to content

Conversation

@adamdelman
Copy link
Member

@adamdelman adamdelman commented Nov 30, 2025

Description

Remove all legacy Kubeflow Pipelines (KFP v1) resources from mlrun-ce, fully switch to the pipelines-v2 subchart, and update installation profiles, dependencies, and supporting values. Also introduce Nuclio URL fields, improve MinIO defaults, and ensure MPI CRDs are created in non-admin ClusterIP installs.

This should only be merged after this PR is merged.

Changes Made

1. Pipelines v2 integration

  • Added pipelines-v2 dependency to requirements.yaml and requirements.lock.
  • Introduced new pipelines-v2 values block (service, CRDs, DB security contexts, persistence, managed storage).
  • Removed obsolete legacy KFP image and MinIO fields.
  • Removed kfp2.yaml and associated compatibility stubs.

2. Non-admin ClusterIP installation updates

  • Switched mpi-operator.crd.create from falsetrue.

3. Nuclio and MinIO adjustments

  • Added:
    • mlrun.nuclio.uiURL
    • mlrun.nuclio.apiURL
  • Added MinIO memory requests/limits.
  • Removed hard-coded MinIO credentials.

4. Removal of all legacy Pipelines templates

https://iguazio.atlassian.net/browse/CEML-550

@adamdelman adamdelman marked this pull request as ready for review November 30, 2025 12:14
@adamdelman adamdelman marked this pull request as draft November 30, 2025 13:21
fullnameOverride: mpi-operator
crd:
create: false
create: true
Copy link
Member

Choose a reason for hiding this comment

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

this should be changed to false

dependencies:
- name: nuclio
version: "0.21.6"
version: "0.19.26"
Copy link
Member

Choose a reason for hiding this comment

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

undo change

Comment on lines -382 to -383
repository: minio/minio
tag: "RELEASE.2025-10-15T17-29-55Z"
Copy link
Member

Choose a reason for hiding this comment

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

make sure the image is set as mentioned here on the minio chart - deployment

@adamdelman adamdelman requested a review from liranbg December 1, 2025 13:39
@adamdelman adamdelman marked this pull request as ready for review December 1, 2025 14:01
apiVersion: v1
name: mlrun-ce
version: 0.10.0-rc5
version: 0.10.0-rc6
Copy link
Member

Choose a reason for hiding this comment

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

fyi @GiladShapira94 and @adamdelman this needs to be 0.11.0-rc0 as development will trigger 1.11 and 0.10.x needs to be released with 1.10.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants