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

MTU size #4422

Open
haribhusal2025 opened this issue Dec 19, 2024 · 1 comment
Open

MTU size #4422

haribhusal2025 opened this issue Dec 19, 2024 · 1 comment
Assignees
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. kind/feature

Comments

@haribhusal2025
Copy link

What would you like to be added?

Spiderpool should allow user to define a custom mtu like how it does allow to define/select 'master interface' and other setting. Below is an example of how multus allows defining mtu size for this additional interface.

cat <<EOF | kubectl create -f -
apiVersion: "k8s.cni.cncf.io/v1"
kind: NetworkAttachmentDefinition
metadata:
  name: macvlan-conf-1
spec:
  config: '{
            "cniVersion": "0.3.0",
            "type": "macvlan",
            "master": "eth1",
            "mode": "bridge",
            "mtu": "1480",
            "ipam": {
                "type": "host-local",
                "ranges": [
                    [ {
                         "subnet": "10.10.0.0/16",
                         "rangeStart": "10.10.1.20",
                         "rangeEnd": "10.10.3.50",
                         "gateway": "10.10.0.254"
                    } ]
                ]
            }
        }'
EOF

Why is this needed?

Our use case is, by default interfaces on cluster nodes are set at 9000 but some applications/destination require smaller mtu size for it to be successful due to legacy system/equipments etc. We have been able to solve it by defining mtu size in cilium cni on per node basis but having it managed by spiderpool/multus would be great as we wont have to change settings in cilium and we could have as many additional interfaces we need.

How to implement it (if possible)?

multus configuration should give some idea.

Additional context

No response

@cyclinder
Copy link
Collaborator

Hi @haribhusal2025, thanks for your contribution. We are considering implementing this this month. also please let me know if you are interested in this. you can follow this: 4ed834b#diff-fa1085a973033fc72b29474a726eea75e7937cd7a341fd01bd8f406077262e3b

@cyclinder cyclinder added the good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. label Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. kind/feature
Projects
None yet
Development

No branches or pull requests

2 participants