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

Crash setting up aws_vpc_peering_connection #3145

Closed
rvangraan opened this issue Sep 2, 2015 · 7 comments
Closed

Crash setting up aws_vpc_peering_connection #3145

rvangraan opened this issue Sep 2, 2015 · 7 comments

Comments

@rvangraan
Copy link

I got this crash while deploying a new aws_vpc_peering_connection resource that uses the outputs of separate modules using Terraform v0.6.3:

resource "aws_vpc_peering_connection" "vpc-peer-pci-core-zone" {
    peer_owner_id = "${var.core-vpc-owner-id}"
    peer_vpc_id = "${module.core.core-zone-vpc-id}"
    vpc_id = "${module.pci.pci-zone-vpc-id}"
    auto_accept = true
}

Edit from @catsby: moved crash log to https://gist.github.com/catsby/a4dfde333e8bb540f019

@rvangraan
Copy link
Author

It looks like the peering connection got created, but then something went haywire.
screen shot 2015-09-02 at 14 23 31

@rvangraan
Copy link
Author

This has to do with

    auto_accept = true

when I removed it, the crash disappeared.

@apparentlymart
Copy link
Contributor

Thanks for the report and the debugging effort, @rvangraan.

Looks like this is failing trying to dereference one of the AcceptVpcPeeringConnectionInput response fields:

Not familiar at all with this API, so no strong theories yet. Just wondering, though: were you trying to peer between two VPCs in the same AWS account, or across two accounts?

@dsalazar-carecloud
Copy link

I experienced the same issue. And in my scenario it was across two accounts. I'm assuming that it has to do with the inability to prove that you are the owner of that other account and execute the auto acceptance of the peer request of the account vpc you're trying to peer in to within the context of the run.

@rvangraan
Copy link
Author

@apparentlymart: This was with two VPCs on the same account.

@catsby
Copy link
Contributor

catsby commented Oct 13, 2015

Hello –

I wasn't able to reproduce this directly, but I did open and merge #3490 to add better error handling and check for nil references.

Please let me know if you're still hitting this.
Thanks!

@catsby catsby closed this as completed Oct 13, 2015
@ghost
Copy link

ghost commented Apr 30, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants