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

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
Labels
area/alerting/notifications Issues when sending alert notifications area/alerting Grafana Alerting good first issue Good for newcomers type/bug

Comments

@gelicia
Copy link
Contributor

gelicia commented Feb 15, 2024

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 becomes https://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.

@armandgrillet
Copy link
Contributor

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.

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
gaurav1999 added a commit to gaurav1999/grafana that referenced this issue May 16, 2024
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
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
gaurav1999 added a commit to gaurav1999/grafana that referenced this issue May 17, 2024
gaurav1999 pushed a commit to gaurav1999/terraform-provider-grafana that referenced this issue May 20, 2024
gaurav1999 added a commit to gaurav1999/terraform-provider-grafana that referenced this issue May 21, 2024
yuri-tceretian pushed a commit to gaurav1999/grafana that referenced this issue May 21, 2024
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
Projects
Archived in project
2 participants