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

v1.9 backports 2021-06-16 #16564

Closed
wants to merge 3 commits into from
Closed

v1.9 backports 2021-06-16 #16564

wants to merge 3 commits into from

Conversation

nathanjsweet
Copy link
Member 

$ for pr in 14594 16190 16342; do contrib/backporting/set-labels.py $pr done 1.9; done

jrajahalme and others added 3 commits June 16, 2021 11:15
@jrajahalme @nathanjsweet
datapath: Do not use proxy original source address with tunneling
2e6e119 
[ upstream commit 4b769cb ]

Tunnel headers carry the source security ID so the use of original
source address on Envoy upstream connections is not needed when
tunneling. This commit disables the use of original source address
when tunneling is used, which allows Envoy redirection to work also
when using Kind to simulate multiple nodes in a single docker host.

Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
Signed-off-by: Nate Sweet <nathanjsweet@pm.me>
@aanm @nathanjsweet
pkg/k8s: add pod IP event change
f2f9455 
[ upstream commit e92dc6a ]

This is a follow up of 6bd98ad ("handle IP addresses modification in running nodes and CEPs")
for more information read the commit description of that commit.

Signed-off-by: André Martins <andre@cilium.io>
Signed-off-by: Nate Sweet <nathanjsweet@pm.me>
@aditighag @nathanjsweet
docs: Clarify LRP loop related note
b06af0b 
[ upstream commit 2783833 ]

The previous document note can lead to confusion for readers, whereby
its misinterpreted as node-local translation being skipped, but regular translation
can happen. Clarify how we avoid forming a loop by rewording the note.

Signed-off-by: Aditi Ghag <aditi@cilium.io>
Signed-off-by: Nate Sweet <nathanjsweet@pm.me>
@nathanjsweet nathanjsweet requested a review from a team as a code owner June 16, 2021 16:36
@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.9 kind/backports This PR provides functionality previously merged into master. labels Jun 16, 2021
@nathanjsweet nathanjsweet requested a review from aanm June 16, 2021 16:42
@pchaigno pchaigno marked this pull request as draft June 16, 2021 18:53
@pchaigno
Copy link
Member

I had the first and third PRs included in #16568. I had skipped the second because I was not able to resolve the conflict. I just cherry-picked it from your backport 🙏

Closing in favor of #16568.

@pchaigno pchaigno closed this Jun 16, 2021
@pchaigno pchaigno deleted the pr/v1.9-backport-2021-06-16 branch June 16, 2021 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrajahalme jrajahalme Awaiting requested review from jrajahalme

@aanm aanm Awaiting requested review from aanm

@aditighag aditighag Awaiting requested review from aditighag

Assignees

@aanm aanm

@jrajahalme jrajahalme

@aditighag aditighag

Labels
kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@nathanjsweet @pchaigno @aanm @jrajahalme @aditighag