Skip to content

Commit 1ba2f7c

Browse files
committed
Consolidate all variables into region.hcl and fix variable references
1 parent abfeb53 commit 1ba2f7c

File tree

8 files changed

+239
-127
lines changed

8 files changed

+239
-127
lines changed

infrastructure/dev/us-east-2/compute/parallel-cluster/terragrunt.hcl

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -79,24 +79,19 @@ inputs = {
7979

8080
# EFA Configuration
8181
efa = {
82-
enabled = true
83-
gdr_support = false # Disabled for dev
82+
enabled = local.efa_config.enabled
83+
gdr_support = local.efa_config.enable_gpudirect
8484
}
8585

8686
# Tags
87-
tags = {
87+
tags = merge(local.common_tags, {
8888
Name = local.cluster_name
8989
Type = "ParallelCluster"
9090
Purpose = "HPC-Compute"
91-
Environment = local.environment
92-
Region = local.region
93-
}
91+
})
9492

9593
# Additional variables for local Terraform resources
9694
environment = local.environment
9795
region = local.region
98-
common_tags = {
99-
Environment = local.environment
100-
Region = local.region
101-
}
96+
common_tags = local.common_tags
10297
}

infrastructure/dev/us-east-2/monitoring/cloudwatch/terragrunt.hcl

