Skip to content

Commit 67d8e5f

Browse files
switched to the v1beta API for modelarmor and add google_model_armor_floorsetting Resource (#14457) (#10359)
[upstream:c01b20fec6deb05853282bf9254f5c8ded815b55] Signed-off-by: Modular Magician <[email protected]>
1 parent 7229c76 commit 67d8e5f

15 files changed

+1755
-36
lines changed

.changelog/14457.txt

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
```release-note:new-resource
2+
`google_model_armor_floorsetting`
3+
```
4+
5+
6+
```release-note:enhancement
7+
modelarmor: switched to the v1beta API for `google_model_armor_*` resources
8+
```

google-beta/fwmodels/provider_model.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,7 @@ type ProviderModel struct {
155155
MigrationCenterCustomEndpoint types.String `tfsdk:"migration_center_custom_endpoint"`
156156
MLEngineCustomEndpoint types.String `tfsdk:"ml_engine_custom_endpoint"`
157157
ModelArmorCustomEndpoint types.String `tfsdk:"model_armor_custom_endpoint"`
158+
ModelArmorGlobalCustomEndpoint types.String `tfsdk:"model_armor_global_custom_endpoint"`
158159
MonitoringCustomEndpoint types.String `tfsdk:"monitoring_custom_endpoint"`
159160
NetappCustomEndpoint types.String `tfsdk:"netapp_custom_endpoint"`
160161
NetworkConnectivityCustomEndpoint types.String `tfsdk:"network_connectivity_custom_endpoint"`

google-beta/fwprovider/framework_provider.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -793,6 +793,12 @@ func (p *FrameworkProvider) Schema(_ context.Context, _ provider.SchemaRequest,
793793
transport_tpg.CustomEndpointValidator(),
794794
},
795795
},
796+
"model_armor_global_custom_endpoint": &schema.StringAttribute{
797+
Optional: true,
798+
Validators: []validator.String{
799+
transport_tpg.CustomEndpointValidator(),
800+
},
801+
},
796802
"monitoring_custom_endpoint": &schema.StringAttribute{
797803
Optional: true,
798804
Validators: []validator.String{

google-beta/provider/provider.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -703,6 +703,11 @@ func Provider() *schema.Provider {
703703
Optional: true,
704704
ValidateFunc: transport_tpg.ValidateCustomEndpoint,
705705
},
706+
"model_armor_global_custom_endpoint": {
707+
Type: schema.TypeString,
708+
Optional: true,
709+
ValidateFunc: transport_tpg.ValidateCustomEndpoint,
710+
},
706711
"monitoring_custom_endpoint": {
707712
Type: schema.TypeString,
708713
Optional: true,
@@ -1281,6 +1286,7 @@ func ProviderConfigure(ctx context.Context, d *schema.ResourceData, p *schema.Pr
12811286
config.MigrationCenterBasePath = d.Get("migration_center_custom_endpoint").(string)
12821287
config.MLEngineBasePath = d.Get("ml_engine_custom_endpoint").(string)
12831288
config.ModelArmorBasePath = d.Get("model_armor_custom_endpoint").(string)
1289+
config.ModelArmorGlobalBasePath = d.Get("model_armor_global_custom_endpoint").(string)
12841290
config.MonitoringBasePath = d.Get("monitoring_custom_endpoint").(string)
12851291
config.NetappBasePath = d.Get("netapp_custom_endpoint").(string)
12861292
config.NetworkConnectivityBasePath = d.Get("network_connectivity_custom_endpoint").(string)

google-beta/provider/provider_mmv1_resources.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,7 @@ import (
121121
"github.com/hashicorp/terraform-provider-google-beta/google-beta/services/migrationcenter"
122122
"github.com/hashicorp/terraform-provider-google-beta/google-beta/services/mlengine"
123123
"github.com/hashicorp/terraform-provider-google-beta/google-beta/services/modelarmor"
124+
"github.com/hashicorp/terraform-provider-google-beta/google-beta/services/modelarmorglobal"
124125
"github.com/hashicorp/terraform-provider-google-beta/google-beta/services/monitoring"
125126
"github.com/hashicorp/terraform-provider-google-beta/google-beta/services/netapp"
126127
"github.com/hashicorp/terraform-provider-google-beta/google-beta/services/networkconnectivity"
@@ -594,9 +595,9 @@ var handwrittenIAMDatasources = map[string]*schema.Resource{
594595
}
595596

596597
// Resources
597-
// Generated resources: 685
598+
// Generated resources: 686
598599
// Generated IAM resources: 339
599-
// Total generated resources: 1024
600+
// Total generated resources: 1025
600601
var generatedResources = map[string]*schema.Resource{
601602
"google_folder_access_approval_settings": accessapproval.ResourceAccessApprovalFolderSettings(),
602603
"google_organization_access_approval_settings": accessapproval.ResourceAccessApprovalOrganizationSettings(),
@@ -1329,6 +1330,7 @@ var generatedResources = map[string]*schema.Resource{
13291330
"google_migration_center_preference_set": migrationcenter.ResourceMigrationCenterPreferenceSet(),
13301331
"google_ml_engine_model": mlengine.ResourceMLEngineModel(),
13311332
"google_model_armor_template": modelarmor.ResourceModelArmorTemplate(),
1333+
"google_model_armor_floorsetting": modelarmorglobal.ResourceModelArmorGlobalFloorsetting(),
13321334
"google_monitoring_alert_policy": monitoring.ResourceMonitoringAlertPolicy(),
13331335
"google_monitoring_service": monitoring.ResourceMonitoringGenericService(),
13341336
"google_monitoring_group": monitoring.ResourceMonitoringGroup(),
@@ -1899,6 +1901,7 @@ func UseGeneratedProducts() {
18991901
var _ = migrationcenter.ProductName
19001902
var _ = mlengine.ProductName
19011903
var _ = modelarmor.ProductName
1904+
var _ = modelarmorglobal.ProductName
19021905
var _ = monitoring.ProductName
19031906
var _ = netapp.ProductName
19041907
var _ = networkconnectivity.ProductName

google-beta/services/modelarmor/resource_model_armor_template_generated_meta.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ resource: 'google_model_armor_template'
22
generation_type: 'mmv1'
33
source_file: 'products/modelarmor/Template.yaml'
44
api_service_name: 'modelarmor.googleapis.com'
5-
api_version: 'v1'
5+
api_version: 'v1beta'
66
api_resource_type_kind: 'Template'
77
autogen_status: true
88
fields:

google-beta/services/modelarmor/resource_model_armor_template_sweeper.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ func listAndActionModelArmorTemplate(action sweeper.ResourceAction) error {
117117
}
118118

119119
// Prepare list URL
120-
listTemplate := strings.Split("https://modelarmor.{{location}}.rep.googleapis.com/v1/projects/{{project}}/locations/{{location}}/templates", "?")[0]
120+
listTemplate := strings.Split("https://modelarmor.{{location}}.rep.googleapis.com/v1beta/projects/{{project}}/locations/{{location}}/templates", "?")[0]
121121
listUrl, err := tpgresource.ReplaceVars(mockConfig, config, listTemplate)
122122
if err != nil {
123123
log.Printf("[INFO][SWEEPER_LOG] error preparing sweeper list url: %s", err)
@@ -198,7 +198,7 @@ func deleteResourceModelArmorTemplate(config *transport_tpg.Config, d *tpgresour
198198
return nil
199199
}
200200

201-
deleteTemplate := "https://modelarmor.{{location}}.rep.googleapis.com/v1/projects/{{project}}/locations/{{location}}/templates/{{template_id}}"
201+
deleteTemplate := "https://modelarmor.{{location}}.rep.googleapis.com/v1beta/projects/{{project}}/locations/{{location}}/templates/{{template_id}}"
202202

203203
url, err := tpgresource.ReplaceVars(d, config, deleteTemplate)
204204
if err != nil {

google-beta/services/modelarmor/resource_model_armor_template_test.go

Lines changed: 8 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -17,31 +17,15 @@
1717
package modelarmor_test
1818

1919
import (
20-
"bytes"
2120
"fmt"
2221
"testing"
23-
"text/template"
2422

2523
"github.com/hashicorp/terraform-plugin-testing/helper/resource"
2624
"github.com/hashicorp/terraform-plugin-testing/plancheck"
2725

2826
"github.com/hashicorp/terraform-provider-google-beta/google-beta/acctest"
2927
)
3028

31-
// Helper function to expand a template
32-
func expandTemplate(tmplStr string, data map[string]interface{}) (string, error) {
33-
tmpl, err := template.New("config").Parse(tmplStr)
34-
if err != nil {
35-
return "", err
36-
}
37-
var buf bytes.Buffer
38-
err = tmpl.Execute(&buf, data)
39-
if err != nil {
40-
return "", err
41-
}
42-
return buf.String(), nil
43-
}
44-
4529
func TestAccModelArmorTemplate_basic(t *testing.T) {
4630
t.Parallel()
4731

@@ -58,13 +42,7 @@ func TestAccModelArmorTemplate_basic(t *testing.T) {
5842
CheckDestroy: testAccCheckModelArmorTemplateDestroyProducer(t),
5943
Steps: []resource.TestStep{
6044
{
61-
Config: func() string {
62-
cfg, err := testAccModelArmorTemplate_basic_config(basicContext)
63-
if err != nil {
64-
t.Fatalf("Failed to expand basic config template: %v", err)
65-
}
66-
return cfg
67-
}(),
45+
Config: testAccModelArmorTemplate_basic_config(basicContext),
6846
},
6947
{
7048
ResourceName: "google_model_armor_template.template-basic",
@@ -75,19 +53,18 @@ func TestAccModelArmorTemplate_basic(t *testing.T) {
7553
})
7654
}
7755

78-
func testAccModelArmorTemplate_basic_config(context map[string]interface{}) (string, error) {
79-
const basic_template = `
56+
func testAccModelArmorTemplate_basic_config(context map[string]interface{}) string {
57+
return acctest.Nprintf(`
8058
resource "google_model_armor_template" "template-basic" {
81-
location = "{{.location}}"
82-
template_id = "{{.templateId}}"
59+
location = "%{location}"
60+
template_id = "%{templateId}"
8361
filter_config {
8462
8563
}
8664
template_metadata {
8765
8866
}
89-
}`
90-
return expandTemplate(basic_template, context)
67+
}`, context)
9168
}
9269

9370
func TestAccModelArmorTemplate_update(t *testing.T) {
@@ -96,6 +73,7 @@ func TestAccModelArmorTemplate_update(t *testing.T) {
9673
templateId := fmt.Sprintf("modelarmor-test-update-%s", acctest.RandString(t, 5))
9774

9875
context := map[string]interface{}{
76+
"location": "us-central1",
9977
"templateId": templateId,
10078
}
10179

@@ -134,7 +112,7 @@ func TestAccModelArmorTemplate_update(t *testing.T) {
134112
func testAccModelArmorTemplate_initial(context map[string]interface{}) string {
135113
return acctest.Nprintf(`
136114
resource "google_model_armor_template" "test-resource" {
137-
location = "us-central1"
115+
location = "%{location}"
138116
template_id = "%{templateId}"
139117
labels = {
140118
"test-label" = "env-testing-initial"
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
// Copyright (c) HashiCorp, Inc.
2+
// SPDX-License-Identifier: MPL-2.0
3+
4+
// ----------------------------------------------------------------------------
5+
//
6+
// *** AUTO GENERATED CODE *** Type: MMv1 ***
7+
//
8+
// ----------------------------------------------------------------------------
9+
//
10+
// This file is automatically generated by Magic Modules and manual
11+
// changes will be clobbered when the file is regenerated.
12+
//
13+
// Please read more about how to change this file in
14+
// .github/CONTRIBUTING.md.
15+
//
16+
// ----------------------------------------------------------------------------
17+
18+
// Package modelarmorglobal contains resources, datasources, etc. for the model armor service.
19+
package modelarmorglobal
20+
21+
const ProductName = "modelarmorglobal"

0 commit comments

Comments
 (0)