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

Add Helm chart value to configure custom CA certificates #173

Merged
merged 2 commits into from
May 12, 2023

Conversation

alexsomesan
Copy link
Member

Description

Some TFE installations use TLS certificates signed by private Certificate Authorities.

In order for the Operator to work with these installations, it needs to be configured with custom CA bundles that include the root certificates for such private CAs. These are installed through Helm, as a ConfigMap object mounted as a volume on top of the container image for the Operator process.

Usage Example

$ helm install \
  demo hashicorp/terraform-cloud-operator \
  --namespace tfc-operator-system \
  --create-namespace \
  --set customCAcertificates=letsencrypt-bundle.crt

Release Note

Release note for CHANGELOG:

* Add Helm chart value to configure custom CA certificates

References

TFECO-3990

Community Note

  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

@github-actions github-actions bot added documentation Improvements or additions to documentation size/S and removed size/XS labels May 12, 2023
@alexsomesan alexsomesan marked this pull request as ready for review May 12, 2023 10:14
@alexsomesan alexsomesan requested a review from a team as a code owner May 12, 2023 10:14
Copy link
Contributor

@arybolovlev arybolovlev left a comment

Choose a reason for hiding this comment

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

👍🏻

@alexsomesan alexsomesan merged commit 21d8779 into main May 12, 2023
@alexsomesan alexsomesan deleted the custom-ca-bundle branch May 12, 2023 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation helm-chart size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants