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

provider/aws: Add aws_alb data source #10196

Merged
merged 2 commits into from
Nov 21, 2016

Conversation

vancluever
Copy link
Contributor

This adds the aws_alb data source for getting information on an AWS
Application Load Balancer.

The schema is nearly the same as the resource of the same name, with
most of the resource population logic de-coupled into its own function
so that they can be shared between the resource and data source.

@vancluever
Copy link
Contributor Author

Just a quick update to this one to fix language in an error message and docs, this one is good to go as well!

@stack72
Copy link
Contributor

stack72 commented Nov 18, 2016

Hi @vancluever

Sorry to ask, but any chance of a rebase on this? Your other PR got merged and caused the conflict

P.

@vancluever
Copy link
Contributor Author

@stack72 sounds good, will do ASAP!

Chris Marchesi added 2 commits November 19, 2016 08:47
This adds the aws_alb data source for getting information on an AWS
Application Load Balancer.

The schema is nearly the same as the resource of the same name, with
most of the resource population logic de-coupled into its own function
so that they can be shared between the resource and data source.
 * Multiple/zero result error slightly updated to be a bit more
   specific.
 * Fixed relic of the copy of the resource docs (resource -> data
   source)
@vancluever
Copy link
Contributor Author

@stack72 sorry for the delay, this is now done and pushed! Should be good to go!

@stack72
Copy link
Contributor

stack72 commented Nov 21, 2016

Hat tip sir! This looks good :)

% make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccDataSourceAWSALB_'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2016/11/21 10:25:04 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccDataSourceAWSALB_ -timeout 120m
=== RUN   TestAccDataSourceAWSALB_basic
--- PASS: TestAccDataSourceAWSALB_basic (88.51s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	88.523s

Thanks for the work as normal

@stack72 stack72 merged commit 4845f8d into hashicorp:master Nov 21, 2016
@vancluever
Copy link
Contributor Author

Awesome! Always a pleasure! 😀

gusmat pushed a commit to gusmat/terraform that referenced this pull request Dec 6, 2016
* provider/aws: Add aws_alb data source

This adds the aws_alb data source for getting information on an AWS
Application Load Balancer.

The schema is nearly the same as the resource of the same name, with
most of the resource population logic de-coupled into its own function
so that they can be shared between the resource and data source.

* provider/aws: aws_alb data source language revisions

 * Multiple/zero result error slightly updated to be a bit more
   specific.
 * Fixed relic of the copy of the resource docs (resource -> data
   source)
fatmcgav pushed a commit to fatmcgav/terraform that referenced this pull request Feb 27, 2017
* provider/aws: Add aws_alb data source

This adds the aws_alb data source for getting information on an AWS
Application Load Balancer.

The schema is nearly the same as the resource of the same name, with
most of the resource population logic de-coupled into its own function
so that they can be shared between the resource and data source.

* provider/aws: aws_alb data source language revisions

 * Multiple/zero result error slightly updated to be a bit more
   specific.
 * Fixed relic of the copy of the resource docs (resource -> data
   source)
@ghost
Copy link

ghost commented Apr 19, 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 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants