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 2021/01/04 16:11:47 UTC
[GitHub] [airflow] jmcarp opened a new pull request #13463: Add aws ses send email operator.
jmcarp opened a new pull request #13463:
URL: https://github.com/apache/airflow/pull/13463
We have an SES hook, but I don't see an SES operator yet. This patch adds the boilerplate operator to pass templated arguments through to the hook.
----------------------------------------------------------------
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] jmcarp closed pull request #13463: Add aws ses send email operator.
Posted by GitBox <gi...@apache.org>.
jmcarp closed pull request #13463:
URL: https://github.com/apache/airflow/pull/13463
----------------------------------------------------------------
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] jmcarp commented on pull request #13463: Add aws ses send email operator.
Posted by GitBox <gi...@apache.org>.
jmcarp commented on pull request #13463:
URL: https://github.com/apache/airflow/pull/13463#issuecomment-754181822
So, write an ses email backend? That sounds reasonable. I might want to use multiple ses credentials for different tasks, so I would like this to include an optional connection argument, if that makes sense. Other email backends seem to use environment variables or special configuration variables for credentials. Any reason not to use connections instead for ses?
----------------------------------------------------------------
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] jmcarp commented on pull request #13463: Add aws ses send email operator.
Posted by GitBox <gi...@apache.org>.
jmcarp commented on pull request #13463:
URL: https://github.com/apache/airflow/pull/13463#issuecomment-782563513
Handled in another PR!
----------------------------------------------------------------
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] github-actions[bot] commented on pull request #13463: Add aws ses send email operator.
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #13463:
URL: https://github.com/apache/airflow/pull/13463#issuecomment-754117881
[The Workflow run](https://github.com/apache/airflow/actions/runs/461429058) is cancelling this PR. Building image for the PR has been cancelled
----------------------------------------------------------------
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] mik-laj commented on pull request #13463: Add aws ses send email operator.
Posted by GitBox <gi...@apache.org>.
mik-laj commented on pull request #13463:
URL: https://github.com/apache/airflow/pull/13463#issuecomment-754227211
we don't have a lot of email backend alternatives, so we don't have good practices. if you present the implementation, I think there will be no problems to accept it.
ps. i thought to drop the use of environment variables in sendgrid backend.
----------------------------------------------------------------
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] mik-laj commented on pull request #13463: Add aws ses send email operator.
Posted by GitBox <gi...@apache.org>.
mik-laj commented on pull request #13463:
URL: https://github.com/apache/airflow/pull/13463#issuecomment-754135220
@jmcarp What do you think to add SES support to EmailOperator?
----------------------------------------------------------------
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] github-actions[bot] commented on pull request #13463: Add aws ses send email operator.
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #13463:
URL: https://github.com/apache/airflow/pull/13463#issuecomment-754116240
[The Workflow run](https://github.com/apache/airflow/actions/runs/461586948) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.
----------------------------------------------------------------
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