-
Notifications
You must be signed in to change notification settings - Fork 11.6k
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
Allow user to choose between Pagerduty global and Pagerduty EU endpoint #82741
Closed
gelicia opened this issue
Feb 15, 2024
Discussed in
#67187
· 1 comment
· Fixed by grafana/alerting#174, #88007 or grafana/terraform-provider-grafana#1576
Closed
Allow user to choose between Pagerduty global and Pagerduty EU endpoint #82741
gelicia opened this issue
Feb 15, 2024
Discussed in
#67187
· 1 comment
· Fixed by grafana/alerting#174, #88007 or grafana/terraform-provider-grafana#1576
Labels
area/alerting/notifications
Issues when sending alert notifications
area/alerting
Grafana Alerting
good first issue
Good for newcomers
type/bug
Comments
gelicia
added
type/feature-request
migrated-discussions
area/alerting
Grafana Alerting
labels
Feb 15, 2024
This is due to https://github.com/grafana/alerting/blob/bc622368256dbf116357d232441b6d1ad6166363/receivers/pagerduty/pagerduty.go#L38 being hardcored, we should fix that to be a configurable field. |
armandgrillet
added
type/bug
area/alerting/notifications
Issues when sending alert notifications
good first issue
Good for newcomers
and removed
type/feature-request
migrated-discussions
labels
Mar 13, 2024
gaurav1999
added a commit
to gaurav1999/alerting
that referenced
this issue
Mar 22, 2024
gaurav1999
added a commit
to gaurav1999/alerting
that referenced
this issue
Mar 22, 2024
gaurav1999
added a commit
to gaurav1999/alerting
that referenced
this issue
Mar 22, 2024
gaurav1999
added a commit
to gaurav1999/alerting
that referenced
this issue
Mar 22, 2024
gaurav1999
added a commit
to gaurav1999/alerting
that referenced
this issue
May 16, 2024
gaurav1999
pushed a commit
to gaurav1999/grafana
that referenced
this issue
May 16, 2024
- Fixes: grafana#82741 - Attached with grafana/alerting#174
gaurav1999
added a commit
to gaurav1999/grafana
that referenced
this issue
May 16, 2024
- Fixes: grafana#82741 - Attached with grafana/alerting#174
gaurav1999
added a commit
to gaurav1999/alerting
that referenced
this issue
May 16, 2024
gaurav1999
added a commit
to gaurav1999/grafana
that referenced
this issue
May 16, 2024
- Fixes: grafana#82741 - Attached with grafana/alerting#174
gaurav1999
added a commit
to gaurav1999/alerting
that referenced
this issue
May 16, 2024
gaurav1999
added a commit
to gaurav1999/grafana
that referenced
this issue
May 17, 2024
- Fixes: grafana#82741 - Attached with grafana/alerting#174
gaurav1999
added a commit
to gaurav1999/grafana
that referenced
this issue
May 17, 2024
- Fixes: grafana#82741 - Attached with grafana/alerting#174
gaurav1999
pushed a commit
to gaurav1999/terraform-provider-grafana
that referenced
this issue
May 20, 2024
- Fixes: grafana/grafana#82741 - Linked PR: grafana/grafana#88007
gaurav1999
added a commit
to gaurav1999/terraform-provider-grafana
that referenced
this issue
May 21, 2024
- Fixes: grafana/grafana#82741 - Linked PR: grafana/grafana#88007
yuri-tceretian
pushed a commit
to gaurav1999/grafana
that referenced
this issue
May 21, 2024
- Fixes: grafana#82741 - Attached with grafana/alerting#174
julienduchesne
added a commit
to grafana/terraform-provider-grafana
that referenced
this issue
May 22, 2024
* Remove hardocded pagerduty APIURL - Fixes: grafana/grafana#82741 - Linked PR: grafana/grafana#88007 * CI Fixes for docs generation added * go generate * Add pack unpack for pagerduty url * Add test --------- Co-authored-by: Julien Duchesne <julien.duchesne@grafana.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/alerting/notifications
Issues when sending alert notifications
area/alerting
Grafana Alerting
good first issue
Good for newcomers
type/bug
Discussed in #67187
Originally posted by guusbeckett April 25, 2023
At our company we use the PagerDuty EU endpoint, but it is not yet possible to use the EU endpoint of PagerDuty.
As PagerDuty and PagerDuty EU are separate platforms, we cannot simply connect to the global endpoint and expect it to work.
I currently have a custom build that just changes the PagerDuty url to the EU one;
https://events.pagerduty.com/v2/enqueue
becomeshttps://events.eu.pagerduty.com/v2/enqueue
Is it an idea to provide users with a new alert manager recipient and add PagerDuty EU or would you prefer something else?
I'd be happy to implement this in the code.
The text was updated successfully, but these errors were encountered: