Template extra volumes in helm chart

[nirroz93]

same as #29357

---

Same as #29357, seems like it was missed
^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

[boring-cyborg]

Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst)
Here are some useful points:

• Pay attention to the quality of your code (ruff, mypy and type annotations). Our pre-commits will help you with that.
• In case of a new feature add useful documentation (in docstrings or in docs/ directory). Adding a new operator? Check this short guide Consider adding an example DAG that shows how users should use it.
• Consider using Breeze environment for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
• Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
• Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
• Be sure to read the Airflow Coding style.
  Apache Airflow is a community-driven project and together we are making it better 🚀.
  In case of doubts contact the developers at:
  Mailing List: [email protected]
  Slack: https://s.apache.org/airflow-slack

[hussein-awala]

in the PR you mention, the values file was updated by adding an example for each extra volumes field and adding a comment before the example:

It can be templated like in the following example:

You need to do the same thing for gitSync, dagProcessor, flower, createUserJob and pgbouncer?

[jedcunningham]

Sorry for the delay here @nirroz93. This looks good. Can you resolve the conflict so I can get this merged? Thanks!

[nirroz93]

@jedcunningham - done, thank you!

[jedcunningham]

@hussein-awala, want to take another look?

[hussein-awala]

Looks good

[jedcunningham]

Looks like static checks and one test are failing. I'd suggest that you set up the pre-commit hooks, they help catch the static check issues early 👍.

[eladkal]

@nirroz93 can you address the failing static tests?

[nirroz93]

Yes, I will look at it later today (sorry for disappearing)

[boring-cyborg]

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

[jedcunningham]

Thanks @nirroz93! Congrats on your first commit 🎉