[GitHub] [airflow] ephraimbuddy opened a new pull request #15661: Add --executor option to breeze kind-cluster deploy command

[GitBox <gi...@apache.org>]

ephraimbuddy opened a new pull request #15661:
URL: https://github.com/apache/airflow/pull/15661


   This change will enable us easily deploy airflow to kubernetes cluster and test it using different executors.
   Example usage:
      ./breeze kind-cluster --executor CeleryExecutor deploy
   
   Note: 
   When running the kubernetes tests, the task `task_with_volume` in this [example dag](https://github.com/apache/airflow/blob/master/airflow/example_dags/example_kubernetes_executor_config.py) fails for all other executors except KubernetesExecutor. 
   I think it's because the task is performing a volume mount? Maybe we should modify this example dag to work with all executors and have a CI job that runs some select executors in k8 cluster
   
   ---
   **^ Add meaningful description above**
   
   Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)** for more information.
   In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)) is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in [UPDATING.md](https://github.com/apache/airflow/blob/master/UPDATING.md).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] alexshires commented on pull request #15661: Add --executor option to breeze kind-cluster deploy command

[GitBox <gi...@apache.org>]

alexshires commented on pull request #15661:
URL: https://github.com/apache/airflow/pull/15661#issuecomment-834317712


   Hi all - the executor shell line is incorrect here as it has a space either side of the = statement (line 653 of breeze)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk merged pull request #15661: Add --executor option to breeze kind-cluster deploy command

[GitBox <gi...@apache.org>]

potiuk merged pull request #15661:
URL: https://github.com/apache/airflow/pull/15661


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org