airflow-helm / charts

The User-Community Airflow Helm Chart is the standard way to deploy Apache Airflow on Kubernetes with Helm. Originally created in 2017, it has since helped thousands of companies create production-ready deployments of Airflow on Kubernetes.
https://github.com/airflow-helm/charts/tree/main/charts/airflow
Apache License 2.0
630 stars 474 forks source link

Resources for init container not specified into helpers/_pods.tpl #808

Closed zippo761 closed 2 months ago

zippo761 commented 7 months ago

Checks

Motivation

In my compary use strong rules into k8s production cluster that resource quota must be specified for every container. So every deploy without resources into init-containers i catch WARNING like this for every pods (that contain init-container/check-db/wait-for-db-migrations/install-pip-packages: image_2023-12-04_19-40-54

Implementation

Im propose specified resource into template charts/airflow/templates/_helpers/pods.tpl and fill values for template somethink like this:

.Values.airflow.

# resource for check-db init container
checkDb:
  resources: {}

# resource for wait-for-db-migrations container
waitForDbMigrations:
  resources: {}

# resource for install-pip-packages container
installPipPackages:
  resources: {} 

and users after this update use only values for avoid this warning

Are you willing & able to help?