Skip to content

Fix mtu setting for tunnel interface in init.sh #20552

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

Merged
merged 1 commit into from
Jul 18, 2022

Conversation

ChengyuanLiCY
Copy link
Contributor

@ChengyuanLiCY ChengyuanLiCY commented Jul 17, 2022

The mtu setting could be overwritten to default value (1500) if interface is
re-created (in TUNNEL_PORT is not nil case),
so move mtu setting after tunnel interface is re-created.

Signed-off-by: Li Chengyuan [email protected]

The mtu setting could be overwritten to default value (1500) if interface is
re-created (in  TUNNEL_PORT isnot nil case),
so move mtu setting after tunnel interface is re-created.

Signed-off-by: Li Chengyuan <[email protected]>
@ChengyuanLiCY ChengyuanLiCY requested a review from a team as a code owner July 17, 2022 02:29
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 17, 2022
@brb brb requested a review from NikAleksandrov July 18, 2022 07:58
@pchaigno pchaigno added kind/bug This is a bug in the Cilium logic. area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/bug This PR fixes an issue in a previous release of Cilium. labels Jul 18, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 18, 2022
@pchaigno
Copy link
Member

/test

@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 18, 2022
@ldelossa ldelossa merged commit 4afdae8 into cilium:master Jul 18, 2022
@pchaigno
Copy link
Member

Added backport labels given it's a bug.

@ldelossa ldelossa added backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed backport-pending/1.12 labels Jul 18, 2022
@tklauser tklauser added backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. and removed backport-pending/1.11 labels Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. kind/bug This is a bug in the Cilium logic. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants