Commit Graph

9271 Commits

Author SHA1 Message Date
Martin Atkins 2e33f5311c Treat each consul key as having its own lifecycle
Previously this resource managed the set of keys as a whole rather than
the individual keys, and so it was unable to recognize when a particular
managed key is removed and delete just that one key from Consul.

Here this is addressed by recognizing that each key actually has its own
lifecycle, and detecting when individual keys are added and removed
without replacing the entire consul_keys instance.

Additionally this restores the behavior of updating the "value" attribute
on read, but restricts it only to blocks that already had a value so as
to avoid the quirkiness seen previously when we updated blocks that were
intended to be read-only. Updating the value is important now, because we
rely on this to detect and repair discrepancies between values stored in
Consul and values given in the configuration.

This change produces a change in the handling of the "delete" attribute.
Before it was considered only when the entire consul_keys resource was
deleted, but now it is considered also when a particular key block is
removed from within a resource.
2016-03-10 07:52:43 -08:00
Martin Atkins df2ce588bc Specialized client for interacting with Consul keys
This deals with some of the quirks of interacting with the Consul API,
with the goal of making the consul_keys resource implementation, and
later the consul_keys data source, less noisy to read.
2016-03-10 07:51:27 -08:00
Paul Frybarger d8c9e448d4 Fix create/delete statuses in load balancing resources for openstack provider. 2016-03-10 09:35:38 -06:00
Clint 99ddea503d Merge pull request #5495 from nicolai86/bugfix/aws_security_group-protocol-case-sensitivity
provider/aws Always transform aws_security_group protocol to lower case
2016-03-10 09:34:55 -06:00
Clint 5b6637e607 Merge pull request #5512 from serbaut/cs-vpc-domain
provider/cloudstack docs: add vpc.network_domain
2016-03-10 09:26:42 -06:00
Clint eb7f821213 Update CHANGELOG.md 2016-03-10 08:57:16 -06:00
Clint 239b3e4f5f Merge pull request #5533 from hashicorp/pr-5184
provider/aws: Fix EC2 Classic SG Rule issue
2016-03-10 08:56:29 -06:00
Paul Hinze 0b9537b942 Update CHANGELOG.md 2016-03-10 08:30:55 -06:00
Paul Hinze cadd0c5767 Merge pull request #5553 from TimeIncOSS/b-fix-retry-err-bug
helper/resource: Error shouldn't be returned in case of success
2016-03-10 08:28:56 -06:00
Radek Simko 034287fdc2 helper/resource: Error shouldn't be returned in case of success 2016-03-10 14:14:14 +00:00
Radek Simko 8582f4df9f helper/resource: Fix TestRetry 2016-03-10 14:13:23 +00:00
Radek Simko 9f6b487791 provider/aws: Document lambda_function updates 2016-03-10 12:31:36 +00:00
Radek Simko d777141a7b provider/aws: Add tests for Lambda function updates 2016-03-10 12:31:35 +00:00
Radek Simko fdc21aad25 provider/aws: Simplify update logic for Lambda function 2016-03-10 12:31:35 +00:00
Chris Marchesi b5c7521f52 aws_lambda_function: Force code update on SHA sum mismatch 2016-03-10 12:31:35 +00:00
Rob Zienert 4aeb5bb8e9 Adding support for Lambda function updates 2016-03-10 12:31:35 +00:00
James Nugent 2dc1c022d3 Merge pull request #5548 from afomi/minor-spelling
docs: fix spelling
2016-03-10 04:02:59 +00:00
Kraig Amador 76b3c74a58 Providing a more informative error message when combining kms_key_id and etag as well as using aws_kms_key to test aws_s3_bucket's kms_key_id. 2016-03-09 19:06:36 -08:00
Ryan Wold 1d3ba5b2ef fix spelling 2016-03-09 17:10:16 -08:00
Paul Hinze 3508ff9bd2 Update CHANGELOG.md 2016-03-09 18:55:44 -06:00
Paul Hinze 5ece31dc88 Merge pull request #5501 from evandbrown/vpnval
provider/google: Validate VPN tunnel peer_ip
2016-03-09 18:55:10 -06:00
Paul Hinze c1f8dda863 Merge pull request #5497 from evandbrown/flakeygcs
provider/google: Address flaky GCS acceptance tests.
2016-03-09 18:48:44 -06:00
Paul Hinze 5abc514434 Update CHANGELOG.md 2016-03-09 18:31:00 -06:00
Paul Hinze d12944797a Merge pull request #3928 from TimeIncOSS/aws-kms
provider/aws: Add support for KMS
2016-03-09 18:30:06 -06:00
Paul Hinze f691b89b05 Update CHANGELOG.md 2016-03-09 18:18:04 -06:00
Paul Hinze f8c6781982 Merge pull request #5543 from hashicorp/phinze/retryerror-pointers-are-evil
builtin: Refactor resource.Retry to clarify return
2016-03-09 18:15:51 -06:00
Pablo Porto b3ecf8eb73 Fix missing double quotes in postgresql_database resource 2016-03-09 23:42:59 +00:00
Paul Hinze 108ccf0007 builtin: Refactor resource.Retry to clarify return
Change the `RetryFunc` from a plain `error` return type to a
specialized `RetryError` which must decide whether it is
retryable or not.

