[GitHub] [airflow] mik-laj commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator)

2019-08-13 Thread GitBox
mik-laj commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator) URL: https://github.com/apache/airflow/pull/5489#issuecomment-520858118 No additional approvals are needed. This is an auto

[GitHub] [airflow] mik-laj commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator)

2019-08-12 Thread GitBox
mik-laj commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator) URL: https://github.com/apache/airflow/pull/5489#issuecomment-520485656 @akki I tested this operator. I was able to run DAG correctly when I configured the TLS correctly. ```

[GitHub] [airflow] mik-laj commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator)

2019-07-14 Thread GitBox
mik-laj commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator) URL: https://github.com/apache/airflow/pull/5489#issuecomment-511208426 I looked at this code and the _execute method seems to me unreadable. It seems to me that it is worth creating two p

[GitHub] [airflow] mik-laj commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator)

2019-07-01 Thread GitBox
mik-laj commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator) URL: https://github.com/apache/airflow/pull/5489#issuecomment-507540728 @potiuk Can you look at - this, our Docker expert?