Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Added SES SMTP user for Airflow #163

Merged
merged 1 commit into from
Sep 11, 2018
Merged

Added SES SMTP user for Airflow #163

merged 1 commit into from
Sep 11, 2018

Conversation

xoen
Copy link
Contributor

@xoen xoen commented Sep 10, 2018

The ses_smtp_user can be used to create an IAM user with
permission to send emails from the specified SES Domain Identity.

NOTE: The SMTP password for the user is not specified but it's
an output of the IAM Access Key associated with the IAM user

Luckily terraform's iam_access_key has an attribute to get hold of the SMTP password without having to do anything weird.

Ticket: https://trello.com/c/tTZ1DG3y/1322-fix-airflow-email-notifications

The `ses_smtp_user` can be used to create an IAM user with
permission to send emails from the specified SES Domain Identity.

NOTE: The SMTP password for the user is not specified but it's
an output of the IAM Access Key associated with the IAM user
@xoen xoen requested a review from kerin September 10, 2018 13:50
Copy link
Contributor

@kerin kerin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@xoen
Copy link
Contributor Author

xoen commented Sep 11, 2018

Thanks @kerin :)!

@xoen xoen merged commit 9d262d0 into master Sep 11, 2018
@xoen xoen deleted the ag--airflow-SMTP-users branch September 11, 2018 09:38
davidread pushed a commit that referenced this pull request Jul 29, 2020
davidread pushed a commit that referenced this pull request Jul 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants