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

Ensure that TLS migration works in a WAN environment #3075

Closed
slackpad opened this issue May 25, 2017 · 2 comments
Closed

Ensure that TLS migration works in a WAN environment #3075

slackpad opened this issue May 25, 2017 · 2 comments
Assignees
Labels
theme/operator-usability Replaces UX. Anything related to making things easier for the practitioner

Comments

@slackpad
Copy link
Contributor

While integrating the latest oss/master into Consul Enterprise I realized that since we don't have the same "use TLS" flag in the WAN Serf tags, we have some remaining work to do in order to get live TLS upshift working for federated Consul clusters. The logic should be the same, I think we just need to add the plumbing.

On the Consul Enterprise side we also need to resolve this as it also has a site that does a dial. Here's the temporary patch that needs to be resolved:

diff --git a/consul/gossip_endpoint.go b/consul/gossip_endpoint.go
index 47f60f2d..efb94ebc 100644
--- a/consul/gossip_endpoint.go
+++ b/consul/gossip_endpoint.go
@@ -25,7 +25,11 @@ func (g *GossipDialer) DialTimeout(datacenter string, addr string, timeout time.
 		return nil, fmt.Errorf("failed to resolve %q: %v", addr, err)
 	}
 
-	conn, _, err := g.pool.DialTimeout(datacenter, target, timeout)
+	// TODO (slackpad) - Set this to false when merging, but we should
+	// improve the TLS support for network areas to make sure that we can
+	// transition these without a down time as well.
+	const useTLS = false
+	conn, _, err := g.pool.DialTimeout(datacenter, target, timeout, useTLS)
 	if err != nil {
 		return nil, fmt.Errorf("failed to connect to %q: %v", addr, err)
 	}
@slackpad slackpad added the theme/operator-usability Replaces UX. Anything related to making things easier for the practitioner label May 25, 2017
@slackpad
Copy link
Contributor Author

slackpad commented Jun 9, 2017

@kyhavlov confirmed that WAN uses the same setup as LAN, so this will work fine in OSS. Consul Enterprise was updated to address the TODO and make TLS depend on verify_outgoing, which fixes TLS in the non-upshift case, so kicking this forward to take a look at making network areas support TLS upshift in the next release of Consul Enterprise.

@kyhavlov
Copy link
Contributor

Closed with #3274

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/operator-usability Replaces UX. Anything related to making things easier for the practitioner
Projects
None yet
Development

No branches or pull requests

2 participants