3b5c8a44ea
Merge pull request #685 from svanharmelen/f-add-test-for-diffing-diffs
...
core: adding a test for the change in PR #681 (diffing diffs)
2014-12-16 10:23:42 -08:00
8347569396
Adding a test for the change in PR #681
...
Should have already done that together with that PR…
2014-12-16 19:15:07 +01:00
2ba827ee3a
update CHANGELOG
2014-12-16 09:16:11 -08:00
af52e744da
Merge pull request #682 from svanharmelen/f-provider/aws-fix-tests
...
provider/aws: updating the tests and refactoring the code a little
2014-12-16 09:14:41 -08:00
e39aaa6604
Merge pull request #678 from hashicorp/track-map-element-count
...
helper/schema: track map element counts
2014-12-16 09:09:35 -08:00
df05483cf6
helper/schema: test pass from SvH
2014-12-16 09:05:16 -08:00
ef60ac5b8c
terraform: update comment
2014-12-16 08:24:21 -08:00
e5f07567c6
terraform: module input to another module doesn't error [GH-659]
2014-12-16 08:23:19 -08:00
9797881ed8
Updating the tests and refactoring the code a little
...
There was an error in the goamz package the prevented updating the
availability zones correctly. So PR #181 should be merged before this
one can be merged…
2014-12-16 15:21:25 +01:00
e1201f079a
Merge pull request #681 from svanharmelen/f-core-fix-diffing-diffs
...
core: adding a small fix to the Same() func (diffing diffs)
2014-12-16 15:19:03 +01:00
d24ab685d0
Adding a small fix to the Same() func
...
This is a little tricky, but when a diff contains a computed list or
set that can only be interpolated after the apply command has created
the dependant resources, it could turn out that the result is actually
the same as the existing state which would remove the key from the diff
2014-12-16 13:58:31 +01:00
462cc24ce6
Merge pull request #680 from svanharmelen/f-provider/aws-update-aws-elb-resource
...
provider/aws: updating the resource to use a set instead of a list
2014-12-16 13:19:46 +01:00
3aeba87e38
Updating the resource to use a set instead of a list
...
By using a set for the availability zones, you can use things like
`availability_zones = ["${aws_instance.web.*.availability_zone}"]`
where is very likely multiple of the same zones will be added to the
set. If you use a list here, the list will say it’s changed (even if
you add the same zone) which will force a new resource.
2014-12-16 13:13:59 +01:00
7e091dd7bd
Merge pull request #677 from svanharmelen/f-provider-aws-fix-route53-toggling
...
provider/aws: fixing the flipflop problem for the aws_route53 resource
2014-12-16 11:22:54 +01:00
a7667986f9
Updated the code so it makes use of the available functionality
...
The `helper/schema` package already knows how to create a set from a
slice with set items. So let’s use that functionality…
2014-12-16 11:14:33 +01:00
40ca4c1c81
Merge pull request #676 from svanharmelen/f-fix-expand-bug
...
core: fix expand bug
2014-12-16 11:04:25 +01:00
8426f5140b
Update CHANGELOG
2014-12-15 22:14:08 -08:00
675eab11d3
Merge pull request #679 from hashicorp/b-fix-module=type
...
config: validate that module variables can go to ints, convert [GH-624]
2014-12-15 22:13:02 -08:00
d24082da1e
config: validate that module variables can go to ints, convert [GH-624]
2014-12-15 22:10:16 -08:00
913f9a923e
helper/schema: remove unnecessary code
2014-12-15 17:39:07 -08:00
e5877543b2
helper/schema: track map element counts
...
This adds "field.#" values to the state/diff with the element count of a
map. This fixes a major issue around not knowing when child elements are
computed when doing variable access of a computed map.
Example, if you have a schema like this:
"foo": &Schema{
Type: TypeMap,
Computed: true,
}
And you access it like this in a resource:
${type.name.foo.computed-field}
Then Terraform will error that "field foo could not be found on resource
type.name". By adding that "foo.#" is computed, Terraform core will pick
up that it WILL exist, so its okay.
2014-12-15 17:35:16 -08:00
41dabd7abc
Update CHANGELOG
2014-12-15 16:41:09 -08:00
4f70630c52
Merge pull request #525 from pmoust/patch-2
...
provider/aws: update to support more regions
2014-12-15 16:40:35 -08:00
43a30e7d13
Finishing up the needed changes
...
Also executed the acceptance tests successfully now.
2014-12-16 00:59:47 +01:00
ee7b33acf4
Fixing the flipflop problem
...
Actually meant as a PoC, but it works perfectly so let’s just merge
this…
2014-12-16 00:40:43 +01:00
5987ce30f9
up version for dev
2014-12-15 15:36:19 -08:00
935bc08003
update CHANGELOG
2014-12-15 15:36:07 -08:00
c2385a5012
Merge pull request #632 from KushalP/master
...
config: do not read temporary editor files, fixes #548
2014-12-15 15:07:03 -08:00
528167513b
Looking at issue #639 this seems to be the root cause
...
After fixing this part, there also needs to be made a change to the
example given or the schema of aws_elb.
2014-12-15 23:38:46 +01:00
820476ef61
Merge pull request #675 from svanharmelen/f-fix-typo/style
...
Fixed a type and reordered the case statements
2014-12-15 23:11:05 +01:00
40fa6c3aaa
Fixed a type and reordered the case statements
...
Making in more inline with the existing code…
2014-12-15 23:02:16 +01:00
6a663796d5
Merge pull request #663 from svanharmelen/f-fix/change-set-logic
...
core: refactoring the way sets work internally v2
2014-12-15 13:38:00 -08:00
f6bfbac3d1
website: Add link to heroku_cert in the docs
2014-12-15 12:14:00 -08:00
f923368fc5
Merge pull request #672 from plan3-labs/heroku_cert_support
...
Heroku SSL certificate support
2014-12-15 12:12:57 -08:00
9f0c8874b9
Merge pull request #674 from svanharmelen/f-make-func-names-consistent
...
Making all function names consistent
2014-12-15 15:57:02 +01:00
4ace4865d7
Making all function names consistent
...
Before all providers were using the helper.Schema approach the helper
function had these names. Now they all use names consistent with the Go
naming conventions except for these last few…
2014-12-15 15:26:17 +01:00
765cd1f494
Heroku SSL support: fixes after code review of @svanharmelen
2014-12-15 15:10:59 +01:00
3c0e9970e5
Heroku SSL certificate support added
2014-12-15 12:21:47 +01:00
6098809ec5
Merge pull request #665 from gosuri/master
...
Fix for #664 , crashed due to ig.Attachements being 0
2014-12-14 18:15:27 -08:00
5535767989
Merge pull request #666 from donjohnson/patch-1
...
Usage info for graph command was actually that of output
2014-12-14 17:43:30 -08:00
9b9d147585
Setting the VPC id to blank for unattached igws
2014-12-14 14:05:38 -08:00
cbd913259a
Merge pull request #667 from svanharmelen/f-fix-schema-aws-internet-gateway
...
provider/aws: fixing the aws_internet_gateway resource
2014-12-14 12:42:13 +01:00
27a97c4498
Small gofmt update...
2014-12-14 12:32:17 +01:00
d9af954c60
provider/aws: fixing the aws_internet_gateway resource
...
The resource is build so it can attach and detach the Internet Gateway
from a VPC, but as the schema has `Required` and `ForceNew` both set
to `true` for the vpc_id field it will never use these capabilities.
2014-12-14 12:20:59 +01:00
e02c62347d
Usage info for graph command was actually that of output
2014-12-14 17:21:43 +09:00
3883e47902
Fix for #664 , crashed due to ig.Attachements being 0
2014-12-13 22:14:23 -08:00
83c760fcb3
core: refactoring the way sets work internally v2
...
This is a refactored solution for PR #616 . Functionally this is still
the same change, but it’s implemented a lot cleaner with less code and
less changes to existing parts of TF.
2014-12-12 23:21:20 +01:00
69b2c245dd
Merge pull request #662 from svanharmelen/master
...
core: fixing new test
2014-12-12 15:44:49 +01:00
4c23019ab7
core: fixing new test
...
Removed the ‘hashcode’ function so we didn’t need to import it here,
but also removed the needed type assertion.
2014-12-12 15:42:01 +01:00
db640cb1dc
Merge pull request #661 from svanharmelen/f-fix-difflist-logic-error
...
core: fixing a small logic bug in diffList
2014-12-12 15:31:22 +01:00
Mitchell Hashimoto