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.

Sign up for GitHub

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

Jump to bottom

Adjust MTU only for pods using the egress gateway or SRv6 encapsulations #22295

Open
pchaigno opened this issue Nov 21, 2022 · 1 comment
Open

Adjust MTU only for pods using the egress gateway or SRv6 encapsulations #22295

pchaigno opened this issue Nov 21, 2022 · 1 comment
Labels
feature/egress-gateway Impacts the egress IP gateway feature. feature/srv6 Impacts the SRv6 feature. kind/feature This introduces new functionality. pinned These issues are not marked stale by our issue bot. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.

Comments

@pchaigno
Copy link
Member

Today, whenever egress gateway is used, we set the MTU of pods to account for the additional overhead of the tunnel (to the gateway node) regardless of whether the pods in question are actually going to use the egress gateway. We are planning to do the same for SRv6 (cf. #22294).

We however know, for both of those features, when pods are going to use encapsulation. This information is given by the podSelector in the CRD used for configuration. We should thus be able to dynamically configure the MTU according to whether encapsulation is going to be used.
@pchaigno pchaigno added kind/feature This introduces new functionality. feature/egress-gateway Impacts the egress IP gateway feature. feature/srv6 Impacts the SRv6 feature. labels Nov 21, 2022
@github-actions

This comment was marked as resolved.

Sign in to view
@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Jan 21, 2023
@pchaigno pchaigno added pinned These issues are not marked stale by our issue bot. and removed stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. labels Jan 21, 2023
@ti-mo ti-mo added the sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. label Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature/egress-gateway Impacts the egress IP gateway feature. feature/srv6 Impacts the SRv6 feature. kind/feature This introduces new functionality. pinned These issues are not marked stale by our issue bot. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pchaigno @ti-mo