Skip to content

Conversation

@thesuperzapper
Copy link
Member

What issues does your PR fix?

What does your PR do?

This PR allows you to set the resource request/limits for the few remaining init-containers for which there was not already values.

This PR adds the following values:

  • airflow.initContainers.checkDb.resources (default: {})
    • Sets resources requests/limits for check-db init-containers
  • airflow.initContainers.waitForDbMigrations.resources (default: {})
    • Sets resources requests/limits for wait-for-db-migrations init-containers
  • airflow.initContainers.installPipPackages.resources (default: {})
    • Sets resources requests/limits for install-pip-packages init-containers

Checklist

For all Pull Requests

@thesuperzapper thesuperzapper changed the title feat: add resources values for init-containers feat: add values for init-container resources Apr 29, 2024
@thesuperzapper thesuperzapper added this to the airflow-8.9.0 milestone Apr 29, 2024
@thesuperzapper thesuperzapper added the status/ready-to-merge status - this will be merged into next release label Apr 29, 2024
@thesuperzapper thesuperzapper changed the title feat: add values for init-container resources feat: add values to set init-container resources May 1, 2024
@thesuperzapper thesuperzapper merged commit a13412a into airflow-helm:main May 1, 2024
@thesuperzapper thesuperzapper deleted the add-resources-values-for-init-containers branch May 1, 2024 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/ready-to-merge status - this will be merged into next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resources for init container not specified into helpers/_pods.tpl add values to specify resources on init-containers

1 participant