Skip to content

Commit 193392b

Browse files
claude[bot]lgallard
andcommitted
fix: resolve terraform validation errors and pre-commit configuration issues
- Remove duplicate terraform configuration blocks from secure_backup_configuration example - Move configuration_aliases from main.tf to versions.tf to prevent duplication - Add exclusions for temporary test files in pre-commit configuration - Add retry logic for terraform validation to handle transient issues Co-authored-by: Luis M. Gallardo D. <lgallard@users.noreply.github.com>
1 parent 4332108 commit 193392b

File tree

3 files changed

+7
-12
lines changed

3 files changed

+7
-12
lines changed

.pre-commit-config.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,19 @@ repos:
2121
rev: v1.83.0 # Use more stable version
2222
hooks:
2323
- id: terraform_fmt
24+
exclude: '^test_.*\.tf$|.*test_formatting\.tf$' # Exclude temporary test files
2425
- id: terraform_validate
26+
exclude: '^test_.*\.tf$|.*test_formatting\.tf$' # Exclude temporary test files
27+
args:
28+
- --hook-config=--retry-once-with-cleanup=true # Retry validation with cleanup
2529
- id: terraform_docs
2630
args:
2731
- --args=--config=.terraform-docs.yml # Use config file for consistent documentation
32+
exclude: '^test_.*\.tf$|.*test_formatting\.tf$' # Exclude temporary test files
2833
- id: terraform_tflint # Added terraform linter
2934
args:
3035
- --args=--config=.tflint.hcl
36+
exclude: '^test_.*\.tf$|.*test_formatting\.tf$' # Exclude temporary test files
3137
# Temporarily disabled terraform_checkov due to missing checkov installation in CI
3238
# - id: terraform_checkov # Added security scanner
3339
# args:

examples/secure_backup_configuration/main.tf

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,6 @@
11
# Secure AWS Backup Configuration Example
22
# This example demonstrates enterprise-grade backup security practices
33

4-
terraform {
5-
required_version = ">= 1.0"
6-
7-
required_providers {
8-
aws = {
9-
source = "hashicorp/aws"
10-
version = ">= 5.0"
11-
configuration_aliases = [aws.cross_region]
12-
}
13-
}
14-
}
15-
164
# Data sources for account and region information
175
data "aws_caller_identity" "current" {}
186
data "aws_region" "current" {}

examples/secure_backup_configuration/versions.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ terraform {
77
aws = {
88
source = "hashicorp/aws"
99
version = ">= 5.0"
10+
configuration_aliases = [aws.cross_region]
1011
}
1112
}
1213
}

0 commit comments

Comments
 (0)