diff --git a/mmv1/products/accesscontextmanager/AccessLevelCondition.yaml b/mmv1/products/accesscontextmanager/AccessLevelCondition.yaml index 39b8d84b1a7f..72d37ccdbe90 100644 --- a/mmv1/products/accesscontextmanager/AccessLevelCondition.yaml +++ b/mmv1/products/accesscontextmanager/AccessLevelCondition.yaml @@ -62,7 +62,7 @@ async: !ruby/object:Provider::Terraform::PollAsync actions: ['create'] autogen_async: true exclude_validator: true -exclude_import: true # no unique way to specify +exclude_import: true # no unique way to specify # Skipping the sweeper due to the non-standard base_url and because this is fine-grained under AccessLevel skip_sweeper: true id_format: '{{access_level}}' diff --git a/mmv1/products/accesscontextmanager/ServicePerimeters.yaml b/mmv1/products/accesscontextmanager/ServicePerimeters.yaml index 80cbd9a25a61..ce49ebbabf0d 100644 --- a/mmv1/products/accesscontextmanager/ServicePerimeters.yaml +++ b/mmv1/products/accesscontextmanager/ServicePerimeters.yaml @@ -11,7 +11,7 @@ # See the License for the specific language governing permissions and # limitations under the License. ---- !ruby/object:Api::Resource # This is the plural of `ServicePerimeter`, any changes here should be made to `ServicePerimeter` as well +--- !ruby/object:Api::Resource # This is the plural of `ServicePerimeter`, any changes here should be made to `ServicePerimeter` as well name: 'ServicePerimeters' base_url: '{{parent}}/servicePerimeters:replaceAll' update_url: '{{parent}}/servicePerimeters:replaceAll' diff --git a/mmv1/products/activedirectory/Domain.yaml b/mmv1/products/activedirectory/Domain.yaml index 784fd9dcba52..f53c5bb810ac 100644 --- a/mmv1/products/activedirectory/Domain.yaml +++ b/mmv1/products/activedirectory/Domain.yaml @@ -16,7 +16,7 @@ name: 'Domain' kind: 'activedirectory#domain' base_url: projects/{{project}}/locations/global/domains create_url: projects/{{project}}/locations/global/domains?domainName={{domain_name}} -delete_url: projects/{{project}}/locations/global/domains/{{domain_name}} # equivalent to {{name}}, but makes sweepers work +delete_url: projects/{{project}}/locations/global/domains/{{domain_name}} # equivalent to {{name}}, but makes sweepers work update_verb: :PATCH update_mask: true self_link: '{{name}}' diff --git a/mmv1/products/apigateway/ApiConfig.yaml b/mmv1/products/apigateway/ApiConfig.yaml index 6d0fe6960759..2c795186cbed 100644 --- a/mmv1/products/apigateway/ApiConfig.yaml +++ b/mmv1/products/apigateway/ApiConfig.yaml @@ -54,7 +54,7 @@ examples: primary_resource_id: 'api_cfg' primary_resource_name: 'fmt.Sprintf("tf-test-my-api%s", context["random_suffix"]), fmt.Sprintf("tf-test-my-config%s", - context["random_suffix"])' # Need to pass 2 ids into a Sprintf - parent resource id also needed to identify primary resource + context["random_suffix"])' # Need to pass 2 ids into a Sprintf - parent resource id also needed to identify primary resource vars: api_id: 'my-api' config_id: 'my-config' diff --git a/mmv1/products/apigateway/product.yaml b/mmv1/products/apigateway/product.yaml index f956f4da9171..e29f3d971c92 100644 --- a/mmv1/products/apigateway/product.yaml +++ b/mmv1/products/apigateway/product.yaml @@ -30,10 +30,10 @@ async: !ruby/object:Api::OpAsync resource_inside_response: true status: !ruby/object:Api::OpAsync::Status path: 'done' - complete: True + complete: true allowed: - - True - - False + - true + - false error: !ruby/object:Api::OpAsync::Error path: 'error' message: 'message' diff --git a/mmv1/products/apigee/KeystoresAliasesSelfSignedCert.yaml b/mmv1/products/apigee/KeystoresAliasesSelfSignedCert.yaml index 0d506372d399..fc19123713ba 100644 --- a/mmv1/products/apigee/KeystoresAliasesSelfSignedCert.yaml +++ b/mmv1/products/apigee/KeystoresAliasesSelfSignedCert.yaml @@ -44,7 +44,7 @@ examples: test_env_vars: org_id: :ORG_ID billing_account: :BILLING_ACCT - # Resource creation race +# Resource creation race timeouts: !ruby/object:Api::Timeouts insert_minutes: 30 delete_minutes: 30 diff --git a/mmv1/products/beyondcorp/product.yaml b/mmv1/products/beyondcorp/product.yaml index b5fd411939c1..eea11e715b7c 100644 --- a/mmv1/products/beyondcorp/product.yaml +++ b/mmv1/products/beyondcorp/product.yaml @@ -1,4 +1,3 @@ -!ruby/object:Api::Product # Copyright 2022 Google Inc. # Licensed under the Apache License, Version 2.0 (the "License"); # you may not use this file except in compliance with the License. @@ -11,6 +10,7 @@ # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. +--- !ruby/object:Api::Product name: Beyondcorp display_name: BeyondCorp versions: diff --git a/mmv1/products/bigquery/DatasetAccess.yaml b/mmv1/products/bigquery/DatasetAccess.yaml index 51f35b8df5f4..7f0e91e398c6 100644 --- a/mmv1/products/bigquery/DatasetAccess.yaml +++ b/mmv1/products/bigquery/DatasetAccess.yaml @@ -53,14 +53,14 @@ mutex: '{{dataset_id}}' examples: - !ruby/object:Provider::Terraform::Examples name: 'bigquery_dataset_access_basic_user' - skip_test: true # not importable + skip_test: true # not importable primary_resource_id: 'access' vars: dataset_id: 'example_dataset' account_name: 'bqowner' - !ruby/object:Provider::Terraform::Examples name: 'bigquery_dataset_access_view' - skip_test: true # not importable + skip_test: true # not importable primary_resource_id: 'access' vars: dataset_id: 'example_dataset' @@ -68,14 +68,14 @@ examples: table_id: 'example_table' - !ruby/object:Provider::Terraform::Examples name: 'bigquery_dataset_access_authorized_dataset' - skip_test: true # not importable + skip_test: true # not importable primary_resource_id: 'access' vars: private: 'private' public: 'public' - !ruby/object:Provider::Terraform::Examples name: 'bigquery_dataset_access_authorized_routine' - skip_test: true # not importable + skip_test: true # not importable primary_resource_type: 'google_bigquery_dataset_access' primary_resource_id: 'authorized_routine' vars: diff --git a/mmv1/products/bigquery/Job.yaml b/mmv1/products/bigquery/Job.yaml index cbdd368a6759..66043f56b0af 100644 --- a/mmv1/products/bigquery/Job.yaml +++ b/mmv1/products/bigquery/Job.yaml @@ -70,7 +70,7 @@ examples: primary_resource_id: 'job' vars: job_id: 'job_load' - bucket_name: 'bq-geojson' # Keep small(er) to avoid downstream acctest having too-long a bucket name + bucket_name: 'bq-geojson' # Keep small(er) to avoid downstream acctest having too-long a bucket name test_env_vars: project: :PROJECT_NAME ignore_read_extra: @@ -85,7 +85,7 @@ examples: - 'etag' - 'load.0.destination_table.0.table_id' - 'status.0.state' - skip_docs: true # there are a lot of examples for this resource, so omitting some that are similar to others + skip_docs: true # there are a lot of examples for this resource, so omitting some that are similar to others - !ruby/object:Provider::Terraform::Examples name: 'bigquery_job_copy' primary_resource_id: 'job' @@ -115,7 +115,7 @@ examples: - 'copy.0.source_tables.0.table_id' - 'copy.0.source_tables.1.table_id' - 'status.0.state' - skip_docs: true # there are a lot of examples for this resource, so omitting some that are similar to others + skip_docs: true # there are a lot of examples for this resource, so omitting some that are similar to others - !ruby/object:Provider::Terraform::Examples name: 'bigquery_job_extract' primary_resource_id: 'job' @@ -135,7 +135,7 @@ examples: - 'etag' - 'extract.0.source_table.0.table_id' - 'status.0.state' - skip_docs: true # there are a lot of examples for this resource, so omitting some that are similar to others + skip_docs: true # there are a lot of examples for this resource, so omitting some that are similar to others custom_code: !ruby/object:Provider::Terraform::CustomCode constants: templates/terraform/constants/bigquery_job.go encoder: templates/terraform/encoders/bigquery_job.go.erb @@ -221,16 +221,16 @@ properties: properties: - !ruby/object:Api::Type::String name: 'resourceUri' + # TODO (mbang): exactly_one_of: resourceUri, inlineCode description: 'A code resource to load from a Google Cloud Storage URI (gs://bucket/path).' - # TODO (mbang): exactly_one_of: resourceUri, inlineCode - !ruby/object:Api::Type::String name: 'inlineCode' + # TODO (mbang): exactly_one_of: resourceUri, inlineCode description: | An inline resource that contains code for a user-defined function (UDF). Providing a inline code resource is equivalent to providing a URI for a file containing the same code. - # TODO (mbang): exactly_one_of: resourceUri, inlineCode - !ruby/object:Api::Type::Enum name: 'createDisposition' description: | diff --git a/mmv1/products/bigquery/Routine.yaml b/mmv1/products/bigquery/Routine.yaml index 91dea402ee6d..5be24ffa7224 100644 --- a/mmv1/products/bigquery/Routine.yaml +++ b/mmv1/products/bigquery/Routine.yaml @@ -130,9 +130,9 @@ properties: - :IN - :OUT - :INOUT - # This is a string instead of a NestedObject because schemas contain ColumnSchemas, - # which can contain nested StandardSqlDataType. - # We'll have people provide the json blob for the schema instead. + # This is a string instead of a NestedObject because schemas contain ColumnSchemas, + # which can contain nested StandardSqlDataType. + # We'll have people provide the json blob for the schema instead. - !ruby/object:Api::Type::String name: 'dataType' description: | diff --git a/mmv1/products/bigqueryconnection/Connection.yaml b/mmv1/products/bigqueryconnection/Connection.yaml index 679f663b42e2..5bac947ba7dd 100644 --- a/mmv1/products/bigqueryconnection/Connection.yaml +++ b/mmv1/products/bigqueryconnection/Connection.yaml @@ -71,7 +71,7 @@ examples: oics_vars_overrides: deletion_protection: 'false' ignore_read_extra: - - 'cloud_sql.0.credential' # password removed + - 'cloud_sql.0.credential' # password removed - !ruby/object:Provider::Terraform::Examples name: 'bigquery_connection_full' pull_external: true @@ -89,7 +89,7 @@ examples: oics_vars_overrides: deletion_protection: 'false' ignore_read_extra: - - 'cloud_sql.0.credential' # password removed + - 'cloud_sql.0.credential' # password removed - !ruby/object:Provider::Terraform::Examples name: 'bigquery_connection_aws' pull_external: true diff --git a/mmv1/products/cloudbuild/Trigger.yaml b/mmv1/products/cloudbuild/Trigger.yaml index 947dfb1b3177..02fd8b18a1d8 100644 --- a/mmv1/products/cloudbuild/Trigger.yaml +++ b/mmv1/products/cloudbuild/Trigger.yaml @@ -29,7 +29,7 @@ docs: !ruby/object:Provider::Terraform::Docs import_format: - 'projects/{{project}}/triggers/{{trigger_id}}' - 'projects/{{project}}/locations/{{location}}/triggers/{{trigger_id}}' - # For global triggers, the id format is changed to projects/{{project}}/triggers/{{trigger_id}} via code overrides. +# For global triggers, the id format is changed to projects/{{project}}/triggers/{{trigger_id}} via code overrides. id_format: 'projects/{{project}}/locations/{{location}}/triggers/{{trigger_id}}' schema_version: 2 examples: diff --git a/mmv1/products/cloudids/Endpoint.yaml b/mmv1/products/cloudids/Endpoint.yaml index 6af460d423b2..b45d08d4e3b6 100644 --- a/mmv1/products/cloudids/Endpoint.yaml +++ b/mmv1/products/cloudids/Endpoint.yaml @@ -33,7 +33,7 @@ examples: name: 'cloudids_endpoint' primary_resource_id: 'example-endpoint' - #Skip_test set to true since the example is identical to what's in the _test.go file. + # skip_test set to true since the example is identical to what's in the _test.go file. skip_test: true parameters: - !ruby/object:Api::Type::String diff --git a/mmv1/products/compute/Address.yaml b/mmv1/products/compute/Address.yaml index e991fda6051e..ae92f71b14a8 100644 --- a/mmv1/products/compute/Address.yaml +++ b/mmv1/products/compute/Address.yaml @@ -79,7 +79,7 @@ examples: vars: address_name: 'my-internal-address' skip_docs: - true # It is almost identical to internal_with_gce_endpoint + true # It is almost identical to internal_with_gce_endpoint # TODO(rileykarson): Remove this example when instance is supported - !ruby/object:Provider::Terraform::Examples name: 'instance_with_ip' diff --git a/mmv1/products/compute/BackendService.yaml b/mmv1/products/compute/BackendService.yaml index 865727c9651b..99c44679532f 100644 --- a/mmv1/products/compute/BackendService.yaml +++ b/mmv1/products/compute/BackendService.yaml @@ -1244,7 +1244,7 @@ properties: properties: - !ruby/object:Api::Type::ResourceRef name: 'clientTlsPolicy' - resource: 'Region' # TODO: 'Region' is incorrect and should be 'ClientTlsPolicy' + resource: 'Region' # TODO: 'Region' is incorrect and should be 'ClientTlsPolicy' imports: 'name' description: | ClientTlsPolicy is a resource that specifies how a client should authenticate diff --git a/mmv1/products/compute/Image.yaml b/mmv1/products/compute/Image.yaml index 1f36645321c1..e106ddaf3352 100644 --- a/mmv1/products/compute/Image.yaml +++ b/mmv1/products/compute/Image.yaml @@ -368,4 +368,4 @@ properties: - :READY - :PENDING - :FAILED - # State is not applicable for state convergence. +# State is not applicable for state convergence. diff --git a/mmv1/products/compute/Network.yaml b/mmv1/products/compute/Network.yaml index 17e57bf6592b..bf512a6e9c90 100644 --- a/mmv1/products/compute/Network.yaml +++ b/mmv1/products/compute/Network.yaml @@ -105,8 +105,8 @@ properties: required: true validation: !ruby/object:Provider::Terraform::Validation function: 'verify.ValidateGCEName' - # TODO: Change subnetworks to ResourceRef - !ruby/object:Api::Type::Array + # TODO: Change subnetworks to ResourceRef name: 'subnetworks' description: | Server-defined fully-qualified URLs for all subnetworks in this diff --git a/mmv1/products/compute/RegionTargetHttpsProxy.yaml b/mmv1/products/compute/RegionTargetHttpsProxy.yaml index c4e551f04808..ead7d7e4d931 100644 --- a/mmv1/products/compute/RegionTargetHttpsProxy.yaml +++ b/mmv1/products/compute/RegionTargetHttpsProxy.yaml @@ -92,22 +92,22 @@ properties: # This field is present in the schema but as of 2019 Sep 23 attempting to set it fails with # a 400 "QUIC override is supported only with global TargetHttpsProxy". jamessynge@ said in an # email sent on 2019 Sep 20 that support for this "is probably far in the future." - #- !ruby/object:Api::Type::Enum - # name: 'quicOverride' - # description: | - # Specifies the QUIC override policy for this resource. This determines - # whether the load balancer will attempt to negotiate QUIC with clients - # or not. Can specify one of NONE, ENABLE, or DISABLE. If NONE is - # specified, uses the QUIC policy with no user overrides, which is - # equivalent to DISABLE. Not specifying this field is equivalent to - # specifying NONE. - # values: - # - :NONE - # - :ENABLE - # - :DISABLE - # update_verb: :POST - # update_url: - # 'projects/{{project}}/regions/{{region}}/targetHttpsProxies/{{name}}/setQuicOverride' + # - !ruby/object:Api::Type::Enum + # name: 'quicOverride' + # description: | + # Specifies the QUIC override policy for this resource. This determines + # whether the load balancer will attempt to negotiate QUIC with clients + # or not. Can specify one of NONE, ENABLE, or DISABLE. If NONE is + # specified, uses the QUIC policy with no user overrides, which is + # equivalent to DISABLE. Not specifying this field is equivalent to + # specifying NONE. + # values: + # - :NONE + # - :ENABLE + # - :DISABLE + # update_verb: :POST + # update_url: + # 'projects/{{project}}/regions/{{region}}/targetHttpsProxies/{{name}}/setQuicOverride' - !ruby/object:Api::Type::Array name: 'sslCertificates' description: | diff --git a/mmv1/products/compute/RegionUrlMap.yaml b/mmv1/products/compute/RegionUrlMap.yaml index 06ca96cb42e8..4403e5088e57 100644 --- a/mmv1/products/compute/RegionUrlMap.yaml +++ b/mmv1/products/compute/RegionUrlMap.yaml @@ -128,7 +128,7 @@ examples: l7_xlb_proxy: 'l7-xlb-proxy' l7_xlb_forwarding_rule: 'l7-xlb-forwarding-rule' skip_docs: true - skip_test: true # Similar to other samples + skip_test: true # Similar to other samples min_version: beta parameters: - !ruby/object:Api::Type::ResourceRef diff --git a/mmv1/products/compute/Router.yaml b/mmv1/products/compute/Router.yaml index 879646422470..c8a813683efc 100644 --- a/mmv1/products/compute/Router.yaml +++ b/mmv1/products/compute/Router.yaml @@ -149,7 +149,7 @@ properties: blank to advertise no custom groups. This enum field has the one valid value: ALL_SUBNETS - item_type: Api::Type::String # TODO(#324): enum? + item_type: Api::Type::String # TODO(#324): enum? send_empty_value: true - !ruby/object:Api::Type::Array name: advertisedIpRanges diff --git a/mmv1/products/containerattached/Cluster.yaml b/mmv1/products/containerattached/Cluster.yaml index f3f0aaea8e43..3d64fa3485b5 100644 --- a/mmv1/products/containerattached/Cluster.yaml +++ b/mmv1/products/containerattached/Cluster.yaml @@ -170,12 +170,6 @@ properties: immutable: true validation: !ruby/object:Provider::Terraform::Validation regex: '^projects/[0-9]+$' - # If the user doesn't specify a loggingConfig, the server will supply a default value. Instead of - # letting that happen and allowing the config and state to get mismatched, just manually send an - # empty object if the user doesn't set anything and require the user to explicitly set the field if a - # value is desired. - # If the loggingConfig passed to the server is empty, an empty object is returned, so the diff in that - # case needs to be ignored. - !ruby/object:Api::Type::String name: state description: | @@ -240,6 +234,12 @@ properties: name: loggingConfig description: | Logging configuration. + # If the user doesn't specify a loggingConfig, the server will supply a default value. Instead of + # letting that happen and allowing the config and state to get mismatched, just manually send an + # empty object if the user doesn't set anything and require the user to explicitly set the field if a + # value is desired. + # If the loggingConfig passed to the server is empty, an empty object is returned, so the diff in that + # case needs to be ignored. send_empty_value: true allow_empty_object: true custom_expand: 'templates/terraform/custom_expand/containerattached_cluster_empty_logging.go.erb' diff --git a/mmv1/products/datacatalog/Tag.yaml b/mmv1/products/datacatalog/Tag.yaml index 980ed652d0d3..be768f3dead4 100644 --- a/mmv1/products/datacatalog/Tag.yaml +++ b/mmv1/products/datacatalog/Tag.yaml @@ -86,7 +86,7 @@ examples: force_delete: 'true' oics_vars_overrides: force_delete: 'true' - skip_docs: true # omitting doc as it is almost identical to the case of data_catalog_entry_tag_basic + skip_docs: true # omitting doc as it is almost identical to the case of data_catalog_entry_tag_basic custom_code: !ruby/object:Provider::Terraform::CustomCode custom_import: templates/terraform/custom_import/data_catalog_tag.go.erb encoder: templates/terraform/encoders/data_catalog_tag.go.erb diff --git a/mmv1/products/datacatalog/Taxonomy.yaml b/mmv1/products/datacatalog/Taxonomy.yaml index af56a28236e2..e348865410aa 100644 --- a/mmv1/products/datacatalog/Taxonomy.yaml +++ b/mmv1/products/datacatalog/Taxonomy.yaml @@ -82,7 +82,7 @@ properties: A list of policy types that are activated for this taxonomy. If not set, defaults to an empty list. item_type: - !ruby/object:Api::Type::Enum # TOOD: should this be enum or string + !ruby/object:Api::Type::Enum # TOOD: should this be enum or string name: 'policyType' description: 'Defines policy types where policy tag can be used for' values: diff --git a/mmv1/products/datafusion/Instance.yaml b/mmv1/products/datafusion/Instance.yaml index b3d03bc649c8..3f11db4c3fcc 100644 --- a/mmv1/products/datafusion/Instance.yaml +++ b/mmv1/products/datafusion/Instance.yaml @@ -198,7 +198,7 @@ properties: default_from_api: true - !ruby/object:Api::Type::String name: 'serviceAccount' - min_version: beta # This field is deprecated and needs to be absent in GA provider. + min_version: beta # This field is deprecated and needs to be absent in GA provider. deprecation_message: Use `tenant_project_id` instead to extract the tenant project ID. output: true diff --git a/mmv1/products/deploymentmanager/Deployment.yaml b/mmv1/products/deploymentmanager/Deployment.yaml index 4cbc5c5afea4..00252933f983 100644 --- a/mmv1/products/deploymentmanager/Deployment.yaml +++ b/mmv1/products/deploymentmanager/Deployment.yaml @@ -46,7 +46,7 @@ async: !ruby/object:Api::OpAsync id_format: 'projects/{{project}}/deployments/{{name}}' import_format: ['projects/{{project}}/deployments/{{name}}'] timeouts: - !ruby/object:Api::Timeouts # Very long to support very long deployments + !ruby/object:Api::Timeouts # Very long to support very long deployments insert_minutes: 60 update_minutes: 60 delete_minutes: 60 @@ -84,7 +84,7 @@ examples: deployment_name: 'my-deployment' skip_test: true custom_code: - !ruby/object:Provider::Terraform::CustomCode # post-create failure: Delete deployment if an invalid deployment was created + !ruby/object:Provider::Terraform::CustomCode # post-create failure: Delete deployment if an invalid deployment was created post_create_failure: 'templates/terraform/post_create_failure/delete_on_failure.go.erb' # Custom diff to force new if 'preview' is true constants: 'templates/terraform/constants/deployment_manager_deployment.go.erb' diff --git a/mmv1/products/dialogflow/Agent.yaml b/mmv1/products/dialogflow/Agent.yaml index 416835d30196..127e7df98daf 100644 --- a/mmv1/products/dialogflow/Agent.yaml +++ b/mmv1/products/dialogflow/Agent.yaml @@ -34,7 +34,7 @@ examples: primary_resource_id: 'full_agent' vars: agent_name: 'dialogflow-agent' - # Only one agent per project, API does not have an agent ID +# Only one agent per project, API does not have an agent ID id_format: '{{project}}' import_format: ['{{project}}'] skip_sweeper: true diff --git a/mmv1/products/dialogflow/EntityType.yaml b/mmv1/products/dialogflow/EntityType.yaml index d4721fc77245..59712997b037 100644 --- a/mmv1/products/dialogflow/EntityType.yaml +++ b/mmv1/products/dialogflow/EntityType.yaml @@ -29,7 +29,7 @@ examples: skip_test: true vars: intent_name: 'basic-entity-type' - # Skip sweeper gen since this is a child resource. +# Skip sweeper gen since this is a child resource. skip_sweeper: true id_format: '{{name}}' import_format: ['{{name}}'] diff --git a/mmv1/products/dialogflow/Fulfillment.yaml b/mmv1/products/dialogflow/Fulfillment.yaml index f04e946b010c..a63582dec806 100644 --- a/mmv1/products/dialogflow/Fulfillment.yaml +++ b/mmv1/products/dialogflow/Fulfillment.yaml @@ -35,7 +35,7 @@ examples: skip_test: true vars: fulfillment_name: 'basic-fulfillment' - # Skip sweeper gen since this is a child resource. +# Skip sweeper gen since this is a child resource. skip_sweeper: true id_format: '{{name}}' import_format: ['{{name}}'] diff --git a/mmv1/products/dialogflow/Intent.yaml b/mmv1/products/dialogflow/Intent.yaml index b54b5055c436..477dfee43ccc 100644 --- a/mmv1/products/dialogflow/Intent.yaml +++ b/mmv1/products/dialogflow/Intent.yaml @@ -39,7 +39,7 @@ examples: intent_name: 'full-intent' test_env_vars: org_id: :ORG_ID - # Skip sweeper gen since this is a child resource. +# Skip sweeper gen since this is a child resource. skip_sweeper: true id_format: '{{name}}' import_format: ['{{name}}'] diff --git a/mmv1/products/dlp/DeidentifyTemplate.yaml b/mmv1/products/dlp/DeidentifyTemplate.yaml index 33c4be732c0b..02181d246656 100644 --- a/mmv1/products/dlp/DeidentifyTemplate.yaml +++ b/mmv1/products/dlp/DeidentifyTemplate.yaml @@ -150,7 +150,7 @@ properties: Apply transformation to all findings not specified in other ImageTransformation's selectedInfoTypes. properties: - [] # Meant to be an empty object with no properties - see here : https://cloud.google.com/dlp/docs/reference/rest/v2/projects.deidentifyTemplates#allinfotypes + [] # Meant to be an empty object with no properties - see here : https://cloud.google.com/dlp/docs/reference/rest/v2/projects.deidentifyTemplates#allinfotypes # The fields below are necessary to include the "allInfoTypes" transformation in the payload send_empty_value: true allow_empty_object: true @@ -160,7 +160,7 @@ properties: Apply transformation to all text that doesn't match an infoType. properties: - [] # Meant to be an empty object with no properties - see here : https://cloud.google.com/dlp/docs/reference/rest/v2/projects.deidentifyTemplates#alltext + [] # Meant to be an empty object with no properties - see here : https://cloud.google.com/dlp/docs/reference/rest/v2/projects.deidentifyTemplates#alltext # The fields below are necessary to include the "allText" transformation in the payload send_empty_value: true allow_empty_object: true @@ -532,11 +532,11 @@ properties: description: | Common alphabets. values: - - :FFX_COMMON_NATIVE_ALPHABET_UNSPECIFIED #Unused. - - :NUMERIC #[0-9] (radix of 10) - - :HEXADECIMAL #[0-9A-F] (radix of 16) - - :UPPER_CASE_ALPHA_NUMERIC #[0-9A-Z] (radix of 36) - - :ALPHA_NUMERIC #[0-9A-Za-z] (radix of 62) + - :FFX_COMMON_NATIVE_ALPHABET_UNSPECIFIED # Unused. + - :NUMERIC # [0-9] (radix of 10) + - :HEXADECIMAL # [0-9A-F] (radix of 16) + - :UPPER_CASE_ALPHA_NUMERIC # [0-9A-Z] (radix of 36) + - :ALPHA_NUMERIC # [0-9A-Za-z] (radix of 62) - !ruby/object:Api::Type::String name: 'customAlphabet' description: | @@ -687,7 +687,7 @@ properties: validation: !ruby/object:Provider::Terraform::Validation function: - 'validation.IntBetween(0, 24)' # "An API may choose to allow the value "24:00:00" for scenarios like business closing time." + 'validation.IntBetween(0, 24)' # "An API may choose to allow the value "24:00:00" for scenarios like business closing time." - !ruby/object:Api::Type::Integer name: minutes description: @@ -707,7 +707,7 @@ properties: validation: !ruby/object:Provider::Terraform::Validation function: - 'validation.IntBetween(0, 60)' # "An API may allow the value 60 if it allows leap-seconds." + 'validation.IntBetween(0, 60)' # "An API may allow the value 60 if it allows leap-seconds." - !ruby/object:Api::Type::Integer name: nanos description: @@ -816,7 +816,7 @@ properties: Hours of day in 24 hour format. Should be from 0 to 23. An API may choose to allow the value "24:00:00" for scenarios like business closing time. validation: !ruby/object:Provider::Terraform::Validation - function: 'validation.IntBetween(0, 24)' # "An API may choose to allow the value "24:00:00" for scenarios like business closing time." + function: 'validation.IntBetween(0, 24)' # "An API may choose to allow the value "24:00:00" for scenarios like business closing time." - !ruby/object:Api::Type::Integer name: minutes description: @@ -833,7 +833,7 @@ properties: the value 60 if it allows leap-seconds. validation: !ruby/object:Provider::Terraform::Validation - function: 'validation.IntBetween(0, 60)' # "An API may allow the value 60 if it allows leap-seconds." + function: 'validation.IntBetween(0, 60)' # "An API may allow the value 60 if it allows leap-seconds." - !ruby/object:Api::Type::Integer name: nanos description: @@ -889,7 +889,7 @@ properties: name: redactConfig description: | Redact a given value. For example, if used with an InfoTypeTransformation transforming PHONE_NUMBER, and input 'My phone number is 206-555-0123', the output would be 'My phone number is '. - properties: [] # Meant to be an empty object with no properties - see here : https://cloud.google.com/dlp/docs/reference/rest/v2/projects.deidentifyTemplates#redactconfig + properties: [] # Meant to be an empty object with no properties - see here : https://cloud.google.com/dlp/docs/reference/rest/v2/projects.deidentifyTemplates#redactconfig # The fields below are necessary to include the "redactConfig" transformation in the payload # A side-effect is null values when the field is unused, see: https://github.com/hashicorp/terraform-provider-google/issues/13201 send_empty_value: true @@ -1043,11 +1043,11 @@ properties: description: | Common alphabets. values: - - :FFX_COMMON_NATIVE_ALPHABET_UNSPECIFIED #Unused. - - :NUMERIC #[0-9] (radix of 10) - - :HEXADECIMAL #[0-9A-F] (radix of 16) - - :UPPER_CASE_ALPHA_NUMERIC #[0-9A-Z] (radix of 36) - - :ALPHA_NUMERIC #[0-9A-Za-z] (radix of 62) + - :FFX_COMMON_NATIVE_ALPHABET_UNSPECIFIED # Unused. + - :NUMERIC # [0-9] (radix of 10) + - :HEXADECIMAL # [0-9A-F] (radix of 16) + - :UPPER_CASE_ALPHA_NUMERIC # [0-9A-Z] (radix of 36) + - :ALPHA_NUMERIC # [0-9A-Za-z] (radix of 62) - !ruby/object:Api::Type::String name: 'customAlphabet' description: | @@ -1518,12 +1518,12 @@ properties: name: partToExtract description: The part of the time to keep. values: - - :YEAR # [0-9999] - - :MONTH # [1-12] - - :DAY_OF_MONTH # [1-31] - - :DAY_OF_WEEK # [1-7] - - :WEEK_OF_YEAR # [1-53] - - :HOUR_OF_DAY # [0-23] + - :YEAR # [0-9999] + - :MONTH # [1-12] + - :DAY_OF_MONTH # [1-31] + - :DAY_OF_WEEK # [1-7] + - :WEEK_OF_YEAR # [1-53] + - :HOUR_OF_DAY # [0-23] - !ruby/object:Api::Type::NestedObject name: cryptoHashConfig description: | diff --git a/mmv1/products/dlp/JobTrigger.yaml b/mmv1/products/dlp/JobTrigger.yaml index f35a0ab6fde7..67310d6682b0 100644 --- a/mmv1/products/dlp/JobTrigger.yaml +++ b/mmv1/products/dlp/JobTrigger.yaml @@ -590,7 +590,7 @@ properties: name: 'surrogateType' description: | Message for detecting output from deidentification transformations that support reversing. - properties: [] # Meant to be an empty object with no properties - see here : https://cloud.google.com/dlp/docs/reference/rest/v2/InspectConfig#SurrogateType + properties: [] # Meant to be an empty object with no properties - see here : https://cloud.google.com/dlp/docs/reference/rest/v2/InspectConfig#SurrogateType # The fields below are necessary to include the "surrogateType" in the payload send_empty_value: true allow_empty_object: true diff --git a/mmv1/products/filestore/product.yaml b/mmv1/products/filestore/product.yaml index 56eb47b52471..a6f6546208ac 100644 --- a/mmv1/products/filestore/product.yaml +++ b/mmv1/products/filestore/product.yaml @@ -38,10 +38,10 @@ async: !ruby/object:Api::OpAsync resource_inside_response: true status: !ruby/object:Api::OpAsync::Status path: 'done' - complete: True + complete: true allowed: - - True - - False + - true + - false error: !ruby/object:Api::OpAsync::Error path: 'error' message: 'message' diff --git a/mmv1/products/firebase/AppleApp.yaml b/mmv1/products/firebase/AppleApp.yaml index dd47a666e802..c3bab50b8ce7 100644 --- a/mmv1/products/firebase/AppleApp.yaml +++ b/mmv1/products/firebase/AppleApp.yaml @@ -69,7 +69,7 @@ examples: display_name: 'Display Name Full' bundle_id: 'apple.app.12345' app_store_id: '12345' - team_id: '9987654321' # Has to be a 10-digit number. + team_id: '9987654321' # Has to be a 10-digit number. test_env_vars: org_id: :ORG_ID project_id: :PROJECT_NAME diff --git a/mmv1/products/firebase/WebApp.yaml b/mmv1/products/firebase/WebApp.yaml index 9fd1629ae2d1..e89f594ec365 100644 --- a/mmv1/products/firebase/WebApp.yaml +++ b/mmv1/products/firebase/WebApp.yaml @@ -48,7 +48,7 @@ import_format: ['{{project}} {{name}}'] autogen_async: true skip_sweeper: true examples: - - !ruby/object:Provider::Terraform::Examples # TODO: https://github.com/hashicorp/terraform-provider-google/issues/14158 + - !ruby/object:Provider::Terraform::Examples # TODO: https://github.com/hashicorp/terraform-provider-google/issues/14158 skip_vcr: true name: 'firebase_web_app_basic' min_version: 'beta' diff --git a/mmv1/products/firebasehosting/Release.yaml b/mmv1/products/firebasehosting/Release.yaml index a7f28a992a61..07de657d7c9b 100644 --- a/mmv1/products/firebasehosting/Release.yaml +++ b/mmv1/products/firebasehosting/Release.yaml @@ -17,7 +17,7 @@ min_version: beta base_url: 'sites/{{site_id}}/channels/{{channel_id}}/releases' self_link: 'sites/{{site_id}}/channels/{{channel_id}}/releases/{{release_id}}' create_url: 'sites/{{site_id}}/channels/{{channel_id}}/releases?versionName={{version_name}}' -immutable: true # not updatable +immutable: true # not updatable description: | A Release is a particular collection of configurations that is set to be public at a particular time. references: !ruby/object:Api::Resource::ReferenceLinks diff --git a/mmv1/products/firebasehosting/Version.yaml b/mmv1/products/firebasehosting/Version.yaml index 28da7ac83291..cd4d8eb500c1 100644 --- a/mmv1/products/firebasehosting/Version.yaml +++ b/mmv1/products/firebasehosting/Version.yaml @@ -17,7 +17,7 @@ min_version: beta base_url: sites/{{site_id}}/versions self_link: sites/{{site_id}}/versions/{{version_id}} create_url: sites/{{site_id}}/versions -immutable: true # not updatable +immutable: true # not updatable description: | A `Version` is a configuration which determine how a site is displayed. Static files are not supported at the moment. references: !ruby/object:Api::Resource::ReferenceLinks diff --git a/mmv1/products/firebasestorage/Bucket.yaml b/mmv1/products/firebasestorage/Bucket.yaml index 771f278b8e35..17fcd76af4ae 100644 --- a/mmv1/products/firebasestorage/Bucket.yaml +++ b/mmv1/products/firebasestorage/Bucket.yaml @@ -22,7 +22,7 @@ delete_verb: :POST description: | An association between a Firebase project and a Google Cloud Storage bucket. This association enables integration of Cloud Storage buckets with Firebase such as Firebase SDKS, Authentication, and Security Rules. -immutable: true # Does not support update operation. There is nothing to update. +immutable: true # Does not support update operation. There is nothing to update. references: !ruby/object:Api::Resource::ReferenceLinks guides: 'Official Documentation': 'https://firebase.google.com/docs/storage/' diff --git a/mmv1/products/gkehub/Membership.yaml b/mmv1/products/gkehub/Membership.yaml index e66e02b64840..7634a3a4c0e5 100644 --- a/mmv1/products/gkehub/Membership.yaml +++ b/mmv1/products/gkehub/Membership.yaml @@ -69,7 +69,7 @@ examples: cluster_name: 'basiccluster' test_env_vars: project: :PROJECT_NAME - # Skip sweeper gen since this is a child resource. +# Skip sweeper gen since this is a child resource. skip_sweeper: true id_format: 'projects/{{project}}/locations/global/memberships/{{membership_id}}' import_format: diff --git a/mmv1/products/gkeonprem/VmwareNodePool.yaml b/mmv1/products/gkeonprem/VmwareNodePool.yaml index 9bd0105b0efb..a2767afef417 100644 --- a/mmv1/products/gkeonprem/VmwareNodePool.yaml +++ b/mmv1/products/gkeonprem/VmwareNodePool.yaml @@ -116,7 +116,6 @@ properties: description: | The OS image to be used for each node in a node pool. Currently `cos`, `ubuntu`, `ubuntu_containerd` and `windows` are supported. - required: true - !ruby/object:Api::Type::String name: "image" description: The OS image name in vCenter, only valid when using Windows. @@ -286,4 +285,4 @@ properties: name: "onPremVersion" description: | Anthos version for the node pool. Defaults to the user cluster version. - output: true \ No newline at end of file + output: true diff --git a/mmv1/products/iam2/product.yaml b/mmv1/products/iam2/product.yaml index fc86a4a343d2..c856f35dea6d 100644 --- a/mmv1/products/iam2/product.yaml +++ b/mmv1/products/iam2/product.yaml @@ -37,7 +37,7 @@ async: !ruby/object:Api::OpAsync path: 'targetLink' status: !ruby/object:Api::OpAsync::Status path: 'done' - complete: True + complete: true allowed: - true - false diff --git a/mmv1/products/iambeta/product.yaml b/mmv1/products/iambeta/product.yaml index 808bfe530251..284843a60d2c 100644 --- a/mmv1/products/iambeta/product.yaml +++ b/mmv1/products/iambeta/product.yaml @@ -37,7 +37,7 @@ async: !ruby/object:Api::OpAsync path: 'targetLink' status: !ruby/object:Api::OpAsync::Status path: 'done' - complete: True + complete: true allowed: - true - false diff --git a/mmv1/products/iamworkforcepool/product.yaml b/mmv1/products/iamworkforcepool/product.yaml index 697162961e84..65b859518531 100644 --- a/mmv1/products/iamworkforcepool/product.yaml +++ b/mmv1/products/iamworkforcepool/product.yaml @@ -37,7 +37,7 @@ async: !ruby/object:Api::OpAsync path: 'targetLink' status: !ruby/object:Api::OpAsync::Status path: 'done' - complete: True + complete: true allowed: - true - false diff --git a/mmv1/products/logging/LinkedDataset.yaml b/mmv1/products/logging/LinkedDataset.yaml index 09430b1c3031..286ae6a2ac1f 100644 --- a/mmv1/products/logging/LinkedDataset.yaml +++ b/mmv1/products/logging/LinkedDataset.yaml @@ -67,7 +67,7 @@ parameters: url_param_only: true immutable: true default_from_api: true - - !ruby/object:Api::Type::String # Make this a String for now since we don't have a good way to reference multiple resources. + - !ruby/object:Api::Type::String # Make this a String for now since we don't have a good way to reference multiple resources. name: bucket description: The bucket to which the linked dataset is attached. url_param_only: true diff --git a/mmv1/products/logging/LogView.yaml b/mmv1/products/logging/LogView.yaml index c5fad24f2987..9115fddc428a 100644 --- a/mmv1/products/logging/LogView.yaml +++ b/mmv1/products/logging/LogView.yaml @@ -57,7 +57,7 @@ parameters: url_param_only: true immutable: true default_from_api: true - - !ruby/object:Api::Type::String # Make this a String for now since we don't have a good way to reference multiple resources. + - !ruby/object:Api::Type::String # Make this a String for now since we don't have a good way to reference multiple resources. name: bucket description: The bucket of the resource url_param_only: true diff --git a/mmv1/products/memcache/product.yaml b/mmv1/products/memcache/product.yaml index 269f3f4afcd5..3f96dbca3f47 100644 --- a/mmv1/products/memcache/product.yaml +++ b/mmv1/products/memcache/product.yaml @@ -32,10 +32,10 @@ async: !ruby/object:Api::OpAsync resource_inside_response: true status: !ruby/object:Api::OpAsync::Status path: 'done' - complete: True + complete: true allowed: - - True - - False + - true + - false error: !ruby/object:Api::OpAsync::Error path: 'error' message: 'message' diff --git a/mmv1/products/metastore/Federation.yaml b/mmv1/products/metastore/Federation.yaml index c7797c4a216b..bbf936cadc02 100644 --- a/mmv1/products/metastore/Federation.yaml +++ b/mmv1/products/metastore/Federation.yaml @@ -53,7 +53,7 @@ import_format: examples: - !ruby/object:Provider::Terraform::Examples name: 'dataproc_metastore_federation_basic' - skip_test: true # https://github.com/hashicorp/terraform-provider-google/issues/13710 + skip_test: true # https://github.com/hashicorp/terraform-provider-google/issues/13710 primary_resource_id: 'default' min_version: beta primary_resource_name: @@ -63,7 +63,7 @@ examples: service_id: 'metastore-service' - !ruby/object:Provider::Terraform::Examples name: 'dataproc_metastore_federation_bigquery' - skip_test: true # https://github.com/hashicorp/terraform-provider-google/issues/13710 + skip_test: true # https://github.com/hashicorp/terraform-provider-google/issues/13710 primary_resource_id: 'default' min_version: beta primary_resource_name: diff --git a/mmv1/products/metastore/Service.yaml b/mmv1/products/metastore/Service.yaml index 6cbbd772f0d4..d323946efb01 100644 --- a/mmv1/products/metastore/Service.yaml +++ b/mmv1/products/metastore/Service.yaml @@ -84,7 +84,7 @@ examples: name: 'dataproc_metastore_service_endpoint' min_version: beta skip_docs: true - skip_test: true # https://github.com/hashicorp/terraform-provider-google/issues/13710 + skip_test: true # https://github.com/hashicorp/terraform-provider-google/issues/13710 primary_resource_id: 'endpoint' vars: metastore_service_name: 'metastore-endpoint' diff --git a/mmv1/products/mlengine/Model.yaml b/mmv1/products/mlengine/Model.yaml index 139852bbeedf..9ea33d2aa9f9 100644 --- a/mmv1/products/mlengine/Model.yaml +++ b/mmv1/products/mlengine/Model.yaml @@ -78,8 +78,8 @@ properties: name: 'name' required: true description: The name specified for the version when it was created. - # Even though only one region is supported, keeping this as an array - # to future-proof it. + # Even though only one region is supported, keeping this as an array + # to future-proof it. - !ruby/object:Api::Type::Array name: 'regions' description: | diff --git a/mmv1/products/mlengine/Version.yaml b/mmv1/products/mlengine/Version.yaml index 4a646e58b6fa..0946169aa4d8 100644 --- a/mmv1/products/mlengine/Version.yaml +++ b/mmv1/products/mlengine/Version.yaml @@ -110,7 +110,7 @@ properties: name: 'labels' description: One or more labels that you can add, to organize your model versions - # no etag since it's immutable: true + # no etag since it's immutable: true - !ruby/object:Api::Type::Enum name: 'framework' description: diff --git a/mmv1/products/networksecurity/AddressGroup.yaml b/mmv1/products/networksecurity/AddressGroup.yaml index a262b96c7b63..a883c74f853a 100644 --- a/mmv1/products/networksecurity/AddressGroup.yaml +++ b/mmv1/products/networksecurity/AddressGroup.yaml @@ -23,7 +23,7 @@ update_mask: true description: | AddressGroup is a resource that specifies how a collection of IP/DNS used in Firewall Policy. references: - !ruby/object:Api::Resource::ReferenceLinks # TODO(diogoesteves): change the url to GA once it is available. + !ruby/object:Api::Resource::ReferenceLinks # TODO(diogoesteves): change the url to GA once it is available. guides: "Use AddressGroups": "https://cloud.google.com/vpc/docs/use-address-groups-firewall-policies" api: "https://cloud.google.com/traffic-director/docs/reference/network-security/rest/v1beta1/organizations.locations.addressGroups" @@ -56,7 +56,7 @@ examples: test_env_vars: project: :PROJECT_NAME custom_code: - !ruby/object:Provider::Terraform::CustomCode # Calling custom operation that are different from other network security resources. + !ruby/object:Provider::Terraform::CustomCode # Calling custom operation that are different from other network security resources. post_create: templates/terraform/post_create/network_security_address_group_operation.go.erb post_update: templates/terraform/post_update/network_security_address_group_operation.go.erb post_delete: templates/terraform/post_delete/network_security_address_group_operation.go.erb diff --git a/mmv1/products/networksecurity/AuthorizationPolicy.yaml b/mmv1/products/networksecurity/AuthorizationPolicy.yaml index f70cb906164e..f4af9f533eee 100644 --- a/mmv1/products/networksecurity/AuthorizationPolicy.yaml +++ b/mmv1/products/networksecurity/AuthorizationPolicy.yaml @@ -43,7 +43,8 @@ async: !ruby/object:Api::OpAsync path: 'error' message: 'message' autogen_async: true -import_format: ['projects/{{project}}/locations/{{location}}/authorizationPolicies/{{name}}'] +import_format: + ['projects/{{project}}/locations/{{location}}/authorizationPolicies/{{name}}'] examples: - !ruby/object:Provider::Terraform::Examples min_version: beta @@ -85,7 +86,8 @@ properties: output: true - !ruby/object:Api::Type::KeyValuePairs name: 'labels' - description: Set of label tags associated with the AuthorizationPolicy resource. + description: + Set of label tags associated with the AuthorizationPolicy resource. - !ruby/object:Api::Type::String name: 'description' description: | @@ -94,40 +96,40 @@ properties: name: 'action' description: | The action to take when a rule match is found. Possible values are "ALLOW" or "DENY". - required: true + required: true values: - :ALLOW - :DENY - !ruby/object:Api::Type::Array name: rules description: | - List of rules to match. Note that at least one of the rules must match in order for the action specified in the 'action' field to be taken. + List of rules to match. Note that at least one of the rules must match in order for the action specified in the 'action' field to be taken. A rule is a match if there is a matching source and destination. If left blank, the action specified in the action field will be applied on every request. item_type: !ruby/object:Api::Type::NestedObject properties: - !ruby/object:Api::Type::Array name: sources description: | - List of attributes for the traffic source. All of the sources must match. A source is a match if both principals and ipBlocks match. + List of attributes for the traffic source. All of the sources must match. A source is a match if both principals and ipBlocks match. If not set, the action specified in the 'action' field will be applied without any rule checks for the source. item_type: !ruby/object:Api::Type::NestedObject properties: - !ruby/object:Api::Type::Array name: principals description: | - List of peer identities to match for authorization. At least one principal should match. Each peer can be an exact match, or a prefix match (example, "namespace/*") or a suffix match (example, "*/service-account") or a presence match "*". + List of peer identities to match for authorization. At least one principal should match. Each peer can be an exact match, or a prefix match (example, "namespace/*") or a suffix match (example, "*/service-account") or a presence match "*". Authorization based on the principal name without certificate validation (configured by ServerTlsPolicy resource) is considered insecure. item_type: Api::Type::String - !ruby/object:Api::Type::Array name: ipBlocks description: | - List of CIDR ranges to match based on source IP address. At least one IP block should match. Single IP (e.g., "1.2.3.4") and CIDR (e.g., "1.2.3.0/24") are supported. Authorization based on source IP alone should be avoided. + List of CIDR ranges to match based on source IP address. At least one IP block should match. Single IP (e.g., "1.2.3.4") and CIDR (e.g., "1.2.3.0/24") are supported. Authorization based on source IP alone should be avoided. The IP addresses of any load balancers or proxies should be considered untrusted. item_type: Api::Type::String - !ruby/object:Api::Type::Array name: destinations description: | - List of attributes for the traffic destination. All of the destinations must match. A destination is a match if a request matches all the specified hosts, ports, methods and headers. + List of attributes for the traffic destination. All of the destinations must match. A destination is a match if a request matches all the specified hosts, ports, methods and headers. If not set, the action specified in the 'action' field will be applied without any rule checks for the destination. item_type: !ruby/object:Api::Type::NestedObject properties: @@ -140,7 +142,7 @@ properties: - !ruby/object:Api::Type::Array name: ports description: | - List of destination ports to match. At least one port should match. + List of destination ports to match. At least one port should match. required: true item_type: Api::Type::Integer - !ruby/object:Api::Type::Array @@ -148,11 +150,11 @@ properties: description: | A list of HTTP methods to match. At least one method should match. Should not be set for gRPC services. required: true - item_type: Api::Type::String + item_type: Api::Type::String - !ruby/object:Api::Type::NestedObject name: httpHeaderMatch description: | - Match against key:value pair in http header. Provides a flexible match based on HTTP headers, for potentially advanced use cases. At least one header should match. + Match against key:value pair in http header. Provides a flexible match based on HTTP headers, for potentially advanced use cases. At least one header should match. Avoid using header matches to make authorization decisions unless there is a strong guarantee that requests arrive through a trusted client or proxy. properties: - !ruby/object:Api::Type::String @@ -165,4 +167,3 @@ properties: description: | The value of the header must match the regular expression specified in regexMatch. For regular expression grammar, please see: en.cppreference.com/w/cpp/regex/ecmascript For matching against a port specified in the HTTP request, use a headerMatch with headerName set to Host and a regular expression that satisfies the RFC2616 Host header's port specifier. required: true - diff --git a/mmv1/products/networksecurity/GatewaySecurityPolicies.yaml b/mmv1/products/networksecurity/GatewaySecurityPolicies.yaml index cd2f4d5e05e4..b62d14ec3d69 100644 --- a/mmv1/products/networksecurity/GatewaySecurityPolicies.yaml +++ b/mmv1/products/networksecurity/GatewaySecurityPolicies.yaml @@ -22,7 +22,7 @@ update_mask: true description: | The GatewaySecurityPolicy resource contains a collection of GatewaySecurityPolicyRules and associated metadata. references: - !ruby/object:Api::Resource::ReferenceLinks # TODO(felipegc): change the url to beta once it is available. + !ruby/object:Api::Resource::ReferenceLinks # TODO(felipegc): change the url to beta once it is available. api: 'https://cloud.google.com/secure-web-proxy/docs/reference/network-security/rest/v1alpha1/projects.locations.gatewaySecurityPolicies' async: !ruby/object:Api::OpAsync operation: !ruby/object:Api::OpAsync::Operation diff --git a/mmv1/products/networksecurity/GatewaySecurityPoliciesRules.yaml b/mmv1/products/networksecurity/GatewaySecurityPoliciesRules.yaml index 3f7c34ae5ac9..65b2ba4cf434 100644 --- a/mmv1/products/networksecurity/GatewaySecurityPoliciesRules.yaml +++ b/mmv1/products/networksecurity/GatewaySecurityPoliciesRules.yaml @@ -23,7 +23,7 @@ description: | The GatewaySecurityPolicyRule resource is in a nested collection within a GatewaySecurityPolicy and represents a traffic matching condition and associated action to perform. references: - !ruby/object:Api::Resource::ReferenceLinks # TODO(felipegc): change the url to beta once it is available. + !ruby/object:Api::Resource::ReferenceLinks # TODO(felipegc): change the url to beta once it is available. api: 'https://cloud.google.com/secure-web-proxy/docs/reference/network-security/rest/v1alpha1/projects.locations.gatewaySecurityPolicies.rules' async: !ruby/object:Api::OpAsync operation: !ruby/object:Api::OpAsync::Operation diff --git a/mmv1/products/networksecurity/UrlLists.yaml b/mmv1/products/networksecurity/UrlLists.yaml index 3f1ed9fe9274..2dae75b6c30a 100644 --- a/mmv1/products/networksecurity/UrlLists.yaml +++ b/mmv1/products/networksecurity/UrlLists.yaml @@ -22,7 +22,7 @@ update_mask: true description: | UrlList proto helps users to set reusable, independently manageable lists of hosts, host patterns, URLs, URL patterns. references: - !ruby/object:Api::Resource::ReferenceLinks # TODO(diogoesteves): change the url to beta once it is available. + !ruby/object:Api::Resource::ReferenceLinks # TODO(diogoesteves): change the url to beta once it is available. guides: 'Use UrlLists': ' https://cloud.google.com/secure-web-proxy/docs/use-url-list' diff --git a/mmv1/products/networkservices/EdgeCacheOrigin.yaml b/mmv1/products/networkservices/EdgeCacheOrigin.yaml index 2fe9c8189404..a837ece8c3c9 100644 --- a/mmv1/products/networkservices/EdgeCacheOrigin.yaml +++ b/mmv1/products/networkservices/EdgeCacheOrigin.yaml @@ -96,7 +96,7 @@ properties: When providing an FQDN (hostname), it must be publicly resolvable (e.g. via Google public DNS) and IP addresses must be publicly routable. It must not contain a protocol (e.g., https://) and it must not contain any slashes. If a Cloud Storage bucket is provided, it must be in the canonical "gs://bucketname" format. Other forms, such as "storage.googleapis.com", will be rejected. - !ruby/object:Api::Type::Enum - name: 'protocol' # default http2 from api + name: 'protocol' # default http2 from api description: | The protocol to use to connect to the configured origin. Defaults to HTTP2, and it is strongly recommended that users use HTTP2 for both security & performance. @@ -107,7 +107,7 @@ properties: - :HTTP default_from_api: true - !ruby/object:Api::Type::Integer - name: 'port' # default from api + name: 'port' # default from api description: | The port to connect to the origin on. Defaults to port 443 for HTTP2 and HTTPS protocols, and port 80 for HTTP. @@ -139,7 +139,7 @@ properties: A reference to a Topic resource. diff_suppress_func: 'compareResourceNames' - !ruby/object:Api::Type::Array - name: 'retryConditions' # default CONNECT_FAILURE from api + name: 'retryConditions' # default CONNECT_FAILURE from api description: | Specifies one or more retry conditions for the configured origin. diff --git a/mmv1/products/networkservices/EdgeCacheService.yaml b/mmv1/products/networkservices/EdgeCacheService.yaml index 9a754072ecc5..95bba8ac190f 100644 --- a/mmv1/products/networkservices/EdgeCacheService.yaml +++ b/mmv1/products/networkservices/EdgeCacheService.yaml @@ -89,12 +89,12 @@ properties: name: 'labels' description: 'Set of label tags associated with the EdgeCache resource.' - !ruby/object:Api::Type::Boolean - name: disableQuic # default from api + name: disableQuic # default from api description: | HTTP/3 (IETF QUIC) and Google QUIC are enabled by default. default_from_api: true - !ruby/object:Api::Type::Boolean - name: disableHttp2 # default from api + name: disableHttp2 # default from api description: | Disables HTTP/2. @@ -102,7 +102,7 @@ properties: Some legacy HTTP clients may have issues with HTTP/2 connections due to broken HTTP/2 implementations. Setting this to true will prevent HTTP/2 from being advertised and negotiated. - !ruby/object:Api::Type::Boolean - name: requireTls # default from api + name: requireTls # default from api description: | Require TLS (HTTPS) for all clients connecting to this service. @@ -240,7 +240,7 @@ properties: item_type: !ruby/object:Api::Type::NestedObject properties: - !ruby/object:Api::Type::Boolean - name: ignoreCase # default from api + name: ignoreCase # default from api description: | Specifies that prefixMatch and fullPathMatch matches are case sensitive. default_from_api: true @@ -275,7 +275,7 @@ properties: description: | The value of the header must end with the contents of suffixMatch. - !ruby/object:Api::Type::Boolean - name: invertMatch # default from api + name: invertMatch # default from api description: | If set to false (default), the headerMatch is considered a match if the match criteria above are met. If set to true, the headerMatch is considered a match if the match criteria above are NOT met. @@ -350,7 +350,7 @@ properties: description: | The value of the header to add. - !ruby/object:Api::Type::Boolean - name: replace # default from api + name: replace # default from api description: | Whether to replace all existing headers with the same name. default_from_api: true @@ -376,7 +376,7 @@ properties: description: | The value of the header to add. - !ruby/object:Api::Type::Boolean - name: replace # default from api + name: replace # default from api description: | Whether to replace all existing headers with the same name. default_from_api: true @@ -421,7 +421,7 @@ properties: The policy to use for defining caching and signed request behaviour for requests that match this route. properties: - !ruby/object:Api::Type::Enum - name: 'cacheMode' # default from api + name: 'cacheMode' # default from api description: | Cache modes allow users to control the behaviour of the cache, what content it should cache automatically, whether to respect origin headers, or whether to unconditionally cache all responses. @@ -446,7 +446,7 @@ properties: When the cache mode is set to "USE_ORIGIN_HEADERS" or "BYPASS_CACHE", you must omit this field. A duration in seconds terminated by 's'. Example: "3s". - !ruby/object:Api::Type::String - name: 'defaultTtl' # defalt from api + name: 'defaultTtl' # defalt from api description: | Specifies the default TTL for cached content served by this origin for responses that do not have an existing valid TTL (max-age or s-max-age). @@ -465,7 +465,7 @@ properties: A duration in seconds terminated by 's'. Example: "3s". default_from_api: true - !ruby/object:Api::Type::String - name: 'maxTtl' # defalt from api + name: 'maxTtl' # defalt from api description: | Specifies the maximum allowed TTL for cached content served by this origin. @@ -488,7 +488,7 @@ properties: Defines the request parameters that contribute to the cache key. properties: - !ruby/object:Api::Type::Boolean - name: includeProtocol # default from api + name: includeProtocol # default from api description: | If true, http and https requests will be cached separately. default_from_api: true @@ -503,7 +503,7 @@ properties: excludeQueryParameters is set, the entire query string will be included. - !ruby/object:Api::Type::Boolean - name: excludeHost # default from api + name: excludeHost # default from api default_from_api: true description: | If true, requests to different hosts will be cached separately. @@ -574,7 +574,7 @@ properties: Note that when specifying an explicit negativeCachingPolicy, you should take care to specify a cache TTL for all response codes that you wish to cache. The CDNPolicy will not apply any default negative caching when a policy exists. - !ruby/object:Api::Type::Enum - name: 'signedRequestMode' # default from api + name: 'signedRequestMode' # default from api default_from_api: true description: | Whether to enforce signed requests. The default value is DISABLED, which means all content is public, and does not authorize access. @@ -587,7 +587,7 @@ properties: - :REQUIRE_SIGNATURES - :REQUIRE_TOKENS - !ruby/object:Api::Type::String - name: 'signedRequestKeyset' # resource ref, EdgeCacheKeyset? + name: 'signedRequestKeyset' # resource ref, EdgeCacheKeyset? default_from_api: true description: | The EdgeCacheKeyset containing the set of public keys used to validate signed requests at the edge. @@ -824,7 +824,7 @@ properties: prefixRedirect cannot be supplied together with pathRedirect. Supply one alone or neither. If neither is supplied, the path of the original request will be used for the redirect. - !ruby/object:Api::Type::Enum - name: 'redirectResponseCode' # default from api + name: 'redirectResponseCode' # default from api default_from_api: true description: | The HTTP Status code to use for this RedirectAction. @@ -843,14 +843,14 @@ properties: - :TEMPORARY_REDIRECT - :PERMANENT_REDIRECT - !ruby/object:Api::Type::Boolean - name: 'httpsRedirect' # default from api + name: 'httpsRedirect' # default from api default_from_api: true description: | If set to true, the URL scheme in the redirected request is set to https. If set to false, the URL scheme of the redirected request will remain the same as that of the request. This can only be set if there is at least one (1) edgeSslCertificate set on the service. - !ruby/object:Api::Type::Boolean - name: 'stripQuery' # default from api + name: 'stripQuery' # default from api default_from_api: true description: | If set to true, any accompanying query portion of the original URL is removed prior to redirecting the request. If set to false, the query portion of the original URL is retained. @@ -860,7 +860,7 @@ properties: Specifies the logging options for the traffic served by this service. If logging is enabled, logs will be exported to Cloud Logging. properties: - !ruby/object:Api::Type::Boolean - name: 'enable' #default from api + name: 'enable' # default from api required: true description: | Specifies whether to enable logging for traffic served by this service. diff --git a/mmv1/products/networkservices/GrpcRoute.yaml b/mmv1/products/networkservices/GrpcRoute.yaml index 4d1ca2e2af20..c8bd32e506c9 100644 --- a/mmv1/products/networkservices/GrpcRoute.yaml +++ b/mmv1/products/networkservices/GrpcRoute.yaml @@ -99,25 +99,25 @@ properties: name: hostnames item_type: Api::Type::String description: | - Required. Service hostnames with an optional port for which this route describes traffic. - required: true + Required. Service hostnames with an optional port for which this route describes traffic. + required: true - !ruby/object:Api::Type::Array name: meshes item_type: Api::Type::String description: | - List of meshes this GrpcRoute is attached to, as one of the routing rules to route the requests served by the mesh. + List of meshes this GrpcRoute is attached to, as one of the routing rules to route the requests served by the mesh. - !ruby/object:Api::Type::Array name: gateways item_type: Api::Type::String description: | - List of gateways this GrpcRoute is attached to, as one of the routing rules to route the requests served by the gateway. + List of gateways this GrpcRoute is attached to, as one of the routing rules to route the requests served by the gateway. - !ruby/object:Api::Type::Array name: rules description: | - Rules that define how traffic is routed and handled. + Rules that define how traffic is routed and handled. required: true item_type: !ruby/object:Api::Type::NestedObject - properties: + properties: - !ruby/object:Api::Type::Array name: matches description: | @@ -127,7 +127,7 @@ properties: - !ruby/object:Api::Type::Array name: headers description: | - Specifies a list of HTTP request headers to match against. + Specifies a list of HTTP request headers to match against. item_type: !ruby/object:Api::Type::NestedObject properties: - !ruby/object:Api::Type::String @@ -143,12 +143,12 @@ properties: - !ruby/object:Api::Type::Enum name: 'type' description: | - The type of match. + The type of match. values: - :TYPE_UNSPECIFIED - :EXACT - :REGULAR_EXPRESSION - default_value: :EXACT + default_value: :EXACT - !ruby/object:Api::Type::NestedObject name: method description: | @@ -157,22 +157,22 @@ properties: - !ruby/object:Api::Type::String name: grpcService description: | - Required. Name of the service to match against. + Required. Name of the service to match against. required: true - !ruby/object:Api::Type::String name: grpcMethod description: | - Required. Name of the method to match against. + Required. Name of the method to match against. required: true - !ruby/object:Api::Type::Boolean name: caseSensitive description: | Specifies that matches are case sensitive. The default value is true. - default_value: true + default_value: true - !ruby/object:Api::Type::NestedObject name: action description: | - Required. A detailed rule defining how to route traffic. + Required. A detailed rule defining how to route traffic. properties: - !ruby/object:Api::Type::Array name: destinations @@ -201,24 +201,24 @@ properties: - !ruby/object:Api::Type::String name: fixedDelay description: | - Specify a fixed delay before forwarding the request. + Specify a fixed delay before forwarding the request. - !ruby/object:Api::Type::Integer name: percentage description: | - The percentage of traffic on which delay will be injected. + The percentage of traffic on which delay will be injected. - !ruby/object:Api::Type::NestedObject name: abort description: | - Specification of how client requests are aborted as part of fault injection before being sent to a destination. + Specification of how client requests are aborted as part of fault injection before being sent to a destination. properties: - !ruby/object:Api::Type::Integer name: httpStatus description: | - The HTTP status code used to abort the request. + The HTTP status code used to abort the request. - !ruby/object:Api::Type::Integer name: percentage description: | - The percentage of traffic which will be aborted. + The percentage of traffic which will be aborted. - !ruby/object:Api::Type::String name: timeout description: | @@ -233,17 +233,16 @@ properties: description: | Specifies one or more conditions when this retry policy applies. item_type: !ruby/object:Api::Type::Enum - name: retryCondition - description: Condition when this retry policy applies. - values: - - :connect-failure - - :refused-stream - - :cancelled - - :deadline-exceeded - - :resource-exhausted - - :unavailable + name: retryCondition + description: Condition when this retry policy applies. + values: + - :connect-failure + - :refused-stream + - :cancelled + - :deadline-exceeded + - :resource-exhausted + - :unavailable - !ruby/object:Api::Type::Integer name: numRetries description: | Specifies the allowed number of retries. - diff --git a/mmv1/products/notebooks/product.yaml b/mmv1/products/notebooks/product.yaml index 13cd23c9d159..179553c8f5a9 100644 --- a/mmv1/products/notebooks/product.yaml +++ b/mmv1/products/notebooks/product.yaml @@ -41,10 +41,10 @@ async: !ruby/object:Api::OpAsync resource_inside_response: true status: !ruby/object:Api::OpAsync::Status path: 'done' - complete: True + complete: true allowed: - - True - - False + - true + - false error: !ruby/object:Api::OpAsync::Error path: 'error' message: 'message' diff --git a/mmv1/products/osconfig/GuestPolicies.yaml b/mmv1/products/osconfig/GuestPolicies.yaml index 73593c691b1f..ebc3b9877a72 100644 --- a/mmv1/products/osconfig/GuestPolicies.yaml +++ b/mmv1/products/osconfig/GuestPolicies.yaml @@ -36,21 +36,21 @@ examples: instance_name: 'guest-policy-inst' guest_policy_id: 'guest-policy' ignore_read_extra: - - 'project' # project number is returned in rest response + - 'project' # project number is returned in rest response - !ruby/object:Provider::Terraform::Examples name: 'os_config_guest_policies_packages' primary_resource_id: 'guest_policies' vars: guest_policy_id: 'guest-policy' ignore_read_extra: - - 'project' # project number is returned in rest response + - 'project' # project number is returned in rest response - !ruby/object:Provider::Terraform::Examples name: 'os_config_guest_policies_recipes' primary_resource_id: 'guest_policies' vars: guest_policy_id: 'guest-policy' ignore_read_extra: - - 'project' # project number is returned in rest response + - 'project' # project number is returned in rest response custom_code: !ruby/object:Provider::Terraform::CustomCode post_create: templates/terraform/post_create/set_computed_name.erb diff --git a/mmv1/products/privateca/Certificate.yaml b/mmv1/products/privateca/Certificate.yaml index 52022cce5077..128d75499bf0 100644 --- a/mmv1/products/privateca/Certificate.yaml +++ b/mmv1/products/privateca/Certificate.yaml @@ -106,7 +106,7 @@ properties: The desired lifetime of the CA certificate. Used to create the "notBeforeTime" and "notAfterTime" fields inside an X.509 certificate. A duration in seconds with up to nine fractional digits, terminated by 's'. Example: "3.5s". - default_value: 315360000s # 10 years + default_value: 315360000s # 10 years immutable: true - !ruby/object:Api::Type::NestedObject name: 'revocationDetails' diff --git a/mmv1/products/privateca/CertificateAuthority.yaml b/mmv1/products/privateca/CertificateAuthority.yaml index 3fe913cbb23b..5f5e37d76629 100644 --- a/mmv1/products/privateca/CertificateAuthority.yaml +++ b/mmv1/products/privateca/CertificateAuthority.yaml @@ -86,9 +86,9 @@ examples: deletion_protection: 'false' ignore_read_extra: - 'deletion_protection' - # Skip test because it depends on a beta resource, but PrivateCA does - # not have a beta endpoint - !ruby/object:Provider::Terraform::Examples + # Skip test because it depends on a beta resource, but PrivateCA does + # not have a beta endpoint skip_test: true name: 'privateca_certificate_authority_byo_key' primary_resource_id: @@ -599,7 +599,7 @@ properties: The desired lifetime of the CA certificate. Used to create the "notBeforeTime" and "notAfterTime" fields inside an X.509 certificate. A duration in seconds with up to nine fractional digits, terminated by 's'. Example: "3.5s". - default_value: 315360000s # 10 years + default_value: 315360000s # 10 years immutable: true - !ruby/object:Api::Type::NestedObject name: 'keySpec' diff --git a/mmv1/products/pubsublite/product.yaml b/mmv1/products/pubsublite/product.yaml index 5060d5f2f48f..236999876cb6 100644 --- a/mmv1/products/pubsublite/product.yaml +++ b/mmv1/products/pubsublite/product.yaml @@ -13,7 +13,7 @@ --- !ruby/object:Api::Product name: PubsubLite -display_name: Cloud Pub/Sub # Merges docs under Pub/Sub +display_name: Cloud Pub/Sub # Merges docs under Pub/Sub versions: - !ruby/object:Api::Product::Version name: ga diff --git a/mmv1/products/sql/Instance.yaml b/mmv1/products/sql/Instance.yaml index 72088f3883c0..12148ed943e5 100644 --- a/mmv1/products/sql/Instance.yaml +++ b/mmv1/products/sql/Instance.yaml @@ -269,9 +269,9 @@ properties: The value of the flag. Booleans should be set to on for true and off for false. This field must be omitted if the flag doesn't take a value. - #- !ruby/object:Api::Type::Constant - # name: 'kind' - # value: 'sql#settings' + # - !ruby/object:Api::Type::Constant + # name: 'kind' + # value: 'sql#settings' - !ruby/object:Api::Type::NestedObject name: 'ipConfiguration' description: | diff --git a/mmv1/products/sql/SslCert.yaml b/mmv1/products/sql/SslCert.yaml index 5fbe71f49a94..b84aaa349b77 100644 --- a/mmv1/products/sql/SslCert.yaml +++ b/mmv1/products/sql/SslCert.yaml @@ -16,7 +16,7 @@ name: 'SslCert' kind: 'sql#sslCert' base_url: projects/{{project}}/instances/{{instance}}/sslCerts self_link: 'projects/{{project}}/instances/{{instance}}/sslCerts/{{sha1_fingerprint}}' -readonly: true # we're not enforcing state as it is all server-side driven. +readonly: true # we're not enforcing state as it is all server-side driven. description: | Represents an SSL certificate created for a Cloud SQL instance. To use the SSL certificate you must have the SSL Client Certificate and the diff --git a/mmv1/products/sql/product.yaml b/mmv1/products/sql/product.yaml index 0937d333da1b..42fdd7a2100d 100644 --- a/mmv1/products/sql/product.yaml +++ b/mmv1/products/sql/product.yaml @@ -43,14 +43,14 @@ async: !ruby/object:Api::OpAsync error: !ruby/object:Api::OpAsync::Error path: 'error/errors' message: 'message' - # 'BackupRun' is not idempotent and will not be covered. - # | - !ruby/object:Api::Resource - # | name: 'BackupRun' - # | kind: 'sql#backupRun' - # | base_url: projects/{{project}}/instances/{{instance}}/backupRuns - # 'Operation' is not idempotent and will not be covered. - # (it is used internally to assert the state of operations being performed) - # | - !ruby/object:Api::Resource - # | name: 'Operation' - # | kind: 'sql#operation' - # | base_url: projects/{{project}}/operations +# 'BackupRun' is not idempotent and will not be covered. +# | - !ruby/object:Api::Resource +# | name: 'BackupRun' +# | kind: 'sql#backupRun' +# | base_url: projects/{{project}}/instances/{{instance}}/backupRuns +# 'Operation' is not idempotent and will not be covered. +# (it is used internally to assert the state of operations being performed) +# | - !ruby/object:Api::Resource +# | name: 'Operation' +# | kind: 'sql#operation' +# | base_url: projects/{{project}}/operations diff --git a/mmv1/products/workflows/product.yaml b/mmv1/products/workflows/product.yaml index 07ff8df6dc9f..2f84e8ef20d4 100644 --- a/mmv1/products/workflows/product.yaml +++ b/mmv1/products/workflows/product.yaml @@ -37,10 +37,10 @@ async: !ruby/object:Api::OpAsync resource_inside_response: true status: !ruby/object:Api::OpAsync::Status path: 'done' - complete: True + complete: true allowed: - - True - - False + - true + - false error: !ruby/object:Api::OpAsync::Error path: 'error' message: 'message'