Reformat chart templates part 3

[dnskr]

The PR reformats multiple chart templates as the continuation of #29917 and #29941
Less empty lines rendered when helm template is executed. Aligned indent style for templates.

[potiuk]

LGTM - Pending passing the tests.

[dnskr]

Helm Unit Tests were cancelled after 80 minutes and 98% of progress.
Is it possible to rerun it?

[potiuk]

I will re-run it but this likely means that there is some problem with your change. This is the second time it happens for the PR in a row:

Usually the tests on public runner are a bit faster than 80 minutes (not by long margin, so this might be small difference causing it, but it is a bit disturbing if it is consistently longer than usual:

https://github.com/apache/airflow/actions/runs/4590214027/jobs/8142557270?pr=30168

Let's see. Maybe it's time to seriously look at speeding those tests up. They take a long time to run.

[dnskr]

Helm unit tests are Successful in 69m this time.

I'm not super excited about unit tests for the helm chart. I mean it's quite hard to run and maintain them, but I don't know and cannot find better/faster existing alternative.

[potiuk]

so be it :)

[potiuk]

I'm not super excited about unit tests for the helm chart. I mean it's quite hard to run and maintain them, but I don't know and cannot find better/faster existing alternative.

I have now merged #30672 - maybe not changing the unit tests, but at least grouping them in separate packages, (each one completing under 8 minutes). That should help quite a lot a least with re-running scenarios but also with running bigger chunks of related tests together.