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

aws_eip data provider does not support Classic EIPs #2825

Closed
Ashex opened this issue Jan 2, 2018 · 4 comments
Closed

aws_eip data provider does not support Classic EIPs #2825

Ashex opened this issue Jan 2, 2018 · 4 comments
Labels
bug Addresses a defect in current functionality. service/ec2 Issues and PRs that pertain to the ec2 service.
Milestone

Comments

@Ashex
Copy link

Ashex commented Jan 2, 2018

The following snippet leads to a crash when the EIP is on ec2-classic. It does work if it is a VPC EIP:

Terraform Version

Terraform v0.11.1

  • provider.aws v1.6.0

Affected Resource(s)

  • aws_eip

If this issue appears to affect multiple resources, it may be an issue with Terraform's core, so please mention this.

Terraform Configuration Files

provider "aws" {
    version = "1.6"
}

data "aws_eip" "this" {
  public_ip = "10.10.10.10"
}

Debug Output

https://gist.github.com/Ashex/7035dc2d335dfc00650495e59816b979

Panic Output

https://gist.github.com/Ashex/7035dc2d335dfc00650495e59816b979

Expected Behavior

EIP allocation Id is retriever

Actual Behavior

Terraform crashes with a EOF error

Steps to Reproduce

Please list the steps required to reproduce the issue, for example:

  1. terraform plan

Important Factoids

Verified that this occurs on aws provider 1.5.0

When I run this with a VPC EIP it succeeds, this appears to only fail with a Classic EIP

@jen20 jen20 added the bug Addresses a defect in current functionality. label Jan 2, 2018
@radeksimko radeksimko added service/ec2 Issues and PRs that pertain to the ec2 service. service/ec2-classic labels Jan 28, 2018
@micemuni
Copy link

micemuni commented May 7, 2018

Not only classic IP. We are not able to attach the EIP that created manually. then i am trying to attach to instance creating via terraform

@bflad bflad closed this as completed in aa8b83f Nov 13, 2018
@bflad bflad added this to the v1.44.0 milestone Nov 13, 2018
@bflad
Copy link
Contributor

bflad commented Nov 13, 2018

Hi folks 👋 Very sorry for the lengthy delays with merging this support, there were a few conflicting pull requests to the same code which needed to get sorted out. Good news is that the following will be supported in version 1.44.0 of the AWS provider, likely releasing later today or tomorrow:

  • Search by filter
  • Search by tags
  • EC2-Classic support

@bflad
Copy link
Contributor

bflad commented Nov 15, 2018

This has been released in version 1.44.0 of the AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

@ghost
Copy link

ghost commented Apr 2, 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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked and limited conversation to collaborators Apr 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Addresses a defect in current functionality. service/ec2 Issues and PRs that pertain to the ec2 service.
Projects
None yet
Development

No branches or pull requests

5 participants