Fixed indent on elasticsearch extraVolumes tpl. Was causing parsing errors.

[d33vil]

Fixed indent on elasticsearch extraVolumes tpl. Was causing parsing errors.

[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]

Could you post what your configuration looks like for this? We have a test for this which passes with and without this indentation. In the YAML spec both should be and are valid so I would be interested to see what configuration you used to break this (as it might affect other values).

This is the test:

helm-charts/elasticsearch/tests/elasticsearch_test.py

Lines 309 to 323 in 24ac48a

	def test_adding_a_extra_volume_with_volume_mount():
	config = '''
	extraVolumes: |
	- name: extras
	emptyDir: {}
	extraVolumeMounts: |
	- name: extras
	mountPath: /usr/share/extras
	readOnly: true
	'''
	r = helm_template(config)
	extraVolume = r['statefulset'][uname]['spec']['template']['spec']['volumes']
	assert {'name': 'extras', 'emptyDir': {}} in extraVolume
	extraVolumeMounts = r['statefulset'][uname]['spec']['template']['spec']['containers'][0]['volumeMounts']
	assert {'name': 'extras', 'mountPath': '/usr/share/extras', 'readOnly': True} in extraVolumeMounts

[Crazybus]

jenkins test this please