Add `RetryableError` / `NonRetryableError` factory functions that
callers are meant to use to build up these errors.

This makes it eminently clear whether or not a given error is
retryable from inside the client code.

Goal here is to _not_ change any behavior, simply reflect the
existing behavior with the new, clearer, API.
2016-03-09 17:37:56 -06:00
Paul Stack a9fd7b4f1a Update CHANGELOG.md 2016-03-10 00:29:55 +01:00
Paul Stack ab075bca60 Merge pull request #5193 from innossh/f-aws-instance-security-groups-updates
provider/aws: Support additional changes to security groups of instance without forcing new
2016-03-10 00:28:32 +01:00
Paul Stack 18df66176a Merge pull request #5546 from ricardclau/add_vsphere_folder_in_layout
Documentation: Link to vsphere_folder resource missing in vsphere provider layout
2016-03-10 00:27:09 +01:00
James Nugent 4ba230c56e Update CHANGELOG.md 2016-03-09 23:26:36 +00:00
James Nugent 7436cf7f89 Merge pull request #5376 from stack72/b-aws-ELBListner
provider/aws: Fixes the use of Uppercase chars in ELB Listeners
2016-03-09 23:18:56 +00:00
Ricard Clau 8e486985d3 vsphere_folder resource missing in vsphere provider layout 2016-03-09 23:16:12 +00:00
Paul Hinze ffa5db4f52 Merge pull request #5538 from hashicorp/phinze/retryerror-pointers-are-evil
provider/aws: Fix all pointer RetryError returns
2016-03-09 16:53:19 -06:00
Paul Stack a4c8d869d3 Update CHANGELOG.md 2016-03-09 23:30:27 +01:00
stack72 c5ea6d0fb9 Merge branch 'BSick7-b/4311' 2016-03-09 23:27:38 +01:00
Brad Sickles 3f41515d2b Fixing noop differences in aws_route. 2016-03-09 23:27:09 +01:00
Brad Sickles 16f94ed4b7 Added failing aws_route test that shows diff after apply. 2016-03-09 23:27:09 +01:00
clint shryock f96ec46681 update comments 2016-03-09 15:50:01 -06:00
James Nugent 13a3387996 Merge pull request #5539 from hashicorp/b-route-docs
Make aws_route and aws_route_table docs consistent
2016-03-09 21:16:44 +00:00
Clint 3679d7965e Update CHANGELOG.md 2016-03-09 15:08:34 -06:00
Clint d14920adaf Merge pull request #5515 from hashicorp/b-aws-acc-test-updates
provider/aws: Retry DB Creation on IAM propigation error
2016-03-09 15:08:13 -06:00
James Nugent 8d9126b8e3 Make aws_route and aws_route_table docs consistent
Fixes #5531.
2016-03-09 21:05:08 +00:00
clint shryock 23c42cd6cf Add test for flattenSecurityGroups 2016-03-09 15:00:30 -06:00
Paul Hinze de656942ae provider/aws: Fix all pointer RetryError returns
All of these RetryErrors were meant to fail right away, but instead
caused retry looping because the typecheck in the implementation of
`resource.Retry()` only catches the value type, and not the pointer
type.

Refs #5537
2016-03-09 14:59:55 -06:00
Paul Stack 0e50c7a6f4 Update CHANGELOG.md 2016-03-09 19:21:21 +00:00
Paul Stack edbf310d7a Merge pull request #5535 from Originate/mb-fix-rds-publicly-accessible-read
provider/aws: aws_db_instance doesn't read publicly_accessible from AWS
2016-03-09 19:18:55 +00:00
Maxime Bury 9e0bac6c9f Terraform should keep track of the publicly accessible attribute 2016-03-09 13:33:00 -05:00
ephemeralsnow 54cb5ffe00 provider/aws: Fix EC2 Classic SG Rule issue
Fixes an issue where security groups would fail to update after applying an
initial security_group, because we were improperly saving the id of the group
and not the name (EC2 Classic only).

This is a PR combining https://github.com/hashicorp/terraform/pull/4983 and
https://github.com/hashicorp/terraform/pull/5184 . It's majority
@ephemeralsnow's work.
2016-03-09 09:51:41 -06:00