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 new data source type aws_autoscaling_group #6849

Merged
merged 4 commits into from
Dec 21, 2018

Conversation

deberon
Copy link
Contributor

@deberon deberon commented Dec 14, 2018

Fixes #6758

Changes proposed in this pull request:

  • Add a new data source type called aws_autoscaling_group
  • Add documentation for new data source type

Output from acceptance testing:

$ TESTARGS="-run TestAccAwsAutoScalingGroupDataSource_basic" make testacc
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./... -v -parallel 20 -run TestAccAwsAutoScalingGroupDataSource_basic -timeout 120m
?       github.com/terraform-providers/terraform-provider-aws   [no test files]
=== RUN   TestAccAwsAutoScalingGroupDataSource_basic
=== PAUSE TestAccAwsAutoScalingGroupDataSource_basic
=== CONT  TestAccAwsAutoScalingGroupDataSource_basic
--- PASS: TestAccAwsAutoScalingGroupDataSource_basic (49.77s)
PASS
ok      github.com/terraform-providers/terraform-provider-aws/aws       49.786s

@ghost ghost added size/XL Managed by automation to categorize the size of a PR. provider Pertains to the provider itself, rather than any interaction with AWS. documentation Introduces or discusses updates to documentation. service/autoscaling Issues and PRs that pertain to the autoscaling service. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure. labels Dec 14, 2018
@deberon
Copy link
Contributor Author

deberon commented Dec 14, 2018

And for the sake of consistency, here is the output of the acceptance testing after my second commit:

$ TESTARGS="-run TestAccAwsAutoScalingGroupDataSource_basic" make testacc
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./... -v -parallel 20 -run TestAccAwsAutoScalingGroupDataSource_basic -timeout 120m
?       github.com/terraform-providers/terraform-provider-aws   [no test files]
=== RUN   TestAccAwsAutoScalingGroupDataSource_basic
=== PAUSE TestAccAwsAutoScalingGroupDataSource_basic
=== CONT  TestAccAwsAutoScalingGroupDataSource_basic
--- PASS: TestAccAwsAutoScalingGroupDataSource_basic (49.55s)
PASS
ok      github.com/terraform-providers/terraform-provider-aws/aws       49.565s

@deberon deberon changed the title Added the aws_autoscaling_group data source for more detailed informa… Add new data source type aws_autoscaling_group Dec 14, 2018
Copy link
Member

@bflad bflad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @deberon 👋 Thanks for submitting this! I left some initial feedback below. Please let us know if you have any questions or do not have time to implement these items.

aws/data_source_aws_autoscaling_group.go Outdated Show resolved Hide resolved
aws/data_source_aws_autoscaling_group.go Outdated Show resolved Hide resolved
aws/data_source_aws_autoscaling_group.go Outdated Show resolved Hide resolved
aws/data_source_aws_autoscaling_group.go Outdated Show resolved Hide resolved
aws/data_source_aws_autoscaling_group.go Outdated Show resolved Hide resolved
website/docs/d/autoscaling_group.html.markdown Outdated Show resolved Hide resolved
aws/data_source_aws_autoscaling_group.go Outdated Show resolved Hide resolved
aws/data_source_aws_autoscaling_group.go Outdated Show resolved Hide resolved
aws/data_source_aws_autoscaling_group_test.go Outdated Show resolved Hide resolved
@bflad bflad added the waiting-response Maintainers are waiting on response from community or contributor. label Dec 19, 2018
@deberon
Copy link
Contributor Author

deberon commented Dec 19, 2018

Wow! It was my first time writing in Go, could you tell? How do you prefer me to submit my patches? Should I squash the newer commits or do you guys like the verbose history? Thanks for the friendly feedback I will get started on it immediately.

@ghost ghost removed the waiting-response Maintainers are waiting on response from community or contributor. label Dec 19, 2018
@bflad
Copy link
Member

bflad commented Dec 19, 2018

It was my first time writing in Go, could you tell?

Hah, I don't judge, but you did pretty good considering that new fact to me. Setting up a Go environment for the first time, compiling a large project like this one, and developing/running integration testing is no small feat. Great job!

How do you prefer me to submit my patches? Should I squash the newer commits or do you guys like the verbose history?

Verbose history or squashing is fine. We'll merge the commits either which way.

@bflad bflad added waiting-response Maintainers are waiting on response from community or contributor. new-data-source Introduces a new data source. labels Dec 19, 2018
…e names to fall in line with the attributes of the autoscaling_group resource
@deberon
Copy link
Contributor Author

deberon commented Dec 21, 2018

==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./... -v -parallel 20 -run TestAccAwsAutoScalingGroupDataSource_basic -timeout 120m
? github.com/terraform-providers/terraform-provider-aws [no test files]
=== RUN TestAccAwsAutoScalingGroupDataSource_basic
=== PAUSE TestAccAwsAutoScalingGroupDataSource_basic
=== CONT TestAccAwsAutoScalingGroupDataSource_basic
--- PASS: TestAccAwsAutoScalingGroupDataSource_basic (51.30s)
PASS
ok github.com/terraform-providers/terraform-provider-aws/aws (cached)

Also I see there's a conflict on website/aws.erb. I tried to lay my changes over the most current master copy but it's a popular repo.

@ghost ghost removed the waiting-response Maintainers are waiting on response from community or contributor. label Dec 21, 2018
@bflad bflad added this to the v1.54.0 milestone Dec 21, 2018
Copy link
Member

@bflad bflad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, @deberon, thanks! 🚀

--- PASS: TestAccAwsAutoScalingGroupDataSource_basic (43.93s)

@bflad bflad merged commit d8ffd5f into hashicorp:master Dec 21, 2018
bflad added a commit that referenced this pull request Dec 21, 2018
@bflad
Copy link
Member

bflad commented Dec 21, 2018

This has been released in version 1.54.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 1, 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 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Introduces or discusses updates to documentation. new-data-source Introduces a new data source. provider Pertains to the provider itself, rather than any interaction with AWS. service/autoscaling Issues and PRs that pertain to the autoscaling service. size/XL Managed by automation to categorize the size of a PR. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: Data Source for : autoscaling_group
2 participants