feat: add `topologySpreadConstraints` values

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

665 stars 476 forks source link

feat: add `topologySpreadConstraints` values #752

Closed ron-damon closed 7 months ago

ron-damon commented 1 year ago

What issues does your PR fix?

Resolves https://github.com/airflow-helm/charts/issues/581

What does your PR do?

This PR adds the following values:

airflow.defaultTopologySpreadConstraints (default: [])
airflow.kubernetesPodTemplate.topologySpreadConstraints (default: [])
airflow.dbMigrations.topologySpreadConstraints (default: [])
airflow.sync.topologySpreadConstraints (default: [])
scheduler.topologySpreadConstraints (default: [])
web.topologySpreadConstraints (default: [])
workers.topologySpreadConstraints (default: [])
triggerer.topologySpreadConstraints (default: [])
flower.topologySpreadConstraints (default: [])
pgbouncer.topologySpreadConstraints (default: [])

Checklist

For all Pull Requests

[x] Commits are signed off
[x] Commits have semantic messages
[x] Documentation updated
[X] Passes ct linting

MrBones757 commented 1 year ago

Awesome to see a PR for this - I'm looking for this myself and was going to submit a PR for it if there wasn't already one What is the status of this PR? is there an eta on seeing this released / is there any additional testing / work required on this feature?

Happy to help where needed. Thanks in advance!

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had activity in 60 days. It will be closed in 7 days if no further activity occurs.

Thank you for your contributions.

Issues never become stale if any of the following is true:

they are added to a Project
they are added to a Milestone
they have the lifecycle/frozen label

ron-damon commented 1 year ago

/remove-lifecycle stale

saschajullmann commented 1 year ago

Hey all,

is there any timeline here/ is there anything you need help with in order to allow this to get merged @thesuperzapper ?

jbvaningen commented 1 year ago

We would also love to have this PR merged! This is currently one of the main blockers for us to avoid monkey-patching the chart in our Airflow setup

stale[bot] commented 8 months ago

This issue has been automatically marked as stale because it has not had activity in 60 days. It will be closed in 7 days if no further activity occurs.

Thank you for your contributions.

Issues never become stale if any of the following is true:

they are added to a Project
they are added to a Milestone
they have the lifecycle/frozen label

ron-damon commented 8 months ago

/remove-lifecycle stale