Skip to content

fix: Correct package filename hash count - #768

Open
Kanishkaagrawal006 wants to merge 2 commits into
terraform-aws-modules:masterfrom
Kanishkaagrawal006:fix/767-package-filename-hash-count
Open

fix: Correct package filename hash count#768
Kanishkaagrawal006 wants to merge 2 commits into
terraform-aws-modules:masterfrom
Kanishkaagrawal006:fix/767-package-filename-hash-count

Conversation

@Kanishkaagrawal006

Copy link
Copy Markdown

Description

Closes #767

This PR fixes terraform_data.package_filename_for_hash so that it respects the module's create flag.

Changes

  • Gate terraform_data.package_filename_for_hash behind local.create.
  • Guard source_code_hash from referencing the resource when create = false.
  • Add a regression test covering the relevant create and ignore_source_code_hash combinations.

Testing

The new Terraform test passes successfully:

  • 3 passed
  • 0 failed

@Kanishkaagrawal006 Kanishkaagrawal006 changed the title fix: gate terraform_data.package_filename_for_hash behind create flag(#767) fix: correct package filename hash count Aug 9, 2026
@Kanishkaagrawal006 Kanishkaagrawal006 changed the title fix: correct package filename hash count fix: Correct package filename hash count Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

resource "terraform_data" "package_filename_for_hash" is not flagged behind create variable

2 participants