Skip to content

Conversation

@milinddethe15
Copy link
Member

Part of kubeflow/manifests#2991

Fixes some of the warnings, when applying the manifests for:

  • admission-webhook
  • centraldashboard-angular

For Profile-controller, not sure that the manifests are valid.
https://github.com/kubeflow/dashboard/blob/main/components/profile-controller/config/base/kustomization.yaml#L1

@milinddethe15 milinddethe15 changed the title manifest: Fix Kustomize v5 warnings fix(manifest): Fix Kustomize v5 warnings May 3, 2025
@milinddethe15 milinddethe15 changed the title fix(manifest): Fix Kustomize v5 warnings fix: Fix Kustomize v5 warnings May 3, 2025
@juliusvonkohout
Copy link
Member

juliusvonkohout commented Jun 3, 2025

@akagami-harsh @kunal-511 can you verify that the diff -u of all manifests build with kusstomize build before and after the PR is zero with these changes?

@kunal-511
Copy link
Contributor

kunal-511 commented Jun 3, 2025

For centraldashboard-angular there is no difference before and after the PR

image

@kunal-511
Copy link
Contributor

For admission-webhook

image

@juliusvonkohout
Copy link
Member

juliusvonkohout commented Jun 3, 2025

If it is only one istio label thank you and

/lgtm
/approve

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: juliusvonkohout

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

The pull request process is described here

Details 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

@juliusvonkohout
Copy link
Member

/ok-to-test

@google-oss-prow google-oss-prow bot merged commit 85cb401 into kubeflow:main Jun 3, 2025
10 of 12 checks passed
yehudit1987 pushed a commit to yehudit1987/dashboard that referenced this pull request Jul 3, 2025
* manifest: fix Kustomize 5 warnings in `admission-webhook`

Signed-off-by: milinddethe15 <[email protected]>

* manifest: fix Kustomize 5 warnings in `centraldashboard-angular`

Signed-off-by: milinddethe15 <[email protected]>

---------

Signed-off-by: milinddethe15 <[email protected]>

ci: Enhance the GHA (kubeflow#101)

* Enhance the GHA

Signed-off-by: kunal-511 <[email protected]>

* Updated the trigger paths

Signed-off-by: kunal-511 <[email protected]>

* Updated the format

Signed-off-by: kunal-511 <[email protected]>

* Updated the env and labels

Signed-off-by: kunal-511 <[email protected]>

* Updated gha to fix errors

Signed-off-by: kunal-511 <[email protected]>

* Added logs to know error

Signed-off-by: kunal-511 <[email protected]>

* Removed logs

Signed-off-by: kunal-511 <[email protected]>

* Update the end point call

Signed-off-by: kunal-511 <[email protected]>

* Updated the deployment name

Signed-off-by: kunal-511 <[email protected]>

* Updated the validate-webhook function to use the correct mutating webhook configuration name

Signed-off-by: kunal-511 <[email protected]>

* updated pod testing

Signed-off-by: kunal-511 <[email protected]>

* Updated the the kind version

Signed-off-by: kunal-511 <[email protected]>

* updated namespace

Signed-off-by: kunal-511 <[email protected]>

* Fixed certificate error

Signed-off-by: kunal-511 <[email protected]>

* Fixed the casing in docker file

Signed-off-by: kunal-511 <[email protected]>

* Updated the kfam deployment

Signed-off-by: kunal-511 <[email protected]>

* Updated the service name

Signed-off-by: kunal-511 <[email protected]>

* Updated the api call

Signed-off-by: kunal-511 <[email protected]>

* updated endpoints

Signed-off-by: kunal-511 <[email protected]>

* Updated the env for end to end test

Signed-off-by: kunal-511 <[email protected]>

* Updated port forwarding

Signed-off-by: kunal-511 <[email protected]>

* Port mapping updated

Signed-off-by: kunal-511 <[email protected]>

* removed logs

Signed-off-by: kunal-511 <[email protected]>

* Added timeout

Signed-off-by: kunal-511 <[email protected]>

* Updated the logs

Signed-off-by: kunal-511 <[email protected]>

* Updated the naming

Signed-off-by: kunal-511 <[email protected]>

* Updated the naming

Signed-off-by: kunal-511 <[email protected]>

* Fixed naming

Signed-off-by: kunal-511 <[email protected]>

---------

Signed-off-by: kunal-511 <[email protected]>

testing

Migrate kubeflow/kubeflow centraldb_docker_publish.yaml GitHub Action to kubeflow/dashboard - initial

Signed-off-by: Yehudit Kerido <[email protected]>

testing

testing

testing

testing

ci: Migrate kubeflow/kubeflow centraldb_docker_publish.yaml GitHub Action to kubeflow/dashboard

Signed-off-by: Yehudit Kerido <[email protected]>
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