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

conrad/teach proxy compute tls #10717

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

conradludgate
Copy link
Contributor

https://github.com/neondatabase/cloud/issues/23008

For TLS between proxy and compute, we are using an internally provisioned CA to sign the compute certificates. This change ensures that proxy will load them from a supplied env var pointing to the correct file - this file and env var will be configured later, using a kubernetes secret.

To use TLS, we need a hostname. For now, we decided to use the kubernetes service name as it seemed most appropriate - eg compute-id.default.svc.cluster.local. (see https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/). This is very neon specific and would break if we ever decided to use different namespaces for each endpoint. Since control-plane is what creates the service, we should update the wake compute response appropriately, but I think this is a suitable first pass

Compute doesn't offer TLS yet, so I have it set to "Prefer", which allows TLS but is fine if it's not enabled - this allows us to roll out. This should be configurable.

Copy link

github-actions bot commented Feb 7, 2025

7425 tests run: 7067 passed, 0 failed, 358 skipped (full report)


Flaky tests (2)

Postgres 16

Postgres 14

Code coverage* (full report)

  • functions: 33.2% (8585 of 25827 functions)
  • lines: 49.1% (72297 of 147290 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
762dafc at 2025-02-07T17:51:21.910Z :recycle:

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

Successfully merging this pull request may close these issues.

1 participant