change secretName field

[hxquangnhat]

• Chart version not bumped (the versions are all bumped and released at the same time)
• README.md updated with any new values or changes
• Updated template tests in ${CHART}/tests/*.py
• Updated integration tests in ${CHART}/examples/*/test/goss.yaml

[elasticmachine]

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

[Crazybus]

Thank you for finding this 🤦‍♂. We do actually have a test for it but unfortunately it uses the same name and secretName so the test doesn't catch this. Could you also update the test to use a different value for the secretName here:

helm-charts/filebeat/tests/filebeat_test.py

Lines 147 to 153 in 6f97893

	def test_adding_a_secret_mount():
	config = '''
	secretMounts:
	- name: elastic-certificates
	secretName: elastic-certificates
	path: /usr/share/filebeat/config/certs
	'''

This bug also exists in the Kibana and Elasticsearch chart too btw! I'll open a separate PR to fix those charts unless you would like to add the changes here too.

[Crazybus]

LGTM! Thank you for fixing up the tests and other charts too! 🙏

[Crazybus]

jenkins test this please

[Crazybus]

jenkins test this please

[Crazybus]

jenkins test this please

[Crazybus]

jenkins test this please