CAAPIM / apim-charts

Helm Charts for Layer7 API Management components.
MIT License
11 stars 48 forks source link

Removed the word "hazelcast" from hazelcast-client address as if rele… #143

Closed DamandeepToor closed 2 years ago

DamandeepToor commented 2 years ago

Removed the word "hazelcast" from hazelcast-client address as if release-name had "hazelcast", hazelcast-client is unable to connect with gateway

Description of the change

Benefits

Drawbacks

Applicable issues

Additional information

Checklist

gvermeulen7205 commented 2 years ago

This includes hazelcast because in this context we are targeting a subChart address. Removing -hazelcast will point to a server that does not exist.

<address>{{ .Release.Name }}-hazelcast.{{ .Release.Namespace }}.svc.cluster.local:{{ .Values.hazelcast.service.port }}</address>

Are you sure this is not a configuration issue?

Gazza7205 commented 2 years ago

This is covered in #154 from #141