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

[aws] Switch to TLS NLB #30

Open
captn3m0 opened this issue Jan 25, 2019 · 2 comments
Open

[aws] Switch to TLS NLB #30

captn3m0 opened this issue Jan 25, 2019 · 2 comments

Comments

@captn3m0
Copy link
Contributor

captn3m0 commented Jan 25, 2019

Creating this issue to start a discussion.

Pros

  • L4 Load Balancer
  • You can assign internal static IPs to the NLB
  • Certs can be assigned to the NLB instead via ACM, and remove the burden of setting --cacert on the client side (possible today, but not sure if can be supported)
  • Because this retains TCP Source IP, etcd logs actual IPs

Cons

  • Pricing is significantly different, might need to be re-evaluated.

Reference:

@Quentin-M
Copy link
Owner

Thanks for the detailed suggestion. NLBs are good, better than ELBs for the use-case obviously for some of the reasons you stated above, but also a few others regarding flow reliability. However, after only looking a few seconds at the articles, I do not see any motion of client certificate authentication. Is that only straight-up TLS termination only?

There is also the debate of having unencrypted traffic flowing between the NLB hosts and your etcd hosts, which may be a security concern for several companies out there.

@captn3m0
Copy link
Contributor Author

I do not see any mention of client certificate authentication

Good catch. I saw some other comments asking for this as well, but I guess this will have to be tested.

There is also the debate of having unencrypted traffic flowing between the NLB hosts and your etcd hosts

No plans of doing that. Want to setup TLS through and through. So the etcd-server cert is self-signed, and marked as origin cert on the NLB (I think it supports this, but need to check).

Will check on support for both of these and report back.

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

No branches or pull requests

2 participants