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 2020/12/05 20:26:38 UTC

[GitHub] [airflow] XD-DENG commented on pull request #12825: Enable to expand the key file path in ssh hooks

XD-DENG commented on pull request #12825:
URL: https://github.com/apache/airflow/pull/12825#issuecomment-739407314


   > These variables maybe change in the future and it's better not to do hard code in the connection.
   
   This doesn't convince me: anything can change, including the non-envvar parts. File paths here should be _explicit_ and DAG author should have full responsibility for that.
   


----------------------------------------------------------------
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