Skip to content

fix: test examples/complete subnet #272

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: test examples/complete subnet #272

wants to merge 1 commit into from

Conversation

@oycyc
Copy link

@oycyc oycyc commented Oct 17, 2025

Tests are failing

Error: Unsupported argument
on .terraform/modules/subnets/main.tf line 286, in resource "aws_eip" "default":
286:   vpc = true

An argument named "vpc" is not expected here.

Fix by upgrading to the latest module for the examples/complete

@oycyc oycyc requested review from a team as code owners October 17, 2025 20:25
@oycyc oycyc requested review from Gowiem and joe-niland October 17, 2025 20:25
@mergify mergify bot added the triage Needs triage label Oct 17, 2025
@oycyc
Copy link
Author

oycyc commented Oct 18, 2025

/terratest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Gowiem Gowiem Awaiting requested review from Gowiem Gowiem is a code owner automatically assigned from cloudposse/contributors

@joe-niland joe-niland Awaiting requested review from joe-niland joe-niland is a code owner automatically assigned from cloudposse/contributors

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

triage Needs triage

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@oycyc