Skip to content

Should not use loopback addresses#5036

Merged
istio-testing merged 3 commits intoistio:masterfrom
hzxuzhonghu:dns
Sep 23, 2019
Merged

Should not use loopback addresses#5036
istio-testing merged 3 commits intoistio:masterfrom
hzxuzhonghu:dns

Conversation

@hzxuzhonghu
Copy link
Copy Markdown
Member

@hzxuzhonghu hzxuzhonghu requested a review from a team as a code owner September 21, 2019 07:13
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Sep 21, 2019
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 21, 2019
@frankbu frankbu requested a review from rshriram September 23, 2019 15:42
@istio-testing istio-testing merged commit 4a3d0c2 into istio:master Sep 23, 2019
@hzxuzhonghu
Copy link
Copy Markdown
Member Author

/cherry-pick release-1.3

@istio-testing
Copy link
Copy Markdown
Contributor

@hzxuzhonghu: new pull request created: #5044

Details

In response to this:

/cherry-pick release-1.3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@hzxuzhonghu hzxuzhonghu deleted the dns branch September 24, 2019 01:28
@albatross0
Copy link
Copy Markdown

@hzxuzhonghu
It does not work.
Linux kernel returns ENETUNREACH if the destination address of the TCP connection is a multicast address.

$ kubectl exec --context=$CTX_CLUSTER1 $SLEEP_POD -n foo -c sleep -- curl -I httpbin.bar.global:8000/headers
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Couldn't connect to server
command terminated with exit code 7

@hzxuzhonghu
Copy link
Copy Markdown
Member Author

Thanks for the report@albatross0

@hzxuzhonghu
Copy link
Copy Markdown
Member Author

hzxuzhonghu commented Sep 25, 2019

multicast address can not be used, as by default there is no route to these address.
we should use class E addresses 240.0.0.0 ~ 255.255.255.254, which is preserved for future use

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

virtualInbound listener breaks down replicated control plane multi-cluster

7 participants