fiskaltrust / product-de-bring-your-own-datacenter

Information about the fiskaltrust "Bring your own Data Center" product, which enables hosting the fiskaltrust.Middleware in data centers.
5 stars 2 forks source link

Latest helm chart forces to change redis host name #53

Closed darek-phorest closed 3 years ago

darek-phorest commented 3 years ago

Describe the bug

latest chart version 1.3.18 breaks our redis url and doesn't allow to override the full redis host

version: 1.3.18 in templates/manifest/byodc.yml

  env:
            - name: Redis__ConnectionString
              value: {{ .Values.byodc.config.redis.host }}.{{ .Release.Namespace }}:{{ .Values.byodc.config.redis.port }}

vs 1.3.16

 - name: Redis__ConnectionString
              value: {{ .Values.byodc.config.redis.host }}:{{ .Values.byodc.config.redis.port }}

To Reproduce Steps to reproduce the behavior:

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

STDOUT/STDERR If applicable, add LogOutput from affected BackendPOD.

POSSystem (please complete the following information):

Cashbox Information (please complete the following information):

Additional context Add any other context about the problem here.

volllly commented 3 years ago

Hello @darek-phorest,

Sorry for the inconvenience. We will provide a fix for this issue in the next version (which will probably be released later this week).

TSchmiedlechner commented 3 years ago

Hi @darek-phorest, we've released version 1.3.22 of BYODC which fixes this issue in our tests. Please note that it's also required to update the Chart (not just the container).

Enjoy your weekend! Tom

darek-phorest commented 3 years ago

hi @TSchmiedlechner

thanks for the update, i assume the new chart version is not released yet: https://charts.fiskaltrust.cloud/ as 1.3.18 is still the latest.

TSchmiedlechner commented 3 years ago

Good morning, sorry, looks like the deployment was waiting in the approval stage. The latest chart version has now been deployed.