Ninir
048a932abd
provider/aws: Added Lambda function guard when needed attributes are not set ( #10663 )
2016-12-12 15:12:56 +00:00
Jan Schumann
a01a93067f
tests for #5907 ( #10669 )
2016-12-12 14:42:04 +00:00
Paul Stack
d133cd82e8
Update CHANGELOG.md
2016-12-12 12:57:33 +00:00
Ninir
3f5e12ff7e
provider/aws: Set the DBName when restoring from a snapshot ( #10664 )
2016-12-12 12:56:56 +00:00
Sean Chittenden
1a93309e50
Expand postgresql_role support to include all known options.
2016-12-12 01:37:00 -08:00
Sean Chittenden
4c6c52ee8d
Nuke some whitespace.
2016-12-12 01:36:14 -08:00
Sean Chittenden
6deb61b0cf
Enable import support for PostgreSQL's extensions.
2016-12-12 01:33:02 -08:00
Sean Chittenden
e36827c5fe
Change the default for `valid_until` to `infinity` to match the default.
2016-12-12 01:18:13 -08:00
Paul Stack
a2db4ad4bb
Update CHANGELOG.md
2016-12-12 09:15:14 +00:00
Joe Topjian
fd40eb3421
provider/openstack: More Import and Region Fixes ( #10662 )
...
This has a few more import and region fixes from #10509
2016-12-12 09:14:38 +00:00
Sander van Harmelen
ae002d99c4
Update CHANGELOG.md
2016-12-12 10:13:53 +01:00
Sander van Harmelen
0af3d071d2
Update CHANGELOG.md
2016-12-12 10:13:34 +01:00
Sean Chittenden
e9dc92c18d
Change the default for `inherit` from `false` to `true` to match PostgreSQL.
2016-12-12 01:11:07 -08:00
Sean Chittenden
b576a3eda1
Rename variable from `roleSuperUserAttr` to `roleSuperuserAttr`.
2016-12-12 01:07:26 -08:00
Sander van Harmelen
bad3a876ca
provider/cloudstack: add support for multiple NICs with port forwards and set network_domain for networks ( #10638 )
...
* Add support for multiple NICs with port forwards
* Fix issue #9801
2016-12-12 10:06:42 +01:00
Sean Chittenden
15cd542392
Remove a duplicate `connect_timeout` from a rebase+stash.
2016-12-12 00:53:06 -08:00
Rui Gonçalves
5016a56fd4
provider/statuscake: fix StatusCake tests ( #10660 )
...
* Update vendored statuscake SDK
* Set all attributes when upserting statuscake tests
2016-12-11 19:41:49 +00:00
Mitchell Hashimoto
42edd22566
helper/variables: address go vet
2016-12-10 20:32:50 -05:00
Mitchell Hashimoto
b346ba32d1
terraform: dependent provider resources are destroyed first in modules
...
This extends the prior commit to also verify (and fix) that resources of
dependent providers are destroyed first even when they're within
modules.
2016-12-10 20:22:12 -05:00
Mitchell Hashimoto
14d079f914
terraform: destroy resources in dependent providers first
...
Fixes #4645
This is something that never worked (even in legacy graphs), but as we
push forward towards encouraging multi-provider usage especially with
things like the Vault data source, I want to make sure we have this
right for 0.8.
When you have a config like this:
```
resource "foo_type" "name" {}
provider "bar" { attr = "${foo_type.name.value}" }
resource "bar_type" "name" {}
```
Then the destruction ordering MUST be:
1. `bar_type`
2. `foo_type`
Since configuring the client for `bar_type` requires accessing data from
`foo_type`. Prior to this PR, these two would be done in parallel. This
properly pushes forward the dependency.
There are more cases I want to test but this is a basic case that is
fixed.
2016-12-10 20:11:24 -05:00
Mitchell Hashimoto
3ba9720b3e
config: validate invalid variable keys
...
Fixes #9416
A simple change to verify that only valid keys for `variable` blocks are
used.
2016-12-10 19:27:01 -05:00
Mitchell Hashimoto
cabcc4b5b8
terraform: allow indexing into a computed list for multi-count resources
...
Fixes #8695
When a list count was computed in a multi-resource access
(foo.bar.*.list), we were returning the value as empty string. I don't
actually know the histocal reasoning for this but this can't be correct:
we must return unknown.
When changing this to unknown, the new tests passed and none of the old
tests failed. This leads me further to believe that the return empty
string is probably a holdover from long ago to just avoid crashes or
UUIDs in the plan output and not actually the correct behavior.
2016-12-10 19:17:29 -05:00
Mitchell Hashimoto
5d684b399c
config: error when loading multiple lifecycle blocks
...
Fixes #8776
This introduces an error when multiple `lifecycle` blocks exist on a
resource in the configuration.
2016-12-10 18:52:13 -05:00
James Turnbull
c02b2471d6
Fixed typoes in Consul key_prefix docs ( #10655 )
2016-12-10 21:45:47 +00:00
Mitchell Hashimoto
da418b2582
website: note about map/var merging
2016-12-10 15:05:58 -05:00
Mitchell Hashimoto
83e72b0361
helper/variables: ParseInput for consistent parsing
2016-12-10 14:44:17 -05:00
Mitchell Hashimoto
77efacf30e
command: use helper/variables for flags and parsing
2016-12-10 14:30:40 -05:00
Mitchell Hashimoto
39e3d8ea9b
helper/variables: helpers for working with vars
2016-12-10 14:30:36 -05:00
KOJIMA Kazunori
e6b1cda7db
documentation/aws: Fix example syntax ( #10651 )
2016-12-10 09:36:06 +00:00
Sean Chittenden
e2448473cb
Clean up the DatabaseCreate call.
2016-12-10 12:32:21 +11:00
Sean Chittenden
db5d7b0438
Style nit
2016-12-10 12:32:21 +11:00
Sean Chittenden
bfc2a2d42f
Commit WIP re: updated postgresql_role provider.
...
*Read() and *Update() still need to be updated.
2016-12-10 12:32:20 +11:00
Sean Chittenden
2e529146a5
Remove unused code.
2016-12-10 12:32:20 +11:00
Sean Chittenden
6b540ecb55
Don't use d.GetOk() when the zero value is a required attribute.
...
Add "pathological" test.
2016-12-10 12:32:20 +11:00
Sean Chittenden
37fdc958b3
Always remove the IS_TEMPLATE attribute before dropping a database.
2016-12-10 12:32:20 +11:00
Sean Chittenden
e9b2b38288
Remove SetId() call from *Read(), this isn't required for import to work.
2016-12-10 12:32:20 +11:00
Sean Chittenden
f253fc9eff
Commit miss earlier, fix syntax from unstaged commit.
2016-12-10 12:32:20 +11:00
Sean Chittenden
5280c37bea
`postgresql_database` resource provider should now be feature complete.
...
* Add support to import databases. See docs.
* Add support for renaming databases
* Add support for all known PostgreSQL database attributes, including:
* "allow_connections"
* "lc_ctype"
* "lc_collate"
* "connection_limit"
* "encoding"
* "is_template"
* "owner"
* "tablespace_name"
* "template"
2016-12-10 12:32:20 +11:00
Sean Chittenden
59f4ad6fd1
Alpha sort the resources in the PostgreSQL map.
2016-12-10 12:32:19 +11:00
Sean Chittenden
300ef2bc97
Add `connect_timeout` support to the PostgreSQL provider.
2016-12-10 12:32:19 +11:00
Sean Chittenden
3750bf7af2
Depreciate the PostgreSQL provider's `ssl_mode` option in favor of `sslmode`.
...
Both libpq(3) and github.com/lib/pq both use `sslmode`. Prefer this vs
the non-standard `ssl_mode`. `ssl_mode` is supported for compatibility
but should be removed in the future.
Changelog: yes
2016-12-10 12:32:19 +11:00
Sean Chittenden
a200899d93
Allow the PostgreSQL provider to talk to a non-default database.
2016-12-10 12:32:19 +11:00
Sean Chittenden
602a908d7b
Add a `fallback_application_name` to the PostgreSQL DSN.
2016-12-10 12:32:19 +11:00
Sean Chittenden
44d907a3de
Revert "Prefer PGHOSTADDR over PGHOST, if set." lib/pq doesn't support PGHOSTADDR.
...
d8eeeb8bae/conn.go (L1851)
This reverts commit 26a2a974f2b2f2cefd41b426b8469fa3ea72b33e.
2016-12-10 12:32:19 +11:00
Sean Chittenden
55061d190a
Prefer PGHOSTADDR over PGHOST, if set.
2016-12-10 12:32:19 +11:00
Sean Chittenden
3779dfffa9
Use a string instead of the `%t` modifier for printing a bool in SQL
2016-12-10 12:32:19 +11:00
Sean Chittenden
5b66bf0745
ForceNew is required when changing the locale, ctype, and encoding.
...
This will cause someone some grief. TODO: Figure out how to prevent
someone from blowing off their foot if they twiddle this after the
fact.
2016-12-10 12:32:18 +11:00
Sean Chittenden
547dcf27e1
Decorate the computed attribute where appropriate
2016-12-10 12:32:18 +11:00
Sean Chittenden
242405bdf1
Factor out the validate function for connection limits
2016-12-10 12:32:18 +11:00
Sean Chittenden
02dea2edd9
Add missing descriptions to owner and name
2016-12-10 12:32:18 +11:00