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

Add support for tags and filters to aws_eip data source #4441

Closed
wants to merge 1 commit into from

Conversation

UnsafePointer
Copy link

This should close #2932

@ghost ghost added the size/S Managed by automation to categorize the size of a PR. label May 3, 2018
@bflad bflad added enhancement Requests to existing resources that expand the functionality or scope. service/ec2 Issues and PRs that pertain to the ec2 service. labels May 5, 2018
@ghost ghost added size/M Managed by automation to categorize the size of a PR. and removed size/S Managed by automation to categorize the size of a PR. labels May 5, 2018
@UnsafePointer
Copy link
Author

Hey 👋, this is my first contribution and a bit of help would be really appreciated. I can't make the acceptance tests pass, I get the following:

testing.go:518: Step 0 error: Error refreshing: 2 error(s) occurred:

                * data.aws_eip.by_filter: 1 error(s) occurred:

                * data.aws_eip.by_filter: data.aws_eip.by_filter: no matching Elastic IP found
                * data.aws_eip.by_tags: 1 error(s) occurred:

                * data.aws_eip.by_tags: data.aws_eip.by_tags: no matching Elastic IP found

As far as I can understand, the data sources I added can't match the Elastic IP resource created. Using the exact same code works if I compile my branch and deploy it as a plugin. What am I missing?

@ghost ghost added the size/M Managed by automation to categorize the size of a PR. label May 7, 2018
@ghost ghost added the size/M Managed by automation to categorize the size of a PR. label May 7, 2018
@UnsafePointer
Copy link
Author

Managed to found the error myself. I believe the data source was trying to fetch the resource before it was created. Fixed it by adding a reference of the resource to the data source:

eip_tags {
  Name = "${aws_eip.test.tags["Name"]}"
}

I think this should be ready for review now, I added tests and updated the docs.

@UnsafePointer UnsafePointer changed the title [WIP] Add support for tags and filters to aws_eip data source Add support for tags and filters to aws_eip data source May 7, 2018
@UnsafePointer
Copy link
Author

@bflad is there anything I can do to move this forward?

@UnsafePointer
Copy link
Author

My bad, I didn't notice that #2932 and #3525 are tackling this already.

Closing in favour of those.

@UnsafePointer UnsafePointer deleted the aws_eip_tags branch May 28, 2018 15:00
@ghost
Copy link

ghost commented Apr 5, 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 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/ec2 Issues and PRs that pertain to the ec2 service. size/M Managed by automation to categorize the size of a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for adding tags to Elastic IPs
2 participants