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

add link-local IP to veth0 for istio #3588

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

cyclinder
Copy link
Collaborator

Thanks for contributing!

What type of PR is this?

  • release/bug

What this PR does / why we need it:

The iptables redirect module uses the IP address of the incoming interface as the dst to modify the packet, but if the interface is not configured with an IP address, the packet will be dropped.

NOTE: this is for istio sidecar

Which issue(s) this PR fixes:

Fixes #3568

Special notes for your reviewer:

Copy link

codecov bot commented Jun 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.16%. Comparing base (b6e4746) to head (b91c9dd).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3588   +/-   ##
=======================================
  Coverage   81.16%   81.16%           
=======================================
  Files          50       50           
  Lines        4391     4391           
=======================================
  Hits         3564     3564           
  Misses        670      670           
  Partials      157      157           
Flag Coverage Δ
unittests 81.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@weizhoublue
Copy link
Collaborator

cherry pick ?

@weizhoublue weizhoublue added cherrypick-release-v0.8 Cherry-pick the PR to branch release-v0.8. cherrypick-release-v0.9 labels Jun 19, 2024
@weizhoublue
Copy link
Collaborator

@ty-dc CI failed, is it caused by the upgrade CI ?

@ty-dc
Copy link
Collaborator

ty-dc commented Jun 19, 2024

@ty-dc CI failed, is it caused by the upgrade CI ?

no rebase main

Signed-off-by: cyclinder <qifeng.guo@daocloud.io>
@weizhoublue weizhoublue changed the title coordinator: add ipAddress for pod's veth0 add link-local IP to veth0 for istio Jun 21, 2024
@weizhoublue weizhoublue merged commit 13c95f0 into spidernet-io:main Jun 21, 2024
50 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 21, 2024
coordinator: add ipAddress for pod's veth0
Signed-off-by: robot <tao.yang@daocloud.io>
github-actions bot pushed a commit that referenced this pull request Jun 21, 2024
coordinator: add ipAddress for pod's veth0
Signed-off-by: robot <tao.yang@daocloud.io>
ty-dc pushed a commit to ty-dc/spiderpool that referenced this pull request Jun 23, 2024
…address

coordinator: add ipAddress for pod's veth0
Signed-off-by: robot <tao.yang@daocloud.io>
ty-dc pushed a commit that referenced this pull request Jun 24, 2024
Signed-off-by: tao.yang <tao.yang@daocloud.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

coordinator 生成的 veth0 网卡默认没有配置 ipv4 的本地链路地址,istio 有问题
3 participants