Skip to content

Commit a19bba1

Browse files
Remove write-only resource validation warnings (#14385) (#23417)
[upstream:b2ddc6816d172ecaf87be6aa8fdf013912cfbe89] Signed-off-by: Modular Magician <[email protected]>
1 parent 551393d commit a19bba1

File tree

4 files changed

+9
-13
lines changed

4 files changed

+9
-13
lines changed

.changelog/14385.txt

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
```release-note:bug
2+
sql: no longer surface persistent warnings recommending write-only field when using `password` on `google_sql_user`
3+
```
4+
```release-note:bug
5+
secretmanager: no longer surface persistent warnings recommending write-only field when using `secret_data` on `google_secret_manager_secret_version`
6+
```
7+
```release-note:bug
8+
bigquerydatatransfer: no longer surface persistent warnings recommending write-only field when using `secret_access_key` on `google_bigquery_data_transfer_config`
9+
```

google/services/bigquerydatatransfer/resource_bigquery_data_transfer_config.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ import (
3232
"github.com/hashicorp/go-cty/cty"
3333
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/customdiff"
3434
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
35-
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
3635

3736
"github.com/hashicorp/terraform-provider-google/google/tpgresource"
3837
transport_tpg "github.com/hashicorp/terraform-provider-google/google/transport"
@@ -135,9 +134,6 @@ func ResourceBigqueryDataTransferConfig() *schema.Resource {
135134
paramsCustomizeDiff,
136135
tpgresource.DefaultProviderProject,
137136
),
138-
ValidateRawResourceConfigFuncs: []schema.ValidateRawResourceConfigFunc{
139-
validation.PreferWriteOnlyAttribute(cty.GetAttrPath("sensitive_params").IndexInt(0).GetAttr("secret_access_key"), cty.GetAttrPath("sensitive_params").IndexInt(0).GetAttr("secret_access_key_wo")),
140-
},
141137

142138
Schema: map[string]*schema.Schema{
143139
"data_source_id": {

google/services/secretmanager/resource_secret_manager_secret_version.go

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@ import (
3131

3232
"github.com/hashicorp/go-cty/cty"
3333
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
34-
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
3534

3635
"github.com/hashicorp/terraform-provider-google/google/tpgresource"
3736
transport_tpg "github.com/hashicorp/terraform-provider-google/google/transport"
@@ -90,10 +89,6 @@ func ResourceSecretManagerSecretVersion() *schema.Resource {
9089
Delete: schema.DefaultTimeout(20 * time.Minute),
9190
},
9291

93-
ValidateRawResourceConfigFuncs: []schema.ValidateRawResourceConfigFunc{
94-
validation.PreferWriteOnlyAttribute(cty.GetAttrPath("secret_data"), cty.GetAttrPath("secret_data_wo")),
95-
},
96-
9792
Schema: map[string]*schema.Schema{
9893
"secret_data_wo_version": {
9994
Type: schema.TypeInt,

google/services/sql/resource_sql_user.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,6 @@ func ResourceSqlUser() *schema.Resource {
7878
tpgresource.DefaultProviderProject,
7979
),
8080

81-
ValidateRawResourceConfigFuncs: []schema.ValidateRawResourceConfigFunc{
82-
validation.PreferWriteOnlyAttribute(cty.GetAttrPath("password"), cty.GetAttrPath("password_wo")),
83-
},
84-
8581
SchemaVersion: 1,
8682
MigrateState: resourceSqlUserMigrateState,
8783

0 commit comments

Comments
 (0)