Skip to content

Commit

Permalink
feat(terraform): VOL-5223 Add timeouts for some batch commands, and a…
Browse files Browse the repository at this point in the history
… default timeout for the commands we don't have
  • Loading branch information
matbeales committed Jul 30, 2024
1 parent f7e3339 commit 171a23f
Show file tree
Hide file tree
Showing 3 changed files with 32 additions and 2 deletions.
31 changes: 30 additions & 1 deletion infra/terraform/environments/dev/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -253,22 +253,27 @@ module "service" {
{
name = "clean-up-variations",
commands = ["batch:clean-up-variations"],
timeout = 43200,
},
{
name = "cns",
commands = ["batch:cns"],
timeout = 43200,
},
{
name = "create-psv-licence-surrender-task",
commands = ["batch:create-psv-licence-surrender-task"],
timeout = 43200,
},
{
name = "psv-operator-list-export",
commands = ["batch:data-gov-uk-export", "-v", "--report-name=psv-operator-list", "--path=/tmp/"],
timeout = 43200,
},
{
name = "international-goods-export",
commands = ["batch:data-gov-uk-export", "-v", "--report-name=international-goods", "--path=/tmp/"],
timeout = 43200,
},
{
name = "data-retention-populate",
Expand All @@ -293,22 +298,27 @@ module "service" {
{
name = "digital-continuation-reminders",
commands = ["batch:digital-continuation-reminders"],
timeout = 43200,
},
{
name = "duplicate-vehicle-warning",
commands = ["batch:duplicate-vehicle-warning"],
timeout = 43200,
},
{
name = "enqueue-ch-compare"
name = "enqueue-ch-compare",
commands = ["batch:enqueue-ch-compare"],
timeout = 1800,
},
{
name = "expire-bus-registration",
commands = ["batch:expire-bus-registration"],
timeout = 43200,
},
{
name = "flag-urgent-tasks",
commands = ["batch:flag-urgent-tasks"],
timeout = 1800,
},
{
name = "import-users-from-csv",
Expand All @@ -317,18 +327,22 @@ module "service" {
{
name = "inspection-request-email",
commands = ["batch:inspection-request-email"],
timeout = 1800,
},
{
name = "interim-end-date-enforcement",
commands = ["batch:interim-end-date-enforcement"],
timeout = 43200,
},
{
name = "last-tm-letter",
commands = ["batch:last-tm-letter"],
timeout = 43200,
},
{
name = "licence-status-rules",
commands = ["batch:licence-status-rules"],
timeout = 1800,
},
{
name = "process-cl",
Expand All @@ -337,18 +351,22 @@ module "service" {
{
name = "process-inbox",
commands = ["batch:process-inbox"],
timeout = 43200,
},
{
name = "process-ntu",
commands = ["batch:process-ntu"],
timeout = 43200,
},
{
name = "remove-read-audit",
commands = ["batch:remove-read-audit"],
timeout = 43200,
},
{
name = "resolve-payments",
commands = ["batch:resolve-payments"],
timeout = 150,
},
{
name = "system-parameter",
Expand All @@ -361,46 +379,57 @@ module "service" {
{
name = "close-expired-windows",
commands = ["permits:close-expired-windows"],
timeout = 43200,
},
{
name = "mark-expired-permits",
commands = ["permits:mark-expired-permits"],
timeout = 43200,
},
{
name = "process-queue-general",
commands = ["queue:process-queue", "--exclude", "que_typ_ch_compare,que_typ_create_gds_vehicle_list,que_typ_create_psv_vehicle_list,que_typ_disc_printing,que_typ_print,que_typ_disc_printing_print,que_typ_create_com_lic,que_typ_remove_deleted_docs,que_typ_permit_generate,que_typ_permit_print,que_typ_run_ecmt_scoring,que_typ_accept_ecmt_scoring,que_typ_irhp_permits_allocate"],
timeout = 60,
},
{
name = "process-queue-community-licences",
commands = ["queue:process-queue", "--type", "que_typ_create_com_lic"],
timeout = 60,
},
{
name = "process-queue-disc-generation",
commands = ["queue:process-queue", "--type", "que_typ_create_gds_vehicle_list,que_typ_create_psv_vehicle_list,que_typ_disc_printing"],
timeout = 60,
},
{
name = "process-queue-disc-print",
commands = ["queue:process-queue", "--type", "que_typ_disc_printing_print", "--queue-duration", "840"],
timeout = 60,
},
{
name = "process-queue-ecmt-accept",
commands = ["queue:process-queue", "--type", "que_typ_accept_ecmt_scoring"],
timeout = 60,
},
{
name = "process-queue-irhp-allocate",
commands = ["queue:process-queue", "--type", "que_typ_run_ecmt_scoring"],
timeout = 60,
},
{
name = "process-queue-permit-generation",
commands = ["queue:process-queue", "--type", "que_typ_permit_generate"],
timeout = 60,
},
{
name = "process-queue-permit-print",
commands = ["queue:process-queue", "--type", "que_typ_permit_print", "--queue-duration", "840"],
timeout = 60,
},
{
name = "process-queue-print",
commands = ["queue:process-queue", "--type", "que_typ_print"],
timeout = 60,
},
{
name = "process-company-profile",
Expand Down
2 changes: 1 addition & 1 deletion infra/terraform/modules/service/batch.tf
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ locals {
}
})

attempt_duration_seconds = 60
attempt_duration_seconds = job.timeout
retry_strategy = local.default_retry_policy
} }
}
Expand Down
1 change: 1 addition & 0 deletions infra/terraform/modules/service/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ variable "batch" {
commands = list(string)
cpu = optional(number, 1)
memory = optional(number, 2048)
timeout = optional(number, 300)
}))
})
}

0 comments on commit 171a23f

Please sign in to comment.