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

ptp: Enabling L2 mode #987

Open
Tjaarda1 opened this issue Nov 28, 2023 · 2 comments
Open

ptp: Enabling L2 mode #987

Tjaarda1 opened this issue Nov 28, 2023 · 2 comments

Comments

@Tjaarda1
Copy link

Tjaarda1 commented Nov 28, 2023

Current Situation:

When utilizing the PTP plugin to establish a connection between two containers, the current workflow necessitates the use of IP Address Management (IPAM) to assign IP addresses to the connected veth pair.

Enhancement Proposal:

It would greatly enhance the usability of the PTP plugin if there were an option to create the veth pair without explicitly specifying IP addresses. This would parallel the simplicity offered by bridge configurations, making the PTP plugin more versatile. In certain scenarios, the use of PTP proves to be a superior alternative to bridge configurations, and eliminating the mandatory requirement for IP addresses would streamline the setup process.

Expected Behavior:

  • Users should have the flexibility to create a PTP connection between containers without the obligation to provide explicit IP addresses.

  • This enhancement would align the PTP plugin's functionality more closely with the user-friendly experience offered by bridge configurations.

@mccv1r0
Copy link
Member

mccv1r0 commented Nov 28, 2023

Can't you just leave IPAM plugin out?

@Tjaarda1
Copy link
Author

Tjaarda1 commented Nov 28, 2023

Can't you just leave IPAM plugin out?

No, the documentation specifies it is a required field. Leaving it as an empty dictionary will result in an unexpected behavior, with the message "no plugin name provided". (While if you fill the IPAM fields, it will work as expected). This behavior has been tested both in a microk8s cluster and a kubeadm one.

Edit: I've checked the code and in the bridge plugin this option is contemplated by checking the value of the IPAM field, while in ptp, it's not the case.

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

No branches or pull requests

2 participants