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

Merge release-1.20 branch #1021

Closed
wants to merge 14 commits into from

Conversation

jwendell
Copy link
Member

No description provided.

istio-testing and others added 14 commits May 22, 2024 09:22
…AS=false (#51241)

Co-authored-by: John Howard <john.howard@solo.io>
… route in host ns. (#51334)

* Change getPodNetNs logic to not rely on /32 route in host ns.

Now it iterates over each process, switches to the net ns of that
process and checks for a veth with an address matching the pod ip.
This handles cases where there is multilayer routing or EBPF usage.

* go fmt

* lint

* Address comments. Switched to netns.Do() for eth0 read.

* Addressed comments: checks every interface for match instead of eth0 specifically

* lint

* Comments: change string IP comparison to net type comparison, and switched to use WithNetNSPath

* gci + make gen

---------

Co-authored-by: Brian Sonnenberg <bsonnenberg@google.com>
@jwendell jwendell added the tide/merge-method-merge Denotes a PR that should use a standard merge by tide when it merges. label Jun 13, 2024
Copy link

openshift-ci bot commented Jun 13, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress size/L tide/merge-method-merge Denotes a PR that should use a standard merge by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants