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

Multus/service mesh(istio, linkerd and so on) Interworking #982

Closed
s1061123 opened this issue Dec 5, 2022 · 4 comments
Closed

Multus/service mesh(istio, linkerd and so on) Interworking #982

s1061123 opened this issue Dec 5, 2022 · 4 comments
Labels

Comments

@s1061123
Copy link
Member

s1061123 commented Dec 5, 2022

Currently multus and service mesh, istio, linkerd, open service mesh, does not have interworking appropriately. This issue is parent issue to track the issue.

The Goal

Service mesh with secondary network

Currently each service mesh does not recognize secondary network (as far as I know of). Kubernetes Network Plumbing WG is still working on how to integrate service mesh with multus for a while and it is not concluded yet. @plwhite, @dougbtv and @nicklesimba are working on that.

Current activities:

Service Mesh and Multi-Network enhancements proposal

Provide consistent install/interworking method (e.g. net-attach-def usage)

Currently some service mesh provides a way to interworking with multus, however, these methods are different, not unified. Some method provides a method which is not appropriate way and it could be improved by design.

#971 is the one of the issue.

Hence we need to come up with a optimized way (from multus point of view) and try to propagate to various service mesh projects.

ormergi added a commit to ormergi/kubevirtci that referenced this issue Dec 13, 2022
Since Multus v4 it expects CNI net conf file to exist at '/etc/cni/net.d'
but isitio generates then at '/etc/cni/multus/net.d'.

Copy istio CNI net conf to where multus expects on cluster-up in order
to enable deploying both Isitio and Multus until proper solution is
available, tracking issue [1] [2].

[1] kubevirt#906
[2] k8snetworkplumbingwg/multus-cni#982

Signed-off-by: Or Mergi <[email protected]>
ormergi added a commit to ormergi/kubevirtci that referenced this issue Dec 13, 2022
Since Multus v4 it expects CNI net conf file to exist at '/etc/cni/net.d'
but isitio generates then at '/etc/cni/multus/net.d'.

Copy istio CNI net conf to where multus expects on cluster-up in order
to enable deploying both Isitio and Multus v4 until proper solution is
available, tracking issue [1] [2].

[1] kubevirt#906
[2] k8snetworkplumbingwg/multus-cni#982

Signed-off-by: Or Mergi <[email protected]>
ormergi added a commit to ormergi/kubevirtci that referenced this issue Dec 13, 2022
Since Multus v4 it expects CNI net conf file to exist at '/etc/cni/net.d'
but isitio generates then at '/etc/cni/multus/net.d'.

Copy istio CNI net conf to where multus expects on cluster-up in order
to enable deploying both Isitio and Multus v4 until proper solution is
available, tracking issue [1] [2].

[1] kubevirt#906
[2] k8snetworkplumbingwg/multus-cni#982

Signed-off-by: Or Mergi <[email protected]>
kubevirt-bot pushed a commit to kubevirt/kubevirtci that referenced this issue Dec 14, 2022
Since Multus v4 it expects CNI net conf file to exist at '/etc/cni/net.d'
but isitio generates then at '/etc/cni/multus/net.d'.

Copy istio CNI net conf to where multus expects on cluster-up in order
to enable deploying both Isitio and Multus v4 until proper solution is
available, tracking issue [1] [2].

[1] #906
[2] k8snetworkplumbingwg/multus-cni#982

Signed-off-by: Or Mergi <[email protected]>

Signed-off-by: Or Mergi <[email protected]>
@github-actions
Copy link

github-actions bot commented Mar 6, 2023

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Mar 6, 2023
@s1061123
Copy link
Member Author

s1061123 commented Mar 7, 2023

revive

@s1061123 s1061123 reopened this Mar 15, 2023
@leerin-ruby
Copy link

Hi, is there any plan to fix this?

@ormergi
Copy link

ormergi commented Jan 14, 2025

Hi, is there a workaround or planned fix for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants