Skip to content

Commit

Permalink
Increase timeouts (#199)
Browse files Browse the repository at this point in the history
<!-- Thank you for submitting a Pull Request. Please fill out the
template below.-->
## Overview/Summary

Increase timeouts to help with ADO eventual consistency issue

## This PR fixes/adds/changes/removes

1. Azure/terraform-azurerm-avm-ptn-alz#157
2. Azure/ALZ-PowerShell-Module#269

### Breaking Changes

None

## Testing Evidence

Please provide any testing evidence to show that your Pull Request
works/fixes as described and planned (include screenshots, if
appropriate).

## As part of this Pull Request I have

- [x] Checked for duplicate [Pull
Requests](https://github.com/Azure/alz-terraform-accelerator/pulls)
- [x] Associated it with relevant
[issues](https://github.com/Azure/alz-terraform-accelerator/issues), for
tracking and closure.
- [x] Ensured my code/branch is up-to-date with the latest changes in
the `main`
[branch](https://github.com/Azure/alz-terraform-accelerator/tree/main)
- [x] Performed testing and provided evidence.
- [x] Updated relevant and associated documentation.
  • Loading branch information
jaredfholgate authored Jan 11, 2025
1 parent d55348c commit 072dae3
Showing 1 changed file with 34 additions and 10 deletions.
44 changes: 34 additions & 10 deletions templates/platform_landing_zone/modules/management_groups/locals.tf
Original file line number Diff line number Diff line change
Expand Up @@ -14,31 +14,55 @@ locals {
} }
} }
default_retries = {
policy_assignments = {
error_message_regex = ["AuthorizationFailed", "The policy definition specified in policy assignment '.+' is out of scope"]
management_groups = {
error_message_regex = ["AuthorizationFailed"]
}
role_definitions = {
error_message_regex = ["AuthorizationFailed"]
}
policy_definitions = {
error_message_regex = ["AuthorizationFailed"]
}
policy_set_definitions = {
error_message_regex = ["AuthorizationFailed"]
}
policy_assignments = {
error_message_regex = ["AuthorizationFailed", "The policy definition specified in policy assignment '.+' is out of scope"]
}
policy_role_assignments = {
error_message_regex = ["AuthorizationFailed", "ResourceNotFound"]
}
hierarchy_settings = {
error_message_regex = ["AuthorizationFailed"]
}
subscription_placement = {
error_message_regex = ["AuthorizationFailed"]
}
}
default_timeouts = {
policy_assignments = {
create = "10m"
management_group = {
create = "60m"
read = "60m"
}
policy_definitions = {
create = "10m"
role_definition = {
create = "60m"
read = "60m"
}
policy_set_definitions = {
create = "10m"
policy_assignment = {
create = "60m"
read = "60m"
}
policy_role_assignments = {
create = "10m"
policy_definition = {
create = "60m"
read = "60m"
}
policy_set_definition = {
create = "60m"
read = "60m"
}
policy_role_assignment = {
create = "60m"
read = "60m"
}
}
}

0 comments on commit 072dae3

Please sign in to comment.