klesh commented on issue #356:
URL:
https://github.com/apache/incubator-devlake-helm-chart/issues/356#issuecomment-3691396865
> I had the same problem here.
>
> I would recommend to add a "clusterName" field in the values.yaml to let
this to configured.
>
> In the mean time, overwritting this value using an additional env variable
in the commonEnvs would work as it is loaded after this hardcoded value. On the
other hand, this unnecessary environment variable would get defined in the
other container which are not using it, so not that lovely.
>
> e.g. values.yaml
>
> ```
> clusterName: "<<your k8s cluster domain name>>"
>
> #the common environments for all pods except grafana, grafana needs to be
set in grafana section seperately
> commonEnvs:
> TZ: "UTC"
> DEVLAKE_ENDPOINT: '{{ include "devlake.fullname" . }}-lake.{{
.Release.Namespace }}.svc.{{ .Values.clusterName }}:{{ .Values.lake.port }}'
> ```
>
> The clusterName value can be also defined using a --set parameter during
the helm chart call such as the extra environment variable if you are not
loading an external values.yaml file to overwrite the default values.
Hi, Thanks for the information.
Would you like to submit a PR to fix the issue?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
