Skip to content

Commit

Permalink
updated expiration_time in tests to use year < 2038, undeprecated it (#…
Browse files Browse the repository at this point in the history
…3034) (#1691)

Signed-off-by: Modular Magician <[email protected]>
  • Loading branch information
modular-magician authored Jan 29, 2020
1 parent 7efce11 commit 6c50cbb
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 4 deletions.
Empty file removed .changelog/3030.txt
Empty file.
3 changes: 3 additions & 0 deletions .changelog/3034.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:bug
sql: undeprecated `settings.ip_configuration.authorized_networks.expiration_time`
```
5 changes: 2 additions & 3 deletions google-beta/resource_sql_database_instance.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,8 @@ const privateNetworkLinkRegex = "projects/(" + ProjectRegex + ")/global/networks
var sqlDatabaseAuthorizedNetWorkSchemaElem *schema.Resource = &schema.Resource{
Schema: map[string]*schema.Schema{
"expiration_time": {
Type: schema.TypeString,
Optional: true,
Deprecated: "This property is only applicable to First Generation instances, and First Generation instances are now deprecated.",
Type: schema.TypeString,
Optional: true,
},
"name": {
Type: schema.TypeString,
Expand Down
2 changes: 2 additions & 0 deletions google-beta/resource_sql_database_instance_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -779,6 +779,7 @@ resource "google_sql_database_instance" "instance" {
authorized_networks {
value = "108.12.12.12"
name = "misc"
expiration_time = "2037-11-15T16:19:00.094Z"
}
}
Expand Down Expand Up @@ -941,6 +942,7 @@ resource "google_sql_database_instance" "instance" {
authorized_networks {
value = "108.12.12.12"
name = "misc"
expiration_time = "2037-11-15T16:19:00.094Z"
}
}
}
Expand Down
1 change: 0 additions & 1 deletion website/docs/r/sql_database_instance.html.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ the above documentation:
* `tier`
Remove any fields that are not applicable to Second-generation instances:
* `settings.crash_safe_replication`
* `settings.ip_configuration.authorized_networks.expiration_time`
* `settings.replication_type`
* `settings.authorized_gae_applications`
And change values to appropriate values for Second-generation instances for:
Expand Down

0 comments on commit 6c50cbb

Please sign in to comment.