fix: delete tab character

[yigang01]

fix：#3153

[shawnh2]

/retest

[yigang01]

/重新测试

/retest

May I ask how to operate it，again commit code？

[shahar-h]

Proposal: you can add add a script that validates there are no tabs in chart yaml files by executing the following for each file:

grep -q $'\t' <yaml file>

This will prevent such cases in future.

[liorokman]

This should have been caught by the yamllint tool. Maybe something is wrong with how it's being invoked as part of the build?

[zirain]

/retest

[zirain]

This should have been caught by the yamllint tool. Maybe something is wrong with how it's being invoked as part of the build?

why there's no difference in test/helm/default.yaml

[shahar-h]

This should have been caught by the yamllint tool. Maybe something is wrong with how it's being invoked as part of the build?

That's because chart is ignored:

gateway/tools/linter/yamllint/.yamllint

Lines 3 to 8 in c943042

	ignore: |
	# This directory fails checks since many files
	# are templated. Instead, we run the linter
	# after running `make generate-manifests` which creates
	# the Install YAML in bin/
	charts/gateway-helm/

[liorokman]

But the linter should still be run on the output of the template, after make generate-manifests is run.

[zirain]

This should have been caught by the yamllint tool. Maybe something is wrong with how it's being invoked as part of the build?

That's because chart is ignored:

gateway/tools/linter/yamllint/.yamllint

Lines 3 to 8 in c943042

	ignore: |
	# This directory fails checks since many files
	# are templated. Instead, we run the linter
	# after running `make generate-manifests` which creates
	# the Install YAML in bin/
	charts/gateway-helm/

there's a render result located test/helm/default.yaml, so I think helm will ignore that.
anyway it's fine to remove it.

[shahar-h]

Right, but the output is generated correctly with space indentation.