Skip to content

Conversation

@namejsjeongkr
Copy link

@namejsjeongkr namejsjeongkr commented Jan 13, 2026

Pull Request Template for Kubeflow Manifests

✏️ Summary of Changes

Describe the changes you have made, including any refactoring or feature additions.

Add ui-virtual-service.yaml to katib-external-db install profile to enable Istio routing for Katib UI (/katib/ path). Cherry-picked from katib-with-kubeflow/ui-virtual-service.yaml to fix "Sorry, /katib/ is not a valid page" error when using external database deployment.

Changes:

  • Add ui-virtual-service.yaml at installs/katib-external-db/ root
  • Update kustomization.yaml to include the VirtualService
  • No breaking changes - preserves existing flat structure and functionality

📦 Dependencies

List any dependencies or related PRs (e.g., "Depends on #123").

🐛 Related Issues

Link any issues that are resolved or affected by this PR.

✅ 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.).

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.

@github-actions
Copy link

Welcome to the Kubeflow Manifests 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 synchronizes 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/manifests to test the platform integration.

Community Resources:

Thanks again for helping to improve Kubeflow.

@google-oss-prow
Copy link

[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:

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

juliusvonkohout commented Jan 13, 2026

Welcome to the Kubeflow Manifests 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 synchronizes 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/manifests 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.

Please do not modify the upstream folders as explained here. You can use this as a test PR, but the actual PR has to be raised against kubeflow/katib. You also have to sign your commits according to the DCO.

@namejsjeongkr
Copy link
Author

@juliusvonkohout Please check below PR.

kubeflow/katib#2599

1 similar comment
@namejsjeongkr
Copy link
Author

@juliusvonkohout Please check below PR.

kubeflow/katib#2599

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants