Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

[Feature Request] Fix the tflint recommended preset findings and enforce the full preset

Đang mở
  • Có gì không ổn?
#321 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
45/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
terraform
Lĩnh vực
devops, infrastructure

Hướng nghiên cứu

Start with .tflint.hcl and tools/tflint.sh, then run tflint to reproduce the 66 findings and the evaluation failure in blueprints/fedramp-high/cloud-scheduler-job. Work through the reported unused declarations, deprecated interpolations, and untyped variable, recording notes for removed variables that may still appear in tfvars files. Done when the preset reports no findings, the blueprint loads successfully, and the script enforces the full preset.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Enhancement Level of Effort - Medium Priority - Medium Scope - Blueprint Scope - Fast Scope - Module

Feature Description

Fix the findings that the tflint recommended preset reports today, then let tools/tflint.sh enforce the whole preset instead of only the terraform_naming_convention rule.

Use Case

  • The .tflint.hcl file enables the recommended preset, but CI never ran tflint until #310, so findings accumulated.
  • On main, the preset reports 66 findings: 62 terraform_unused_declarations, 3 terraform_deprecated_interpolation and 1 terraform_typed_variables.
  • tflint also fails to load blueprints/fedramp-high/cloud-scheduler-job, because it evaluates base64encode(var.data) with a null default.

Proposed Solution

  • Remove or use the unused variables and locals, convert the deprecated interpolations, and type the remaining variable.
  • Fix the cloud-scheduler-job evaluation, for example by guarding the null value.
  • Drop --only=terraform_naming_convention from tools/tflint.sh once the findings reach zero.
  • Removing an unused variable breaks tfvars files that still set it, so each removal needs a note.

Compliance & Deployment Context

  • Target Deployment Type(s):
    • US Region Restricted (e.g., Access Policy constraint)
    • FedRAMP Moderate
    • FedRAMP High
    • DoD IL4
    • DoD IL5
    • All / General
  • Relevant NIST 800-53r5 Controls: CM-2 baseline configuration

Reusability Check

Stellar Engine prioritizes reusability.

  • I have checked if this functionality can be achieved by extending an existing module or blueprint.
  • I have verified that this does not duplicate existing functionality.

Alternatives Considered

  • Disable the noisy rules in .tflint.hcl, which hides real dead code.
  • Keep only the naming rule, which leaves the preset unenforced.

Additional Context

  • Related: #309, #310 and #311.
Ngôn ngữ chính
HCL
Star
58
Fork
25
Merge trung bình
1 ngày 17 giờ
Pull request đã merge (30 ngày)
60

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của google/stellar-engine

Tất cả issue của google/stellar-engine

Issue tương tự

Thêm issue về DevOps

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.