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

[blocked] provider/aws: Allow ordering of cache_behaviors in CF distribution #7395

Conversation

radeksimko
Copy link
Member

@radeksimko radeksimko commented Jun 28, 2016

Fixes #7253

Depends on (i.e. blocked by) #7485

Summary

  • cache_behavior + default_cache_behavior were changed from TypeSet to TypeList to reflect ordering given by the HCL config
  • allowed_methods + cached_methods were changed from TypeList to TypeSet because AWS API returns these in pretty much random order which is generating extra diffs + updates
  • state migration + test added

Test plan

make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSCloudFrontDistribution'
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSCloudFrontDistribution -timeout 120m
=== RUN   TestAccAWSCloudFrontDistribution_S3Origin
--- PASS: TestAccAWSCloudFrontDistribution_S3Origin (946.92s)
=== RUN   TestAccAWSCloudFrontDistribution_customOrigin
--- PASS: TestAccAWSCloudFrontDistribution_customOrigin (810.02s)
=== RUN   TestAccAWSCloudFrontDistribution_multiOrigin
--- PASS: TestAccAWSCloudFrontDistribution_multiOrigin (896.46s)
=== RUN   TestAccAWSCloudFrontDistribution_noOptionalItemsConfig
--- PASS: TestAccAWSCloudFrontDistribution_noOptionalItemsConfig (934.08s)
=== RUN   TestAccAWSCloudFrontDistribution_noCustomErrorResponseConfig
--- PASS: TestAccAWSCloudFrontDistribution_noCustomErrorResponseConfig (955.18s)
=== RUN   TestAccAWSCloudFrontDistribution_cacheBehaviorPrecedence
--- PASS: TestAccAWSCloudFrontDistribution_cacheBehaviorPrecedence (1015.78s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    5558.474s

@radeksimko radeksimko force-pushed the b-aws-cloudfront-behaviour-precedence branch 2 times, most recently from ce4efd3 to a3df330 Compare June 29, 2016 18:04
@radeksimko radeksimko changed the title [blocked] provider/aws: Allow ordering of cache_behaviors in CF distribution provider/aws: Allow ordering of cache_behaviors in CF distribution Jun 29, 2016
@radeksimko radeksimko changed the title provider/aws: Allow ordering of cache_behaviors in CF distribution [blocked] provider/aws: Allow ordering of cache_behaviors in CF distribution Jul 8, 2016
@radeksimko radeksimko force-pushed the b-aws-cloudfront-behaviour-precedence branch 2 times, most recently from 5bf6f0b to b85a4ab Compare October 2, 2016 07:13
@adamb0mb
Copy link

adamb0mb commented Oct 3, 2016

I'd love to see this merged!

@PayscaleNateW PayscaleNateW mentioned this pull request Oct 12, 2016
@radeksimko radeksimko force-pushed the b-aws-cloudfront-behaviour-precedence branch from b85a4ab to 79b1765 Compare February 10, 2017 17:13
@radeksimko radeksimko force-pushed the b-aws-cloudfront-behaviour-precedence branch from 79b1765 to 3d25157 Compare February 13, 2017 10:19
@apparentlymart
Copy link
Contributor

As part of the the Terraform 0.10 release earlier this year, all of the Terraform providers were moved to their own repositories in the terraform-providers GitHub organization, and removed from the Terraform Core repository.

Unfortunately due to the fact that new issues and pull requests are being opened constantly, it was not possible for the various provider maintainers to merge all outstanding pull requests before this split, and there is no automatic way to migrate a pull request to a new repository.

As a result, this pull request can sadly no longer be applied as-is, and so I'm going to close it.

If someone has the time and motivation to apply same changes to the aws provider repository and open a new PR there, the maintainers of that provider should be able to review and merge it.

Thanks again for working on this, and sorry it was not able to be merged before the provider repository changes.

@ghost
Copy link

ghost commented Apr 6, 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 6, 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.

Allow configuring CloudFront cache_behavior precedence
3 participants