helm: add Kubeflow Dashboard chart by danish9039 · Pull Request #3524 · kubeflow/community-distribution

danish9039 · 2026-07-02T22:48:43Z

Summary of Changes

Adds the Kubeflow Dashboard Helm wrapper chart as a co-located application chart under applications/dashboard/helm.

This PR keeps the first reviewable slice focused on Kustomize parity for the customer-facing Dashboard platform install path. It retains only the platform scenario, which maps to applications/dashboard/overlays/istio, and removes the exploratory split Dashboard CI scenarios from the chart PR scope.

The PR also updates Helm/Kustomize comparison wiring to the new chart path, uses Helm 4 in the comparison workflow, and extends scripts/synchronize-dashboard-manifests.sh so future Dashboard upstream syncs also refresh the Helm chart appVersion and regenerated static platform template.

The chart is placed under applications/dashboard/helm because Dashboard is an official application component, and this is the application-side equivalent of placing common wrappers under common/<component>/helm. The sync script owns applications/dashboard/upstream, while the Helm chart lives as a sibling under applications/dashboard/helm.

Dependencies

Conceptually follows the foundation and wrapper chart PRs for runtime installation order:

Render parity validation for this PR is self-contained.

Related Issues

Part of the Project 5 Helm chart work.

Contributor Checklist

I have tested these changes with kustomize. See Installation Prerequisites.
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.).

Validation

helm lint applications/dashboard/helm
./tests/helm_kustomize_compare.sh kubeflow-dashboard platform
./tests/helm_kustomize_compare_all.sh kubeflow-dashboard
bash -n scripts/synchronize-dashboard-manifests.sh tests/helm_kustomize_compare.sh tests/helm_kustomize_compare_all.sh
python3 -m py_compile tests/helm_kustomize_compare.py
git diff --check

Also verified with temporary Helm v4.2.2:

helm lint applications/dashboard/helm
./tests/helm_kustomize_compare_all.sh kubeflow-dashboard

google-oss-prow · 2026-07-02T22:48:49Z

[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 juliusvonkohout for approval. For more information see the Kubernetes Code Review Process.

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

Details

Needs approval from an approver in each of these files:

OWNERS

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

github-actions · 2026-07-02T22:48:51Z

Welcome to the Kubeflow Community Distribution Repository

Thanks for opening your first PR. Your contribution means a lot to the Kubeflow community.

Before making more PRs:
Please ensure your PR follows our Contributing Guide.
Please also be aware that many components are synchronized from upstream via the scripts in /scripts.
So in some cases you have to fix the problem in the upstream repositories first, but you can use a PR against kubeflow/community-distribution to test the platform integration.

Community Resources:

CNCF Slack channels: https://www.kubeflow.org/docs/about/community/#kubeflow-slack-channels
Community meetings: https://www.kubeflow.org/docs/about/community/

Thanks again for helping to improve Kubeflow.

Copilot

Pull request overview

Adds a co-located Helm wrapper chart for the Kubeflow Dashboard application and wires it into the existing Helm↔Kustomize render-parity validation and synchronization flow.

Changes:

Introduce applications/dashboard/helm as a static, scenario-gated Helm wrapper chart rendering the current Dashboard overlays/istio (“platform”) install.
Extend Helm/Kustomize comparison scripts to include the new kubeflow-dashboard component and scenario.
Update the Dashboard synchronization script to also refresh the Helm chart appVersion and regenerated static platform.yaml template; update the comparison workflow to install Helm 4.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
`tests/helm_kustomize_compare.sh`	Adds `kubeflow-dashboard` component wiring for parity comparison.
`tests/helm_kustomize_compare.py`	Allows `kubeflow-dashboard` as a supported component in the comparison tool.
`tests/helm_kustomize_compare_all.sh`	Adds `kubeflow-dashboard` to the “all” comparison matrix.
`scripts/synchronize-dashboard-manifests.sh`	Extends upstream sync automation to also refresh Helm chart metadata and static template output.
`applications/dashboard/helm/values.yaml`	Defines default chart values (scenario selection).
`applications/dashboard/helm/templates/validate.yaml`	Adds scenario validation to fail fast on unsupported values.
`applications/dashboard/helm/templates/platform.yaml`	Provides the static Kustomize-parity rendered payload for the “platform” scenario.
`applications/dashboard/helm/README.md`	Documents install shape, caveats, and parity validation commands.
`applications/dashboard/helm/ci/values-platform.yaml`	CI fixture values for the “platform” scenario.
`applications/dashboard/helm/Chart.yaml`	Introduces the Dashboard wrapper chart metadata.
`.github/workflows/helm-kustomize-comparison.yml`	Triggers parity CI for Dashboard changes and installs Helm v4.2.2 via setup action.

danish9039 · 2026-07-02T23:06:49Z

    declare -a failed_components=()

-    for comp in katib hub kserve-models-web-app; do
+    for comp in katib hub kserve-models-web-app kubeflow-dashboard; do


Fixed in the latest commit.

danish9039 · 2026-07-02T23:06:50Z

        print("Usage: python compare.py <kustomize_file> <helm_file> <component> <scenario> [namespace] [--verbose]")
-        print("Components: katib, hub, kserve-models-web-app")
+        print("Components: katib, hub, kserve-models-web-app, kubeflow-dashboard")


Fixed in the latest commit.

danish9039 · 2026-07-02T23:06:51Z

+        sed -i "s|^appVersion:.*|appVersion: ${COMMIT}|" "$chart_yaml"
+    fi
+
+    temporary_platform_template="$(mktemp)"


Fixed in the latest commit.

Copilot

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.

            "Usage: python compare.py <kustomize_file> <helm_file> <component> <scenario> [namespace] [--verbose]"
        )
