Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Email notifications #1442

Closed
SamuelAdamsMcGuire opened this issue Sep 26, 2023 · 4 comments
Closed

Email notifications #1442

SamuelAdamsMcGuire opened this issue Sep 26, 2023 · 4 comments
Assignees
Labels
kind: feature New feature request scope: backend Backend task scope: design scope: frontend Frontend task
Milestone

Comments

@SamuelAdamsMcGuire
Copy link

Is your proposal related to a problem?

We have non-slack users that are also interested in getting alerts via email

Describe the solution you'd like

Add a simple field to specific alerts that a user can enter an email to receive updates

Describe alternatives you've considered

Using something like python logging handlers or whatever is available in the respective data entity

Additional context

NA

@SamuelAdamsMcGuire SamuelAdamsMcGuire added the kind: feature New feature request label Sep 26, 2023
@RamanDamayeu RamanDamayeu added this to the 0.18 milestone Sep 27, 2023
@AndreyNenashev
Copy link
Contributor

AndreyNenashev commented Oct 25, 2023

Hey, @SamuelAdamsMcGuire! Right now we don't support directive notifications on specific alerts - it requires to enhance current notifications system at all levels and it's on roadmap.
Proposal solution:

  • We will add possiblity to add list of emails on data entity alerts notification settings. This email list will apply the same rules as other notifications.

@AndreyNenashev AndreyNenashev added the scope: frontend Frontend task label Oct 25, 2023
@SamuelAdamsMcGuire
Copy link
Author

@AndreyNenashev Thanks for the feedback. Keep me posted. I currently have a simple work around and use the email alerts from Great Expectations. However a general alert from the platform would be great.

@Vladysl
Copy link
Contributor

Vladysl commented Oct 30, 2023

Implemented in #1489

@Vladysl Vladysl closed this as completed Oct 30, 2023
@Vladysl
Copy link
Contributor

Vladysl commented Oct 30, 2023

Implemented email smpt notifications
to use it you should fill this properties in odd-platform-api/src/main/resources/application.yml

notifications:
  enabled: true
   email:
      sender: your_sender_email
      password: password
      smtp: smtp_host
      port: smtp_port
      notification:
        emails: "yourFirst@gmail.com,yourSecond@gmail.com"

Or you can specify Environment variables:
NOTIFICATIONS_ENABLED=true
NOTIFICATIONS_RECEIVERS_EMAIL_SENDER={your_sender_email}
NOTIFICATIONS_RECEIVERS_EMAIL_PASSWORD={password}
NOTIFICATIONS_RECEIVERS_EMAIL_SMTP={smtp_host}
NOTIFICATIONS_RECEIVERS_EMAIL_PORT={smtp_port}
NOTIFICATIONS_RECEIVERS_EMAIL_NOTIFICATION_EMAILS={"yourFirst@gmail.com,yourSecond@gmail.com"}

Also, in case you want to receive a link to a data entity where an alert occurs, you should specify the PLATFORM_HOST_URL environment variable (by default, it is localhost).
Docs:
google smpt - https://support.google.com/a/answer/176600?hl=en
(in case you have google 2factor auth - https://support.google.com/accounts/answer/185833?visit_id=638338654265262930-3223924282&p=InvalidSecondFactor&rd=1)
aws smpt - https://repost.aws/knowledge-center/ses-set-up-connect-smtp

sender, password, smpt and notification.emails is mandatory in case you want to enable email notifications
Email subject : ODD Platform - ${alertType} Alert
alertType can be - BACKWARDS_INCOMPATIBLE_SCHEMA, FAILED_DQ_TEST, FAILED_JOB, DISTRIBUTION_ANOMALY
Email example:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: feature New feature request scope: backend Backend task scope: design scope: frontend Frontend task
Projects
None yet
Development

No branches or pull requests

4 participants