Skip to content

Add support redis mtls for TFE init/setting #174

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

sandrampeter
Copy link
Collaborator

@sandrampeter sandrampeter commented May 28, 2025

Background

Add support for following env :

These vars will be used to connect to redis via mtls:

TFE_REDIS_CA_CERT_PATH
TFE_REDIS_CLIENT_CERT_PATH
TFE_REDIS_CLIENT_KEY_PATH
TFE_REDIS_USE_MTLS

How has this been tested?

Through CI release test and local

TFE Modules

Did you add a new setting?

@sandrampeter sandrampeter requested a review from a team as a code owner May 28, 2025 16:35
@@ -53,6 +53,15 @@ echo $certificate_data_b64 | base64 --decode > ${tls_bootstrap_cert_pathname}
echo "[$(date +"%FT%T")] [Terraform Enterprise] Skipping TlsBootstrapCert configuration" | tee -a $log_pathname
%{ endif ~}

%{ if enable_redis_mtls == true ~}

Choose a reason for hiding this comment

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

Do we need to do this for azure and gcp as well?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ideally yes, just enabling it for AWS for our internal tests

@sandrampeter sandrampeter changed the title add redis mtls changes Add support redis mtls for TFE init/setting Jun 6, 2025
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.

2 participants