You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2018/09/05 23:09:31 UTC

[GitHub] ndmar edited a comment on issue #2708: [AIRFLOW-1746] Add a Nomad operator to trigger job from Airflow

ndmar edited a comment on issue #2708: [AIRFLOW-1746] Add a Nomad operator to trigger job from Airflow
URL: https://github.com/apache/incubator-airflow/pull/2708#issuecomment-418907808
 
 
   @etrabelsi Indeed, have a wonderful honeymoon!
   
   I'll try to take a look at this in the meantime. If you don't mind, I'll push directly to this forked version.
   
   @Fokko From a cursory glance, it appears that aside from the failing CI (which I believe is just due to the missing the dependency in the `ci/requirements.txt` file), this item might still be outstanding:
   - https://github.com/apache/incubator-airflow/pull/2708#discussion_r194507088
   
   Is there anything else I should address?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services