Skip to content

Conversation

kimminw00
Copy link
Contributor

Description of the change

  • Ingress hostname is now rendered with common.tplvalues.render anywhere it is referenced in ingress.yaml and ingress-tls-secrets.yaml.
  • External S3 bucket value is rendered via common.tplvalues.render in the S3 helper.
  • External database host is rendered via common.tplvalues.render in the DB helper.

Benefits

  • Enhanced Flexibility
    Allows for dynamic configuration of resource names (e.g., hostnames, buckets) using Go template syntax like {{ .Release.Name }} instead of static values.

  • Improved Chart Reusability
    Enables deploying the same chart across multiple environments (e.g., dev, staging) from a single configuration without causing naming collisions.

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

Signed-off-by: Bitnami Bot <[email protected]>
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Sep 28, 2025
@github-actions github-actions bot removed the triage Triage is needed label Sep 28, 2025
@github-actions github-actions bot removed the request for review from javsalgar September 28, 2025 08:03
@github-actions github-actions bot requested a review from migruiz4 September 28, 2025 08:03
Signed-off-by: Bitnami Bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress mlflow verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants