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.

Sign up for GitHub

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

Jump to bottom

[BUG] Difficult to diagnose invalid sendgrid email failures #3150

Open
2 tasks done
katrogan opened this issue Dec 13, 2022 · 2 comments · May be fixed by flyteorg/flyteadmin#503
Open
2 tasks done

[BUG] Difficult to diagnose invalid sendgrid email failures #3150

katrogan opened this issue Dec 13, 2022 · 2 comments · May be fixed by flyteorg/flyteadmin#503
Assignees
@katrogan
Labels
bug Something isn't working flyteadmin Issue for FlyteAdmin Service

Comments

@katrogan
Copy link
Contributor

Describe the bug

When using launch plan notifications with a sengrid emailer, emails can fail with difficult to diagnose issues, such as

Sendgrid sent email {\"errors\":[{\"message\":\"The from email does not contain a valid address

Expected behavior

It should be easy to pinpoint invalid notification emails from logs that report email failures

Additional context to reproduce

No response

Screenshots

No response

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes
@katrogan katrogan added bug Something isn't working flyteadmin Issue for FlyteAdmin Service labels Dec 13, 2022
@katrogan katrogan self-assigned this Dec 13, 2022
@katrogan katrogan linked a pull request Dec 13, 2022 that will close this issue
Open
8 tasks
@github-actions
Copy link

Hello 👋, This issue has been inactive for over 9 months. To help maintain a clean and focused backlog, we'll be marking this issue as stale and will close the issue if we detect no activity in the next 7 days. Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot added the stale label Sep 10, 2023
@github-actions
Copy link

Hello 👋, This issue has been inactive for over 9 months and hasn't received any updates since it was marked as stale. We'll be closing this issue for now, but if you believe this issue is still relevant, please feel free to reopen it. Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 18, 2023
@eapolinario eapolinario reopened this Nov 2, 2023
@github-actions github-actions bot removed the stale label Nov 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@katrogan katrogan

Labels
bug Something isn't working flyteadmin Issue for FlyteAdmin Service
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Detailed sendgrid email error logging flyteorg/flyteadmin
2 participants
@eapolinario @katrogan