omusavi commented on issue #4121: [AIRFLOW-2568] Azure Container Instances
operator
URL:
https://github.com/apache/incubator-airflow/pull/4121#issuecomment-449416747
Thanks to all the previous commenters! The PR has now been updated to
include full unit testing for all the hooks. I have a
omusavi commented on issue #4121: [AIRFLOW-2568] Azure Container Instances
operator
URL:
https://github.com/apache/incubator-airflow/pull/4121#issuecomment-44285
Rebased again to get a new build. @Fokko, please take a look when you get a
chance, hope it is ready to go. Thanks!
--
omusavi commented on issue #4121: [AIRFLOW-2568] Azure Container Instances
operator
URL:
https://github.com/apache/incubator-airflow/pull/4121#issuecomment-441690305
Flaky tests caused a build failure, could somebody with write access restart
the job that failed? It passed in our forked r
omusavi commented on issue #4121: [AIRFLOW-2568] Azure Container Instances
operator
URL:
https://github.com/apache/incubator-airflow/pull/4121#issuecomment-441670842
Noticed there were some merge conflicts in integration.rst since last week
so I fixed that and pushed again
--
omusavi commented on issue #4121: [AIRFLOW-2568] Azure Container Instances
operator
URL:
https://github.com/apache/incubator-airflow/pull/4121#issuecomment-439809982
Apologies for the bad iteration after splitting out the files @Fokko .
Everything should be up to date now and I have verif
omusavi commented on issue #4121: [AIRFLOW-2568] Azure Container Instances
operator
URL:
https://github.com/apache/incubator-airflow/pull/4121#issuecomment-438516567
Removed extra line and rebased onto master so it is all up to date. Thanks
@Fokko
---
omusavi commented on issue #4121: [AIRFLOW-2568] Azure Container Instances
operator
URL:
https://github.com/apache/incubator-airflow/pull/4121#issuecomment-436809335
Updated PR, thanks for the comments @Fokko! Let me know if there is anything
else I can fix
--