Emily Ye
6b0dbf5bbe
check d.Set errors for complex values for container cluster
2018-06-22 10:35:46 -07:00
Emily Ye
f8c3a10809
Merge branch 'master' of https://github.com/terraform-providers/terraform-provider-google into container_cluster_list_param_multifix
2018-06-22 10:31:06 -07:00
Paddy
d9fa83f6db
Merge pull request #1692 from terraform-providers/paddy_storage_bucket_acl_custom_diff
...
Fix the perma-diff in storage bucket ACLs
2018-06-22 09:39:29 -07:00
Paddy
9ffe753bb4
Merge pull request #1693 from terraform-providers/paddy_dns_managed_zone_data_source
...
Update DNS managed zone in tests.
2018-06-22 09:38:37 -07:00
Paddy
fe0950d8e1
Merge pull request #1684 from ndmckinley/sql-disk
...
Reject disk shrinkage on SQL instances
2018-06-22 00:30:57 -07:00
Paddy Carver
2edf3087a9
Update DNS managed zone in tests.
...
Move from using qa.test.com, a domain we don't own, to qa.tf-test.club,
a domain we do own, so the domain validation doesn't cause our tests to
fail anymore.
2018-06-22 00:00:30 -07:00
Paddy Carver
1c8d56eca2
Fix the perma-diff in storage bucket ACLs.
...
Add a CustomDiff function to storage bucket ACLs that will ignore a diff
if the config and state have the same role_entities, even if they're in
a different order.
Fixes #1525 .
2018-06-21 23:42:25 -07:00
The Magician
a3964baed6
Fix some unit test failures in generated code where custom expanders are used. ( #1691 )
...
<!-- This change is generated by MagicModules. -->
/cc @ndmckinley
2018-06-21 17:46:50 -07:00
Emily Ye
f3b092b3ca
make sure TypeList keys still have empty values
2018-06-20 16:45:49 -07:00
Nathan McKinley
a64056156c
Reject disk shrinkage.
2018-06-20 15:17:49 -07:00
emily
a2b95e35e6
Add diff suppress for empty/default pod_security_policy_config ( #1670 )
...
* add diff suppress for pod_security_policy_config
* change key comparison to exact
2018-06-19 16:37:10 -07:00
The Magician
57a2e984ca
Autogenerate Subnetwork. ( #1661 )
2018-06-19 13:56:06 -07:00
Bastien Cadiot
751a3d5c9e
Add project variable for GCS service account datasource ( #1677 )
2018-06-19 10:30:20 -07:00
Jonathan Pentecost
f8a3335bf9
service_account_key: regression fix for v1.14 ( #1664 )
...
Commit 8f31fec
introduced a bug for the 'service_account_key' resource
where it required a project be set either in the provider or in the
resource for 'service_account_key', but a project isn't required if the
service account is a service account fully qualified name or a service
account email.
This PR relaxes the requirement that a project needs to be set for the
'service_account_key' resource, 'service_account' datasource and
'service_account_key' datasource, but will error if we try to build a
fully qualified name from a service account id when no project can be
found.
This also cleans up 'serviceAccountFQN' so it is slightly easier to
follow and return an error if there is no project but we need one to
build the service account fully qualified name.
Fixes : #1655
2018-06-18 13:37:41 -07:00
Jonathan Pentecost
d0890dc2b7
container_cluster: added disk_type node configuration
...
Added node config 'disk_type' which can either be 'pd-standard' or
'pd-ssd', if left blank 'pd-standard' will be the default used by google
cloud.
Closes : #1656
2018-06-15 21:21:51 +01:00
Gareth Evans
fee1c7f682
Type conversion issue
2018-06-15 11:39:08 +01:00
Gareth Evans
e14aa8bf39
Added ability to configure resource labels on a GKE cluster
2018-06-15 11:10:25 +01:00
The Magician
43cccf3e2c
Minor cleanup of disk's labelFingerprint. ( #1651 )
2018-06-14 20:34:10 -05:00
Dana Hoffman
d884312f88
don't error if service account key is already gone on delete ( #1659 )
...
Fixes #1657
2018-06-14 14:49:24 -07:00
The Magician
f4e4d880c5
Autogenerate Route resource. ( #1647 )
2018-06-14 15:22:02 -05:00
Daisuke Fujita
b7f2025fb9
Add project to container_node_pool import name ( #1653 )
...
## What
As well as https://github.com/terraform-providers/terraform-provider-google/pull/1282 , make `resource_container_node_pool` importer accept `{project}/{zone}/{cluster}/{name}` format to specify the project where the node pool belongs to actually.
## Why
Sometimes I want to import container pool in different project from default SA's. However, currently there is no way to specify project the target node pool belongs to, Terraform tries to retrieve node pool from SA's project, then it fails due to `You cannot import non-existent resources using Terraform import.` error.
2018-06-14 09:54:08 -07:00
Dana Hoffman
d424cf05ba
fix iot updatemask and docs ( #1640 )
2018-06-13 12:37:21 -07:00
Dana Hoffman
0497eec580
use same mutex for project_iam_policy as the other project_iam resources ( #1645 )
2018-06-13 12:35:49 -07:00
The Magician
d4b4436feb
Store address region as short name instead of full URL. Fixes currently failing test TestAccComputeVpnTunnel_defaultTrafficSelectors. ( #1646 )
...
<!-- This change is generated by MagicModules. -->
/cc @danawillow
2018-06-13 11:35:04 -07:00
The Magician
50f00fa6dc
Autogenerate the Address resource. ( #1634 )
2018-06-11 17:31:33 -07:00
Dana Hoffman
f0c5b34a1d
add error checking for bigquery dataset id ( #1638 )
...
* add error checking for bigquery dataset id
* use existing regexes for import validation
2018-06-11 14:20:11 -07:00
The Magician
605c950b18
Add DiffSuppress for global address ip_version. IPV4 and empty are equivalent, but the API will return whichever one was sent on creation. Fixes #1637 . ( #1639 )
2018-06-11 14:17:04 -07:00
Dana Hoffman
13eba21650
update sql user docs/comments ( #1635 )
2018-06-11 13:47:01 -07:00
Paddy
430b735dd0
Merge pull request #1621 from terraform-providers/paddy_app_engine_app
...
Make App Engine applications updatable
2018-06-11 13:09:50 -07:00
Paddy
8f9d0785f2
Merge pull request #1622 from terraform-providers/paddy_undeprecate_ssl_name_prefix
...
Undeprecate name_prefix for ssl_certificate.
2018-06-11 13:09:08 -07:00
Dana Hoffman
c505255556
changes to make it easier to autogen address ( #1607 )
2018-06-08 16:35:36 -07:00
The Magician
877f8107f8
Check for done redis operations before waiting on them. Fixes #1630 . ( #1633 )
2018-06-08 16:32:24 -07:00
Dana Hoffman
d00e55f11b
check for done operations before waiting on them ( #1632 )
2018-06-08 16:04:55 -07:00
The Magician
d7d72398b6
zone field in compute disk should be optional ( #1631 )
2018-06-08 15:20:39 -07:00
Vincent Roseberry
4f89945cce
Store v1 self_link for network and subnetwork ( #1629 )
2018-06-08 09:55:21 -07:00
Olve S. Hansen
d605e0f170
Fixes #1624 - issues with consecutive whitespace. ( #1625 )
2018-06-08 09:37:29 -07:00
Vincent Roseberry
32d965062d
Support update for DNS Managed Zone ( #1617 )
...
* vendor google.golang.org/api/dns/v1beta2
* Support update for DNS Managed Zone
2018-06-08 09:27:08 -07:00
Paddy
470f27b659
Undeprecate name_prefix for ssl_certificate.
...
As discussed in #1326 , we're not going to remove name_prefix for
compute_ssl_certificate, because it makes the common use case more
ergonomic by a good amount, and the only cost is it's harder to maintain
the autogenerated code, and we've decided the benefits outweigh the
costs in this circumstance.
2018-06-07 18:29:45 -07:00
Paddy
8296c16168
Remove extraneous Exists checks in App Engine tests.
2018-06-07 18:16:48 -07:00
Paddy
4176a98949
terraform fmt all configs in project_test.
2018-06-07 18:14:43 -07:00
Paddy
6910d8277e
SetPartial on app_engine.
2018-06-07 18:11:07 -07:00
Paddy
4999637399
Remove unnecessary CustomizeDiff function.
2018-06-07 18:09:51 -07:00
Paddy
16ab5bb14b
Reorder fields.
2018-06-07 18:09:14 -07:00
Paddy Carver
6d040db9d8
Really fix location_id force new.
2018-06-07 15:37:30 -07:00
Sébastien GLON
d766990ce9
Add new google_compute_regions ( #1603 )
...
* Add new google_compute_regions
* Add docs
* correct doc
2018-06-07 09:52:12 -07:00
Dana Hoffman
e92fddd33e
Fix forwarding rule data source test ( #1606 )
2018-06-06 18:40:42 -07:00
The Magician
edb34aaaf7
Fix redis authorized network and tests. The Redis API currently only accepts partial links. The tests weren't failing because they weren't actually using the network (oops). There were a few other test issues that I fixed while I was there. Fixes #1571 . ( #1599 )
...
<!-- This change is generated by MagicModules. -->
/cc @danawillow
2018-06-06 10:55:04 -07:00
Paddy Carver
53cc357b4c
Fix network_tier tests.
...
Make network_tier computed, so when it's not specified we don't get a
permadiff.
Always store v1 self_links, so we don't get a permadiff.
2018-06-05 17:49:33 -07:00
Paddy Carver
9ebeeda1d2
Fix forcenew for location_id.
2018-06-05 15:31:52 -07:00
Dana Hoffman
2f5c77cd7f
add support for network tiers ( #1530 )
...
* add support for network tiers
* Update resource_compute_forwarding_rule.go
2018-06-05 12:35:44 -07:00
Jamie Lennox
f8eb752809
Allow using in repo configuration for cloudbuild trigger ( #1557 )
...
* Allow using in repo configuration for cloudbuild trigger
Cloudbuild triggers have a complex configuration that can be defined
from the API. When using the console, the more typical way of doing this
is to defined the configuration within the repository and point the
configuration to the file that defines the config.
This can be supported by sending the filename parameter instead of the
build parameter, however only one can be sent.
* Acceptance testing for cloudbuild trigger with filename
Ensure that when a cloudbuild repo trigger is created with a filename,
that filename is what actually ends up in the cloud.
* Don't specify "by default" in cloudbuild-trigger.
The docs shouldn't say that "cloudbuild.yaml" is used by default. There
is no default from the APIs, but the console suggest using this value.
Just say it's the typical value in documentation.
2018-06-05 10:17:20 -07:00
The Magician
164936e4a5
add update support for redis ( #1590 )
...
* add update support for redis
* add test for redis update
2018-06-04 17:41:48 -07:00
Tenorio Orta JM
6864f1f9e5
Added GCP Netblock Data Source ( #1416 ) ( #1580 )
...
* Added GCP Netblock Data Source (#1416 )
* Added docs for google_netblock_ip_ranges (#1416 )
* Code review changes (#1416 )
2018-06-04 16:31:09 -07:00
Mikaël Gibert
d60381a693
Implement multiple version in instance group manager ( #1499 )
...
Hi there,
Here is an attempt to implement canary releases ( #1252 ). This is the first time I write golang and make a terraform contribution, I opened the PR to obtain feedback and advices so please let me know how I can improve this code!
In addition I used `make fmt` to format the code but left some lines bigger than 80 characters, do I need to split them ?
I tested the feature against a project with the following configuration:
```
resource "google_compute_health_check" "mikael-hackathon-healthcheck" {
name = "mikael-hackathon-healthcheck"
check_interval_sec = 1
timeout_sec = 1
healthy_threshold = 2
unhealthy_threshold = 10
http_health_check {
request_path = "/"
port = "80"
}
}
resource "google_compute_instance_template" "mikael-hackaton-template" {
name_prefix = "mikael-hackaton-"
description = "This template is used to create app server instances."
tags = ["loadbalanced", "internal-web", "hackaton"]
labels = {
environment = "hackaton"
}
instance_description = "Hackaton demo rolling upgrade"
machine_type = "n1-standard-1"
can_ip_forward = false
scheduling {
automatic_restart = true
on_host_maintenance = "MIGRATE"
}
disk {
source_image = "debian-cloud/debian-9"
disk_type = "pd-standard"
disk_size_gb = 20
auto_delete = true
boot = true
}
network_interface {
network = "default"
access_config = {}
}
service_account {
email = "${google_service_account.mikael-hackaton.email}"
scopes = ["cloud-platform"]
}
lifecycle {
create_before_destroy = true
}
metadata_startup_script = "apt-get update && apt-get install -y apache2 && echo I am stable version at $(hostname) > /var/www/html/index.html"
}
resource "google_compute_instance_template" "mikael-hackaton-template-canary" {
name_prefix = "mikael-hackaton-canary"
description = "This template is used to create app server instances."
tags = ["loadbalanced", "internal-web", "hackaton"]
labels = {
environment = "hackaton"
}
instance_description = "Hackaton demo rolling upgrade"
machine_type = "n1-standard-1"
can_ip_forward = false
scheduling {
automatic_restart = true
on_host_maintenance = "MIGRATE"
}
disk {
source_image = "debian-cloud/debian-9"
disk_type = "pd-standard"
disk_size_gb = 20
auto_delete = true
boot = true
}
network_interface {
network = "default"
access_config = {}
}
service_account {
email = "${google_service_account.mikael-hackaton.email}"
scopes = ["cloud-platform"]
}
lifecycle {
create_before_destroy = true
}
metadata_startup_script = "apt-get update && apt-get install -y apache2 && echo I am a canary at $(hostname) > /var/www/html/index.html"
}
resource "google_compute_target_pool" "mikael-hackaton-target-pool" {
name = "mikael-hackaton-target-pool"
}
resource "google_compute_instance_group_manager" "mikael-hackaton-manager" {
name = "mikael-hackaton-manager"
base_instance_name = "mikael-hackaton"
#instance_template = "${google_compute_instance_template.mikael-hackaton-template.self_link}"
update_strategy = "ROLLING_UPDATE"
zone = "${var.zone}"
target_pools = ["${google_compute_target_pool.mikael-hackaton-target-pool.self_link}"]
target_size = 5
version {
name = "primary"
instance_template = "${google_compute_instance_template.mikael-hackaton-template.self_link}"
}
version {
name = "canary"
instance_template = "${google_compute_instance_template.mikael-hackaton-template-canary.self_link}"
target_size_fixed = 1
}
named_port {
name = "http"
port = 80
}
auto_healing_policies {
health_check = "${google_compute_health_check.mikael-hackathon-healthcheck.self_link}"
initial_delay_sec = 10
}
rolling_update_policy {
type = "PROACTIVE"
minimal_action = "REPLACE"
max_surge_percent = 100
max_unavailable_percent = 50
min_ready_sec = 5
}
}
```
2018-06-04 15:34:48 -07:00
Dana Hoffman
ac6e30d335
allow beta kubernetes logging; add monitoring validation ( #1589 )
...
Fixes #1577
2018-06-04 15:09:52 -07:00
The Magician
b85473619c
Correct issue with Disk encryption. ( #1584 )
...
* Correct issue with Disk encryption.
* Update to test to make it less permissive.
2018-06-04 10:01:52 -07:00
The Magician
dd92a3732e
Adds handleNotFoundError to all resource's Delete methods. ( #1581 )
2018-06-02 17:33:30 -03:00
The Magician
fd208dc7ca
Autogenerate Disk resource ( #1521 ).
2018-05-31 18:10:06 -07:00
Jonathan Pentecost
8f31fec857
New datasource: service account and service account key ( #1535 )
2018-05-31 17:31:45 -07:00
Seth Vargo
aff6bc6519
Retry while listing enabled services ( #1573 )
...
This fixes GH-1562 for realz
2018-05-31 14:50:38 -07:00
Paddy Carver
d0a6b2b5b5
Make App Engine applications updatable.
...
No longer ForceNew when adding an App Engine application to a project,
when modifying the auth domain, modifying the serving status, or
modifying the feature settings.
2018-05-31 14:26:24 -07:00
Seth Vargo
40094ba417
Lions, tigers, and services being enabled with "precondition failed", oh my! ( #1565 )
...
* Use errwrap to retain original error
* Use built-in Page function, only return names when listing services
This removes the custom logic on pagination and uses the built-in Page function in the SDK to make things a bit simpler. Additionally, I added a field filter to only return service names, which drastically reduces the size of the API call (important for slow connections, given how frequently this function is executed).
Also added errwrap to better trace where errors originate.
* Add helper function for diffing string slices
This just looked really nasty inline
* Batch 20 services at a time, handle precondition failed, better errwrap
This commit does three things:
1. It batches services to be enabled 20 at a time. The API fails if you try to enable more than 20 services, and this is documented in the SDK and API. I learned this the hard way. I think Terraform should "do the right thing" here and batch them in series' of twenty, which is what this does. Each batch is tried in serial, but I think making it parallelized is not worth the complexity tradeoffs.
2. Handle the precondition failed error that occurs randomly. This just started happened, but it affects at least two APIs consistently, and a rudimentary test showed that it failed 78% of the time (78/100 times in an hour). We should fix this upstream, but that failure rate also necessitates (in my opinion) some mitigation on the Terraform side until a fix is in place at the API level.
3. Use errwrap on errors for better tracing. It was really difficult to trace exactly which error was being throw. That's fixed.
* Updates from code review
2018-05-31 09:26:40 -07:00
The Magician
6240cf2ab4
Add support for SSL policy to google_compute_target_ssl_proxy ( #1568 )
2018-05-30 16:00:53 -07:00
Tenorio Orta JM
8a77e42698
Added support for restoring default organization policies ( #1477 )
...
* Added support for restoring default organization policies
* Added support for restoring default folder/project organization policies
* Refactored code
* Fixes #1239
* Clarify docs
* Clarify docs
* Clarify docs
2018-05-30 15:48:08 -07:00
Tenorio Orta JM
c6b0cfcbfa
Added user label support in sql_database_instance ( #1372 ) ( #1567 )
...
* Added user label support in sql_database_instance (#1372 )
* Fixes #1372
2018-05-30 15:32:11 -07:00
Vincent Roseberry
ebbae91b77
Convert instance_group self_link to v1 ( #1549 )
2018-05-30 15:25:07 -07:00
Ian L
b4be2fa124
Add basic update for google_kms_crypto_key
resource ( #1511 )
...
* Add basic update for `google_kms_crypto_key` resource
Prior to this commit, any changes to `rotation_period` would
force a new resource as no `Update` was defined for the resource.
This commit introduces a basic `Update` through calling the
`Patch` service method. It only modifies the `rotation_period`,
and `next_rotation_time` at the moment, but this is reflective
of what is "allowed" on https://console.cloud.google.com/security/kms .
* Remove unused `Purpose` value in `CryptoKey`
We are only patching the `rotation_period`, and `next_rotation_time`,
so that value will not be affected.
* nit: format `Patch` operation to be in a single line
* Extend `TestAccKmsCryptoKey_rotation` test steps
- Test change in rotation period
- Test removal of rotation period
* Do not parse `NextRotationTime` if it is not set
* remove ForceNew: false
2018-05-29 17:20:32 -07:00
Vincent Roseberry
e4c9b2d4b1
Add a DiffSupress for ipv6 shortening ( #1551 )
2018-05-29 14:43:39 -07:00
Vincent Roseberry
8bb494520f
project field for google_project_services can use the default provider project ( #1553 )
2018-05-29 14:26:58 -07:00
Vincent Roseberry
8c2f47ba69
Remove label from bucket ( #1550 )
...
* Remove label from bucket
* Remove debugging statement
* Fix typos
2018-05-29 14:23:37 -07:00
Dana Hoffman
dc46348d67
Add support for custom request headers in backend services ( #1537 )
...
* revendor compute beta apis
* changes to subnetwork iam from api revendoring
* add support for custom request headers in backend services
2018-05-29 13:29:40 -07:00
Dana Hoffman
1af48cddf5
add disabled property to firewall ( #1536 )
...
* add disabled property to firewall
* add test and change update to patch
2018-05-29 13:27:29 -07:00
Jamie Lennox
cd83bab7dc
Use the project defined in the trigger_template when creating trigger ( #1556 )
...
When creating a trigger by using the project defined in the schema we
enforce that the repo must be in that same project. We should be looking
at the project defined in the trigger_template data and falling back to
that first project if not found.
Closes : #1555
2018-05-29 11:38:12 -07:00
Vincent Roseberry
f0857a09d5
Accept subnetwork name only in forwarding rule ( #1552 )
...
* Accept subnetwork name only in forwarding rule
* Fix acceptance test
2018-05-29 11:00:04 -07:00
Dana Hoffman
974c3f7646
Add GKE Shared VPC support ( #1528 )
...
All this needed was to allow setting the network/subnetwork to a self link.
~WIP for now because I want to run the tests in CI.~ Tests are passing.
2018-05-23 16:47:17 -07:00
Paddy
53450244c3
Merge pull request #1523 from terraform-providers/paddy_project_read_fix
...
Ignore API not enabled errors when reading project
2018-05-23 14:53:48 -07:00
Paddy
23acaf2dd3
Merge pull request #1524 from terraform-providers/paddy_eventual_services
...
Guard against eventually consistent services
2018-05-23 14:53:14 -07:00
The Magician
488b9e86f3
Redis resource ( #1485 )
...
<!-- This change is generated by MagicModules. -->
/cc @danawillow
2018-05-23 11:25:48 -07:00
Paddy Carver
4ec59fccea
Guard against eventually consistent services
...
When enabling services, after the waiter returns, list the enabled
services and ensure the ones we enabled are in there. If not, retry. May
not always resolve #1393 , but should help. Unfortunately, the real
answer is probably either:
1. For us to try and get the API updated to only return the waiter when
the service will consistently be available. I don't know how feasible
this is, but I'm willing to open a ticket.
2. For us to build retries into ~all our resources to retry for a set
amount of time when a service not enabled error is returned. This would
greatly slow down the provider in the case of the service legitimately
not being enabled, but is how other providers handle this class of
problem.
Unfortunately, due to the eventual consistency at play, this is a hard
issue to reproduce and prove, though it matches with my
experience--while testing this patch, one of the tests failed with the
error that the serviceusage API hadn't been enabled, but only on step 4
of the test, when calls had already succeeded. Which suggests eventual
consistency, to me. Regardless, this patch shouldn't _hurt_ and should
mostly be an imperceptible change to users, and should make instances
like #1393 less likely.
2018-05-23 04:01:05 -07:00
Dana Hoffman
20616e424d
Use new serviceusage API for google_project_service[s]
( #1522 )
...
* vendor service usage api
* use serviceusage api instead of servicemanagement for project services
* add bigquery-json to test
* add import for project service
* add serviceusage_operation.go
2018-05-22 17:45:22 -07:00
Paddy Carver
4c40febd4b
Ignore API not enabled errors when reading project
...
When reading a project, both App Engine and Billing would fail if
_neither_ the default project the provider was configured with nor the
project being targeted had the App Engine Admin or Billing APIs
(respectively) enabled. We didn't catch this because our source project
obviously has both enabled.
This fixes the matter by checking the error returned from each of those,
and if it looks like an API not enabled error, logging it at warning
level instead of returning it as an error, which will let the read
proceed as usual.
2018-05-22 16:59:33 -07:00
emily
acaf767f33
Fix panic on empty maintenance window ( #1507 )
...
* fix maintenance window panic
* tiny cleanup
2018-05-22 13:31:39 -07:00
Shinichi TAMURA
028ec27ccf
Fixed region_backend_service to calc hash using relative path ( #1491 )
...
Along the change on region_backend_service.backend[].group to use DiffSuppressFunc (#1487 ), we also need to change hash function, as [pointed out](https://github.com/terraform-providers/terraform-provider-google/pull/1487#issuecomment-389000239 ).
Fixed this in a same way regular backend services do.
https://github.com/terraform-providers/terraform-provider-google/blob/v1.12.0/google/resource_compute_backend_service_migrate.go#L102-L103
2018-05-22 13:05:38 -07:00
Tenorio Orta JM
9ac7acc164
Added name and project_id regex validations ( #1498 ) ( #1519 )
2018-05-22 12:45:28 -07:00
Nathan McKinley
ca83ea530a
Revert #1510 and #1508 . ( #1515 )
2018-05-21 13:09:15 -07:00
Dana Hoffman
bc8f3046ad
initial tf changes needed to support redis ( #1513 )
2018-05-21 12:01:06 -07:00
Paddy
cd7364d649
Merge pull request #1503 from terraform-providers/paddy_app_engine_app
...
Add App Engine Application support
2018-05-18 17:12:34 -07:00
Paddy Carver
98b6323342
Fix suggestions from @ndmckinley.
2018-05-18 17:01:40 -07:00
Nathan McKinley
6cc720cad3
File moves to prepare for removing 'resourceManager' hack in Magic Modules. ( #1508 )
2018-05-18 16:32:05 -07:00
Christoph Tavan
0bcb01266c
Add google_logging_project_exclusion resource ( #990 )
...
Adds support for log exclusions in billingAccounts, organizations,
folders and projects, see:
https://cloud.google.com/logging/docs/exclusions
```
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./google -v -run=Exclusion -timeout 120m
=== RUN TestAccLoggingBillingAccountExclusion_basic
=== PAUSE TestAccLoggingBillingAccountExclusion_basic
=== RUN TestAccLoggingBillingAccountExclusion_update
=== PAUSE TestAccLoggingBillingAccountExclusion_update
=== RUN TestAccLoggingFolderExclusion_basic
=== PAUSE TestAccLoggingFolderExclusion_basic
=== RUN TestAccLoggingFolderExclusion_folderAcceptsFullFolderPath
=== PAUSE TestAccLoggingFolderExclusion_folderAcceptsFullFolderPath
=== RUN TestAccLoggingFolderExclusion_update
=== PAUSE TestAccLoggingFolderExclusion_update
=== RUN TestAccLoggingOrganizationExclusion_basic
=== PAUSE TestAccLoggingOrganizationExclusion_basic
=== RUN TestAccLoggingOrganizationExclusion_update
=== PAUSE TestAccLoggingOrganizationExclusion_update
=== RUN TestAccLoggingProjectExclusion_basic
=== PAUSE TestAccLoggingProjectExclusion_basic
=== RUN TestAccLoggingProjectExclusion_disablePreservesFilter
=== PAUSE TestAccLoggingProjectExclusion_disablePreservesFilter
=== RUN TestAccLoggingProjectExclusion_update
=== PAUSE TestAccLoggingProjectExclusion_update
=== CONT TestAccLoggingBillingAccountExclusion_basic
=== CONT TestAccLoggingOrganizationExclusion_update
=== CONT TestAccLoggingProjectExclusion_update
=== CONT TestAccLoggingFolderExclusion_folderAcceptsFullFolderPath
=== CONT TestAccLoggingOrganizationExclusion_basic
--- PASS: TestAccLoggingProjectExclusion_update (3.60s)
--- PASS: TestAccLoggingOrganizationExclusion_update (4.40s)
=== CONT TestAccLoggingFolderExclusion_update
--- PASS: TestAccLoggingOrganizationExclusion_basic (1.90s)
=== CONT TestAccLoggingFolderExclusion_basic
--- PASS: TestAccLoggingBillingAccountExclusion_basic (6.21s)
=== CONT TestAccLoggingBillingAccountExclusion_update
--- PASS: TestAccLoggingBillingAccountExclusion_update (5.90s)
=== CONT TestAccLoggingProjectExclusion_disablePreservesFilter
--- PASS: TestAccLoggingProjectExclusion_disablePreservesFilter (3.90s)
=== CONT TestAccLoggingProjectExclusion_basic
--- PASS: TestAccLoggingFolderExclusion_folderAcceptsFullFolderPath (16.67s)
--- PASS: TestAccLoggingProjectExclusion_basic (1.96s)
--- PASS: TestAccLoggingFolderExclusion_basic (15.30s)
--- PASS: TestAccLoggingFolderExclusion_update (18.35s)
PASS
ok github.com/terraform-providers/terraform-provider-google/google 22.810s
```
2018-05-18 08:55:04 -07:00
The Magician
56481d8316
Add liens resource. ( #1484 )
2018-05-17 16:33:30 -07:00
Paddy
f78de6b76e
Drop the resources we can't support.
...
IAP has no reasonable support policy, because PATCH is broken, and IAP
must be configured with an OAuth2 client ID and secret that belongs to
the project the app is associated with. There's no programmatic way to
create Clients. But we create the project and the app at the same time,
and we can't update because PATCH is broken. So this just drops IAP. It
also forces all our updates to ForceNew, because we can't update.
Also, adds more test coverage and docs, and fixes import by not relying
on the config for setting app engine info in state.
2018-05-17 14:47:34 -07:00
emily
d360236a8d
Allow bigquery regional location ( #1496 )
...
* Allow bigquery regional location
* remove stale comment about location
2018-05-15 14:36:12 -07:00
emily
3cc868b4a9
Fix order of getting project for storage notification topic ( #1490 )
...
* fix getting project for storage notification topic
* add more documentation for storage notification topic
2018-05-15 13:30:59 -07:00
Shinichi TAMURA
2d580ea0a2
Compare region_backend_service.backend[].group as a relative path ( #1487 )
2018-05-14 16:59:27 -07:00
Nathan McKinley
d47687f574
Check current status during apply-time diff. ( #1474 )
2018-05-14 16:31:24 -07:00
leo-baltus
4a0849eb86
Fixed typo ( #1488 )
...
google_project_iam_policy_binding and google_project_iam_policy_member should
read google_project_iam_binding and google_project_iam_policy_member
2018-05-14 10:11:11 -07:00
Vincent Roseberry
dc029dd329
Improve testing for forwarding rule ( #1482 )
2018-05-14 10:07:59 -07:00
The Magician
5e0d39225f
Release generated SSLPolicy ( #1478 )
2018-05-14 09:52:33 -07:00
Paddy Carver
02a4259c39
Add basic test and fix bugs.
...
Add a test case that exercises the obvious path, and fix the some of the
bugs it exposed.
2018-05-13 22:49:26 -07:00
The Magician
722c21d78f
Re-release generated vpn_gateway ( #1479 )
2018-05-11 12:06:17 -07:00
The Magician
fff3d612d6
Codegen update: allow all properties to be used in Updates. ( #1443 )
2018-05-10 15:34:24 -07:00
Nao YONASHIRO
1e38b21854
fix: replica_configuration.password to be sensitive ( #1468 )
2018-05-10 15:31:07 -07:00
Paddy Carver
765d9af0a3
Beginnings of App Engine app support.
...
This should have all the code, but who really knows if it works or not,
tbh.
2018-05-10 15:01:22 -07:00
Dana Hoffman
6672e09e9d
add oslogin to the project services tests ( #1471 )
2018-05-10 10:02:31 -07:00
Dana Hoffman
63277caf39
fix dns import tests ( #1470 )
2018-05-10 09:26:02 -07:00
Dana Hoffman
93689454f6
use google_project_service for tests that enable the compute api ( #1464 )
2018-05-09 16:57:30 -07:00
Dana Hoffman
d78725f866
fix ipallocationpolicy test ( #1465 )
2018-05-09 16:57:01 -07:00
The Magician
da0b6f9d48
Add support for ssl_policy to google_target_https_proxy ( #1466 )
2018-05-09 15:55:17 -07:00
Nathan McKinley
560e180693
Allow use of provider region, with big caveats in docs. ( #1463 )
2018-05-09 14:16:49 -07:00
Dana Hoffman
08b5804b70
add ability to retry cloud functions on failure ( #1452 )
2018-05-09 13:57:54 -07:00
Nathan McKinley
95e84aed83
Change in firewall rule handling so that only one service account (for source / target) is permitted. ( #1462 )
2018-05-09 13:38:00 -07:00
Shinichi TAMURA
73fe8bcec8
mark google_compute_instance_template's nat_ip as ForceNew ( #1444 )
2018-05-09 13:18:08 -07:00
Dana Hoffman
337a913cac
use correct region for guest accelerator import test ( #1461 )
2018-05-09 13:01:23 -07:00
Nathan McKinley
e37c25abe5
Can't move from region to region without a recreate. ( #1456 )
2018-05-09 12:55:28 -07:00
Dana Hoffman
8907321d60
Roll forward beta scaffolding PR ( #1453 )
...
* Revert "Merge pull request #1434 from terraform-providers/paddy_revert_beta"
This reverts commit 118cd71201
, reversing
changes made to d59fcbbc59
.
* add ConvertSelfLinkToV1 calls to places where beta links are stored
2018-05-09 11:24:40 -07:00
Nathan McKinley
4c4b4be413
When a disk shrinks, require ForceNew. ( #1460 )
2018-05-09 10:59:48 -07:00
Nathan McKinley
04a475d73d
Check provider level zone as well when parsing zonal values. ( #1455 )
2018-05-08 16:30:31 -07:00
Nathan McKinley
11c8a52d35
Modify resourcemanager API to be more consistent with compute API. ( #1454 )
2018-05-08 15:49:57 -07:00
Rene Kroon
dda26771d3
#843 : Add policy support to storage buckets ( #1190 )
2018-05-08 14:00:48 -07:00
Nathan McKinley
b3ceef76ee
Get region from provider in case of import (or in case of not-set-in-resource) ( #1447 )
2018-05-08 11:23:35 -07:00
Vincent Roseberry
9cb159f992
ValidateFunc accepts blank value for cluster_ipv4_cidr ( #1446 )
...
* ValidateFunc accepts blank value for cluster_ipv4_cidr in google_container_cluster
* Simplify tests
2018-05-07 15:34:56 -07:00
Vincent Roseberry
b7ae1e77bd
Remove stale comments and checks for beta API in address ( #1442 )
...
* Remove stale comments and check around beta API
* Add the check for empty addresstype
2018-05-07 15:04:20 -07:00
Vincent Roseberry
038cd0b7d1
Add support for regional cluster in datasource ( #1441 )
2018-05-07 15:02:14 -07:00
Dan Couture
4f7a6c828f
Add new google_cloudbuild_trigger
resource ( #1357 )
2018-05-07 12:53:34 -07:00
Paddy Carver
a91951d3f8
Revert "remove dead api version code and move things around"
...
This reverts commit 7e291ce8392dc2766a158dd8d70007dcf791aa44.
2018-05-04 07:54:08 -07:00
Paddy Carver
f91c62d598
Revert "remove switch statements between api versions for container"
...
This reverts commit 9c85e57b8c7e84de385995df7ca04dd93e843f5b.
2018-05-04 07:52:42 -07:00
Paddy Carver
aa2626b490
Revert "remove switch statements between api versions for compute"
...
This reverts commit bf1b3a24e8c1cac832e74d9d0e145c7010a4c2f1.
2018-05-04 07:51:45 -07:00
Dana Hoffman
d59fcbbc59
Remove switch statements between api versions for container; clean up dead version code ( #1427 )
...
* remove switch statements between api versions for container
* remove dead api version code and move things around
2018-05-03 21:51:54 -07:00
The Magician
9c5f5f63a3
Disable autogeneration for google_compute_vpn_gateway ( #1432 )
...
* Disable autogeneration for google_compute_vpn_gateway
* Revert unrelated change to backend bucket
2018-05-03 17:50:48 -07:00
Vincent Roseberry
1567ac0879
Store region name only for vpn gateway ( #1431 )
2018-05-03 17:44:36 -07:00
Paddy
afcd482297
Merge pull request #1387 from terraform-providers/paddy_spanner_instance_iam
...
Add support for IAM on Spanner Instances.
2018-05-03 15:46:31 -07:00
Vincent Roseberry
f8ad4a2054
Always parse fixed64 string to int64 ( #1429 )
2018-05-03 15:35:17 -07:00
Dana Hoffman
30409d315e
Remove switch statements between api versions for compute ( #1422 )
2018-05-03 12:24:21 -07:00
Vincent Roseberry
135f5c4144
Allow setting folder_id to empty string on google_project ( #1425 )
...
* Allow setting folder_id to empty string on google_project
* Simplify logic
* Update Changelog
2018-05-03 09:53:22 -07:00
Paddy
325a118053
Merge branch 'master' into paddy_flow_logs
2018-05-02 11:47:43 -07:00
Paddy
5ad4414931
Don't reimplement policy translation.
...
We already have this in spaner_database_iam, no need to reimplement it
here.
2018-05-02 11:44:58 -07:00
Paddy Carver
6cb9f4c51c
Merge branch 'master' into paddy_spanner_instance_iam
2018-05-02 11:39:08 -07:00
Paddy Carver
0ae3daaa5d
Merge branch 'master' into paddy_spanner_database_iam
2018-05-02 10:44:39 -07:00
Paddy
87a7af9c2b
Merge branch 'master' into paddy_spanner_instance_iam
2018-05-02 10:40:20 -07:00
The Magician
77a02b2b68
Generate google_target_tcp_proxy using Magic Modules ( #1415 )
2018-05-02 10:01:37 -07:00
The Magician
7eac32ab1f
Generate google_target_tcp_proxy using Magic Module ( #1406 )
2018-05-01 15:04:56 -07:00
The Magician
1d05aa3a3a
Generate google_vpn_gateway using Magic Module ( #1409 )
2018-05-01 13:48:56 -07:00
The Magician
2759e8a801
Add more test cases to TestReplaceVars ( #1413 )
2018-05-01 13:41:35 -07:00
Vincent Roseberry
7fd468c034
Delete dead code ( #1411 )
2018-05-01 12:31:43 -07:00
Vincent Roseberry
bc0b25e0c2
StateFunc for GetResourceNameFromSelfLink ( #1412 )
2018-05-01 12:31:18 -07:00
Paddy
b5ec48cdcc
Merge pull request #1408 from terraform-providers/paddy_list_policy_nil
...
Check that org listpolicy isn't nil.
2018-04-30 16:24:09 -07:00
Paddy
c5580ca12d
Merge pull request #1407 from terraform-providers/paddy_sql_err
...
Check error in SQL user test.
2018-04-30 16:23:54 -07:00
The Magician
f04a1d0eba
Don't silently fail when d.Set fails at read time ( #1403 )
2018-04-30 16:18:17 -07:00
Paddy Carver
1a169a5900
Check that org listpolicy isn't nil.
...
Fix a panic in our test that is caused by a ListPolicy being nil. I
assume, but cannot verify, that this is an API change in that it may now
send back a nil listpolicy if a default is used.
2018-04-30 16:15:27 -07:00