-
Notifications
You must be signed in to change notification settings - Fork 63
Closed
Labels
Description
Problem
After merging PR #205, there are still 7 failing CI checks that need to be addressed to achieve a clean CI status for the repository.
Failing CI Checks
The following checks are currently failing:
Check | Status | Terraform | AWS Provider |
---|---|---|---|
Terraform Validate | ❌ FAIL | 1.3.0 | 5.0.0 |
Terraform Validate | ❌ FAIL | 1.3.0 | 5.70.0 |
Terraform Validate | ❌ FAIL | 1.5.0 | 5.0.0 |
Terraform Validate | ❌ FAIL | 1.5.0 | 5.70.0 |
Terraform Validate | ❌ FAIL | 1.9.0 | 5.0.0 |
Terraform Validate | ❌ FAIL | 1.9.0 | 5.70.0 |
Pre-commit | ❌ FAIL | - | - |
Root Cause Analysis Needed
The failures appear to be related to:
- Terraform formatting requirements across different versions
- Validation errors in the module or examples
- Pre-commit hook configuration issues
Tasks to Complete
- Investigate specific terraform validation errors in CI logs
- Fix any terraform formatting issues (
terraform fmt
) - Resolve validation errors across all terraform/provider version combinations
- Fix pre-commit workflow failures
- Verify all examples pass validation
- Ensure CI matrix tests pass for all supported versions
- Update documentation if any breaking changes are required
Success Criteria
- ✅ All terraform validation checks passing (6 version combinations)
- ✅ Pre-commit workflow passing
- ✅ Clean CI status on master branch
- ✅ All examples validating successfully
Related Context
- PR fix: resolve terraform validation errors found by pre-commit workflow #205 resolved the
test_formatting.tf
malformed file issue - PR fix: remove malformed test_formatting.tf causing CI failures #213 removed the problematic test file from master
- Security scan checks are currently passing ✅
- The issue appears to be with terraform code formatting or validation requirements
Priority
High Priority - These failures are blocking a clean CI status and may prevent future PRs from being merged successfully.
Environment Details
- Terraform versions tested: 1.3.0, 1.5.0, 1.9.0
- AWS Provider versions tested: 5.0.0, 5.70.0
- Pre-commit hooks: terraform_fmt, terraform_validate, terraform_docs, terraform_tflint
Next Steps
- Review CI logs for specific error messages
- Run
terraform fmt -recursive .
locally - Test with multiple terraform versions locally
- Fix any validation errors
- Update pre-commit configuration if needed
- Create PR with fixes