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/05/24 21:02:06 UTC

[GitHub] [airflow] mans2singh opened a new issue #8999: Tutorial documentation with bash_command correction

mans2singh opened a new issue #8999:
URL: https://github.com/apache/airflow/issues/8999


   **Description:**
   
   Correction in tutorial documentation regarding bash_command
   
   **Issue:**
   
   - The bash_command with shell file is as follows:
   
   > ``bash_command='templated_command.sh'``, where the file location is relative to
   
   But perhaps should be:
   
   > ``bash_command='templated_command.sh '``, where the file location is relative to
   
   
   
   


----------------------------------------------------------------
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] turbaszek commented on issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
turbaszek commented on issue #8999:
URL: https://github.com/apache/airflow/issues/8999#issuecomment-658837685


   Not sure what the issue is, @mans2singh can you explain?


----------------------------------------------------------------
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] eladkal edited a comment on issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
eladkal edited a comment on issue #8999:
URL: https://github.com/apache/airflow/issues/8999#issuecomment-769320120


   The text is quoted from the [tutorial](https://github.com/apache/airflow/blob/master/docs/apache-airflow/tutorial.rst#templating-with-jinja
   ) which explains using `bash_command` with relative path. Using relative path does not require additional space.
   
   for more information check BashOperator [docs](https://airflow.apache.org/docs/apache-airflow/stable/howto/operator/bash.html#jinja-template-not-found)
   
   


----------------------------------------------------------------
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] boring-cyborg[bot] commented on issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on issue #8999:
URL: https://github.com/apache/airflow/issues/8999#issuecomment-633299207


   Thanks for opening your first issue here! Be sure to follow the issue template!
   


----------------------------------------------------------------
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] eladkal edited a comment on issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
eladkal edited a comment on issue #8999:
URL: https://github.com/apache/airflow/issues/8999#issuecomment-769320120


   I assume @mans2singh is referring to the issue related to Jinja where you need to[ add space](https://airflow.apache.org/docs/apache-airflow/stable/howto/operator/bash.html#jinja-template-not-found) at the end of the script. This is mentioned at the Jinja section of the docs but not in the tutorial.
   
   The problem refers to is doc:
   https://github.com/apache/airflow/blob/master/docs/apache-airflow/tutorial.rst#templating-with-jinja
   
   I'll fix it soon
   
   


----------------------------------------------------------------
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] eladkal commented on issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
eladkal commented on issue #8999:
URL: https://github.com/apache/airflow/issues/8999#issuecomment-769320120


   I assume @mans2singh is referring to the issue related to Jinja where you need to[ add space](https://airflow.apache.org/docs/apache-airflow/stable/howto/operator/bash.html#jinja-template-not-found) at the end of the script. This is mentioned at the Jinja section of the doc.
   
   The problem refers to is doc:
   https://github.com/apache/airflow/blob/master/docs/apache-airflow/tutorial.rst#templating-with-jinja
   
   I'll fix it soon
   
   


----------------------------------------------------------------
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] eladkal edited a comment on issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
eladkal edited a comment on issue #8999:
URL: https://github.com/apache/airflow/issues/8999#issuecomment-769320120


   The text is quoted from the [tutorial](https://github.com/apache/airflow/blob/master/docs/apache-airflow/tutorial.rst#templating-with-jinja
   ) which explains with relative path. Using relative path does not require additional space.
   
   for more information check BashOperator [docs](https://airflow.apache.org/docs/apache-airflow/stable/howto/operator/bash.html#jinja-template-not-found)
   
   


----------------------------------------------------------------
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] eladkal closed issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
eladkal closed issue #8999:
URL: https://github.com/apache/airflow/issues/8999


   


----------------------------------------------------------------
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] eladkal edited a comment on issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
eladkal edited a comment on issue #8999:
URL: https://github.com/apache/airflow/issues/8999#issuecomment-769320120


   I assume @mans2singh is referring to the issue related to Jinja where you need to[ add space](https://airflow.apache.org/docs/apache-airflow/stable/howto/operator/bash.html#jinja-template-not-found) after the script name. This is mentioned at the Jinja section of the docs but not in the tutorial.
   
   The problem refers to is doc:
   https://github.com/apache/airflow/blob/master/docs/apache-airflow/tutorial.rst#templating-with-jinja
   
   I'll fix it soon
   
   


----------------------------------------------------------------
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] eladkal edited a comment on issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
eladkal edited a comment on issue #8999:
URL: https://github.com/apache/airflow/issues/8999#issuecomment-769320120






----------------------------------------------------------------
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] eladkal closed issue #8999: Tutorial documentation with bash_command correction

Posted by GitBox <gi...@apache.org>.
eladkal closed issue #8999:
URL: https://github.com/apache/airflow/issues/8999


   


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