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

🆕 Support Dynamic Routing #878

Closed
NargiT opened this issue Jun 6, 2024 · 3 comments
Closed

🆕 Support Dynamic Routing #878

NargiT opened this issue Jun 6, 2024 · 3 comments

Comments

@NargiT
Copy link
Contributor

NargiT commented Jun 6, 2024

It's still in early access, but I would love to see this supported.

image
image

@NargiT
Copy link
Contributor Author

NargiT commented Jun 7, 2024

Migration issue

ℹ️ Right now the interface allow to add a dynamic or static routing even if there are existing rules.

image

But terraform use an api that does not allow it.

│ Error: PUT API call to https://api.eu.pagerduty.com/event_orchestrations/167e5733-a62b-49ff-9008-50d498bff948/router failed 400 Bad Request. Code: 0, Errors: map[sets:[map[rules:[map[actions:map[route_to:[route_to and dynamic_route_to are mutually exclusive actions in Router rules]]] map[] map[] map[] map[]]]]], Message: Invalid payload
│ 
│   with pagerduty_event_orchestration_router.event_orchestration_router_nagios_prod,
│   on auto-orchestration-router-nagios.tf line 1, in resource "pagerduty_event_orchestration_router" "event_orchestration_router_nagios_prod":
│    1: resource "pagerduty_event_orchestration_router" "event_orchestration_router_nagios_prod" {

Because other wise, we need to do 2 steps instead of one to migrate.

  1. delete all rules
    wait....
  2. add the new one
    wait...

This is not acceptable from operation point of view as it can lead to miss event between step 1 and 2.

@NargiT
Copy link
Contributor Author

NargiT commented Jul 25, 2024

I will test #885 and close this if it solve the problem

@NargiT
Copy link
Contributor Author

NargiT commented Jul 29, 2024

works !

@NargiT NargiT closed this as completed Jul 29, 2024
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

No branches or pull requests

1 participant