Commit Graph

17313 Commits

Author SHA1 Message Date
James Bardin
2cffa25235 Add test to verify that Validation isn't called
The apply won't succeed because we don't have a valid plan, but this
verifies that providing a plan file prevents Validation.
2017-03-27 18:39:18 -04:00
James Bardin
8027fe9e08 Don't Validate if we have an execution plan
The plan file should contain all data required to execute the apply
operation. Validation requires interpolation, and the `file()`
interpolation function may fail if the module files are not present.
This is the case currently with how TFE executes plans.
2017-03-27 17:11:50 -04:00
Adam Dehnel
3e8a8c5e23 Just adding the id attribute (#13090) 2017-03-27 15:50:33 +03:00
Ján Dzurek
15abbe21c0 fixed modules docs typo (#13087) 2017-03-27 15:34:05 +03:00
Jake Champlin
c95f25a1d0 Merge pull request #13057 from hashicorp/b-getting-started-docs
docs: Update getting started docs to use a valid AMI in us-east-1, vpc
2017-03-27 08:06:43 -04:00
Paul Stack
0e9f84b30b Update CHANGELOG.md 2017-03-27 13:17:34 +03:00
Sergey
0ff734f157 provider/digitalocean: Support disk only resize (#13059)
Allow to resize a droplet permanently (i.e. apply disk resize)
if previously it was resized temporarily (CPU and RAM only).

Fixes: #13007

```
$ make testacc TEST=./builtin/providers/digitalocean TESTARGS='-run=TestAccDigitalOceanDroplet_ResizeOnlyDisk'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/03/25 03:54:23 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/digitalocean -v -run=TestAccDigitalOceanDroplet_ResizeOnlyDisk -timeout 120m
=== RUN   TestAccDigitalOceanDroplet_ResizeOnlyDisk
--- PASS: TestAccDigitalOceanDroplet_ResizeOnlyDisk (198.62s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/digitalocean	198.638s
```
2017-03-27 13:16:01 +03:00
Paul Stack
8735e936d4 Update CHANGELOG.md 2017-03-27 13:01:36 +03:00
Raphael Randschau
403ea9f6d7 provider/scaleway: work around parallel request limitation (#13045)
according to the official scaleway support, requests within the same session can
not be parallelized.

While I do not know for sure that this is a write-only limitation, I've
implemented it as a write-only limitation for now.

Previously requests like this would produce a 500 internal server error:

```
resource "scaleway_ip" "test_ip" {
  count = 2
}
```

now this limitation should be lifted, for all scaleway resources
2017-03-27 13:00:11 +03:00
Paul Stack
47c8fc456a Update CHANGELOG.md 2017-03-27 12:51:29 +03:00
Joe Topjian
0edbedd1a8 random_pet resource (#12903)
* vendor: adding golang-petname for random_pet resource

* provider/random: random_pet resource
2017-03-27 12:50:39 +03:00
Kit Ewbank
08c0ac68e9 Correct handling of network ACL default IPv6 ingress/egress rules. (#12835) 2017-03-27 12:42:49 +03:00
Paul Stack
9210222f3c Update CHANGELOG.md 2017-03-26 14:35:33 +03:00
Joe Topjian
fafd488a1c provider/openstack: 409 Response on Pool Create (#13074)
This commit accounts for a 409 response when a LBaaS v2 pool is
being created. Rather than error out, this should be considered a
pending state.
2017-03-26 14:35:01 +03:00
Paul Stack
d89055154f Update CHANGELOG.md 2017-03-26 12:36:06 +03:00
Joe Topjian
89905368b9 provider/openstack: Don't log the catalog (#13075)
The OS_DEBUG feature has worked out great, but frequent logging of
the service catalog during client initialization can make logging
very chatty. This commit omits the service catalog in the logs.
2017-03-26 12:35:40 +03:00
Paul Stack
86d599cb3c Update CHANGELOG.md 2017-03-25 23:37:19 +02:00
Paul Stack
25d128ca08 Update CHANGELOG.md 2017-03-25 22:44:05 +02:00
Jasmin Gacic
d899709004 Handling missing resources (#13053) 2017-03-25 22:43:41 +02:00
Paul Stack
e50fbfe779 Update CHANGELOG.md 2017-03-25 22:43:33 +02:00
Joe Topjian
9af4c1da67 provider/openstack: Fix monitor_id typo in LBaaS v1 Pool (#13069) 2017-03-25 22:42:18 +02:00
Paul Stack
bbc6ea6f6c Update CHANGELOG.md 2017-03-25 13:10:01 +02:00
Joe Topjian
de255d43d6 provider/openstack: Decprecating Instance Floating IP attribute (#13063)
This commit deprecates the floating_ip attributes from the
openstack_compute_instance_v2 resource. It is recommended to use
either the openstack_compute_floatingip_associate resource or
configure an openstack_networking_port_v2 resource with a floating
IP.
2017-03-25 13:09:35 +02:00
Paul Stack
a097a2ef5b Update CHANGELOG.md 2017-03-25 13:09:12 +02:00
Joe Topjian
de8e2d7f2b provider/openstack: Deprecating Instance Volume attribute (#13062)
This commit deprecates the volume attribute in the
openstack_compute_instance_v2 resource. It's recommended to either
use the block_device attribute or the openstack_compute_volume_attach_v2
resource.
2017-03-25 13:08:07 +02:00
Joe Topjian
ee0e23b242 Merge pull request #13060 from jtopjian/openstack-secgrouprule-timeout-fix
provider/openstack: Fixing typo in secgroup rule timeout test
2017-03-24 19:52:32 -06:00
Joe Topjian
0d3190fd8f provider/openstack: Fixing typo in secgroup rule timeout test 2017-03-25 01:34:42 +00:00
Sean Chittenden
32db4d184f Add consul check type to circonus_check resource (#13030)
* Add the `consul` check type to the `circonus_check` resource.

* Fix a tab-complete fail.

`Parse` != `Path`, but lexically close.

* Dept of 2nd thoughts: `s/service_name/service/g`
2017-03-24 15:05:19 -07:00
clint shryock
f6a3e7d2b2 docs: Update getting started docs to use a valid AMI in us-east-1, vpc
Supersedes #12932 , populating the entire guide with the correct AMI(s)
2017-03-24 15:51:56 -05:00
Clint
c5935ab1ad Update CHANGELOG.md 2017-03-24 14:06:17 -05:00
Jon Oden
c18ad6ed25 fix bool vs Compatibool ref used in flattenBackends (#12538)
* fix bool vs Compatobool value used in flattendBackends

* remove log line

* update test case
2017-03-24 14:05:29 -05:00
Paul Stack
f895302630 Update CHANGELOG.md 2017-03-24 19:50:17 +02:00
Paul Stack
6b44a1b9ba provider/aws: Only allow 1 value in alb_listener_rule condition (#13051)
Fixes: #12983

```
% make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSALBListenerRule_'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/03/24 19:31:26 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSALBListenerRule_ -timeout 120m
=== RUN   TestAccAWSALBListenerRule_basic
--- PASS: TestAccAWSALBListenerRule_basic (247.76s)
=== RUN   TestAccAWSALBListenerRule_multipleConditionThrowsError
--- PASS: TestAccAWSALBListenerRule_multipleConditionThrowsError (0.02s)
PASS
ok	github.com/hashicorp/terraform/builtin/providers/aws	247.815s
```
2017-03-24 19:49:27 +02:00
Paul Stack
812369e0cd Update CHANGELOG.md 2017-03-24 19:36:04 +02:00
Paul Stack
48a4b3deaa provider/aws: aws_network_acl_rule treat all and -1 for protocol the (#13049)
same

Fixes: #13012

```
% make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSNetworkAclRule_allProtocol'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/03/24 18:42:05 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSNetworkAclRule_allProtocol -timeout 120m
=== RUN   TestAccAWSNetworkAclRule_allProtocol
--- PASS: TestAccAWSNetworkAclRule_allProtocol (53.95s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	53.974s
```
2017-03-24 19:35:36 +02:00
Matthew Frahry
7c21b6ad49 Merge pull request #13047 from hashicorp/p-aws-route-53-zone
Fixes route53 test
2017-03-24 10:46:04 -06:00
Tom Harvey
68e32c165c Switching to a simpler ARM Template (#13043) 2017-03-24 18:34:40 +02:00
James Bardin
d5f81f8022 Merge pull request #13048 from hashicorp/jbardin/travis-consul
remove consul tests from travis runs
2017-03-24 12:30:37 -04:00
=
84a9bcde9a Fixes route53 test 2017-03-24 10:22:50 -06:00
James Bardin
96617b609b remove consul tests from travis runs
they have timing out too often
2017-03-24 12:14:05 -04:00
James Bardin
399715e1b7 Merge pull request #13046 from hashicorp/jbardin/GH-12892
reify the list values before validation
2017-03-24 12:13:49 -04:00
James Bardin
2e6a44d5ff reify the list values before validation
If the list was marked as computed, all values will be raw config
values. Fetch the individual keys from the config to get any known
values before validating.
2017-03-24 12:04:18 -04:00
James Bardin
99a12f5df3 interpolation strings were beeing validated
Interpolation strings for non-computed values in a list were being
passed to the schema's ValidateFunc.
2017-03-24 12:04:18 -04:00
Paul Stack
17f60ac682 Update CHANGELOG.md 2017-03-24 13:06:00 +02:00
demonwy
11768bcf5b provider alicloud:add new rds resource and some bugs fix (#12913)
* add new rds resource and some bugs fix

* add docs

* fix validator conflix

fix validator conflix
2017-03-24 13:04:56 +02:00
Paul Stack
4fe7ee16e6 provider/aws: Migrate the state for AWS VPC after IPv6 changes (#13041)
Fixes: #13035

It was pointed out in the issue that the addition of a new parameter
with a default value AND a ForceNew: true is causing Terraform to try
and recreate the VPC

This PR migrates the state to add the default value of false for `assign_generated_ipv6_cidr_block`

```
% make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAWSVpcMigrateState'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/03/24 12:51:41 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAWSVpcMigrateState -timeout 120m
=== RUN   TestAWSVpcMigrateState
2017/03/24 12:52:26 [INFO] Found AWS VPC State v0; migrating to v1
2017/03/24 12:52:26 [DEBUG] Attributes before migration: map[string]string{"assign_generated_ipv6_cidr_block":"true"}
2017/03/24 12:52:26 [DEBUG] Attributes after migration: map[string]string{"assign_generated_ipv6_cidr_block":"false"}
2017/03/24 12:52:26 [INFO] Found AWS VPC State v0; migrating to v1
2017/03/24 12:52:26 [DEBUG] Attributes before migration: map[string]string{}
2017/03/24 12:52:26 [DEBUG] Attributes after migration: map[string]string{"assign_generated_ipv6_cidr_block":"false"}
--- PASS: TestAWSVpcMigrateState (0.00s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	0.024s
```
2017-03-24 13:02:11 +02:00
Naveen Nalam
fc04f4b5a0 Fix zone value in example (#13023)
As experienced during testing, the value of the zone needs to be the zone name, and not the NS1 internal ID.
2017-03-24 12:29:27 +02:00
Paul Stack
8b548fe30d Update CHANGELOG.md 2017-03-23 19:17:28 +02:00
Paul Stack
1a80044397 provider/aws: Specify that aws_network_acl_rule requires a cidr block (#13013)
Fixes: #13011

```
% make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSNetworkAclRule_'           2 ↵ ✚
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/03/23 17:45:25 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSNetworkAclRule_ -timeout 120m
=== RUN   TestAccAWSNetworkAclRule_basic
--- PASS: TestAccAWSNetworkAclRule_basic (41.10s)
=== RUN   TestAccAWSNetworkAclRule_missingParam
--- PASS: TestAccAWSNetworkAclRule_missingParam (21.21s)
=== RUN   TestAccAWSNetworkAclRule_ipv6
--- PASS: TestAccAWSNetworkAclRule_ipv6 (53.00s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	115.333s
```
2017-03-23 19:16:56 +02:00
Peter McAtominey
6a13d70d40 Update CHANGELOG.md 2017-03-23 16:45:32 +00:00