Skip to content
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.

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

terraform-aws-vpc timeouts when attaching VPG #39

Closed
disaac opened this issue Mar 8, 2019 · 1 comment
Closed

terraform-aws-vpc timeouts when attaching VPG #39

disaac opened this issue Mar 8, 2019 · 1 comment
Assignees

Comments

@disaac
Copy link
Member

disaac commented Mar 8, 2019

Currently when running complete:overrides:apply and complete:peer-vpc:apply an error is often reported due to a lack of dependency between the vpg being attached and the route table creation since they are referenced in separate modules. The VPG id can exist before it is attached allowing the route table to create but will error out due propagating_vgws = ["${compact(var.vgw_ids)}"] being present but not attached.

To address this issue there is a workaround that is mentioned here by brikis98 I applied this fix/workaround and verified that the issue is no longer seen. Previously this was addressed by simply running the apply twice so that the next time around it will succeed.

Since the fix is more of a workaround I have pushed a branch for reference but will address the recent failures when applying complete:peer-vpc:apply by also running it twice since this will avoid the drift that would be seen on deployments if the null resource fix is used.

A refactor that leverages the aws_vpn_gateway_route_propagation resource instead of the route_table:propagating_vgws may also be a means of ensuring that the az module can create its resources without the vgws presence.

disaac added a commit that referenced this issue Mar 8, 2019
* Addresses [issue #39](#39) by creating a null_resource that causes a dependency on the vpg being attachment in the az module when creating the routing tables.
disaac added a commit that referenced this issue Mar 8, 2019
* Run all tasks that rely on the VPG attachment twice.
* Added link to issue for reference.
disaac added a commit that referenced this issue Mar 9, 2019
…-release

Addresses timeout in CI runs from issue #39
@disaac disaac self-assigned this Mar 9, 2019
@disaac
Copy link
Member Author

disaac commented Mar 9, 2019

Addressed in release v0.4.0

@disaac disaac closed this as completed Mar 9, 2019
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

No branches or pull requests

1 participant