-        print("Components: katib, hub, kserve-models-web-application, cert-manager")
+        print(
+            "Components: katib, hub, kserve-models-web-application, cert-manager, kubeflow-dashboard"


Signed-off-by: danish9039 <danishsiddiqui040@gmail.com>

Copilot AI review requested due to automatic review settings July 2, 2026 22:48

google-oss-prow Bot added the do-not-merge/work-in-progress label Jul 2, 2026

google-oss-prow Bot requested review from kimwnasptd and tarekabouzeid July 2, 2026 22:48

google-oss-prow Bot added the size/XXL label Jul 2, 2026

danish9039 mentioned this pull request Jul 2, 2026

helm: add Kubeflow Notebooks chart #3525

Draft

3 tasks

Copilot started reviewing on behalf of danish9039 July 2, 2026 22:49 View session

Copilot AI reviewed Jul 2, 2026

View reviewed changes

danish9039 force-pushed the gsoc/dashboard-wrapper-chart branch from c3c85e9 to bd02c7a Compare July 3, 2026 09:30

juliusvonkohout requested a review from Copilot July 5, 2026 10:57

Copilot started reviewing on behalf of juliusvonkohout July 5, 2026 10:58 View session

Copilot AI reviewed Jul 5, 2026

View reviewed changes

danish9039 added 4 commits July 5, 2026 20:20

helm: add kubeflow dashboard chart

07c4f35

Signed-off-by: danish9039 <danishsiddiqui040@gmail.com>

helm: move kubeflow-dashboard chart to applications/dashboard/helm

0ea818e

Signed-off-by: danish9039 <danishsiddiqui040@gmail.com>

test: address dashboard helm review comments

cf7c5cb

Signed-off-by: danish9039 <danishsiddiqui040@gmail.com>

helm: update chart repository metadata

c3b6c0c

Signed-off-by: danish9039 <danishsiddiqui040@gmail.com>

danish9039 force-pushed the gsoc/dashboard-wrapper-chart branch from bd02c7a to c3b6c0c Compare July 5, 2026 14:56

test: wait for cert-manager deployments during install

bb8a74f

Signed-off-by: danish9039 <danishsiddiqui040@gmail.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

helm: add Kubeflow Dashboard chart#3524

helm: add Kubeflow Dashboard chart#3524
danish9039 wants to merge 5 commits into
kubeflow:masterfrom
danish9039:gsoc/dashboard-wrapper-chart

danish9039 commented Jul 2, 2026

Uh oh!

google-oss-prow Bot commented Jul 2, 2026

Uh oh!

github-actions Bot commented Jul 2, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

danish9039 Jul 2, 2026

Uh oh!

danish9039 Jul 2, 2026

Uh oh!

danish9039 Jul 2, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

danish9039 commented Jul 2, 2026

Summary of Changes

Dependencies

Related Issues

Contributor Checklist

Validation

Uh oh!

google-oss-prow Bot commented Jul 2, 2026

Uh oh!

github-actions Bot commented Jul 2, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

danish9039 Jul 2, 2026

Choose a reason for hiding this comment

Uh oh!

danish9039 Jul 2, 2026

Choose a reason for hiding this comment

Uh oh!

danish9039 Jul 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants