[Bug Report] headless service name missconfigured about helm-charts

[Crazybus]

Did you get any errors when applying the change via helm? It sounds like the service was updated but helm failed to update the statefulset.

Can you give me the full steps to reproduce this? Did you first deploy with the clusterName of elasticsearch then apply a change with it set to test? When I tried this a new statefulset with new pvcs was created. It worked, but it of course means that all of the data was wiped too.

[Sherler]

@Crazybus
it's too easy to reproduce:

here is what is shown in shell after I run the helm install command:

and here is the logs of es without any mannual changes:

what is shown next is the configuration of the es statefulset:

as you can see, the headless service-name in the env-config didn't change. but after I mannually change it to test-master-headless , everything goes well

helm version

[Crazybus]

Ahh I see the problem now. It looks like you are using an old version of the helm chart. Can you please try installing by explicitly pointing to the correct version? If you don't do it helm ignores the alpha tags.

From: https://github.com/elastic/helm-charts/tree/master/elasticsearch#installing

helm install --name elasticsearch elastic/elasticsearch --version 7.0.0-alpha1

If you don't specify the chart version you will get an older version before we started adding in the alpha tags.

I noticed this because discovery.zen.ping.unicast.hosts should not be set for clusters that are running Elasticsearch 7.

[botelastic]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[botelastic]

This issue has been automatically closed because it has not had recent activity since being marked as stale.