Add ingress to the helm chart

[aksakalli]

For some reason #8777 did not include an ingress record in the helm chart for production. astronomer/airflow-chart has an ingress support but it has additional subdomain logic that is not needed for most of the deployments. Thus I adopted the templates from stable/airflow.

---

^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, 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 UPDATING.md.

[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/master/CONTRIBUTING.rst)
Here are some useful points:

• Pay attention to the quality of your code (flake8, pylint 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://apache-airflow-slack.herokuapp.com/

[kaxil]

cc @schnie if you have time

[alikhtag]

Just thinking should this enabled conditional moved down to each resource?
So ingress.web.enabled and ingress.flower.enabled
I may want web enabled, but flower ingress disabled if I have Celery as executor for example.

[kaxil]

@aksakalli WDYT about this suggestion?

[aksakalli]

There is no option to disable flower if you use CeleryExecutor (unlike the stable/helm). I don't think it is common that someone would like to deploy it with an ingress record for webserver but not for flower.

I can make that change if you think that is a necessary use case.

[saikatharryc]

I have another finding, it would be great to add this same thing in Notes.txt as well

[alikhtag]

I have another finding, it would be great to add this same thing in Notes.txt as well

Indeed this was missing, I've added it! Thanks!

[saikatharryc]

anyone has any idea when it can get merged? eagerly waiting for this!

[boring-cyborg]

Awesome work, congrats on your first merged pull request!