chore: drop the namespaces to resolve kustomize conflicts#1005
Conversation
Link: kubeflow/manifests#3430 (comment) Signed-off-by: Christian Heusel <christian@heusel.eu>
|
/ok-to-test |
andyatmiami
left a comment
There was a problem hiding this comment.
/lgtm
/approve
Based on the comment by @thesuperzapper here:
I can confirm this PR accomplishes its stated goal.
I ran the app via tilt and confirmed (unsurprisingly) that everything still works locally with these namespace resources being removed.
This should also help me when I (eventually) finish documenting deployment instructions here:
As I imagine this same issue was going to fail the (very outdated) instructions as written 💯
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andyatmiami, ederign The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Link: kubeflow/manifests#3430 (comment) Signed-off-by: Christian Heusel <christian@heusel.eu>
Link: kubeflow/manifests#3430 (comment)
Todo: