-
Notifications
You must be signed in to change notification settings - Fork 216
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
auto_pause_notifications_parameters - not apply #574
Comments
Hi @elihai-risk, could you please also share an extract of your It should be something like this:
Also, debug logs produced with the
|
`resource "pagerduty_service" "service" { dynamic "alert_grouping_parameters" { dynamic "incident_urgency_rule" { dynamic "incident_urgency_rule" {
} dynamic "support_hours" { auto_pause_notifications_parameters { |
module.technical_service_staging.pagerduty_service.service["account-event-linking-service"]: Modifying... [id=PVJKT7N] -----------------------------------------------------: timestamp=2022-09-29T13:42:46.025+0300 { { |
Thanks for the logs. The parameters are clearly sent to the PagerDuty API:
Could you also share the response to the |
Hi @elihai-risk, this is the request to the API, not the response from the API. You should have something just after that in the logs. |
---[ RESPONSE ]-------------------------------------- { |
Ok, thanks for the logs. So in your case, the PagerDuty API is not returning the I just re-ran the acceptance test, and it does work properly:
Notice how the Tweaking the acceptance test, I also confirmed updates to existing services (PUT request), like seems your case, do work:
Notice too how the I suggest that you open a support case to PagerDuty pointing them to this issue. PS: I'm just a contributor here, so I have no idea what's going on the server side of the Pagerduty API. |
@elihai-risk A patch for solving is now available on version v2.12.1 of PagerDuty Terraform Provider. Hey @pdecat thank you so much again for your frequent interest on supporting this Terraform Provider users and development lifecycle 🎉 . On top that, I encourage you to do the upgrade and if you still face the issue, please don't hesitate on re-opening this issue ✌🏽 😎 |
Hi there,
This Feature doesn't apply by Terraform
in the Terraform plan it show it is going to add it but in the UI it's still off.
if i changing manually it doesn't show me drift
Terraform v1.2.3
PagerDuty providers - 2.6.2
If this issue appears to affect multiple resources, it may be an issue with Terraform's core, so please mention this.
Terraform Configuration Files
Debug Output
Please provider a link to a GitHub Gist containing the complete debug output: https://www.terraform.io/docs/internals/debugging.html. Please do NOT paste the debug output in the issue; just paste a link to the Gist.
Panic Output
If Terraform produced a panic, please provide a link to a GitHub Gist containing the output of the
crash.log
.Expected Behavior
What should have happened?
Actual Behavior
What actually happened?
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:
terraform apply
Important Factoids
Are there anything atypical about your accounts that we should know? For example: Running in EC2 Classic? Custom version of OpenStack? Tight ACLs?
References
Are there any other GitHub issues (open or closed) or Pull Requests that should be linked here? For example:
The text was updated successfully, but these errors were encountered: