-
Notifications
You must be signed in to change notification settings - Fork 892
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
Upgrade istio-cni to v.1.23.2 #2903
Upgrade istio-cni to v.1.23.2 #2903
Conversation
Signed-off-by: Tarek Abouzeid <[email protected]>
I restarted the failed seldon test |
Thank you for the PR. rebased to master and fixed the merge conflict and will approve afterwards. We should use the new "Values.gateways.seccompProfile" to get rid of the manual patch at manifests/.github/workflows/pss_test.yaml Line 38 in 26d970d
both for istio and istio-cni and and try to switch to istio-cni by default in a follow up PR. So it is related to #2528 |
…ation (kubeflow#2906) Signed-off-by: varodrig <[email protected]> Signed-off-by: juliusvonkohout <[email protected]>
Signed-off-by: Tarek Abouzeid <[email protected]>
/lgtm |
[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
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Pull Request Template for Kubeflow manifests Issues
✏️ A brief description of the changes
📦 List any dependencies that are required for this change
🐛 If this PR is related to an issue, please put the link to the issue here.
✅ Contributor checklist
DCO
check)