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

enable dual CNI plugin dataplanes #9743

Conversation

VadimEisenberg
Copy link

@VadimEisenberg VadimEisenberg commented Jan 22, 2025

Description

Currently, for CNI plugin it is possible to use either the default system dataplane or a grpc one. In NeuReality, our hardware, employs a dual network stack for different types of traffic. One is the standard Linux stack, for which the default system dataplane is used. For the other network stack, we developed our own dataplane (external).

We propose to change Calico code to enable using two dataplanes simultaneously. The default system dataplane will be designated as the primary one, the external dataplane will be designated as the secondary one. Calico will send messages to both dataplanes.

The change should not be disruptive to the current users of Calico since they will continue to specify either default system dataplane or a grpc dataplane. In case a dataplane option "useAsSecondary": "true" in addition to "type": "grpc", two dataplanes will be used simultaneously.

Related issues/PRs

Todos

  • Tests
  • Documentation
  • Release note

Release Note

@VadimEisenberg VadimEisenberg requested a review from a team as a code owner January 22, 2025 16:00
@marvin-tigera marvin-tigera added this to the Calico v3.30.0 milestone Jan 22, 2025
@marvin-tigera marvin-tigera added release-note-required Change has user-facing impact (no matter how small) docs-pr-required Change is not yet documented labels Jan 22, 2025
@VadimEisenberg VadimEisenberg changed the title enable dual CNI plugin dataplane enable dual CNI plugin dataplanes Jan 22, 2025
@VadimEisenberg
Copy link
Author

This functionality can be handled by CNI plugin chaining, by adding an additional CNI plugin.

@marvin-tigera marvin-tigera removed release-note-required Change has user-facing impact (no matter how small) docs-pr-required Change is not yet documented labels Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants