Skip to content

Conversation

ds-jkreutzfeld
Copy link
Member

Description

This PR enhances the Umbrella Chart with TLS configuration support, making it easier to deploy a secure Tractus-X environment with HTTPS.

1. Added Comprehensive TLS Configuration Example

  • Created a dedicated file that provides ready-to-use TLS configurations for all components values-tls.yaml
  • Configured ACME issuer integration with cert-manager for automated certificate management
  • Set up proper TLS host configurations across all components

2. Enhanced TLS Documentation

  • Expanded the TLS setup documentation with instructions for both ACME and self-signed certificate workflows
  • Added warnings about using domains with ACME issuers .test
  • Provided clear examples of how to apply TLS configurations manually or using the new values file

3. Fixed Ingress and TLS Configuration Issues

  • Corrected TLS configurations for various components
  • Fixed domain name consistency (replaced incorrect sdfactory.tx.test.net with sdfactory.tx.test)

4. Improved Default Values

  • Updated default values to make TLS configuration more intuitive
  • Standardized TLS configurations across all components

Usage

Users can now enable TLS in two ways:

  1. Use the provided TLS example file: helm upgrade --install umbrella charts/umbrella --namespace umbrella -f charts/umbrella/values-tls.yaml
  2. Manually configure TLS by following the expanded documentation in docs/user/network/tls.md

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

@ds-jkreutzfeld ds-jkreutzfeld marked this pull request as ready for review September 15, 2025 07:01
@ds-jhartmann
Copy link
Contributor

Hi @mgarciaLKS did you have a chance to review this PR already?

@mgarciaLKS
Copy link
Contributor

Hi @mgarciaLKS did you have a chance to review this PR already?

Still on it, by the end of the week will have it tested.

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

Successfully merging this pull request may close these issues.

3 participants