[BUG][opensearch] dead "minimum_master_nodes" setting still listed

[rursprung]

Describe the bug
the opensearch chart still contains the commented-out discovery.zen.minimum_master_nodes:

helm-charts/charts/opensearch/values.yaml

Lines 50 to 52 in 12e6af6

	# # minimum_master_nodes need to be explicitly set when bound on a public IP
	# # set to 1 to allow single node clusters
	# discovery.zen.minimum_master_nodes: 1

however, since Elasticsearch 7.0 this setting has been ignored: release notes for 7.0 (see also elastic/elasticsearch#37868)

accordingly this should probably be removed (unclear: should something else be added instead as a recommendation?)

To Reproduce
check the default config provided in the helm charts.

Expected behavior
only settings are listed which are still supported and which have an effect.

Chart Name
opensearch

Screenshots
n/a

Host/Environment (please complete the following information):
n/a

Additional context
i saw that this has also been touched upon in #54

[TheAlgo]

Yes this needs to be removed from the values.yaml as per my knowledge to avoid any confusion. I will raise a fix for this to remove it if @rursprung is not planning any PR

[rursprung]

i'm not planning to contribute a PR here because i'm not familiar enough with what the proper settings would be for this use-case.
i'm actually hoping to see what you'll be doing :D

[peterzhuamazon]

@TheAlgo any progress?
Thanks.

[peterzhuamazon]

@TheAlgo Any updates?
Thanks.