Skip to content

Conversation

@oycyc
Copy link
Contributor

@oycyc oycyc commented Oct 1, 2025

Tests are failing because of the examples/complete/: https://github.com/cloudposse/terraform-aws-mq-broker/actions/runs/17283152983/job/49055266176

Error: Unsupported argument
on .terraform/modules/subnets/nat-gateway.tf line 19, in resource "aws_eip" "default":
19: vpc = true

An argument named "vpc" is not expected here.

The newer module of the cloudposse/dynamic-subnets/aws addresses it by not using the deprecated vpc attribute.

This will fix tests.

@oycyc oycyc requested review from a team as code owners October 1, 2025 13:50
@oycyc oycyc requested review from gberenice and jamengual October 1, 2025 13:50
@mergify mergify bot added the triage Needs triage label Oct 1, 2025
@oycyc
Copy link
Contributor Author

oycyc commented Oct 1, 2025

/terratest

@oycyc
Copy link
Contributor Author

oycyc commented Oct 1, 2025

/terratest

@oycyc
Copy link
Contributor Author

oycyc commented Oct 1, 2025

/terratest

@oycyc
Copy link
Contributor Author

oycyc commented Oct 1, 2025

/terratest

@oycyc oycyc changed the title fix(tests): updated version of the subnets module due to provider fix(tests): available engine version and updated version of the subnets module due to provider Oct 1, 2025
@mergify mergify bot removed the triage Needs triage label Oct 1, 2025
@oycyc oycyc merged commit 9aa93a9 into main Oct 1, 2025
26 checks passed
@oycyc oycyc deleted the fix/tests-subnet-module branch October 1, 2025 14:49
@github-actions
Copy link
Contributor

github-actions bot commented Oct 1, 2025

These changes were released in v3.5.1.

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.

3 participants