Skip to content

Commit 01756ac

Browse files
abid-nhsNogaNHS
andauthored
[PRMP-1375] ods-update ECS task - increase CPU/memory (#227)
* PRMP-1375 - upgrade the cpu and memory providions for the ecs task * [PRMP-1375] Disable ods weekly update schedule --------- Co-authored-by: NogaNHS <[email protected]>
1 parent ce2db3b commit 01756ac

File tree

6 files changed

+10
-9
lines changed

6 files changed

+10
-9
lines changed

infrastructure/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
| Name | Version |
1010
|------|---------|
11-
| <a name="provider_aws"></a> [aws](#provider\_aws) | 5.66.0 |
11+
| <a name="provider_aws"></a> [aws](#provider\_aws) | 5.77.0 |
1212

1313
## Modules
1414

@@ -282,7 +282,7 @@
282282
|------|-------------|------|---------|:--------:|
283283
| <a name="input_auth_session_dynamodb_table_name"></a> [auth\_session\_dynamodb\_table\_name](#input\_auth\_session\_dynamodb\_table\_name) | The name of dynamodb table to store user login sessions | `string` | `"AuthSessionReferenceMetadata"` | no |
284284
| <a name="input_auth_state_dynamodb_table_name"></a> [auth\_state\_dynamodb\_table\_name](#input\_auth\_state\_dynamodb\_table\_name) | The name of dynamodb table to store the state values (for CIS2 authorisation) | `string` | `"AuthStateReferenceMetadata"` | no |
285-
| <a name="input_availability_zones"></a> [availability\_zones](#input\_availability\_zones) | This is a list that specifies all the Availability Zones that will have a pair of public and private subnets | `list(string)` | <pre>[<br> "eu-west-2a",<br> "eu-west-2b",<br> "eu-west-2c"<br>]</pre> | no |
285+
| <a name="input_availability_zones"></a> [availability\_zones](#input\_availability\_zones) | This is a list that specifies all the Availability Zones that will have a pair of public and private subnets | `list(string)` | <pre>[<br/> "eu-west-2a",<br/> "eu-west-2b",<br/> "eu-west-2c"<br/>]</pre> | no |
286286
| <a name="input_bulk_upload_report_dynamodb_table_name"></a> [bulk\_upload\_report\_dynamodb\_table\_name](#input\_bulk\_upload\_report\_dynamodb\_table\_name) | The name of dynamodb table to store bulk upload status | `string` | `"BulkUploadReport"` | no |
287287
| <a name="input_certificate_domain"></a> [certificate\_domain](#input\_certificate\_domain) | n/a | `string` | n/a | yes |
288288
| <a name="input_certificate_subdomain_name_prefix"></a> [certificate\_subdomain\_name\_prefix](#input\_certificate\_subdomain\_name\_prefix) | Prefix to add to subdomains on certification configurations, dev envs use api-{env}, prod envs use api.{env} | `string` | `"api-"` | no |

infrastructure/ecs.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,10 +90,10 @@ module "ndr-ecs-fargate-ods-update" {
9090
"value" : terraform.workspace
9191
}
9292
]
93-
ecs_container_definition_memory = 512
94-
ecs_container_definition_cpu = 256
95-
ecs_task_definition_memory = 512
96-
ecs_task_definition_cpu = 256
93+
ecs_container_definition_memory = 5120
94+
ecs_container_definition_cpu = 1024
95+
ecs_task_definition_memory = 5120
96+
ecs_task_definition_cpu = 1024
9797
}
9898

9999
resource "aws_iam_role" "ods_weekly_update_task_role" {

infrastructure/modules/ecs/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ No modules.
6363
| <a name="input_ecs_task_definition_cpu"></a> [ecs\_task\_definition\_cpu](#input\_ecs\_task\_definition\_cpu) | n/a | `number` | `1024` | no |
6464
| <a name="input_ecs_task_definition_memory"></a> [ecs\_task\_definition\_memory](#input\_ecs\_task\_definition\_memory) | n/a | `number` | `2048` | no |
6565
| <a name="input_environment"></a> [environment](#input\_environment) | n/a | `string` | n/a | yes |
66-
| <a name="input_environment_vars"></a> [environment\_vars](#input\_environment\_vars) | n/a | `list` | <pre>[<br> null<br>]</pre> | no |
66+
| <a name="input_environment_vars"></a> [environment\_vars](#input\_environment\_vars) | n/a | `list` | <pre>[<br/> null<br/>]</pre> | no |
6767
| <a name="input_is_autoscaling_needed"></a> [is\_autoscaling\_needed](#input\_is\_autoscaling\_needed) | n/a | `bool` | `true` | no |
6868
| <a name="input_is_lb_needed"></a> [is\_lb\_needed](#input\_is\_lb\_needed) | n/a | `bool` | `false` | no |
6969
| <a name="input_is_service_needed"></a> [is\_service\_needed](#input\_is\_service\_needed) | n/a | `bool` | `true` | no |

infrastructure/modules/lambda/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ No modules.
3333
| Name | Description | Type | Default | Required |
3434
|------|-------------|------|---------|:--------:|
3535
| <a name="input_api_execution_arn"></a> [api\_execution\_arn](#input\_api\_execution\_arn) | n/a | `string` | n/a | yes |
36-
| <a name="input_default_policies"></a> [default\_policies](#input\_default\_policies) | n/a | `list` | <pre>[<br> "arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole",<br> "arn:aws:iam::aws:policy/CloudWatchLambdaInsightsExecutionRolePolicy"<br>]</pre> | no |
36+
| <a name="input_default_policies"></a> [default\_policies](#input\_default\_policies) | n/a | `list` | <pre>[<br/> "arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole",<br/> "arn:aws:iam::aws:policy/CloudWatchLambdaInsightsExecutionRolePolicy"<br/>]</pre> | no |
3737
| <a name="input_handler"></a> [handler](#input\_handler) | n/a | `string` | n/a | yes |
3838
| <a name="input_http_methods"></a> [http\_methods](#input\_http\_methods) | n/a | `list(string)` | `[]` | no |
3939
| <a name="input_iam_role_policy_documents"></a> [iam\_role\_policy\_documents](#input\_iam\_role\_policy\_documents) | n/a | `list(string)` | `[]` | no |

infrastructure/schedules.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ resource "aws_scheduler_schedule" "ods_weekly_update_ecs" {
6161
count = local.is_sandbox ? 0 : 1
6262
name_prefix = "${terraform.workspace}_ods_weekly_update_ecs"
6363
description = "A weekly trigger for the ods update run"
64+
state = "DISABLED"
6465

6566
flexible_time_window {
6667
mode = "OFF"

virusscanner/terraform/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
| Name | Version |
1010
|------|---------|
11-
| <a name="provider_aws"></a> [aws](#provider\_aws) | 5.43.0 |
11+
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 4.0 |
1212

1313
## Modules
1414

0 commit comments

Comments
 (0)