manifests icon indicating copy to clipboard operation
manifests copied to clipboard

Add various Helm charts POCs

Open kunal-511 opened this issue 10 months ago • 5 comments

Pull Request Template for Kubeflow Manifests

✏️ Summary of Changes

Introduces Helm chart structure for Kubeflow components starting with spark-operator, including umbrella chart pattern and component-specific charts. Implements comprehensive verification system to ensure Helm produces functionally equivalent manifests to existing Kustomize approach with CI/CD validation. Created experimental/helm/kubeflow/ umbrella chart with Chart.yaml, values.yaml Created experimental/helm/charts/spark-operator/ component chart referencing upstream v2.2.0

🐛 Related Issues

Deploying Kubeflow with helm

✅ Contributor Checklist

  • [ ] I have tested these changes with kustomize. See Installation Prerequisites.
  • [x] All commits are signed-off to satisfy the DCO check.
  • [ ] I have considered adding my company to the adopters page to support Kubeflow and help the community, since I expect help from the community for my issue (see 1. and 2.).

You can join the CNCF Slack and access our meetings at the Kubeflow Community website. Our channel on the CNCF Slack is here #kubeflow-platform.

kunal-511 avatar Jun 03 '25 12:06 kunal-511

[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 assign kimwnasptd for approval. 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:

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

google-oss-prow[bot] avatar Jun 03 '25 12:06 google-oss-prow[bot]

Please can you check is this the right way now @juliusvonkohout

kunal-511 avatar Jun 04 '25 15:06 kunal-511

@juliusvonkohout do i make the sync helm scripts to be same as the kustomize one which creates a seperate pr for that or include them directly here ?

kunal-511 avatar Jun 09 '25 18:06 kunal-511

CC @kromanow94 @chasecadet @varodrig

juliusvonkohout avatar Jun 13 '25 10:06 juliusvonkohout

Rebase to master

kunal-511 avatar Jun 23 '25 19:06 kunal-511

@kunal-511 @andreyvelich what is blocking the PR now ?

juliusvonkohout avatar Jul 08 '25 14:07 juliusvonkohout

CC @chasecadet @varodrig

juliusvonkohout avatar Jul 08 '25 14:07 juliusvonkohout

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Sep 07 '25 00:09 github-actions[bot]

@kunal-511 please rebase and get this read to merge.

juliusvonkohout avatar Sep 26 '25 13:09 juliusvonkohout