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

Support IPv6 terminated at the router with internal IPv4 #13663

Merged
merged 3 commits into from
Jun 9, 2017

Conversation

imcsk8
Copy link
Contributor

@imcsk8 imcsk8 commented Apr 6, 2017

Allow haproxy to listen to ipv6 interfaces.
@imcsk8
Copy link
Contributor Author

imcsk8 commented Apr 6, 2017

PTAL @openshift/networking

@sghosh151
Copy link

Make v6 support configurable instead of always on?

Related keepalived config?

Copy link
Contributor

@knobunc knobunc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks, but addressing @sghosh151's concerns would be good.

@knobunc
Copy link
Contributor

knobunc commented Apr 25, 2017

[test]

1 similar comment
@imcsk8
Copy link
Contributor Author

imcsk8 commented May 11, 2017

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 00fd6b6

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/1363/) (Base Commit: 2628c77)

@knobunc
Copy link
Contributor

knobunc commented Jun 7, 2017

[merge] But @imcsk8 please put a comment in here explaining the testing you did that shows that if there is no IPv6 address on the router node it still works correctly.

@lihongan
Copy link
Contributor

lihongan commented Jun 8, 2017

Is it possible to add env like "ROUTER_TERMINATING_IPV6" to control if enable IPv6 ?

@knobunc
Copy link
Contributor

knobunc commented Jun 8, 2017

@lihongan: Why do we need to disable v6? If the host supports it, what's the reason not to always listen on the v6 interface too? @imcsk8 tested to make sure that if v6 was disabled that it worked correctly still. We automatically bind to an http and an https port, there's no way to disable it (but you can control which port). If someone has v6, and the host is reachable by v6, and the DNS records serve an AAAA address, but they still want to forbid the router from answering, then they can block that in iptables... (or, more likely, not enable it in iptables).

@lihongan
Copy link
Contributor

lihongan commented Jun 9, 2017

Probably I didn't express it clearly before. I means we can bind ipv4 only by default (current implementation), so router won't answer ipv6 request. If someone want to terminate ipv6 request on router, then can add env to let router bind both ipv4 and ipv6 address.

@knobunc
Copy link
Contributor

knobunc commented Jun 9, 2017

@lihongan, no, I understood your point and I agree that it is a minor change in behavior. But I am asking why we care? The router admin needs to have IPv6 working, have DNS configured to serve IPv6 responses (AAAA records), and needs to have the firewall open to allow IPv6 in to the router ports. If they don't do all of that it won't work anyway... why also make them configure something in the router?

What harm is there if we do not add another config knob?

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 00fd6b6

@openshift-bot
Copy link
Contributor

openshift-bot commented Jun 9, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/937/) (Base Commit: cc2ed8f) (Image: devenv-rhel7_6333)

@openshift-bot openshift-bot merged commit 1565480 into openshift:master Jun 9, 2017
@lihongan
Copy link
Contributor

@knobunc thanks for your explanation, understand now.

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

Successfully merging this pull request may close these issues.

6 participants