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

Inbound email integration #837

Merged
merged 25 commits into from
Mar 16, 2023
Merged

Conversation

vstpme
Copy link
Member

@vstpme vstpme commented Nov 11, 2022

This PR add Inbound Email integration.

It designed to support some variety of ESPs, but in prod we will use Mailgun, so locally I tested it only with mailgun ESP.

Important:
To make it work on different clusters I'm planning to provide different email domains for different regions, like ....@us.oncall.grafana.net, ...@eu.oncall.grafana.net

@vstpme vstpme changed the title Restore inbound email integration Inbound email integration Mar 7, 2023
@Konstantinov-Innokentii Konstantinov-Innokentii marked this pull request as ready for review March 13, 2023 14:00
@Konstantinov-Innokentii Konstantinov-Innokentii requested review from a team March 13, 2023 14:00
For now I want to make sure that InboundEmail is working with mailgun. Other ESPs stability is for next iteration.
## Grouping and auto-resolve

Alert groups will be grouped by email subject and auto-resolved if email message text equals "OK".
This behaviour can be modified via custom templates.
Copy link
Contributor

Choose a reason for hiding this comment

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

Alert groups will be grouped by email subject and auto-resolved if the email message text equals "OK".

CHANGELOG.md Outdated
@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## Unreleased

- Inbound email integration
Copy link
Contributor

Choose a reason for hiding this comment

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

May be refer to this PR? :)

@@ -50,3 +50,4 @@ opentelemetry-exporter-otlp-proto-grpc==1.15.0
pyroscope-io==0.8.1
django-dbconn-retry==0.1.7
django-ipware==4.0.2
django-anymail==8.6
Copy link
Contributor

Choose a reason for hiding this comment

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

License ok 👍

@@ -0,0 +1,41 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you also please add OSS configuration instruction here? https://grafana.com/docs/oncall/latest/open-source/#email-setup

@Konstantinov-Innokentii Konstantinov-Innokentii deleted the vadimkerr/inbound-email branch March 16, 2023 05:59
brojd pushed a commit that referenced this pull request Sep 18, 2024
This PR add Inbound Email integration.

It designed to support some variety of ESPs, but in prod we will use
Mailgun, so locally I tested it only with mailgun ESP.

**Important:**
To make it work on different clusters I'm planning to provide different
email domains for different regions, like ....@us.oncall.grafana.net,
...@eu.oncall.grafana.net

---------

Co-authored-by: Innokentii Konstantinov <innokenty.konstantinov@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants