c602f024f4
Fix the remaining `postgresql_extension` unit test.
2016-12-12 13:40:49 -08:00
daa951434f
Teach postgresql_extension about schemas.
2016-12-12 13:15:57 -08:00
d1c9ebb6c2
Add PostgreSQL schema support
2016-12-12 13:11:47 -08:00
1a93309e50
Expand postgresql_role support to include all known options.
2016-12-12 01:37:00 -08:00
4c6c52ee8d
Nuke some whitespace.
2016-12-12 01:36:14 -08:00
6deb61b0cf
Enable import support for PostgreSQL's extensions.
2016-12-12 01:33:02 -08:00
e36827c5fe
Change the default for `valid_until` to `infinity` to match the default.
2016-12-12 01:18:13 -08:00
e9dc92c18d
Change the default for `inherit` from `false` to `true` to match PostgreSQL.
2016-12-12 01:11:07 -08:00
b576a3eda1
Rename variable from `roleSuperUserAttr` to `roleSuperuserAttr`.
2016-12-12 01:07:26 -08:00
15cd542392
Remove a duplicate `connect_timeout` from a rebase+stash.
2016-12-12 00:53:06 -08:00
e2448473cb
Clean up the DatabaseCreate call.
2016-12-10 12:32:21 +11:00
db5d7b0438
Style nit
2016-12-10 12:32:21 +11:00
bfc2a2d42f
Commit WIP re: updated postgresql_role provider.
...
*Read() and *Update() still need to be updated.
2016-12-10 12:32:20 +11:00
2e529146a5
Remove unused code.
2016-12-10 12:32:20 +11:00
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
37fdc958b3
Always remove the IS_TEMPLATE attribute before dropping a database.
2016-12-10 12:32:20 +11:00
e9b2b38288
Remove SetId() call from *Read(), this isn't required for import to work.
2016-12-10 12:32:20 +11:00
f253fc9eff
Commit miss earlier, fix syntax from unstaged commit.
2016-12-10 12:32:20 +11:00
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
59f4ad6fd1
Alpha sort the resources in the PostgreSQL map.
2016-12-10 12:32:19 +11:00
300ef2bc97
Add `connect_timeout` support to the PostgreSQL provider.
2016-12-10 12:32:19 +11:00
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
a200899d93
Allow the PostgreSQL provider to talk to a non-default database.
2016-12-10 12:32:19 +11:00
602a908d7b
Add a `fallback_application_name` to the PostgreSQL DSN.
2016-12-10 12:32:19 +11:00
44d907a3de
Revert "Prefer PGHOSTADDR over PGHOST, if set." lib/pq doesn't support PGHOSTADDR.
...
d8eeeb8bae/conn.go (L1851)
2016-12-10 12:32:19 +11:00
55061d190a
Prefer PGHOSTADDR over PGHOST, if set.
2016-12-10 12:32:19 +11:00
3779dfffa9
Use a string instead of the `%t` modifier for printing a bool in SQL
2016-12-10 12:32:19 +11:00
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
547dcf27e1
Decorate the computed attribute where appropriate
2016-12-10 12:32:18 +11:00
242405bdf1
Factor out the validate function for connection limits
2016-12-10 12:32:18 +11:00
02dea2edd9
Add missing descriptions to owner and name
2016-12-10 12:32:18 +11:00
deb91f61ce
Update the descriptions to mostly match the official PostgreSQL docs.
2016-12-10 12:32:18 +11:00
68cadd3c2a
Update the FIXME/TODO style to match Google style guides
2016-12-10 12:32:18 +11:00
f3add9e7ef
Flesh out the CREATE DATABASE for PostgreSQL.
2016-12-10 12:32:18 +11:00
2aee081e4b
Eh, specify default values lib/pq zero initializes itself
2016-12-10 12:32:17 +11:00
7e5ccc089b
Spell `ssl_mode` like `sslmode`
2016-12-10 12:32:17 +11:00
655617ed6a
Remove the PGPASSWORD requirement for tests.
...
If they're required and the value is missing, the test will fail. There's
no need to enforce that in the test itself.
2016-12-10 12:32:17 +11:00
ec130d538c
Add comments requesting that the testing-specific environment variables
...
can be removed in Terraform 0.8.0
2016-12-10 12:32:17 +11:00
b68ef2c40b
Fall through to using the defaults from github.com/lib/pq
2016-12-10 12:32:17 +11:00
f31ebff10e
Change the PostgreSQL PGSSLMODE option to sslmode to match PostgreSQL idioms.
...
Also don't specify the default and rely on github.com/lib/pq (which uses "required"
and is different than what libpq(3) uses, which is "preferred" and unsupported by
github.com/lib/pq).
2016-12-10 12:32:17 +11:00
c69e325a1a
provider/aws: Add support for AWS Lightsail Domain ( #10637 )
...
```
% make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSLightsailDomain_'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2016/12/09 15:19:58 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v
-run=TestAccAWSLightsailDomain_ -timeout 120m
=== RUN TestAccAWSLightsailDomain_basic
--- PASS: TestAccAWSLightsailDomain_basic (16.28s)
=== RUN TestAccAWSLightsailDomain_disappears
--- PASS: TestAccAWSLightsailDomain_disappears (12.71s)
PASS
ok github.com/hashicorp/terraform/builtin/providers/aws29.015s
```
2016-12-09 17:56:48 +00:00
e705cd9a93
Merge pull request #10635 from dougneal/import_vpc_peerconn_accepter_fix
...
provider/aws: aws_vpc_peering_connection: import fix
2016-12-09 11:50:55 -06:00
dd39296d6c
Allow import of aws_security_groups with more than one source_security_group_id rule ( #9477 )
...
* Allow import of aws_security_groups with more than one source_security_group_id rule
* Add acceptable test for security group with multiple source rules.
2016-12-09 15:50:51 +00:00
5f765d9de6
Merge pull request #10624 from hashicorp/pr-9891
...
provider/aws: Add aws_snapshot_create_volume_permission resource (contd. #9891 )
2016-12-09 08:43:18 -06:00
0766074289
Merge pull request #10583 from hashicorp/f-lightsail-key-pair
...
provider/aws: Add Lightsail Key Pair resource
2016-12-09 08:26:00 -06:00
24baa38408
provider/aws: aws_vpc_peering_connection: import fix
...
When importing an `aws_vpc_peering_connection`, the code assumes that
the account under Terraform control is the initiator (requester) of the
VPC peering request. This holds true when the peering connection is
between two VPCs in the same account, or when the peering connection has
been initiated from the controlled account to another.
However, when the peering connection has been initiated from a foreign
account towards the account under management, importing the peering
connection into the statefile results in values of `peer_vpc_id` and
`vpc_id` being the opposite way round to what they should be, and in the
`peer_owner_id` being set to the managed account's ID rather than the
foreign account's ID.
This patch checks the Accepter and Requester Owner IDs against the AWS
connection's reported owner ID, and reverses the mapping if it is
determined that the VPC peering connection is owned by the foreign
account.
2016-12-09 12:05:06 +00:00
6f21b78710
provider/azurerm: fix virtual_machine reading plan as the wrong type ( #10626 )
...
TF_ACC=1 go test ./builtin/providers/azurerm -v -run TestAccAzureRMVirtualMachine_plan -timeout 120m
=== RUN TestAccAzureRMVirtualMachine_plan
--- PASS: TestAccAzureRMVirtualMachine_plan (798.75s)
PASS
ok github.com/hashicorp/terraform/builtin/providers/azurerm 798.835s
2016-12-09 10:41:45 +01:00
7e737dac62
Add support for AWS CA Central 1 Region ( #10618 )
...
* Add support for AWS CA Central 1 Region
* additional metadata for ca-central-1
* add elb metadata for ca-central-1
2016-12-09 10:35:07 +01:00
42057045ff
refactor the test to use caller_identity data source, and new ebs_snapshot resource
2016-12-08 16:43:03 -06:00
185ee439da
add wait after AWS snapshot attr modification
...
This adds up to a 5 minute wait after issuing an add or remove request
to adjust a snapshot's createVolumePermission attribute.
2016-12-08 16:12:25 -06:00
Sean Chittenden