Skip to content

Commit

Permalink
Merge pull request #16248 from CDCgov/ms/devsecops/tf-timeouts
Browse files Browse the repository at this point in the history
bump GET timeout to 60m
  • Loading branch information
devopsmatt authored Oct 16, 2024
2 parents fa29642 + 322b8d7 commit 1ff8ce4
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 1 deletion.
14 changes: 14 additions & 0 deletions operations/app/terraform/modules/storage/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -196,6 +196,13 @@ resource "azurerm_storage_account" "storage_public" {
tags = {
environment = var.environment
}

timeouts {
create = var.timeout_create
read = var.timeout_read
delete = var.timeout_delete
update = var.timeout_update
}
}

resource "azurerm_storage_share" "gh_locks" {
Expand Down Expand Up @@ -251,6 +258,13 @@ resource "azurerm_storage_account" "storage_partner" {
tags = {
environment = var.environment
}

timeouts {
create = var.timeout_create
read = var.timeout_read
delete = var.timeout_delete
update = var.timeout_update
}
}

# Grant the storage account Key Vault access, to access encryption keys
Expand Down
2 changes: 1 addition & 1 deletion operations/app/terraform/modules/storage/~inputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ variable "timeout_create" {
variable "timeout_read" {
description = "Timeout for read operations"
type = string
default = "615m" # module default 5m
default = "60m" # module default 5m
}

variable "timeout_update" {
Expand Down

0 comments on commit 1ff8ce4

Please sign in to comment.