Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kfp-profile-controller's crd_manifests.yaml.j2 incorrectly includes istio objects in 1.8-updates-dev-branch #373

Closed
ca-scribner opened this issue Nov 2, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@ca-scribner
Copy link
Contributor

Bug Description

on the 1.8-updates-dev-branch, kfp-profile-controller's crd_manifests.yaml.j2 includes references to istio destinationrules and authorizationpolicies objects. This was introduced by #331 which updated the file to be more consistent with upstream.

These istio objects are intentionally omitted from charmed kubeflow because we don't currently use them in the kfp-profile-controller (see our sync.py). #331 introduced a bug in the form of the metacontroller having a dependency on istio.

To fix this, we can revert #331's changes to this file.

To Reproduce

Environment

1.8-updates-dev-branch

Relevant log output

-

Additional context

No response

@NohaIhab
Copy link
Contributor

closed by #376

DnPlas added a commit that referenced this issue Jun 5, 2024
This commit ensures the controller is bootstrapped with version 3.5.0 instead of the default
as we have found issues with it.
This is a workaround for canonical/bundle-kubeflow#921.

Fixes #373
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants