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/11/03 14:10:25 UTC

[GitHub] [airflow] KIC opened a new issue #12028: Add `files` to templated fields of `EmailOperator`

KIC opened a new issue #12028:
URL: https://github.com/apache/airflow/issues/12028


   **Description**
   
   Files are not part of the templated fields in the `EmailOperator` https://airflow.apache.org/docs/stable/_modules/airflow/operators/email_operator.html
   
   Whle in fact file names should also be templated .
   
   **Use case / motivation**
   We want to store files accordingly to the DAGs execution date and further send them to email recipients like so:
   
   ```
   send_report_email = EmailOperator(
           ....
           files=[
               "/tmp/Report-A-{{ execution_date.strftime('%Y-%m-%d') }}.csv",
               "/tmp/Reportt-B-{{ execution_date.strftime('%Y-%m-%d') }}.csv"
           ]
   ```
   


----------------------------------------------------------------
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] phraniiac edited a comment on issue #12028: Add `files` to templated fields of `EmailOperator`

Posted by GitBox <gi...@apache.org>.

phraniiac edited a comment on issue #12028:
URL: https://github.com/apache/airflow/issues/12028#issuecomment-727439994


   Hi @turbaszek , I can try to take this up if there is no one else been involved yet. (And if it's okay with @contributor13)


----------------------------------------------------------------
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] brighton1101 commented on issue #12028: Add `files` to templated fields of `EmailOperator`

Posted by GitBox <gi...@apache.org>.

brighton1101 commented on issue #12028:
URL: https://github.com/apache/airflow/issues/12028#issuecomment-729333065


   I have a solution I believe to be working above :)


----------------------------------------------------------------
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 #12028: Add `files` to templated fields of `EmailOperator`

Posted by GitBox <gi...@apache.org>.

boring-cyborg[bot] commented on issue #12028:
URL: https://github.com/apache/airflow/issues/12028#issuecomment-720306284


   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] turbaszek commented on issue #12028: Add `files` to templated fields of `EmailOperator`

Posted by GitBox <gi...@apache.org>.

turbaszek commented on issue #12028:
URL: https://github.com/apache/airflow/issues/12028#issuecomment-727284237


   @contributor13 would you like to work on this one?


----------------------------------------------------------------
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] potiuk closed issue #12028: Add `files` to templated fields of `EmailOperator`

Posted by GitBox <gi...@apache.org>.

potiuk closed issue #12028:
URL: https://github.com/apache/airflow/issues/12028


   


----------------------------------------------------------------
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] contributor13 commented on issue #12028: Add `files` to templated fields of `EmailOperator`

Posted by GitBox <gi...@apache.org>.

contributor13 commented on issue #12028:
URL: https://github.com/apache/airflow/issues/12028#issuecomment-721124464


   I would like to fix this issue. Can anyone please explain that what exactly needs to be done.


----------------------------------------------------------------
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] phraniiac commented on issue #12028: Add `files` to templated fields of `EmailOperator`

Posted by GitBox <gi...@apache.org>.

phraniiac commented on issue #12028:
URL: https://github.com/apache/airflow/issues/12028#issuecomment-727439994


   Hi @turbaszek , I can try to take this up if there is no one else been involved yet.


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