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

New Resource: ECR Lifecycle Policy #1869

Closed
wants to merge 196 commits into from

Conversation

atsushi-ishibashi
Copy link
Contributor

arr-dev and others added 12 commits October 1, 2017 12:17
Fixes: hashicorp#1588

```
% make testacc TEST=./aws TESTARGS='-run=TestAccAWSALBTargetGroup_'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -run=TestAccAWSALBTargetGroup_ -timeout 120m
=== RUN   TestAccAWSALBTargetGroup_basic
--- PASS: TestAccAWSALBTargetGroup_basic (73.55s)
=== RUN   TestAccAWSALBTargetGroup_namePrefix
--- PASS: TestAccAWSALBTargetGroup_namePrefix (72.26s)
=== RUN   TestAccAWSALBTargetGroup_generatedName
--- PASS: TestAccAWSALBTargetGroup_generatedName (74.94s)
=== RUN   TestAccAWSALBTargetGroup_changeNameForceNew
--- PASS: TestAccAWSALBTargetGroup_changeNameForceNew (129.78s)
=== RUN   TestAccAWSALBTargetGroup_changeProtocolForceNew
--- PASS: TestAccAWSALBTargetGroup_changeProtocolForceNew (151.66s)
=== RUN   TestAccAWSALBTargetGroup_changePortForceNew
--- PASS: TestAccAWSALBTargetGroup_changePortForceNew (129.73s)
=== RUN   TestAccAWSALBTargetGroup_changeVpcForceNew
--- PASS: TestAccAWSALBTargetGroup_changeVpcForceNew (112.48s)
=== RUN   TestAccAWSALBTargetGroup_tags
--- PASS: TestAccAWSALBTargetGroup_tags (124.72s)
=== RUN   TestAccAWSALBTargetGroup_updateHealthCheck
--- PASS: TestAccAWSALBTargetGroup_updateHealthCheck (122.61s)
=== RUN   TestAccAWSALBTargetGroup_updateSticknessEnabled
--- PASS: TestAccAWSALBTargetGroup_updateSticknessEnabled (170.23s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	1161.982s
```
It’s a valid combination to set `self false` and `cidr_blocks`. So changing the validation to reflect this as currently you cannot actively set `self` to `false` without getting a validation error.
… documentation to fall in line with changes in AWS
AWS has updated the idle connection timeout to 4000 seconds (http://docs.aws.amazon.com/elasticloadbalancing/latest/classic/config-idle-timeout.html).  This change is to bring the provider current with that setting.
@Ninir Ninir added the enhancement Requests to existing resources that expand the functionality or scope. label Oct 12, 2017
iiska and others added 17 commits October 12, 2017 12:16
Link to Elastic Beanstalk Supported Platforms documentation for solution
stack name reference.
Update ELB create to align with 4000s idle timeout
website: Add AWS documentation link to elastic_beanstalk_solution_stack datasource
resource/aws_spot_fleet_request: Add support for instance_interruption_behaviour
…ry_stream (hashicorp#1605)

* First attempt at resolving hashicorp#1601

* Removing unused method

* Add acceptance test for aws_kinesis_firehose_delivery_stream using Kinesis Stream as source

* Updated "source_configuration" to be "kinesis_source_configuration"
Added missing attribute in db_option_group documentation
aws_instance resources export ipv6_addresses which may be used like:
```
 value  = "${element(aws_instance.consul_server.*.ipv6_addresses[0], count.index)}"
```
…-route-waiters

r/vpn_connection_route: Wait until route is available/deleted
radeksimko and others added 11 commits October 27, 2017 14:59
…dtrail_service_account

Add 'aws_cloudtrail_service_account' data source
…tance-subnet-id

r/spot_instances_request: Fix failing acceptance test (SubnetId)
…latest

vendor: Update hashicorp/terraform to latest
* some test config cleanups

* refactor to re-use the same config
@atsushi-ishibashi
Copy link
Contributor Author

@radeksimko I reflect all of your feedbacks, rebase master and run testacc.

make testacc TEST=./aws TESTARGS='-run=TestAccAWSEcrLifecyclePolicy_basic'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -run=TestAccAWSEcrLifecyclePolicy_basic -timeout 120m
=== RUN   TestAccAWSEcrLifecyclePolicy_basic
--- PASS: TestAccAWSEcrLifecyclePolicy_basic (36.22s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	36.287s

@atsushi-ishibashi
Copy link
Contributor Author

Sorry I made a mistake when rebasing... 🙇 @radeksimko

@atsushi-ishibashi
Copy link
Contributor Author

atsushi-ishibashi commented Oct 29, 2017

@radeksimko I made a mistake when rebasing and it seems difficult for me to fix it...
I created new PR #2096 so could you proceed to review it and close this PR? I'm sorry for bothering you:bow:

@ghost
Copy link

ghost commented Apr 10, 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 10, 2020
@breathingdust breathingdust removed the waiting-response Maintainers are waiting on response from community or contributor. label Sep 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-resource Introduces a new resource.
Projects
None yet
Development

Successfully merging this pull request may close these issues.