ADD git-sync container lifecycle hooks

[getChan]

---

Closes: #40364

[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 Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.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.
• Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
  Apache Airflow is a community-driven project and together we are making it better 🚀.
  In case of doubts contact the developers at:
  Mailing List: dev@airflow.apache.org
  Slack: https://s.apache.org/airflow-slack

[jedcunningham]

Can you add this to values.yaml and values.schema.json?

[getChan]

Can you add this to values.yaml and values.schema.json?

That is already in values.yaml dags.gitSync.containerLifecycleHooks

[romsharon98]

Congrats on your first PR. LGTM

[nevcohen]

What if there is a global .Values.containerLifecycleHooks?
Don't we want something like in here? Or do we not want it defined for this at the global level?

[getChan]

What if there is a global .Values.containerLifecycleHooks? Don't we want something like in here? Or do we not want it defined for this at the global level?

@nevcohen
thanks for opinion.
I added global lifecycle hook condition.

{{- $containerLifecycleHooksGitSync := or .Values.dags.gitSync.containerLifecycleHooks .Values.containerLifecycleHooks }}

It works same as other container hooks.

[jedcunningham]

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

[getChan]

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

thanks. i fixed it

[boring-cyborg]

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