Re: [I] [helm] `strategy.type=Recreate` not fully supported by all deployments [airflow]

2023-11-01 Thread via GitHub
hakuno commented on issue #35227: URL: https://github.com/apache/airflow/issues/35227#issuecomment-1789971069 @jiajie-chen > However, I think it would be nice to have consistent configuration defaults for strategy. Yes, I think so. -- This is an automated message from the

Re: [I] [helm] `strategy.type=Recreate` not fully supported by all deployments [airflow]

2023-11-01 Thread via GitHub
jiajie-chen commented on issue #35227: URL: https://github.com/apache/airflow/issues/35227#issuecomment-1789054672 I'm also concerned that using `workers.strategy.rollingUpdate=null` may not work if the Airflow chart is used as a subchart by a user. It looks like Helm has a bug with

Re: [I] [helm] `strategy.type=Recreate` not fully supported by all deployments [airflow]

2023-10-31 Thread via GitHub
jiajie-chen commented on issue #35227: URL: https://github.com/apache/airflow/issues/35227#issuecomment-1788126499 @hakuno Thanks for the advice. I did see that `workers.strategy.rollingUpdate` has a default value set, while `scheduler.strategy` was set to `~`. This led me to try

[I] [helm] `strategy.type=Recreate` not fully supported by all deployments [airflow]

2023-10-27 Thread via GitHub
jiajie-chen opened a new issue, #35227: URL: https://github.com/apache/airflow/issues/35227 ### Official Helm Chart version 1.11.0 (latest released) ### Apache Airflow version v2.7.1 ### Kubernetes Version v1.27.4 (minikube v1.31.2) ### Helm Chart