[GitHub] [airflow] dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file

2020-02-11 Thread GitBox
dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file URL: https://github.com/apache/airflow/pull/6230#issuecomment-584763785 @davlum congrats! This is an automated

[GitHub] [airflow] dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file

2020-01-10 Thread GitBox
dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file URL: https://github.com/apache/airflow/pull/6230#issuecomment-573262847 > > That's a tough one as that would break a LOT of DAGs, that said it would be nice to have some class

[GitHub] [airflow] dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file

2020-01-10 Thread GitBox
dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file URL: https://github.com/apache/airflow/pull/6230#issuecomment-573241912 That's a tough one as that would break a LOT of DAGs, that said it would be nice to have some class typing

[GitHub] [airflow] dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file

2020-01-09 Thread GitBox
dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file URL: https://github.com/apache/airflow/pull/6230#issuecomment-572807836 Looks like the current failures are in `test_pod_generator`. Once the CI/CD is passing @ashb and I will look

[GitHub] [airflow] dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file

2019-12-13 Thread GitBox
dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file URL: https://github.com/apache/airflow/pull/6230#issuecomment-565633783 @davlum is there any way to break this up into smaller PRs? Might be easier to merge piecemeal.

[GitHub] [airflow] dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file

2019-12-09 Thread GitBox
dimberman commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be configured from JSON/YAML file URL: https://github.com/apache/airflow/pull/6230#issuecomment-563403070 @ashb could you give this one another lookover please?