Lines changed: 15 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -23,42 +23,35 @@ dependency "kms" {
2323
inputs = {
2424
# CloudWatch Log Groups
2525
log_groups = {
26-
hpc_system_logs = {
27-
name = "/aws/hpc/${local.environment}/system"
28-
retention_in_days = local.monitoring.cloudwatch.log_retention_days
29-
kms_key_id = dependency.kms.outputs.kms_key_arn
30-
}
31-
hpc_application_logs = {
32-
name = "/aws/hpc/${local.environment}/application"
33-
retention_in_days = local.monitoring.cloudwatch.log_retention_days
26+
for name, config in local.cloudwatch_config.log_groups : name => {
27+
name = config.name
28+
retention_in_days = config.retention_in_days
3429
kms_key_id = dependency.kms.outputs.kms_key_arn
3530
}
3631
}
3732

3833
# CloudWatch Alarms
3934
alarms = {
4035
high_queue_depth = {
41-
alarm_name = "hpc-${local.environment}-high-queue-depth"
42-
comparison_operator = "GreaterThanThreshold"
43-
evaluation_periods = "2"
44-
metric_name = "JobsQueued"
45-
namespace = "AWS/ParallelCluster"
46-
period = "300"
47-
statistic = "Average"
48-
threshold = "100"
49-
alarm_description = "This metric monitors high queue depth"
36+
alarm_name = local.cloudwatch_config.alarms.high_queue_depth.alarm_name
37+
comparison_operator = local.cloudwatch_config.alarms.high_queue_depth.comparison_operator
38+
evaluation_periods = local.cloudwatch_config.alarms.high_queue_depth.evaluation_periods
39+
metric_name = local.cloudwatch_config.alarms.high_queue_depth.metric_name
40+
namespace = local.cloudwatch_config.alarms.high_queue_depth.namespace
41+
period = local.cloudwatch_config.alarms.high_queue_depth.period
42+
statistic = local.cloudwatch_config.alarms.high_queue_depth.statistic
43+
threshold = local.cloudwatch_config.alarms.high_queue_depth.threshold
44+
alarm_description = local.cloudwatch_config.alarms.high_queue_depth.alarm_description
5045
alarm_actions = [dependency.kms.outputs.sns_topic_arn]
5146
ok_actions = [dependency.kms.outputs.sns_topic_arn]
52-
treat_missing_data = "notBreaching"
47+
treat_missing_data = local.cloudwatch_config.alarms.high_queue_depth.treat_missing_data
5348
}
5449
}
5550

5651
# Tags
57-
tags = {
52+
tags = merge(local.common_tags, {
5853
Name = "hpc-${local.environment}-cloudwatch"
5954
Type = "CloudWatch"
6055
Purpose = "Monitoring"
61-
Environment = local.environment
62-
Region = local.region
63-
}
56+
})
6457
}

infrastructure/dev/us-east-2/networking/efa-sg/terragrunt.hcl

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,14 @@ dependency "vpc" {
2323
inputs = {
2424
cluster_name = "hpc-${local.environment}"
2525
vpc_id = dependency.vpc.outputs.vpc_id
26-
vpc_cidr = local.networking.vpc_cidr
26+
vpc_cidr = local.vpc_config.cidr_block
2727
subnet_id = dependency.vpc.outputs.compute_subnets[0]
2828
availability_zone = local.networking.primary_az
2929

3030
# EFA Configuration
31-
efa_device = "efa0"
32-
mtu_size = 9000
33-
enable_gpudirect = false # Disabled for dev
31+
efa_device = local.efa_config.device_name
32+
mtu_size = local.efa_config.mtu_size
33+
enable_gpudirect = local.efa_config.enable_gpudirect
3434

3535
# Instance Configuration
3636
instance_type = local.instance_types.compute
@@ -69,18 +69,13 @@ inputs = {
6969
ok_actions = [aws_sns_topic.hpc_alerts.arn]
7070

7171
# Tags
72-
tags = {
72+
tags = merge(local.common_tags, {
7373
Component = "EFA-Network"
7474
Tier = "HPC-Compute"
75-
Environment = local.environment
76-
Region = local.region
77-
}
75+
})
7876

7977
# Additional variables for local Terraform resources
8078
environment = local.environment
8179
region = local.region
82-
common_tags = {
83-
Environment = local.environment
84-
Region = local.region
85-
}
80+
common_tags = local.common_tags
8681
}

infrastructure/dev/us-east-2/networking/vpc/terragrunt.hcl

Lines changed: 19 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -18,19 +18,19 @@ terraform {
1818
inputs = {
1919
# VPC Module inputs
2020
name = "hpc-${local.environment}-vpc"
21-
cidr = local.networking.vpc_cidr
21+
cidr = local.vpc_config.cidr_block
2222

2323
azs = local.availability_zones
24-
private_subnets = local.networking.subnet_cidrs.private
25-
public_subnets = local.networking.subnet_cidrs.public
24+
private_subnets = local.subnet_config.private.cidr_blocks
25+
public_subnets = local.subnet_config.public.cidr_blocks
2626

2727
# Additional subnets for HPC
28-
database_subnets = local.networking.subnet_cidrs.storage
29-
compute_subnets = local.networking.subnet_cidrs.compute
28+
database_subnets = local.subnet_config.storage.cidr_blocks
29+
compute_subnets = local.subnet_config.compute.cidr_blocks
3030

3131
# Enable DNS
32-
enable_dns_hostnames = true
33-
enable_dns_support = true
32+
enable_dns_hostnames = local.vpc_config.enable_dns_hostnames
33+
enable_dns_support = local.vpc_config.enable_dns_support
3434

3535
# Enable NAT Gateway (single for dev)
3636
enable_nat_gateway = true
@@ -46,49 +46,36 @@ inputs = {
4646
enable_s3_endpoint = true
4747

4848
# Tags
49-
tags = {
49+
tags = merge(local.common_tags, {
5050
Name = "hpc-${local.environment}-vpc"
5151
Type = "VPC"
52-
Environment = local.environment
53-
Region = local.region
54-
}
52+
})
5553

5654
# Subnet tags
57-
private_subnet_tags = {
55+
private_subnet_tags = merge(local.common_tags, {
5856
Type = "Private-Subnet"
5957
Tier = "Compute"
60-
Environment = local.environment
61-
Region = local.region
62-
}
58+
})
6359

64-
public_subnet_tags = {
60+
public_subnet_tags = merge(local.common_tags, {
6561
Type = "Public-Subnet"
6662
Tier = "Management"
67-
Environment = local.environment
68-
Region = local.region
69-
}
63+
})
7064

71-
database_subnet_tags = {
65+
database_subnet_tags = merge(local.common_tags, {
7266
Type = "Database-Subnet"
7367
Tier = "Storage"
74-
Environment = local.environment
75-
Region = local.region
76-
}
68+
})
7769

78-
compute_subnet_tags = {
70+
compute_subnet_tags = merge(local.common_tags, {
7971
Type = "Compute-Subnet"
8072
Tier = "HPC-Compute"
81-
Environment = local.environment
82-
Region = local.region
83-
}
73+
})
8474

8575
# Additional variables for local Terraform resources
8676
environment = local.environment
8777
region = local.region
88-
vpc_cidr = local.networking.vpc_cidr
89-
common_tags = {
90-
Environment = local.environment
91-
Region = local.region
92-
}
78+
vpc_cidr = local.vpc_config.cidr_block
79+
common_tags = local.common_tags
9380
}
9481

0 commit comments

Comments
 (0)