Skip to content

copy of #6699 for CI/previews#6908

Draft
corneliusroemer wants to merge 8 commits into
mainfrom
helm-host
Draft

copy of #6699 for CI/previews#6908
corneliusroemer wants to merge 8 commits into
mainfrom
helm-host

Conversation

@corneliusroemer

@corneliusroemer corneliusroemer commented Jul 15, 2026

Copy link
Copy Markdown
Contributor
  • feat(networking): add configurable ingress hostnames for services
  • feat!(networking): add new networking root property in the helm chart and group publicHosts, ingressHosts, subdomainSeparator, enforceHTTPS, and traefikVersion under this common group for improved flexibility.
  • chore: cleanup of values.yaml
  • fix(schema): update description for publicHosts to reference networking.subdomainSeparator
  • fix(networking): enhance website host handling with www subdomain support
  • fix(networking): ensure default values for host and ingressHosts when values are not set and improve URL construction
  • fix(networking): ensure redirectUris in keycloak-config-map.yaml actually contains the websiteUrl that end-users will interact with

resolves #

Screenshot

PR Checklist

  • All necessary documentation has been adapted.
  • The implemented feature is covered by appropriate, automated tests.
  • Any manual testing that has been done is documented (i.e. what exactly was tested?)

🚀 Preview: https://helm-host.loculus.org

@corneliusroemer corneliusroemer added the preview Triggers a deployment to argocd label Jul 15, 2026
@claude claude Bot added the deployment Code changes targetting the deployment infrastructure label Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployment Code changes targetting the deployment infrastructure preview Triggers a deployment to argocd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants